[PATCH linux dev-5.3 v2 6/8] fsi: aspeed: Fix link enable
Andrew Jeffery
andrew at aj.id.au
Fri Nov 1 14:08:16 AEDT 2019
On Thu, 31 Oct 2019, at 16:06, Joel Stanley wrote:
> Link enable was saving the value of the write to result, which was
> not checked. It should either have been ignoring the result of the
> write, or erroring out if it failed. This chose to error out.
>
> Signed-off-by: Joel Stanley <joel at jms.id.au>
Lolled-by^W
Reviewed-by: Andrew Jeffery <andrew at aj.id.au>
> ---
> drivers/fsi/fsi-master-aspeed.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/fsi/fsi-master-aspeed.c
> b/drivers/fsi/fsi-master-aspeed.c
> index 8f757eaa6a55..ee760e84c576 100644
> --- a/drivers/fsi/fsi-master-aspeed.c
> +++ b/drivers/fsi/fsi-master-aspeed.c
> @@ -344,7 +344,9 @@ static int aspeed_master_link_enable(struct
> fsi_master *master, int link)
>
> reg = cpu_to_be32(0x80000000 >> bit);
>
> - result = opb_writel(aspeed, ctrl_base + FSI_MSENP0 + (4 * idx), reg);
> + ret = opb_writel(aspeed, ctrl_base + FSI_MSENP0 + (4 * idx), reg);
> + if (ret)
> + return ret;
More information about the openbmc
mailing list