mbox series

[RESEND,v1,0/4] dt-binding: clk: prepare for imx8qm clock support

Message ID 1547207400-28307-1-git-send-email-aisheng.dong@nxp.com (mailing list archive)
Headers show
Series dt-binding: clk: prepare for imx8qm clock support | expand

Message

Dong Aisheng Jan. 11, 2019, 11:55 a.m. UTC
After a bit more investigation on IMX8QM, it seemed SCU clock can be used
in a similar way as IMX8QXP. So update the binding and driver to be a bit
generic for easily extension of IMX8QM support later.

Dong Aisheng (4):
  dt-bindings: fsl: scu: add fallback compatible string for clock
  dt-bindings: fsl: scu: add imx8qm scu clock support
  clk: imx: scu: add fallback compatible string support
  dt-bindings: imx8-clock: add a53 and a72 clock id

 Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 7 +++++--
 drivers/clk/imx/clk-imx8qxp.c                               | 1 +
 include/dt-bindings/clock/imx8-clock.h                      | 6 ++++--
 3 files changed, 10 insertions(+), 4 deletions(-)

Comments

Dong Aisheng Jan. 16, 2019, 5:40 a.m. UTC | #1
Ping...

BTW, as the first two patches change the same firmware binding doc fs.scu.txt.
I'm not sure whether it should go to Shawn's tree for the binding part, or Stephen's clock tree.

Shawn & Stephen,
What's your suggestion?

Regards
Dong Aisheng

> From: Aisheng Dong [mailto:aisheng.dong@nxp.com]
> Sent: Friday, January 11, 2019 7:56 PM
> 
> After a bit more investigation on IMX8QM, it seemed SCU clock can be used in
> a similar way as IMX8QXP. So update the binding and driver to be a bit generic
> for easily extension of IMX8QM support later.
> 
> Dong Aisheng (4):
>   dt-bindings: fsl: scu: add fallback compatible string for clock
>   dt-bindings: fsl: scu: add imx8qm scu clock support
>   clk: imx: scu: add fallback compatible string support
>   dt-bindings: imx8-clock: add a53 and a72 clock id
> 
>  Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 7 +++++--
>  drivers/clk/imx/clk-imx8qxp.c                               | 1 +
>  include/dt-bindings/clock/imx8-clock.h                      | 6 ++++--
>  3 files changed, 10 insertions(+), 4 deletions(-)
> 
> --
> 2.7.4
Shawn Guo Jan. 16, 2019, 11:02 a.m. UTC | #2
On Wed, Jan 16, 2019 at 05:40:01AM +0000, Aisheng Dong wrote:
> Ping...
> 
> BTW, as the first two patches change the same firmware binding doc fs.scu.txt.
> I'm not sure whether it should go to Shawn's tree for the binding part, or Stephen's clock tree.
> 
> Shawn & Stephen,
> What's your suggestion?

I can apply the first two to my tree as long as Rob and Stephen ack on
them.

Shawn

> 
> Regards
> Dong Aisheng
> 
> > From: Aisheng Dong [mailto:aisheng.dong@nxp.com]
> > Sent: Friday, January 11, 2019 7:56 PM
> > 
> > After a bit more investigation on IMX8QM, it seemed SCU clock can be used in
> > a similar way as IMX8QXP. So update the binding and driver to be a bit generic
> > for easily extension of IMX8QM support later.
> > 
> > Dong Aisheng (4):
> >   dt-bindings: fsl: scu: add fallback compatible string for clock
> >   dt-bindings: fsl: scu: add imx8qm scu clock support
> >   clk: imx: scu: add fallback compatible string support
> >   dt-bindings: imx8-clock: add a53 and a72 clock id
> > 
> >  Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 7 +++++--
> >  drivers/clk/imx/clk-imx8qxp.c                               | 1 +
> >  include/dt-bindings/clock/imx8-clock.h                      | 6 ++++--
> >  3 files changed, 10 insertions(+), 4 deletions(-)
> > 
> > --
> > 2.7.4
>
Stephen Boyd Jan. 22, 2019, 7:06 p.m. UTC | #3
Quoting Shawn Guo (2019-01-16 03:02:17)
> On Wed, Jan 16, 2019 at 05:40:01AM +0000, Aisheng Dong wrote:
> > Ping...
> > 
> > BTW, as the first two patches change the same firmware binding doc fs.scu.txt.
> > I'm not sure whether it should go to Shawn's tree for the binding part, or Stephen's clock tree.
> > 
> > Shawn & Stephen,
> > What's your suggestion?
> 
> I can apply the first two to my tree as long as Rob and Stephen ack on
> them.
> 

The first two are documentation updates? I don't care really either way,
so I gave my ack in case you want to pick them up before I get around to
the rest of the series.