[PATCH] staging: et131x: fix invalid fail after the call to eeprom_wait_ready

devendra.aaru devendra.aaru at gmail.com
Sat Mar 9 06:11:08 UTC 2013


Hey Mark,

will you please ACK or NACK this?

On Thu, Feb 21, 2013 at 12:37 PM, Devendra Naga <devendra.aaru at gmail.com> wrote:
> should be err < 0 instead of if (err) which actually the read register
> value can be a positive number
>
> Signed-off-by: Devendra Naga <devendra.aaru at gmail.com>
> ---
>  drivers/staging/et131x/et131x.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c
> index 84bbcd4..6bd4a79 100644
> --- a/drivers/staging/et131x/et131x.c
> +++ b/drivers/staging/et131x/et131x.c
> @@ -607,7 +607,7 @@ static int eeprom_write(struct et131x_adapter *adapter, u32 addr, u8 data)
>          */
>
>         err = eeprom_wait_ready(pdev, NULL);
> -       if (err)
> +       if (err < 0)
>                 return err;
>
>          /*
> @@ -730,7 +730,7 @@ static int eeprom_read(struct et131x_adapter *adapter, u32 addr, u8 *pdata)
>          */
>
>         err = eeprom_wait_ready(pdev, NULL);
> -       if (err)
> +       if (err < 0)
>                 return err;
>         /*
>          * Write to the LBCIF Control Register:  bit 7=1, bit 6=0, bit 3=0,
> --
> 1.8.1.2
>



More information about the devel mailing list