Messages in this thread | | | From | Aisheng Dong <> | Subject | RE: [PATCH V12 3/5] thermal: imx_sc: add i.MX system controller thermal support | Date | Thu, 16 May 2019 10:39:34 +0000 |
| |
[...]
> > > + if (!sensor) > > > + return -ENOMEM; > > > + > > > + ret = thermal_zone_of_get_sensor_id(child, > > > + &sensor_specs, > > > > It looks a bit strange why need sensor_specs as you even did not use it. > > It is because parsing the phandle arguments needs it, such as the > thermal_zone_of_sensor_register() function needs this pointer to be passed > back to check the elements are valid or NOT, so the API does NOT define it as > local structure. The API NOT just return the sensor id, but also the > of_phandle_args address, although we do NOT need it at all here.
The main problem is this one introduced in 2/5 which needs get ack from maintainer. For me, I would suggest to make sensor_specs transparent to user if it's really not needed by users.
Please try to make it either optional or hide to core users as well. And I guess the later one is doable after a glance at the core code.
Regards Dong Aisheng
| |