Re: [patch] radeon: handle errors in radeon_hwmon_init()

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

 



2010/8/7 Rafał Miłecki <zajec5@xxxxxxxxx>:
> 2010/8/8 Dan Carpenter <error27@xxxxxxxxx>:
>> Smatch complained that the ERR_PTR from hwmon_device_register() wasn't
>> handled.  I added some error handling in radeon_hwmon_init() to silence
>> the warning.
>>
>> Unfortunately errors from radeon_pm_init() aren't handled so this
>> doesn't really make a difference beyond silencing the warning.
>>
>> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
>
> Didn't we try to switch from DRM_ERROR to dev_err? I think that's
> preferred method.

Other than that, the patch looks fine.

Reviewed-by: Alex Deucher <alexdeucher@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux