Received-SPF: pass (google.com: domain of linux-kernel+bounces-27630-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Tue, 16 Jan 2024 12:35:44 -0500
From: Frank Li <Frank.li@nxp.com>
To: Conor Dooley <conor.dooley@microchip.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, 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: <Zaa+cLGVVDSB5MYr@lizhi-Precision-Tower-5810>
References: <ZaFbbeQrC7o2dchO@lizhi-Precision-Tower-5810>
 <e3b9aa63-25a5-41cc-9eb7-6e7d1eacb136@linaro.org>
 <ZaFjaWCA6k+tiCSJ@lizhi-Precision-Tower-5810>
 <ZaWLCrWJEMtFx8cR@lizhi-Precision-Tower-5810>
 <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>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240116-achiness-thievish-10a12b3c08cd@wendy>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?DEm/g4SpXWWuHIVI6GGzDn7MWPNyhNpBLakn364C704TyCP7qpH5G8zPKHKd?=
 =?us-ascii?Q?PnWt3eTkpslbKsnYpt6YXmzI9PyhU0Q8k2PuUu6YOq7ycdDNjEnzDiSLKejZ?=
 =?us-ascii?Q?VSD52P7gBhGZ68kr+wq2rNFRWLZjO5X39tArn2I/V4gxT+7nlUMgkth1i0Ig?=
 =?us-ascii?Q?wf0wgeaQV5MrVPZLMr/cn14/0Mx9/ULR7J4GR2I+XG2/cvpXjd+JAMJvGhBv?=
 =?us-ascii?Q?vv+JnNH/TnCuMqe2T0O3lNqiTxdbH8V6lIca8mOVCOQKxcVO60UcBiAhedk0?=
 =?us-ascii?Q?Z23lY3Qbh8K3UXbVyaXZj4ijHQzqM6X1Z6YCJ05KndI4QAGINbxXY4Tcz1lw?=
 =?us-ascii?Q?r5/yvH5KYO58nTah3vxCFWs98dkiAfFufpv+bh7AlIC8qLfION5IsbxXURe2?=
 =?us-ascii?Q?nkYkS+McF1nz6JOvqetUw879kww7D+4XnwG13R/1KFbMUtYU3izfRDOFU3rB?=
 =?us-ascii?Q?ptG5OufaS9nEtYF7LgkIJHlPAKqgXk6EWSuDuo/aErQuJmlGYJmG5N6Xfgfm?=
 =?us-ascii?Q?ccD5u3ouCVIcl79U4q5vxmrinIkwVMBEhQ/ji5I5oKwES0j/38J1oDuAfp57?=
 =?us-ascii?Q?TrFGlxcxoC7Yp3I9ersFkTnfCfMJ1qFpDbxgnOcuZG2u4vGivG+wAX71aArs?=
 =?us-ascii?Q?bRRY7oHo6l2qW8onSONqlDwYYMx4rY1dnN61WjWmhoB3KFGX5hwsLUQO+ueG?=
 =?us-ascii?Q?zZ7YOrDE3C8SR+JluKxjsc6ivfeB+tBAYETT0GuzNCC6dpkOPRT6kpJH+nEp?=
 =?us-ascii?Q?zPIptn2N6M/xlwJVyXx3vOtid8sHlYjbK04pyqrySWL48jOT3OanAGAwsk0y?=
 =?us-ascii?Q?QqaH46Q+By8iq1aLqlmoEmdIeVXDHdwhEm8iOCOcS681fzU0HjQF14K/s65E?=
 =?us-ascii?Q?TbZyDV3pddf9VAjBC4UB978KByMwyK6sm3TiGVHMvmyOmR9W8usY+Qne8UMz?=
 =?us-ascii?Q?v0xaSQAMXrG08EdPMOHAsXYQCOOVyeMNVeBOLaEXyD+yCUbU1ukslGdZ8HOa?=
 =?us-ascii?Q?yYiOzvxLduseyk1WnbqdgXPQRxXzb3WoFRhPNR79BNPzkvakJJWVDaoYW43S?=
 =?us-ascii?Q?eRYw4sqTQnv8zNg9yrYBw3NPLsKFS5IGusjtU30T3ejcXM6e8GyTOoZsDJCC?=
 =?us-ascii?Q?nTnb2evkp5lHU1kYvDcmaM8vGX0iXo7k868KNIIxrbDdzVMKs2bS2K6EKjMP?=
 =?us-ascii?Q?ivNlmxH/0iEaJqFrQh2OFxw4997en5ulYpsIhmXWSRp/bdUOBzUTOeydy0QA?=
 =?us-ascii?Q?0EnGsfn8fCaDXLAqhzjOlh2YjHhDYAiUSigPUhRlHEHiQrIRf+9LH1IS8OUT?=
 =?us-ascii?Q?Kko1kpcWGrTUkakrqLkDNgkOt/c85LqaIxYM1yzV40P5f+Nyc8gnXPe/oaRc?=
 =?us-ascii?Q?rX2Qtne8KgWv5twh1xV3pr2F5CEX+sKVbP9RiMIp8Wo8spEVZsFysVWJuDc+?=
 =?us-ascii?Q?NB90Il+oNvQzmDobTGsLii/QCQQQ2knyUuGzLGOxXSP/OjfvceCUSQ2aFGP3?=
 =?us-ascii?Q?bTFGwPLkdt6ONsA7x20EGzzVOzreuEiDo1q0rHwJOloDCLoV2a9l2TQ4dF/O?=
 =?us-ascii?Q?rN0cerXEDHXB4tGjzig=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 37a208ce-afd9-45bb-606a-08dc16b99658
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jan 2024 17:35:52.8643
 (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: fgA3tmAY1BLQfiNAHrUNBgFcBict0iCFlJBxTTMtXsd9Zb5Y9jCEFPX/v5oD0QlGO6cgY5NyxhWWlyGeDGI2TQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB9440

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'?

Frank

> 
> Thanks,
> Conor.