Re: [PATCH v2 4/4] arm64: dts: renesas: salvator-x: Enable DU

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

On Thursday 11 Aug 2016 11:17:17 Geert Uytterhoeven wrote:
> On Tue, Aug 9, 2016 at 2:29 PM, Laurent Pinchart wrote:
> > Only the VGA output is supported for now.
> > 
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> 
> One question below, though...
> 
> > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> > @@ -135,6 +135,50 @@
> > 
> > +&du {
> > +       status = "okay";
> 
> Don't you need pinctrl properties? Or are you relying on initial settings?
> 
> GPSR initial settings for DU depend on MD[4:1] i.e. SW10[5:8].
> The defaults on my board are not 0000, so all DU pins are GPIOs?
> IPSR initial settings don't seem to be documented.

pinctrl configuration is needed, but isn't available yet in the sh-pfc driver. 
I guess I'll just have to bite the bullet and add it :-)

-- 
Regards,

Laurent Pinchart




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux