[PATCH v2 3/3] dt-bindings: media: imx-jpeg: Add clocks property
Conor Dooley
conor at kernel.org
Tue Jul 25 12:14:02 PDT 2023
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 at ew.tq-group.com>
> > > ---
> > > Changes in v2:
> > > * None
> > >
> > > .../devicetree/bindings/media/nxp,imx8-jpeg.yaml | 8 ++++++++
> > > 1 file changed, 8 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> > > b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml index
> > > 3d9d1db37040..2533e16720f2 100644
> > > --- a/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> > > +++ b/Documentation/devicetree/bindings/media/nxp,imx8-jpeg.yaml
> > >
> > > @@ -46,6 +46,14 @@ properties:
> > > minItems: 2 # Wrapper and 1 slot
> > > maxItems: 5 # Wrapper and 4 slots
> > >
> > > + clocks:
> > > + maxItems: 2
> > > +
> > > + clock-names:
> > > + items:
> > > + - const: per
> > > + - const: ipg
> >
> > What do "per" and "ipg" mean? I assume "per" is peripheral?
>
> Actually I don't know what "ipg" stands for. It's a quite common name on i.MX
> platforms though. I opted for the names currently used in the DT. The driver
> doesn't care for the names currently.
FWIW, my motivation was wondering how someone would know which clock to
put in which.
> But cross-checking the reference manual these clocks seems to be called "jpeg"
> and "ips", individually for both jpeg encoder and decoder.
Hm, that seems confusing TBH. The reference manual is where I would be
going to try and figure out the numbers.
> Mirela (added to recipients): As the original author of the DT nodes, could
> you provide additional information regarding the clock names?
That'd be great, thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230725/9cd5158b/attachment.sig>
More information about the linux-arm-kernel
mailing list