Hi Cao, Thanks for your patch!
On Tue, Oct 2, 2018 at 10:35 AM Cao Van Dong <[email protected]> wrote: > In __hwmon_device_register() function of hwmon.c, we have assigned 'dev' > directly to 'hdev->parent'. > Formerly, when registering the hwmon device, we pass NULL as the device. This > is not affected. > Recently, the developer has replaced the parameter NULL as the device by the > actual device. Do you know the commit ID of this recent change? It's useful for review of your change, and to know to which versions your patch should be backported. > This causes the "No sensors found" error. This patch is to fix this error. On which platform do you see this failure? Thanks again! > --- a/drivers/hwmon/hwmon.c > +++ b/drivers/hwmon/hwmon.c > @@ -610,7 +610,7 @@ __hwmon_device_register(struct device *dev, const char > *name, void *drvdata, > > hwdev->name = name; > hdev->class = &hwmon_class; > - hdev->parent = dev; > + hdev->parent = dev->parent; > hdev->of_node = dev ? dev->of_node : NULL; > hwdev->chip = chip; > dev_set_drvdata(hdev, drvdata); Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
