[PATCH v5 01/16] scmi: refactor the code to hide a channel from devices

Etienne CARRIERE - foss etienne.carriere at foss.st.com
Thu Oct 5 09:07:04 CEST 2023


> From: U-Boot <u-boot-bounces at lists.denx.de> on behalf of AKASHI Takahiro <takahiro.akashi at linaro.org>
> Sent: Tuesday, September 26, 2023 8:57 AM
> 
> The commit 85dc58289238 ("firmware: scmi: prepare uclass to pass channel
> reference") added an explicit parameter, channel, but it seems to make
> the code complex.
> 
> Hiding this parameter will allow for adding a generic (protocol-agnostic)
> helper function, i.e. for PROTOCOL_VERSION, in a later patch.
> 
> Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> v5
> * correct the size for per_child_auto
> v4
> * revive scmi_bind_protocols which was accidentally removed
> * remove .per_child_auto from the driver declaration as it is not needed
> v3
> * fix an issue on ST board (reported by Etienne)
>   by taking care of cases where probed devices are children of
>   SCMI protocol device (i.e. clock devices under CCF)
>   See find_scmi_protocol_device().
> * move "per_device_plato_auto" to a succeeding right patch
> v2
> * new patch
> ---
>  drivers/clk/clk_scmi.c                    |  27 ++----
>  drivers/firmware/scmi/scmi_agent-uclass.c | 105 ++++++++++++++++------
>  drivers/power/regulator/scmi_regulator.c  |  26 ++----
>  drivers/reset/reset-scmi.c                |  19 +---
>  include/scmi_agent.h                      |  15 ++--
>  5 files changed, 104 insertions(+), 88 deletions(-)
> 

Reviewed-by: Etienne Carriere <etienne.carriere at foss.st.com>



More information about the U-Boot mailing list