drivers/cpufreq/scmi-cpufreq.c | 5 ++++- drivers/hwmon/scmi-hwmon.c | 1 + drivers/pinctrl/pinctrl-scmi.c | 11 ++++++++++- drivers/powercap/arm_scmi_powercap.c | 1 + drivers/reset/reset-scmi.c | 8 +++++++- 5 files changed, 23 insertions(+), 3 deletions(-)
SCMI client drivers do not consistently log the number of supported
entities discovered from firmware. This information is useful during
debugging because it shows which domains or resources were exposed by
firmware during probe.
Add logging of the number of supported entities to the SCMI cpufreq,
pinctrl, reset, hwmon, and powercap client drivers after a successful
probe. This aligns these drivers with the existing logging in the SCMI
power and performance domain drivers.
Signed-off-by: Alex Tran <alex.tran@oss.qualcomm.com>
---
Changes in v2:
- Use dev_dbg instead of dev_info log level
- Link to v1: https://lore.kernel.org/r/20260513-scmi-client-probe-log-v1-0-00b47b1be009@oss.qualcomm.com
---
Alex Tran (5):
powercap: arm_scmi_powercap: Log number of powercap domains
cpufreq: scmi-cpufreq: Log number of perf domains
hwmon: scmi-hwmon: Log number of sensors
reset: reset-scmi: Log number of reset domains
pinctrl: pinctrl-scmi: Log number of pins, groups, functions
drivers/cpufreq/scmi-cpufreq.c | 5 ++++-
drivers/hwmon/scmi-hwmon.c | 1 +
drivers/pinctrl/pinctrl-scmi.c | 11 ++++++++++-
drivers/powercap/arm_scmi_powercap.c | 1 +
drivers/reset/reset-scmi.c | 8 +++++++-
5 files changed, 23 insertions(+), 3 deletions(-)
---
base-commit: 1bfaee9d3351b9b32a99766bbfb1f5baed60ddef
change-id: 20260509-scmi-client-probe-log-173cf85d5563
Best regards,
--
Alex Tran <alex.tran@oss.qualcomm.com>
On Wed, 13 May 2026 10:16:53 -0700 Alex Tran <alex.tran@oss.qualcomm.com> wrote: > SCMI client drivers do not consistently log the number of supported > entities discovered from firmware. This information is useful during > debugging because it shows which domains or resources were exposed by > firmware during probe. > > Add logging of the number of supported entities to the SCMI cpufreq, > pinctrl, reset, hwmon, and powercap client drivers after a successful > probe. This aligns these drivers with the existing logging in the SCMI > power and performance domain drivers. > > Signed-off-by: Alex Tran <alex.tran@oss.qualcomm.com> Hi Alex, Just curious but why +CC linux-iio and IIO folk? May be you had a false suggestion to add them from get maintainers. If so be sure to check it's suggestions make sense! Not to worry - we can all hit the delete button ;) Jonathan > --- > Changes in v2: > - Use dev_dbg instead of dev_info log level > - Link to v1: https://lore.kernel.org/r/20260513-scmi-client-probe-log-v1-0-00b47b1be009@oss.qualcomm.com > > --- > Alex Tran (5): > powercap: arm_scmi_powercap: Log number of powercap domains > cpufreq: scmi-cpufreq: Log number of perf domains > hwmon: scmi-hwmon: Log number of sensors > reset: reset-scmi: Log number of reset domains > pinctrl: pinctrl-scmi: Log number of pins, groups, functions > > drivers/cpufreq/scmi-cpufreq.c | 5 ++++- > drivers/hwmon/scmi-hwmon.c | 1 + > drivers/pinctrl/pinctrl-scmi.c | 11 ++++++++++- > drivers/powercap/arm_scmi_powercap.c | 1 + > drivers/reset/reset-scmi.c | 8 +++++++- > 5 files changed, 23 insertions(+), 3 deletions(-) > --- > base-commit: 1bfaee9d3351b9b32a99766bbfb1f5baed60ddef > change-id: 20260509-scmi-client-probe-log-173cf85d5563 > > Best regards,
On 5/14/2026 8:44 AM, Jonathan Cameron wrote: > On Wed, 13 May 2026 10:16:53 -0700 > Alex Tran <alex.tran@oss.qualcomm.com> wrote: > >> SCMI client drivers do not consistently log the number of supported >> entities discovered from firmware. This information is useful during >> debugging because it shows which domains or resources were exposed by >> firmware during probe. >> >> Add logging of the number of supported entities to the SCMI cpufreq, >> pinctrl, reset, hwmon, and powercap client drivers after a successful >> probe. This aligns these drivers with the existing logging in the SCMI >> power and performance domain drivers. >> >> Signed-off-by: Alex Tran <alex.tran@oss.qualcomm.com> > Hi Alex, > > Just curious but why +CC linux-iio and IIO folk? > > May be you had a false suggestion to add them from get maintainers. > If so be sure to check it's suggestions make sense! > > Not to worry - we can all hit the delete button ;) > > Jonathan Hi Jonathan, Originally, there was another patch in this series to add the same functionality to scmi_iio probe but it was dropped. Apparently running b4 prep --auto-to-cc does not prune stale entries from the cover letter. Will manually remove all entries and rerun the command in the future. Alex
On Thu, May 14, 2026 at 02:23:56PM -0700, Alex Tran wrote: > On 5/14/2026 8:44 AM, Jonathan Cameron wrote: > > > On Wed, 13 May 2026 10:16:53 -0700 > > Alex Tran <alex.tran@oss.qualcomm.com> wrote: > > > >> SCMI client drivers do not consistently log the number of supported > >> entities discovered from firmware. This information is useful during > >> debugging because it shows which domains or resources were exposed by > >> firmware during probe. > >> > >> Add logging of the number of supported entities to the SCMI cpufreq, > >> pinctrl, reset, hwmon, and powercap client drivers after a successful > >> probe. This aligns these drivers with the existing logging in the SCMI > >> power and performance domain drivers. > >> > >> Signed-off-by: Alex Tran <alex.tran@oss.qualcomm.com> > > Hi Alex, > > > > Just curious but why +CC linux-iio and IIO folk? > > > > May be you had a false suggestion to add them from get maintainers. > > If so be sure to check it's suggestions make sense! > > > > Not to worry - we can all hit the delete button ;) > > > > Jonathan > Hi Jonathan, > > Originally, there was another patch in this series to add the same > functionality to scmi_iio probe but it was dropped. Apparently running b4 > prep --auto-to-cc does not prune stale entries from the cover letter. Will > manually remove all entries and rerun the command in the future. > I guessed so, but why was it dropped ? I don't agree to adding them elsewhere just curious about why it was dropped in this case. -- Regards, Sudeep
On Thu, May 14, 2026 at 04:44:22PM +0100, Jonathan Cameron wrote:
> On Wed, 13 May 2026 10:16:53 -0700
> Alex Tran <alex.tran@oss.qualcomm.com> wrote:
>
> > SCMI client drivers do not consistently log the number of supported
> > entities discovered from firmware. This information is useful during
> > debugging because it shows which domains or resources were exposed by
> > firmware during probe.
> >
> > Add logging of the number of supported entities to the SCMI cpufreq,
> > pinctrl, reset, hwmon, and powercap client drivers after a successful
> > probe. This aligns these drivers with the existing logging in the SCMI
> > power and performance domain drivers.
> >
> > Signed-off-by: Alex Tran <alex.tran@oss.qualcomm.com>
> Hi Alex,
>
> Just curious but why +CC linux-iio and IIO folk?
>
> May be you had a false suggestion to add them from get maintainers.
> If so be sure to check it's suggestions make sense!
>
My guess, the intention was to add similar logging in scmi_iio_dev_probe()
as well, may have got dropped {un/}intentionally, but the list remained
in place. The author of the driver is also cc-ed.
--
Regards,
Sudeep
On Wed, May 13, 2026 at 10:16:53AM -0700, Alex Tran wrote: > SCMI client drivers do not consistently log the number of supported > entities discovered from firmware. This information is useful during > debugging because it shows which domains or resources were exposed by > firmware during probe. > > Add logging of the number of supported entities to the SCMI cpufreq, > pinctrl, reset, hwmon, and powercap client drivers after a successful > probe. This aligns these drivers with the existing logging in the SCMI > power and performance domain drivers. Too quick between the series, the v1 discussion is still ongoing. Hence this should not be applied (at least yet). Read Submitting Patches and other process documentation that suggest at least 24h period for review and in patch series even week or so. > --- > Changes in v2: > - Use dev_dbg instead of dev_info log level > - Link to v1: https://lore.kernel.org/r/20260513-scmi-client-probe-log-v1-0-00b47b1be009@oss.qualcomm.com -- With Best Regards, Andy Shevchenko
© 2016 - 2026 Red Hat, Inc.