Received-SPF: pass (google.com: domain of linux-kernel+bounces-28266-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, 16 Jan 2024 15:44:52 -0500
From: Frank Li <Frank.li@nxp.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Conor Dooley <conor@kernel.org>,
	Conor Dooley <conor.dooley@microchip.com>, robh@kernel.org,
	alexandre.belloni@bootlin.com, conor.culhane@silvaco.com,
	gregkh@linuxfoundation.org, imx@lists.linux.dev,
	jirislaby@kernel.org, joe@perches.com,
	linux-i3c@lists.infradead.org, linux-kernel@vger.kernel.org,
	linux-serial@vger.kernel.org, miquel.raynal@bootlin.com,
	zbigniew.lukwinski@linux.intel.com, devicetree@vger.kernel.org,
	krzysztof.kozlowski+dt@linaro.org
Subject: Re: [PATCH v2 2/7] dt-bindings: i3c: svc: add compatible string i3c:
 silvaco,i3c-target-v1
Message-ID: <ZabqxHD4wtiPn6ep@lizhi-Precision-Tower-5810>
References: <1b628901-7f71-4c97-9a16-723912988417@linaro.org>
 <ZaXqCoCHPWER94Hh@lizhi-Precision-Tower-5810>
 <d45e31c4-914e-4cea-a145-9775b6f516ab@linaro.org>
 <20240116-bleach-herbicide-48d636967134@wendy>
 <3199c245-3d2d-49e8-951e-2b059de4d683@linaro.org>
 <20240116-achiness-thievish-10a12b3c08cd@wendy>
 <Zaa+cLGVVDSB5MYr@lizhi-Precision-Tower-5810>
 <20240116-retract-conclude-c47a7fc8cb21@spud>
 <ZabVSYgq1Mz3LPpC@lizhi-Precision-Tower-5810>
 <4a9ed1ca-cd13-4b61-af06-a3d7935aeeee@linaro.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4a9ed1ca-cd13-4b61-af06-a3d7935aeeee@linaro.org>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?BDbW1XNKzOztJoi0YLoz70MToet35gJwf9oPCG2a+atkZ08gPTZfNjgXCtCf?=
 =?us-ascii?Q?iQ3Ct/k9IrejKgEDvbkwsM7qJgv6p6KaN658BpgCQBSZ5WLteEOatksfxshL?=
 =?us-ascii?Q?RvyTxfQ0KP9E+aSYmXH3eshCKn6cNreN3asthaZ/ZxBynzTYr4+NKY1XUjUO?=
 =?us-ascii?Q?+UxsDfuuJTmR81CkgC6sWT819ubS/QUxV/vOPlL0qaqsiu6yM19MWk0rnf1c?=
 =?us-ascii?Q?P85riGT7O3L0eTBmFH5dBVYoZ2LbwUyuUI/Q99lbdrYTJd3+Qz/T7F/1BmX2?=
 =?us-ascii?Q?FPrsk/B0bVUkBdBRfm131lTCILaS2d2BBb9UjYHr8TFqPy/+hXcThUnRBLFg?=
 =?us-ascii?Q?Qbf4hj1nebnqwxL39GCTvoWypZCJvnTplFJchOScV2Zw227fjJpIHijmQuCU?=
 =?us-ascii?Q?dLa6rgGPwz8p9zbhaWDjvvu+j+x0iEI0rAuxX9d+h3UJ4bwtR9cECF4EbgUs?=
 =?us-ascii?Q?IYMS1AX36RoxZ1YiaQINbeIUgZQBmglveC6YoxNXSqH59qFnGYdUJi1xfkNC?=
 =?us-ascii?Q?3OcdAE5QTYRAfxj2zPVPcxdeogqzVKOpgbvFBRfhIOB502hwxMvi5J1GluVU?=
 =?us-ascii?Q?ajkEmcinl0F3yqK+nyD3kZiuOxEWB81+QC3ERhOoX2JAUIxgQAXEj+f5DSor?=
 =?us-ascii?Q?vX7EYvEkhoDXEo2VmzqK1/nhCcTUSxBq6wtMCEV/L/aXmUjgkESulRl1Dndk?=
 =?us-ascii?Q?+vEr/bwoh+1JDnnG7yfQe1U7SfyRlxQxKaGjw6nF9uUguRhwCX4ba39ugR/G?=
 =?us-ascii?Q?GeKk1VRjWV4a0J/bWVkYZuwjPqbH6OLrA8hczV917y+HIAWryHXmI0xYDCKB?=
 =?us-ascii?Q?FCSBaVfmmR5VKoT2Ko9EfP+pt3T7DBWlwfp8Yfy4e+IhM+/8JWpw+Dc3AR0w?=
 =?us-ascii?Q?lwAbnKLr16KDuVWcZdCI0tt7xIBkVwdFqWEorbuMcgLBZWoeviyleywmdhE2?=
 =?us-ascii?Q?e/SwZtqRuhJzIiIObLnpKyAx19p1vrAHiY3Qx8oQnQH4vvoFen4a2gsnNcqJ?=
 =?us-ascii?Q?syU0g5gYiXS/m8rfwc4a5O37C8P41Mb9uL7Fc14R8PAK0V98F8hnE2baS8M2?=
 =?us-ascii?Q?/JlvmVmpKxoTEgNpZaMp3h4y9Et0V6y7EOCeq1C7HigUjGRQpcOOUgMZBvpo?=
 =?us-ascii?Q?Szn6nzqWk+R8pAQPZPBU3avaDxZWLLP3Z9dfy8E+YSuvDmb1vOEX6S77tRFh?=
 =?us-ascii?Q?Rf+ufW1TOXp9u6mTgbyTRgvkdk9lizir6koUe5OF95faV+CNNkigZ7CmBn1B?=
 =?us-ascii?Q?Fp2pX4o9kkpIFUrnXrgjBQf6Q47JUosr56YappXhyxIh4iAPSpHMDFI2hmVg?=
 =?us-ascii?Q?Imb+mqI9i4VWphtxadaB2Rug6oa5aVYHFSTVHDBs86ivl+8mSSgk5KtpROGW?=
 =?us-ascii?Q?eQRl1SUVcmjKxZsFN+fA29IHpgdOfnP+dY/tSAG+WBNWJvgrmHeZEchdrsN/?=
 =?us-ascii?Q?k/z7/zGGDsaymInBlukK8Mj4XsbyMY5SEIMmeZ/l8NZ2kzcNJipl62nXX158?=
 =?us-ascii?Q?x00h/2Am/vQdH8JAVpZYEcYD3TFiXx/KP4YaxZogwfH2W0bTj5VR0QO8tubT?=
 =?us-ascii?Q?0LVphZnYJMR+duqAKsk=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: edcced84-767a-45c2-935e-08dc16d4033d
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jan 2024 20:45:02.4503
 (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: F6V4LPcCE0tX5rUD42eDvxnf3PK74NDz4DeOVSBWeE40J3rYAs/5IsjFcnp4IMWPsorR/n3JMIyGE4gykupfWA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB8615

On Tue, Jan 16, 2024 at 09:30:24PM +0100, Krzysztof Kozlowski wrote:
> On 16/01/2024 20:13, Frank Li wrote:
> > On Tue, Jan 16, 2024 at 06:23:09PM +0000, Conor Dooley wrote:
> >> On Tue, Jan 16, 2024 at 12:35:44PM -0500, Frank Li wrote:
> >>> On Tue, Jan 16, 2024 at 09:48:08AM +0000, Conor Dooley wrote:
> >>>> On Tue, Jan 16, 2024 at 10:33:48AM +0100, Krzysztof Kozlowski wrote:
> >>>>> On 16/01/2024 10:30, Conor Dooley wrote:
> >>>>>> On Tue, Jan 16, 2024 at 08:24:20AM +0100, Krzysztof Kozlowski wrote:
> >>>>>>> On 16/01/2024 03:29, Frank Li wrote:
> >>>>>>>>>> 	Patches were accepted after discussion, what you ponit to. So I
> >>>>>>>>>> think everyone agree on the name 'silvaco,i3c-master-v1'.
> >>>>>>>>>> 	I plan send next version to fix auto build error. Any additional
> >>>>>>>>>> comments about this?
> >>>>>>>>>
> >>>>>>>>> I still do not see how did you address Rob's comment and his point is
> >>>>>>>>> valid. You just did not reply to it.
> >>>>>>>>
> >>>>>>>> See https://lore.kernel.org/imx/ZXCiaKfMYYShoiXK@lizhi-Precision-Tower-5810/
> >>>>>>>
> >>>>>>> First of all, that's not the answer to Rob's email, but some other
> >>>>>>> thread which is 99% ignored by Rob (unless he has filters for
> >>>>>>> "@Rob"...). Therefore no, it does not count as valid answer.
> >>>>>>>
> >>>>>>> Second, explanation does not make sense. There is no argument granting
> >>>>>>> you exception from SoC specific compatibles.
> >>>>>>
> >>>>>> The patch could have been applied two months ago had Frank done as
> >>>>>> was requested (multiple times). I don't understand the resistance
> >>>>>> towards doing so given the process has taken way way longer as a result.
> >>>>>
> >>>>> I think that Rob's comment was just skipped and original master binding
> >>>>> was merged without addressing it. I don't want to repeat the same
> >>>>> process for the "target". Indeed I could point this earlier... if I only
> >>>>> knew that Rob pointed out that issue.
> >>>>
> >>>> Oh I think I got confused here. The context for this mail led me to
> >>>> think that this was still trying to push the i3c-master-v1 stuff through
> >>>> and I was commenting on my frustration with the resistance to applying
> >>>> the feedback received. I didn't realise that this was for another
> >>>> patch adding a target.
> >>>>
> >>>> I think you already said it, but NAK to adding any more compatibles here
> >>>> until the soc-specific compatible that was asked for for the imx93 is
> >>>> added.
> >>>
> >>> Is it okay for 'silvaco,i3c-target-imx93'?
> 
> No, because imx93 is product of NXP, not Silvaco.
> 
> You need regular SoC-block compatibles, just like we have for all other
> snps, dwc and cdns.

"nxp,imx93-svc-i3c-target" ? Just little bit strange for binding file name
is silvaco,i3c-master.yaml.

look like "dwc,*" compatitble string's file name is "dwc,*".yaml.

Frank
> 
> 
> Best regards,
> Krzysztof
>