| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <e343e674b87a0b5a0d5fc74adda2f3515f0be895.camel@oss.nxp.com>
Date: Fri, 13 Nov 2020 16:12:24 +0000
From: "Mirela Rabulea (OSS)" <mirela.rabulea@....nxp.com>
To: "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
"ezequiel@...labora.com" <ezequiel@...labora.com>,
"mchehab@...nel.org" <mchehab@...nel.org>,
"shawnguo@...nel.org" <shawnguo@...nel.org>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"hverkuil-cisco@...all.nl" <hverkuil-cisco@...all.nl>
CC: dl-linux-imx <linux-imx@....com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"laurent.pinchart+renesas@...asonboard.com"
<laurent.pinchart+renesas@...asonboard.com>,
Aisheng Dong <aisheng.dong@....com>,
Laurentiu Palcu <laurentiu.palcu@....com>,
"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
Robert Chiras <robert.chiras@....com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"paul.kocialkowski@...tlin.com" <paul.kocialkowski@...tlin.com>,
"mark.rutland@....com" <mark.rutland@....com>,
"niklas.soderlund+renesas@...natech.se"
<niklas.soderlund+renesas@...natech.se>,
Daniel Baluta <daniel.baluta@....com>,
"dafna.hirschfeld@...labora.com" <dafna.hirschfeld@...labora.com>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>
Subject: Re: [EXT] Re: [PATCH v5 05/10] arm64: dts: imx8qxp: Add jpeg
encoder/decoder nodes
Hi Ezequiel,
On Thu, 2020-11-12 at 05:36 -0300, Ezequiel Garcia wrote:
>
> Hi Mirela,
>
> On Thu, 2020-11-12 at 05:05 +0200, Mirela Rabulea (OSS) wrote:
> > From: Mirela Rabulea <mirela.rabulea@....com>
> >
> > Add jpeg decoder/encoder nodes, for now on imx8qxp only.
> > The same should work on imx8qm, but it was not tested.
> >
>
> Does imx8qm need changes in the dt bindings?
>
> Unless you are aware of reasons preventing us from enabling
> it on imx8qm, then we could go for imx8qm as well (reusing
> imx8qxp- compatible).
I think it will be possible to reuse the same compatile for 8qm, too.
There is no dts for 8qm upstream for now, I understand Aisheng has
something started on that. We'll see how it goes.
>
> > Signed-off-by: Mirela Rabulea <mirela.rabulea@....com>
> > ---
> > arch/arm64/boot/dts/freescale/imx8qxp-mek.dts | 8 ++++
> > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 37
> > +++++++++++++++++++
> > 2 files changed, 45 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts
> > b/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts
> > index 46437d3c7a04..a0ad9789e9b8 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8qxp-mek.dts
> > @@ -270,3 +270,11 @@
> > >;
> > };
> > };
> > +
> > +&jpegdec {
> > + status = "okay";
> > +};
> > +
> > +&jpegenc {
> > + status = "okay";
> > +};
>
> Please drop this. See below.
Done for the next version.
>
> > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > index e46faac1fe71..1d9a16388fa8 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > @@ -629,4 +629,41 @@
> > };
> > };
> > };
> > +
> > + img_subsys: bus@...00000 {
> > + compatible = "simple-bus";
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + ranges = <0x58000000 0x0 0x58000000 0x1000000>;
> > +
> > + jpegdec: jpegdec@...00000 {
> > + compatible = "nxp,imx8qxp-jpgdec";
> > + reg = <0x58400000 0x00050000 >;
> > + interrupts = <GIC_SPI 309
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 310
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 311
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 312
> > IRQ_TYPE_LEVEL_HIGH>;
> > + power-domains = <&pd IMX_SC_R_MJPEG_DEC_MP>,
> > + <&pd IMX_SC_R_MJPEG_DEC_S0>,
> > + <&pd IMX_SC_R_MJPEG_DEC_S1>,
> > + <&pd IMX_SC_R_MJPEG_DEC_S2>,
> > + <&pd IMX_SC_R_MJPEG_DEC_S3>;
> > + status = "disabled";
>
> Pure memory-to-memory are typically not enabled per-board,
> but just per-platform.
>
> So you can drop the disabled status here.
Done for the next version.
Thanks,
Mirela
>
> Thanks,
> Ezequiel
>
> > + };
> > +
> > + jpegenc: jpegenc@...50000 {
> > + compatible = "nxp,imx8qxp-jpgenc";
> > + reg = <0x58450000 0x00050000 >;
> > + interrupts = <GIC_SPI 305
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 306
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 307
> > IRQ_TYPE_LEVEL_HIGH>,
> > + <GIC_SPI 308
> > IRQ_TYPE_LEVEL_HIGH>;
> > + power-domains = <&pd IMX_SC_R_MJPEG_ENC_MP>,
> > + <&pd IMX_SC_R_MJPEG_ENC_S0>,
> > + <&pd IMX_SC_R_MJPEG_ENC_S1>,
> > + <&pd IMX_SC_R_MJPEG_ENC_S2>,
> > + <&pd IMX_SC_R_MJPEG_ENC_S3>;
> > + status = "disabled";
> > + };
> > + };
> > };
>
>
Powered by blists - more mailing lists