Received-SPF: pass (google.com: domain of linux-kernel+bounces-137487-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Tue, 9 Apr 2024 14:59:04 -0400
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>,
	Shengjiu Wang <shengjiu.wang@nxp.com>,
	"open list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." <linux-sound@vger.kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>,
	open list <linux-kernel@vger.kernel.org>, imx@lists.linux.dev
Subject: Re: [PATCH v2 1/1] ASoC: dt-bindings: fsl-esai: Convert fsl,esai.txt
 to yaml
Message-ID: <ZhWP+Cxy9yMAYXJC@lizhi-Precision-Tower-5810>
References: <20240322145406.2613256-1-Frank.Li@nxp.com>
 <CAL_JsqLJDhJd_uoL05Z=s3Mc7PRkcJWsQGGjdG0Q0GFp0zA8xw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CAL_JsqLJDhJd_uoL05Z=s3Mc7PRkcJWsQGGjdG0Q0GFp0zA8xw@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?c0VET3NSYlFPcVI1L2Ntb3phUTZIZ2xpVkZOR3EwdnQyK0EvSDFlbnUySWhM?=
 =?utf-8?B?TnVDM0xzS1hYbEpqNU94SFByUTduYjhJajFkWE1CcWpyUkFXdnBIYVdZcHBj?=
 =?utf-8?B?UjdVbFJvMWhYUXpqZ08yKzdoYzB2bVF1NTR6a09JRm00Y1g0OUJRQkFQVEJh?=
 =?utf-8?B?YzBhT25NZVA5RWxYRnBLT1ZBU0lUWU04U0p0VFJVZmx2T1J2akdISEtXQ0p3?=
 =?utf-8?B?SWp6NzFnaUZKNWxUelgwVkpqTnN0NEJmdEljNmpWVHZhSVY5cEJkakZZN1Fu?=
 =?utf-8?B?SWhRUmk5Mkc3WDRXd3dES1EraW5oSVhyb3cyMTNMb1FxU0JBY09tUy9pWjg3?=
 =?utf-8?B?bzFnS2pROUFJaGdGaFlqY2JVMHVnS0t1cng3dUgxZ1VvOXhZOWdMdXNCczZ3?=
 =?utf-8?B?VVlpRVVtUitEZk8zSFhDL01rVDVWR2sybGZUVXJLRGErK1N6bHd2a3RPUng4?=
 =?utf-8?B?VE4vd1BNRVhwNytSZnppeFlKa0poOGxJMXZRZnNFWXB3MllMNnpUb05vbXYz?=
 =?utf-8?B?eXBWalMxWnA2VnRjaEZVS0FldlVPeGt3TnA4eU01R2V4L3dLVEEyU0g3cG5h?=
 =?utf-8?B?UUp3OVJQQ3JoTDAyNG9UU2xDb0V4ZjBMQTZJeVFNdW4zZDc0cXlxQ3FSSGts?=
 =?utf-8?B?eDhsam9TdzZKdzVlNmNERUVucU1JQzVCZ3gyUmMyQ21DVEdLaFpJR3FFaDlX?=
 =?utf-8?B?bGdvOHRoYzBYNGJ3cHFMRG1DSjdYTWpMY2t5OFBiQnZEb1lMYThuam5VRjls?=
 =?utf-8?B?TmJQaDFwZFRHUGpOZ283VFBabHJXNWF6ZXZCeWpBVHZzdWxtVmZoenkxKzZ6?=
 =?utf-8?B?UlNtcTY5czFTaXFwLzlZUHRzd0VINUZnQ0RCbXp3Nk5hMjdid3JaQU1LK3B1?=
 =?utf-8?B?M1duSWxyVXVzbUJaVlI4RzFBZ25JT1BLeHZOeUJJZHF3TC9FSUtZSm1sRVRB?=
 =?utf-8?B?MXVGK20ySUZKZ1ZEdjBDUXRjNksrQXZDU3hBNGZURHJyYjN5c2ozRFpnREpC?=
 =?utf-8?B?eXVHdkxaVThHUFZjRjJETXBha2VvSFk4Z1FhcWZRL3BxQzhBcjI1dk91U1Vr?=
 =?utf-8?B?Vk1NZk5MTTNrcS93R3J3ZXBPOFV5TVdsckJRQXJmWHdyWnhzVUNxbUxaTWlp?=
 =?utf-8?B?a1NsSmxpdmFpMDB1WXFKMnRSOFFEUTRLUVJhdk1SRzlVQ2p0TUpvZlc4eWZE?=
 =?utf-8?B?V09nczBNZk4yUG8xeWRKWUxYZ0xPc3p1elRUSFo2ZG4yNTAwNHFXZ2JhdWpY?=
 =?utf-8?B?c0tpTzQvSU1LaHo1Z1EyeXFXRFd0d0JIcmRrOEtLQlhUclhBOEF3K0Zrd2pU?=
 =?utf-8?B?MW1KS1k1MHhxd2s5SzhqZU5XSjI2NWloWlV1aE9WVG51WnU1NUx5T21UYXJS?=
 =?utf-8?B?OXFsR1RnVW8zd1FHc0UxWnhOZndPU2JiWmRmVzBEU0dELzBvRkl1WTF6R0lT?=
 =?utf-8?B?K0FFVnFML1drVWpROGMyWk93WFo1ZURwSXhLSTlFVHh1Y3M0czlpa2l0VzBn?=
 =?utf-8?B?a1hTV2JaMk1RNnBMeFpYRjc5RkhVbVlMTHVteTZ3c09TWXVqazVtUHgvM2Vq?=
 =?utf-8?B?bnJwTVZHNXY0RFgrT2s5aktZc1dmSjhCVWppOTB2OFV1RFVnajd4dVcrQkV4?=
 =?utf-8?B?d0xRVmZQU0g4WkJ0Q1FFRVJvOXNjNkxQTkp5MXVVWmxyMUJlZzMvUlVKK3hY?=
 =?utf-8?B?TWhQUUZROUxWL0JLNHlVWmNWTW5GVzRTQS9FZHBuMDFFOFJDS1ExaE51My80?=
 =?utf-8?B?dWt5Y3dvei9aazBhallxWnY2dWxjOTFHY1NGaWpHbXJHL0QrQnBUaWdYSXAz?=
 =?utf-8?B?SW5tWFBFSDRYNEpIVDJDdHdLbUhIVXVFWGNUQ3JsUGNqVkhRUHpJOE1HQWRk?=
 =?utf-8?B?NTdndktEc2lJeitucHRFNXZYVzN6aGkydlBxeUowcHl5UHRIVWJOcWRHK0da?=
 =?utf-8?B?U2FBdGIrejc5QUZXbVU5LzhYcUFQbmlQWmxOK00xeHVsY3R4WEFYUXp3aVJp?=
 =?utf-8?B?UmNSSDI2Q0ZPZGx5UlE3WTBUckFZS1p2azJaOEE1bmNlOXhxY204U0pnWlBo?=
 =?utf-8?B?VXZ2bGV3bnVaZGtrbEppYWZVSndHMXFHckRyL2dHVnJBM1duZ0lWOGZQcDVl?=
 =?utf-8?Q?MV8yjwEaS09kR5q1WcgmQkMoP?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 406fa6fa-a8f0-42de-f91e-08dc58c72557
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2024 18:59:13.0368
 (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: TFFd3pXfqt7dhabXNQywHX9co2Aw3UbDnTYpxXL8axyMXh7o92LIyGUPDugHnoZU+ou0Da0By8VFI5IXLhNOTQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7673

On Tue, Apr 09, 2024 at 12:26:52PM -0500, Rob Herring wrote:
> On Fri, Mar 22, 2024 at 9:54 AM Frank Li <Frank.Li@nxp.com> wrote:
> >
> > Convert fsl,esai.txt to yaml. So DTB_CHECK tools can verify dts file about
> > esai part.
> >
> > clock-names 'spba' is optional according to description. So minItems of
> > clocks and clock-names is 3.
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >
> > Notes:
> >     Change from v1 to v2
> >     - alphabetical order compatible string according to rob's suggestion
> >     - clock description move under 'clock' according to kryszof's suggestion
> >     - fix descritpion indent according to rob's suggestion
> >
> >     Pass dt_binding check
> >      make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- -j8  dt_binding_check DT_SCHEMA_FILES=fsl,esai.yaml
> >       DTEX    Documentation/devicetree/bindings/sound/fsl,esai.example.dts
> >       LINT    Documentation/devicetree/bindings
> >       CHKDT   Documentation/devicetree/bindings/processed-schema.json
> >       SCHEMA  Documentation/devicetree/bindings/processed-schema.json
> >       DTC_CHK Documentation/devicetree/bindings/sound/fsl,esai.example.dtb
> 
> Did you run dtbs_check too? Probably not arm32 at least (first number
> is number of warnings):
> 
>     218  esai@2024000: clocks: [[2, 208], [2, 209], [2, 118], [2,
> 208], [2, 156]] is too long
>     218  esai@2024000: clock-names:3: 'spba' was expected
>     218  esai@2024000: clock-names:2: 'fsys' was expected
>     218  esai@2024000: clock-names:1: 'extal' was expected
>     218  esai@2024000: clock-names: ['core', 'mem', 'extal', 'fsys',
> 'spba'] is too long

Yes, I run dtb_check for one platform. Which dtb report this warning?

Frank

> 
> Conversions can leave warnings, but any you think should be fixed in
> the binding should be fixed in the conversion.
> 
> Rob