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.
Attachment:
signature.asc
Description: PGP signature
