linux-kernel - Re: [PATCH v2 1/5] dt-bindings: clock: Add i.MX91 clock support

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]

Message-ID: <ZmD76mp2dPVv6HeA@pengfei-OptiPlex-Tower-Plus-7010>
Date: Wed, 5 Jun 2024 16:59:38 -0700
From: Pengfei Li <pengfei.li_1@....com>
To: Rob Herring <robh@...nel.org>
Cc: krzk+dt@...nel.org, conor+dt@...nel.org, abelvesa@...nel.org,
	mturquette@...libre.com, sboyd@...nel.org, shawnguo@...nel.org,
	s.hauer@...gutronix.de, kernel@...gutronix.de, festevam@...il.com,
	ping.bai@....com, ye.li@....com, peng.fan@....com,
	aisheng.dong@....com, frank.li@....com, tharvey@...eworks.com,
	alexander.stein@...tq-group.com, gregor.herburger@...tq-group.com,
	hiago.franco@...adex.com, joao.goncalves@...adex.com,
	hvilleneuve@...onoff.com, Markus.Niebel@...tq-group.com,
	m.felsch@...gutronix.de, m.othacehe@...il.com, bhelgaas@...gle.com,
	leoyang.li@....com, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org,
	imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 1/5] dt-bindings: clock: Add i.MX91 clock support

On Tue, Jun 04, 2024 at 10:03:06AM -0500, Rob Herring wrote:
> On Wed, May 29, 2024 at 07:26:30PM -0700, Pengfei Li wrote:
> > i.MX91 has similar Clock Control Module(CCM) design as i.MX93.
> > Add a new compatible string for i.MX91.
> > 
> > Signed-off-by: Pengfei Li <pengfei.li_1@....com>
> > Reviewed-by: Frank Li <Frank.Li@....com>
> > ---
> >  Documentation/devicetree/bindings/clock/imx93-clock.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/clock/imx93-clock.yaml b/Documentation/devicetree/bindings/clock/imx93-clock.yaml
> > index ccb53c6b96c1..98c0800732ef 100644
> > --- a/Documentation/devicetree/bindings/clock/imx93-clock.yaml
> > +++ b/Documentation/devicetree/bindings/clock/imx93-clock.yaml
> > @@ -16,6 +16,7 @@ description: |
> >  properties:
> >    compatible:
> >      enum:
> > +      - fsl,imx91-ccm
> >        - fsl,imx93-ccm
> 
> Should fallback to fsl,imx93-ccm? Being a superset should be ok because 
> your DT should never use the non-existent clocks. If not, where is the 
> driver change?
> 
> Rob
> 

Hi Rob Herring,

Due to the different maintainers of the CCM framework and DTS, I have separated 
the CCM driver patch and plan to send the DTS patch first before sending the 
CCM driver patch. If that's possible, could you help merge this patch first? If 
this is not allowed, I can also send out the CCM driver patch at the same time.

BR,
Pengfei Li