Re: [PATCH 5.4 1/2] hwmon: Introduce hwmon_device_register_for_thermal
From: Julian Haller
Date: Wed Jun 22 2022 - 11:39:57 EST
> On Wed, Jun 22, 2022 at 04:49:01PM +0200, Julian Haller wrote:
> > From: Guenter Roeck <linux@xxxxxxxxxxxx>
> >
> > [ upstream commit e5d21072054fbadf41cd56062a3a14e447e8c22b ]
> >
> > The thermal subsystem registers a hwmon driver without providing
> > chip or sysfs group information. This is for legacy reasons and
> > would be difficult to change. At the same time, we want to enforce
> > that chip information is provided when registering a hwmon device
> > using hwmon_device_register_with_info(). To enable this, introduce
> > a special API for use only by the thermal subsystem.
> >
> > Acked-by: Rafael J . Wysocki <rafael@xxxxxxxxxx>
> > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
>
> What is the point of applying those patches to the 5.4 kernel ?
> This was intended for use with new code, not for stable releases.
>
> Guenter
The upstream commit ddaefa209c4ac791c1262e97c9b2d0440c8ef1d5 ("hwmon: Make chip
parameter for with_info API mandatory") was backported to the 5.4 kernel as
part of v5.4.198, see commit 1ec0bc72f5dab3ab367ae5230cf6f212d805a225. This
breaks the hwmon device registration in the thermal drivers as these two
patches here have been left out. We either need to include them as well or
revert the original commit.
I'm also not sure why the original commit found its way into the 5.4 stable
branch.
- Julian