On Tue, May 19, 2015 at 02:55:17PM -0300, Henrique de Moraes Holschuh wrote:
> On Tue, May 19, 2015, at 14:45, Bjørn Mork wrote:
> > The conversion to DEVICE_ATTR_* macros failed to fixup a few cases where
> > the old attribute names didn't match the show/store function names.
> > Instead of renaming the functions, the attributes were renamed. This
> > caused an unintentional API change. The hwmon required 'name' attribute
> > were among the renamed attribute, causing libsensors to fail to detect
> > the hwmon device at all.
> >
> > Fix by using the DEVICE_ATTR macro for these attributes, allowing the
> > show/store functions to keep their system specific prefixes.
> >
> > Fixes: b4dd04ac6ef8 ("thinkpad_acpi: use DEVICE_ATTR_* macros")
> > Cc: Bastien Nocera <hadess@xxxxxxxxxx>
> > Cc: Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>
> > Signed-off-by: Bjørn Mork <bjorn@xxxxxxx>
> > ---
> > v2: kept the original function names, using the DEVICE_ATTR instead
>
> Acked-by: Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>
Thanks for the catch Bjørn, much appreciated. Queued for 4.1 fixes.
--
Darren Hart
Intel Open Source Technology Center
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
ibm-acpi-devel mailing list
ibm-acpi-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel
