Re: [PATCH 2/2] scsi: mpt3sas: add hwmon support

[Guenter Roeck]

On 5/13/26 18:11, Louis Sautier wrote:
> On 13/05/2026 05:57, Guenter Roeck wrote:
> > >    Documentation/hwmon/index.rst        |   1 +
> > >    Documentation/hwmon/mpt3sas.rst      |  57 ++++++++
> >
> > This is not appropriate. The description is wrong and misleading.
> > mpt3sas is _not_ a hwmon driver. It is a chip access driver which
> > happens to support hardware monitoring.
> >
> > If this is part of the mpt3sas code and not a separate driver,
> > please keep it there.
> >
> > Thanks,
> > Guenter
>
> Hi,
>
> Sorry about that, I had assumed this directory was also meant to contain
> documentation for chip drivers that support hardware monitoring.

My recommendation is to implement drivers such as this one as auxiliary drivers.
Most people don't want to do that. I accept that, but then I don't want to carry
the documentation either.

Thanks,
Guenter