From:   Jacky Bai <ping.bai@nxp.com>
To:     Peng Fan <peng.fan@nxp.com>,
        "Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
        "abelvesa@kernel.org" <abelvesa@kernel.org>,
        "abel.vesa@linaro.org" <abel.vesa@linaro.org>,
        "mturquette@baylibre.com" <mturquette@baylibre.com>,
        "sboyd@kernel.org" <sboyd@kernel.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>
CC:     dl-linux-imx <linux-imx@nxp.com>,
        "linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: RE: [PATCH] dt-bindings: clock: imx8m: add interrupts property
Thread-Topic: [PATCH] dt-bindings: clock: imx8m: add interrupts property
Thread-Index: AQHZS+ECotOWgCkTKUua1Qtf9bVuNK7lMhAggAACxACAAAA5UA==
Date:   Wed, 1 Mar 2023 02:35:31 +0000
Message-ID: <AS8PR04MB86427383662B9852BD16477187AD9@AS8PR04MB8642.eurprd04.prod.outlook.com>
References: <20230301020122.3389102-1-peng.fan@oss.nxp.com>
 <AS8PR04MB8642379DB76D1C529CB8960487AD9@AS8PR04MB8642.eurprd04.prod.outlook.com>
 <DU0PR04MB9417700E2F71A06D9A7247BB88AD9@DU0PR04MB9417.eurprd04.prod.outlook.com>
In-Reply-To: <DU0PR04MB9417700E2F71A06D9A7247BB88AD9@DU0PR04MB9417.eurprd04.prod.outlook.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?iO+zm1/NVjcLhzNtFR2o0cZWdJK452gNR3sq2+xhNoRMjrFTfFuheKVswf5h?=
 =?us-ascii?Q?995/G1kte8Adrd2xflitjMg3lcGsZQ9Q2u8O3Vs4Syxawyy71SSoFV/pog2X?=
 =?us-ascii?Q?+zSxe9BODaLTj1W6qTUDc7ijwnDMa5sPUcHvesFRL7Zmgq7NltoD6TBe24Ro?=
 =?us-ascii?Q?Tc83nnmbG5SGJkL7Eb6JymmMrMka988d/SYG2o7k9wZKLXr+WqCA9RHCarEU?=
 =?us-ascii?Q?g7RrjdXenbkqT3vQw4mvT/PAV6lhQ3Ap0toNfEWwFiiKIhQCKjgrxd26w5RR?=
 =?us-ascii?Q?S53qmgLaKDrmnT+FeYOE3fotqdShaIOebN+ir1deo2HVLkFlBKkRqPxGjT6I?=
 =?us-ascii?Q?HdxQ9EtKjRQO8eDmDJsIpmQl/ry5vmuS/A8lSp1LVGCiht23oJgL/UgGxq12?=
 =?us-ascii?Q?e3XzK2hNAG4oo2YHrYPCaUIQnGp4aGAv//QGiVVQyFUxCfWl70ao0P7jmE1b?=
 =?us-ascii?Q?GT+huX8LKM6dGEeRPr+RHOm+85TrTrdWnOwjH2PcIfQOjppjyW5tZSsdKYIB?=
 =?us-ascii?Q?2GLGJvFoxotSUXyReSQ9wv9N4TisGGnIM8LJreOFpzdZKe7Q2XUR4wjTdh6M?=
 =?us-ascii?Q?lO692BmU8d9tTt+aRCDA2e7HbjcXBdLHPHBsdIpAMX5fMgcCkLo9rN655xfD?=
 =?us-ascii?Q?iQAyybRRB/D+WfZ4NmlvUylujf28PJ3mATERmP+T5hinHzDRL0/6isTrcupI?=
 =?us-ascii?Q?4tQFDkKlIS5hpElAQegkKc6S6IefA85c0X5Q+Uo4dYiukqyC1h3SoYgQnoC4?=
 =?us-ascii?Q?N1f4Q4FT9FSB9T1x1ikpjgRMGd/I98VlKWfLy08NXpZvDn81Q9QnowUwSNdo?=
 =?us-ascii?Q?/rYS5awsRqS3xBUKwSdWbzyBW0PtgB8NyebcBZgIxQTaqLt6V3lcV/bol+Vz?=
 =?us-ascii?Q?YlgSLL2OX5Co9vZxb+1NAFoTZCvuuT+8HRA4K4w6aJWYuaPH5s/SKy+bizfq?=
 =?us-ascii?Q?hOkuIeiJEJ3ptEhv+yVXviAfwYLT1DS59A5w8gGjsDuegMLRUm7iHZ1y842M?=
 =?us-ascii?Q?5u5caYHDotIZ+4NmMXd/SF9DaOiez8VMB0eT2PgmVZsW+FDiRt/XGkj5znt7?=
 =?us-ascii?Q?g4PsN9GRAvHkUM+cB29qJEF2pInNJrSlUDXLHhIylmV//dBAepdeITIOTYQj?=
 =?us-ascii?Q?ZfQ5K6ZM+syTVYPTwIi1RP1gT4hgvyYjvBiU8d0VJQm2U2iGABHfgrwGeRLv?=
 =?us-ascii?Q?mX+GaKiTHQmMq7tr3vo/a7NQ6n0awtCbUtwRHf6JlkOywcm6PzERQPxVXPbY?=
 =?us-ascii?Q?xpLffywZyjbiehHWjBvKOnxpoG+FvDz3PBbqGPx7RHUGoUmDSbfu9htUecf2?=
 =?us-ascii?Q?996zDkHJhf+nir7nfS7srU6zuY/vhp6+Ml2XOvUssCTyRWNNRfKonEIAbz2I?=
 =?us-ascii?Q?xt7NLdK4a/Ia4U3gIroL9l+i/38BkJwP9G+gt+ihq66+Gb7X/cWIsa+7O6QA?=
 =?us-ascii?Q?xxnJTDFG1kWluw9FmSTlNWnfCZHTfQSsmIdchOb88aCcHQ/tUcqorxo4YRrO?=
 =?us-ascii?Q?F5Da7jbyemc6cK0u7IxkMvh9JfZYTpQmDtSvpS18L3s+Yier0/Y5nV+6aqn3?=
 =?us-ascii?Q?ojLMKnephKe/ZJDG6g4=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b9d48bfa-dbed-434c-0fb1-08db19fda091
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Mar 2023 02:35:31.3870
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: b6wuNBUF+a2oi3J5IVwuF19mqm93hPlNWWH1cbmqjOy/Jb/KP7FBYibaidEDRM84x7QlAZjcjTp5evZiyZBdRQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB7213
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: RE: [PATCH] dt-bindings: clock: imx8m: add interrupts property
> 
> > Subject: RE: [PATCH] dt-bindings: clock: imx8m: add interrupts
> > property
> >
> > > Subject: [PATCH] dt-bindings: clock: imx8m: add interrupts property
> > >
> > > From: Peng Fan <peng.fan@nxp.com>
> > >
> > > Since the CCM module could trigger interrupt, so add interrupts
> > > property for i.MX8M.
> > >
> >
> > Any chance that these interrupt will be used? More likely such
> > interrupt are useless and has no details about how to use it in RM,
> > just piece of words in the interrupt table.
> 
> This is to clear the System-Ready check warning. This is to describe the
> hardware. Since device tree already has it, and the hardware indeed supports
> interrupt, so let's add it.
> 

Ok, got it, thx.

BR
> Thanks,
> Peng.
> >
> > BR
> >
> > > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > > ---
> > >  Documentation/devicetree/bindings/clock/imx8m-clock.yaml | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> > > b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> > > index 0dbc1433fede..ac7ad8459c79 100644
> > > --- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> > > +++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> > > @@ -39,6 +39,10 @@ properties:
> > >        ID in its "clocks" phandle cell. See
> > > include/dt-bindings/clock/imx8m-clock.h
> > >        for the full list of i.MX8M clock IDs.
> > >
> > > +  interrupts:
> > > +    minItems: 1
> > > +    maxItems: 2
> > > +
> > >  required:
> > >    - compatible
> > >    - reg
> > > --
> > > 2.37.1