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: <35431593.B1LTs037XF@avalon> Date: Fri, 06 Apr 2018 16:45:16 +0300 From: Laurent Pinchart <laurent.pinchart@...asonboard.com> To: Jacopo Mondi <jacopo+renesas@...ndi.org> Cc: horms@...ge.net.au, magnus.damm@...il.com, geert@...ux-m68k.org, niklas.soderlund@...natech.se, sergei.shtylyov@...entembedded.com, dri-devel@...ts.freedesktop.org, linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 5/7] arm64: dts: renesas: eagle: Enable DU Hi Jacopo, Thank you for the patch. On Friday, 6 April 2018 16:08:10 EEST Jacopo Mondi wrote: > Enable DU for Renesas R-Car V3M Eagle board. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@...ndi.org> > --- > arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts index 3c5f598..144b847 > 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts > @@ -76,6 +76,11 @@ > function = "i2c0"; > }; > > + du_pins: du { > + groups = "du_rgb666", "du_sync", "du_oddf", "du_clk_out"; > + function = "du"; > + }; As far as I can tell the DU parallel output isn't used on the Eagle board, but is used on the Eagle expansion board. I would move this to patch 7/7 in this series. > scif0_pins: scif0 { > groups = "scif0_data"; > function = "scif0"; > @@ -93,3 +98,9 @@ > > status = "okay"; > }; > + > +&du { > + pinctrl-0 = <&du_pins>; > + pinctrl-names = "default"; These two properties should be moved to patch 7/7 too. > + status = "okay"; > +}; There's little use for enabling the DU in DT if you have no output port described. I'd move this to patch 6/7. -- Regards, Laurent Pinchart
Powered by blists - more mailing lists