Hi, On 3/30/24 12:24 PM, Maximilian Luz wrote: > The Surface Pro 9 has thermal sensors connected via the Surface > Aggregator Module. Add a device node to support those. > > Signed-off-by: Maximilian Luz <luzmaximilian@xxxxxxxxx> Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note I had to apply this manually do to a conflict with: 3427c443a6dc platform/surface: platform_profile: add fan profile switching which I merged into pdx86/for-next after this series was send out. Please double check I resolved the conflict correct. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/surface/surface_aggregator_registry.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/platform/surface/surface_aggregator_registry.c b/drivers/platform/surface/surface_aggregator_registry.c > index 035d6b4105cd6..c38203c00a705 100644 > --- a/drivers/platform/surface/surface_aggregator_registry.c > +++ b/drivers/platform/surface/surface_aggregator_registry.c > @@ -74,6 +74,12 @@ static const struct software_node ssam_node_tmp_pprof = { > .parent = &ssam_node_root, > }; > > +/* Thermal sensors. */ > +static const struct software_node ssam_node_tmp_sensors = { > + .name = "ssam:01:03:01:00:02", > + .parent = &ssam_node_root, > +}; > + > /* Fan speed function. */ > static const struct software_node ssam_node_fan_speed = { > .name = "ssam:01:05:01:01:01", > @@ -311,6 +317,7 @@ static const struct software_node *ssam_node_group_sp9[] = { > &ssam_node_bat_ac, > &ssam_node_bat_main, > &ssam_node_tmp_pprof, > + &ssam_node_tmp_sensors, > &ssam_node_fan_speed, > &ssam_node_pos_tablet_switch, > &ssam_node_hid_kip_keyboard,
