On Tue, Feb 01, 2022 at 01:57:02PM +0100, Hans de Goede wrote: > Hi, > > On 1/31/22 22:19, Armin Wolf wrote: > > The hardware monitoring solutions supported by the sch5627 and > > sch5636 drivers are Fujitsu-specific. > > After some online searching, i found out that the solution used > > with the SCH5627 is called "Antiope" by Fujitsu, just like the > > "Theseus" solution inside the SCH5636. > > I also found out that "Antiope" and "Theseus" are listed as > > DMI onboard devices on supported Fujitsu devices, so the > > sch56xx_common module can be loaded automatically an check > > for the DMI devices. However some devices like the Esprimo C700 > > have both devices, so after verifying that at least one onboard > > device is present, sch56xx_common still has to detect which chip > > is present. > > This is safe however if at least one device is present. > > > > Tested on a Fujitsu Esprimo P720. > > Thanks, the new version of the entire series looks good to me: > > Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> > Series applied. Thanks, Guenter