linux-kernel - RE: [PATCH 0/2] firmware: arm_scmi: create scmi devices for protocols that not have of

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Wed, 26 Jun 2024 08:13:54 +0000
From: Peng Fan <peng.fan@....com>
To: "Peng Fan (OSS)" <peng.fan@....nxp.com>, Sudeep Holla
	<sudeep.holla@....com>, Cristian Marussi <cristian.marussi@....com>
CC: "arm-scmi@...r.kernel.org" <arm-scmi@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 0/2] firmware: arm_scmi: create scmi devices for protocols
 that not have of_node

Please ignore this patchset, there is an build error that I push the button
earlier.

Regards,
Peng.
> Subject: [PATCH 0/2] firmware: arm_scmi: create scmi devices for
> protocols that not have of_node
> 
> Per
> https://lore.kernel.org/all/20230125141113.kkbowopusikuogx6@bog
> us/
> "
> In short we shouldn't have to add a node if there are no consumers. It
> was one of the topic of discussion initially when SCMI binding was
> added and they exist only for the consumers otherwise we don't need
> it as everything is discoverable from the interface.
> "
> https://lore.kernel.org/all/Y9JLUIioxFPn4BS0@e120937-lin/
> If a node has its own channel, the of_node is still needed.
> 
> i.MX95 SCMI firmware not have dedicated channel for 0x12, and no
> need of_node. This patchset is to support protocol 0x12 without the
> procotol node in device tree.
> 
> Without of_node, still need to create the scmi devices. As of now, it is
> based on an array 'protocols[]' in 'scmi_probe'.
> 
> And no of_node, means no per protocol channel, so reuse the base
> protocol channel. Need patch chan_available to support.
> 
> Signed-off-by: Peng Fan <peng.fan@....com>
> ---
> Peng Fan (2):
>       firmware: arm_scmi: channel unavailable if no of_node
>       firmware: arm_scmi: create scmi_devices that not have of_node
> 
>  drivers/firmware/arm_scmi/driver.c  | 33
> ++++++++++++++++++++++++++++++++-
> drivers/firmware/arm_scmi/mailbox.c |  2 ++
>  drivers/firmware/arm_scmi/optee.c   |  3 +++
>  drivers/firmware/arm_scmi/smc.c     |  7 ++++++-
>  drivers/firmware/arm_scmi/virtio.c  |  3 +++
>  5 files changed, 46 insertions(+), 2 deletions(-)
> ---
> base-commit: d8003eb2eb0200352b5d63af77ec0912a52e79ad
> change-id: 20240626-scmi-driver-96dc61b036a2
> 
> Best regards,
> --
> Peng Fan <peng.fan@....com>