Re: [PATCH v2 0/6] firmware: arm_scmi: imx: Dump syslog and system_info

[Peng Fan <peng.fan@oss.nxp.com>]

Hi Sudeep, Cristian

On Thu, Jul 10, 2025 at 04:33:25PM +0800, Peng Fan wrote:
>Cristian,
>  I only kept your R-b for patch 1, dropped your other R-b tags, because
>  there are some changes to address the comments from Sudeep. Most changes
>  are only to dump the information in probe phase, update to single sysfs
>  single value for syslog.
>
>Sudeep,
>  I saw you sent out PR to Arnd for 6.17, so no rush now for this patchset.
>  Take your time.

This patchset was originally submitted before v6.16-rc6. As the kernel has
now reached v6.17-rc2, I would greatly appreciate it if you could take some
time to review it when convenient. Thank you in advance!

Thanks,
Peng

>
>Hope I not miss any comments from V1.
>
>Thanks,
>Peng
>
>System Manager firmware provides API to dump board, silicon, firmware
>information. It also provides API to dump system sleep, wakeup
>information. So add the interface for Linux to retrieve the information:
>
>patch 1 is to add doc for board information which was missed before.
>
>Signed-off-by: Peng Fan <peng.fan@nxp.com>
>---
>Changes in v2:
>- Add more info in patch 1 for documentation(attribute is reserved for
>  future use, boardname may not align with the board name in device tree)
>- Drop dump silicon revision API
>- Update discover build info, cfg info, board info API to be only used in probe phase
>- Update to use single sysfs single value for syslog.
>- Link to v1: https://lore.kernel.org/r/20250627-sm-misc-api-v1-v1-0-2b99481fe825@nxp.com
>
>---
>Peng Fan (6):
>      firmware: arm_scmi: imx: Add documentation for MISC_BOARD_INFO
>      firmware: arm_scmi: imx: Support discovering buildinfo of MISC protocol
>      firmware: arm_scmi: imx: Support getting cfg info of MISC protocol
>      firmware: arm_scmi: imx: Support getting board info of MISC protocol
>      firmware: arm_scmi: imx: Support getting syslog of MISC protocol
>      firmware: imx: sm-misc: Dump syslog info
>
> .../firmware/arm_scmi/vendors/imx/imx-sm-misc.c    | 190 +++++++++++++++++++++
> drivers/firmware/arm_scmi/vendors/imx/imx95.rst    |  22 +++
> drivers/firmware/imx/sm-misc.c                     | 178 +++++++++++++++++++
> include/linux/scmi_imx_protocol.h                  |  19 +++
> 4 files changed, 409 insertions(+)
>---
>base-commit: e88d17c7335a917e98b7c65776cbe3358210e8ab
>change-id: 20250627-sm-misc-api-v1-85c030c670c6
>
>Best regards,
>-- 
>Peng Fan <peng.fan@nxp.com>
>