Received-SPF: pass (google.com: domain of linux-kernel+bounces-148870-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 17 Apr 2024 12:01:43 -0400
From: Frank Li <Frank.li@nxp.com>
To: Francesco Dolcini <francesco@dolcini.it>
Cc: Rob Herring <robh@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>, devicetree@vger.kernel.org,
	imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 02/10] arm64: dts: imx8qm-ss-audio: add audio nodes
Message-ID: <Zh/yZ2HyF+G8bty7@lizhi-Precision-Tower-5810>
References: <20240415-dts_8qm_audio-v1-0-2976c35a0c52@nxp.com>
 <20240415-dts_8qm_audio-v1-2-2976c35a0c52@nxp.com>
 <20240416065026.GA4165@francesco-nb>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240416065026.GA4165@francesco-nb>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?klBhC4Ycp1JEJEYExKvZ6WjTNxUuxWQCv1XdsXKjXI5q0WjCBoEIvuJtI72D?=
 =?us-ascii?Q?QVysUbSRgCPbjiNnnsrsY8qRAwWY+lyfe8FoR77ykJWac2hjlqXnkuma/xhr?=
 =?us-ascii?Q?dv8oH5zNHeMK401quP4d6VmWT+9AxseFpCcWJcbsii87NLeGUivUBrT+HN6A?=
 =?us-ascii?Q?a2aYcR39BWtqHc4SDzEmu8FtG1Cb7CE9m94FzJ67L1mGQVs2pgew4p4wgjF/?=
 =?us-ascii?Q?EP3q/ZXlTEd4BYeMMi9s9UbpWMnC+eU8yZYpQaF6FjvVYelR0HYxUforWJCA?=
 =?us-ascii?Q?PnYHu0G52k+MqbFF0+VnHnm0IWqHUzB1WXsZfRzsmf8RoP/99dD2MkAnygk2?=
 =?us-ascii?Q?oEaF/5FJKQ1xcK9gtlg/VipzlDGRYWkHd3DxPMj3lf52nO22wz3e5vq9Iw3m?=
 =?us-ascii?Q?bxm/70HjVLpik8bJgSqLxK8udnt67vSi/MdVr1XzWIfxrHrJZ0n+oEv1cycR?=
 =?us-ascii?Q?NciVTNh3rYuJWdmxw2GE6QaQ54uO8eNvci1bhDtRKTBUGXjLNmraO3atrDoe?=
 =?us-ascii?Q?jGGjl9UKQr5UIO3d+p2kjTCoFRhiJPX31FKFlu9RryiIDjWHmFqrXapSxPAS?=
 =?us-ascii?Q?rAVZI1grRHeSYrwXd6ENHBQRJW8FBTix3RY7tGK+91kp51l4Xchn/8N8SoAW?=
 =?us-ascii?Q?4jxsCSr0E6vlqfnhY/hituk8ORaaO6BDZpVugjX/T4IGewK8bELxquWq6LAQ?=
 =?us-ascii?Q?EoBe4XbcMuvYH/ciTpeZuYNMNqXozzoCtyncZIziOuOM7itobqpYS/dukhb3?=
 =?us-ascii?Q?C2Zz0LmPnUnJZR3EQyut5OeJpx3MFjKLO+cSISBheNXWIKoJ5poSaGn17Hpv?=
 =?us-ascii?Q?35jl/wU/2Lsw2+qJDm96IB2kE8JZXDeIsbXYZv9pWhFEeYSJERLUJz7s51dG?=
 =?us-ascii?Q?AH1T8GG20yPWd+qnlY3IsUx8q+mwrihKaODTELEt9pbwGEkgCzkyF4PAyon4?=
 =?us-ascii?Q?9r4UB70Kb/1FwC5CdS/ZdaGQgUGL5Rv9gvDu5F33fsq+q2VC8uqU+MG/T0EM?=
 =?us-ascii?Q?SVY+AGty4ehYE7YfkmbtyYDd6GPU+WKL2uEkgGaVC7tDH/i5Wi65lhY/vgUo?=
 =?us-ascii?Q?SroibiOdO+q3HMKklnwi7MWL8rWj743FB42H3FqmEqK3ExbcORZ5D6H0FO+o?=
 =?us-ascii?Q?lg5XI+dn3wOnmW61WynO5T4XJS+QL8PJyWZOuU2+oeMp5y0/k0SWGUwxsamr?=
 =?us-ascii?Q?aqPKoDwvoqGTBmVl1C3h/KoqNq4gf2VfMzAerEAbK1ibcaY4OvipWCTDHq9u?=
 =?us-ascii?Q?YZbaRy6lr/aOyXpuqFzz0NE/FPK47onRrakaq5SVWnwLNnM3oo/QP8E5+L3g?=
 =?us-ascii?Q?G2IljawymOuhZCjn+OQaDw9Xd3AaG38iECHbdKMmKypwnaTOs7zlynbEdKXA?=
 =?us-ascii?Q?ER3rq7q8vREeMFbHoBxKmB2FZrBvE3pnZ956gLCCXz1RJM1E/6+AwCnlH9ki?=
 =?us-ascii?Q?onh5e8NzBdwUw8YKUHecqKbHxY5a7T/eVteiEfe6BGZZP7rNbbACPdLcltqD?=
 =?us-ascii?Q?SAAIo3fvBKtC3IIGyWIhxT/cY4TzKbF/R3zw9fPGP8vEpOLXzf101y4aQ6Uy?=
 =?us-ascii?Q?WlItD3kf2NAsiN5YHIP4RTPE97Hg6ejQHutBIZhZ?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e43ab70e-28aa-436f-2410-08dc5ef7b1fb
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Apr 2024 16:01:51.6771
 (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: MmHcYIdxyfmo/tsSJkz5t7MZMUJfk5bOKZ44liUPnNcT/YgjrRzfBqCf6Pw7c3tHz1GSn2x+5TWJTrPx/sCOtA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB10002

On Tue, Apr 16, 2024 at 08:50:26AM +0200, Francesco Dolcini wrote:
> Hello Frank,
> 
> On Mon, Apr 15, 2024 at 03:46:38PM -0400, Frank Li wrote:
> > Add i.MX8QM audio related nodes and update eDMA[0,1]'s information.
> > 
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/imx8qm-ss-audio.dtsi | 473 +++++++++++++++++++++
> >  arch/arm64/boot/dts/freescale/imx8qm.dtsi          |  86 ++++
> >  2 files changed, 559 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/imx8qm-ss-audio.dtsi b/arch/arm64/boot/dts/freescale/imx8qm-ss-audio.dtsi
> > new file mode 100644
> > index 0000000000000..ed5a1b4af1d76
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/freescale/imx8qm-ss-audio.dtsi
> > @@ -0,0 +1,473 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> 
> OR MIT?

All other qm related dtsi is only license GPLv2. If add Or MIT, it need
re-license whole include tree files, which out of scope of this patch. I
feel like to keep consistent with other files.

> 
> Normally for device tree files having both is preferred.
> 
> > +/*
> > + * Copyright 2024 NXP
> > + *	Dong Aisheng <aisheng.dong@nxp.com>
> > + */
> > +
> > +/delete-node/ &acm;
> > +/delete-node/ &sai4;
> > +/delete-node/ &sai5;
> > +/delete-node/ &sai4_lpcg;
> > +/delete-node/ &sai5_lpcg;
> 
> Can you explain these delete-node ? This is something that I would
> expect when a dtsi is previously included, not in this case.

We want to avoid some property inherent from parent dtsi file because it is
big difference with common one.

This node will be rewrite totally in this files.

> 
> Francesco
>