On 02/10/2023 13:16, Conor Dooley wrote: > On Mon, Oct 02, 2023 at 11:18:56AM +0200, Hans Verkuil wrote: >> Rob, Conor, >> >> On 09/08/2023 22:43, Mirela Rabulea wrote: >>>> -----Original Message----- >>>> From: Rob Herring <robh@xxxxxxxxxx> >>>> Sent: Wednesday, July 26, 2023 8:02 PM >>>> To: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> >>>> Cc: Conor Dooley <conor@xxxxxxxxxx>; Mirela Rabulea >>>> <mirela.rabulea@xxxxxxx>; Ming Qian <ming.qian@xxxxxxx>; Shijie Qin >>>> <shijie.qin@xxxxxxx>; Eagle Zhou <eagle.zhou@xxxxxxx>; Mauro Carvalho >>>> Chehab <mchehab@xxxxxxxxxx>; Krzysztof Kozlowski >>>> <krzysztof.kozlowski+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; >>>> Shawn Guo <shawnguo@xxxxxxxxxx>; Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>; >>>> Fabio Estevam <festevam@xxxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>; >>>> Anson Huang <Anson.Huang@xxxxxxx>; dl-linux-imx <linux-imx@xxxxxxx>; >>>> Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>; linux- >>>> media@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-arm- >>>> kernel@xxxxxxxxxxxxxxxxxxx; linux-spi@xxxxxxxxxxxxxxx >>>> Subject: [EXT] Re: [PATCH v2 3/3] dt-bindings: media: imx-jpeg: Add clocks >>>> property >>>> >>>> Caution: This is an external email. Please take care when clicking links or >>>> opening attachments. When in doubt, report the message using the 'Report this >>>> email' button >>>> >>>> >>>> On Tue, Jul 25, 2023 at 07:31:55AM +0200, Alexander Stein wrote: >>>>> Am Montag, 24. Juli 2023, 20:26:15 CEST schrieb Conor Dooley: >>>>>> On Mon, Jul 24, 2023 at 02:21:00PM +0200, Alexander Stein wrote: >>>>>>> i.MX8 and i.MX8X both use two clocks for accessing the periphery. >>>>>>> Add clocks and clock-names properties accordingly. >>>>>>> >>>>>>> Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> >> >> Is this patch OK or do you want changes? >> >> It's a bit unclear. > > It's unclear to me too at this point, but I wonder if the names should > just be removed and replaced by descriptions in the clocks property? > > One of the responses here mentioned that the names aren't even needed by > hardware. Right, I'm marking this as "Obsoleted" based on Mirela's reply. Alexander, if you believe this is still needed, then please post a v2. Regards, Hans
