Received-SPF: pass (google.com: domain of linux-kernel+bounces-97490-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Date: Fri, 8 Mar 2024 14:01:29 -0500
From: Frank Li <Frank.li@nxp.com>
To: Rob Herring <robh@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Shengjiu Wang <shengjiu.wang@nxp.com>, linux-sound@vger.kernel.org,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
	Conor Dooley <conor.dooley@microchip.com>
Subject: Re: [PATCH v6 2/4] ASoC: dt-bindings: fsl,imx-asrc: update max
 interrupt numbers
Message-ID: <ZetgiaZ1+wt5lCkB@lizhi-Precision-Tower-5810>
References: <20240308-asrc_8qxp-v6-0-e08f6d030e09@nxp.com>
 <20240308-asrc_8qxp-v6-2-e08f6d030e09@nxp.com>
 <20240308181103.GA855753-robh@kernel.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240308181103.GA855753-robh@kernel.org>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?BTS2vBDq4dq1atuiMQLzAtyKzhD2qa5/B1txylMSidotnf8ckZQHPIq8xGrG?=
 =?us-ascii?Q?8X/hMfS+v2dhYfOHIgamdHWmhUl+sycl5Zpqr9xYU3Y/5yb+ypZBgXJRSuAs?=
 =?us-ascii?Q?2ZHuRLbm34c1APZ3JhKorny1bWy0HOrphimIAujlle9kJ0r1Z+SPs7SQFJkv?=
 =?us-ascii?Q?vcxxw/tpGHTPQv5bBI3QjISfxr7oVgv1hD4An8JFUdN1YrxFXnSRzm0+FOvq?=
 =?us-ascii?Q?CV+dYqpgkDmsgx7qfx6Owv9vf7mUsDzU7tBhlwlbRqA7GqAzgNJZexbqE+2q?=
 =?us-ascii?Q?WGH6+NFExeh1T+cCPzqjDrcjxOWbzTScKCgREYqAc8WAoFJCnvUVWt1saCUl?=
 =?us-ascii?Q?qyQUFM2cqqSub2zTPgnTPUZdcfDn/gltWYMUDIO8Bvy+IQRgwVFxO45TsJ73?=
 =?us-ascii?Q?gQmtvraK/xMDYVhiDjVL2wb5Vp1j7eWnDHZMqiQ/HOxJ8oe6ejHTP3Pye4ge?=
 =?us-ascii?Q?JYLKI99VY10cn39GqI34kcRoQe1FOb7Nydk9ihOyDfxL3eD2UTh8fp7UfHBI?=
 =?us-ascii?Q?VnXoMAtzCR1FxNJ2CcNVykimjLMXMOcwdqGnbyAkiNhRLcofXmCQo0iyimel?=
 =?us-ascii?Q?TwGwRGNucnphtfal23zPg8AvHpsHSIjknBTFJSjrAPLFW03cs9/6KncM9QDA?=
 =?us-ascii?Q?pZJqWcttabU1/GrPsWwHxnWfr+p4FkabsAaNNfusXy6LlZ0tc6PGxuhk2aAt?=
 =?us-ascii?Q?G2sAMDfa12/kNYZidRucuksDH6sRDD4OxLHeNjRdVQpUWPlVABLC8s4nEHW0?=
 =?us-ascii?Q?uAiHeo1GqAS3R5ssgZNwVB7Wn1y6XunTP8QEZjaaJWUlyXGSIAx5AgLTWzxx?=
 =?us-ascii?Q?j5IjZgz9QUKIfKK+TCp7l8UggjHpW7F3d5CCcZjf4NMVZXx9nVRV/fGN09Wk?=
 =?us-ascii?Q?0ISdBIvDzBEKoW/gx6mutI1LUKP4lkBn/ZFYO4KOE+3oWc9K0besEsN+M2Tb?=
 =?us-ascii?Q?7CorymQ9Se0SRmeZBXUHV7Sz7FG5rxU8WM0ttu/N+iCwcfqyTqOUR4olIcLX?=
 =?us-ascii?Q?iT+MKbIZbr7lYHH+/Db1iAJkrybkDNsxZ03+LXnJRGtloNj86BUADsxhUEVS?=
 =?us-ascii?Q?448gy/nK160cc0YFLRa3eiUYxNmEhCd6IjxhugV6V+gnzck6ctJRb4ObVCyx?=
 =?us-ascii?Q?scbbhmTnBsIWXSpOLMVOgNEMdfcnSnJ+i0BoP7NuHouaMoZvhRfFxhP+DshR?=
 =?us-ascii?Q?LhANXpF5I0EFbW8DD2OWlYJCHgSROHb1HQuN8CVR5eju4bgTpsaC8SPG0IQ1?=
 =?us-ascii?Q?l7C1OU4TEyp5m8RfEYsJy3cfW0s0oaskNlMz+MpCsZGhK5/e54RVqraYzyJm?=
 =?us-ascii?Q?lJp9/gVd48IRE4pb9XlIxYd48hER/WKLHakXLFGKGEmdSlpFwgVBpQMlHybG?=
 =?us-ascii?Q?k3H94J/I2T4Vq+YVpuU7OZH6kTrcbwbGGwBTlIgvhZq2b16UiPYXrAz+cfvl?=
 =?us-ascii?Q?NQGj3gwFdKLkEY7wryjsOVj946KmEd04ilJbsvvHOyuk7wc9R9ztY360fLMt?=
 =?us-ascii?Q?lR2BrRdp3FxRG+l132eifZAVISBlgrfw9S6hy2AnEagyS+uZEEo1Am9NvHDY?=
 =?us-ascii?Q?ouVIHNfiMPxdIdmzaKCVmCC2zT56+q7w1YiBVx1e?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 61b04cb3-6091-42ff-e80f-08dc3fa22e17
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2024 19:01:37.1745
 (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: FwtejvAJ7ag/q+SBqu81h9sY1ZEM2CK9xj19T7SsAQYGOnQoPPpfUNKF2eIfzYkq3dObn5+6fWjkUIieLoT+hA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DUZPR04MB9869

On Fri, Mar 08, 2024 at 12:11:03PM -0600, Rob Herring wrote:
> On Fri, Mar 08, 2024 at 10:30:51AM -0500, Frank Li wrote:
> > fsl,imx8qxp-spdif and fsl,imx8qm-spdif have 2 interrupts. Other platforms
> > have 1 interrupt.
> > 
> > Increase max interrupt number to 2 and add restriction for platforms except
> > i.MX8QXP and i.MX8QM.
> > 
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >  .../devicetree/bindings/sound/fsl,spdif.yaml        | 21 ++++++++++++++++++++-
> >  1 file changed, 20 insertions(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/sound/fsl,spdif.yaml b/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
> > index 56f8c0c8afdea..a242f68f99f18 100644
> > --- a/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
> > +++ b/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
> > @@ -31,7 +31,11 @@ properties:
> >      maxItems: 1
> >  
> >    interrupts:
> > -    maxItems: 1
> > +    minItems: 1
> > +    maxItems: 2
> > +    items:
> > +      - description: Combined or receive interrupt
> > +      - description: Transmit interrupt
> 
> Test your patches please because this will have warnings. Or, you can 
> put in *exactly* what I provided because this is not it.
> 
> If you continue to just toss crap at us at the rate you are, the DT 
> maintainers will either just start ignoring your patches or require some 
> trusted review by another NXP colleague first (offhand, not sure who 
> that would be which is part of the problem).

Sorry, I run wrong command to check another dtb file. So have not catch
this problem.

Frank

> 
> Rob