[PATCH v3 2/2] clk: scmi: add compatibility for clock version 3.0 and onwards

Dhruva Gole d-gole at ti.com
Tue Sep 9 12:18:06 CEST 2025


On Aug 25, 2025 at 16:49:30 +0300, Maxim Kochetkov wrote:
> 19.08.2025 10:33, Kamlesh Gurudasani wrote:
> > Clock version 3.0 specified in ARM SCMI Platform design document
> > v3.2, adds extended_config_val parameter in CLOCK_CONFIG_SET.
> > 
> > The SCMI server with clock version 3.0 rejects the message if
> > they don't have this field.
> > 
> > Add support so that both SCMI server with clock version 2.0 and 3.0
> > can be handled.
> > 
> > Signed-off-by: Kamlesh Gurudasani <kamlesh at ti.com>
> > ---
> >   drivers/clk/clk_scmi.c   | 43 ++++++++++++++++++++++++++++++-------------
> >   include/scmi_protocols.h |  6 ++++++
> >   2 files changed, 36 insertions(+), 13 deletions(-)
> > 
> > diff --git a/drivers/clk/clk_scmi.c b/drivers/clk/clk_scmi.c
> > index 83ea7bb4286..b33cc05685d 100644
> > --- a/drivers/clk/clk_scmi.c
> > +++ b/drivers/clk/clk_scmi.c
> > @@ -131,19 +131,36 @@ static int scmi_clk_get_attibute(struct udevice *dev, int clkid, char **name,
> >   static int scmi_clk_gate(struct clk *clk, int enable)
> >   {
> > -	struct scmi_clk_state_in in = {
> > -		.clock_id = clk_get_id(clk),
> > -		.attributes = enable,
> > -	};
> > +	struct scmi_clock_priv *priv = dev_get_priv(clk->dev);
> 
> struct scmi_clock_priv *priv = dev_get_priv(clk->dev->parent);
> 

Could you elaborate?
I think clk->dev should be fine, similar to other instances in the same
file. Maybe I am missing something?

Reviewed-by: Dhruva Gole <d-gole at ti.com>

-- 
Best regards,
Dhruva Gole
Texas Instruments Incorporated


More information about the U-Boot mailing list