Re: [PATCH -next] platform/x86: mlx-platform: Fix copy-paste error in mlxplat_init()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 8, 2018 at 6:56 AM, Vadim Pasternak <vadimp@xxxxxxxxxxxx> wrote:
>
>
>> -----Original Message-----
>> From: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
>> Sent: Wednesday, August 08, 2018 7:01 AM
>> To: Vadim Pasternak <vadimp@xxxxxxxxxxxx>; Darren Hart
>> <dvhart@xxxxxxxxxxxxx>; Andy Shevchenko <andy@xxxxxxxxxxxxx>
>> Cc: Wei Yongjun <weiyongjun1@xxxxxxxxxx>; platform-driver-
>> x86@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx
>> Subject: [PATCH -next] platform/x86: mlx-platform: Fix copy-paste error in
>> mlxplat_init()
>>
>> The return value from platform_device_register_resndata() is not checked
>> correctly. The test is done against a wrong variable. This patch fix it.
>>
>> Fixes: 0378123c5800 ("platform/x86: mlx-platform: Add mlxreg-fan platform
>> driver activation")
>> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
>

Pushed to my review and testing queue, thanks!

> Acked-by: Vadim Pasternak <vadimp@xxxxxxxxxxxx>
>
>> ---
>>  drivers/platform/x86/mlx-platform.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/platform/x86/mlx-platform.c b/drivers/platform/x86/mlx-
>> platform.c
>> index e1f9fce..d89936c 100644
>> --- a/drivers/platform/x86/mlx-platform.c
>> +++ b/drivers/platform/x86/mlx-platform.c
>> @@ -1704,8 +1704,8 @@ static int __init mlxplat_init(void)
>>                                       PLATFORM_DEVID_NONE, NULL, 0,
>>                                       mlxplat_fan,
>>                                       sizeof(*mlxplat_fan));
>> -             if (IS_ERR(priv->pdev_io_regs)) {
>> -                     err = PTR_ERR(priv->pdev_io_regs);
>> +             if (IS_ERR(priv->pdev_fan)) {
>> +                     err = PTR_ERR(priv->pdev_fan);
>>                       goto fail_platform_io_regs_register;
>>               }
>>       }
>



-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux