RE: [PATCH 1/3] arm64: dts: renesas: r8a7795: salvator-xs: enable USB2.0 phy channel 3

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

 



Hi Geert-san,

> -----Original Message-----
> From: Geert Uytterhoeven
> Sent: Monday, October 2, 2017 6:20 PM
> 
> Hi Shimoda-san,
> 
> On Mon, Oct 2, 2017 at 11:14 AM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote:
> >> -----Original Message-----
> >> From: Kieran Bingham
> >> Sent: Monday, October 2, 2017 5:45 PM
> >> On 02/10/17 09:20, Geert Uytterhoeven wrote:
> >> > On Mon, Oct 2, 2017 at 8:29 AM, Yoshihiro Shimoda
> >> > <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote:
> >> >> This patch enables the USB2.0 phy channel 3. R-Car H3 ES2.0 has
> >> >> this channel and we can use it on the Salvator-XS.
> >> >>
> >> >> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> >> >
> >> > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> >> >
> >> >> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts
> >> >> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts
> >> >> @@ -99,6 +99,11 @@
> >> >>                 groups = "usb2";
> >> >>                 function = "usb2";
> >> >>         };
> >> >> +
> >> >> +       usb2_ch3_pins: usb2_ch3 {
> >> >> +               groups = "usb2_ch3";
> >> >> +               function = "usb2_ch3";
> >> >> +       };
> >> >>  };
> >> >
> >> > Using these pins requires changing SW31, disabling the interrupt capabilities
> >> > of the ADV7482? Does the ADV7482 still work after that?
> >
> > Oops, I missed the usb2.0 pins are connected to ADV7482's interruption pins.
> > But,
> >
> >> The ADV7482 is currently working without interrupts at the moment. However, I do
> >> have hotplug and interrupt handling on my todo list, and infact I have already
> >> started a patch adding interrupt support.
> >>
> >>
> >> The DT currently specifies the following interrupts for the ADV7482,
> >>
> >> +     video-receiver@70 {
> >> +             compatible = "adi,adv7482";
> >> +             interrupt-parent = <&gpio6>;
> >> +             interrupt-names = "intrq1", "intrq2";
> >> +             interrupts = <30 IRQ_TYPE_LEVEL_LOW>,
> >> +                          <31 IRQ_TYPE_LEVEL_LOW>;
> >
> > It seems Kieran-san is using non-conflict pins with usb2.0 ch3 :)
> > According to the board datasheet, GP6_30 and GP6_31 are initial setting for ADV7482 and
> > it doesn't conflict with USB 2.0 ch3 (USB2.0 ch3 pins are GP6_04 and GP6_21).
> 
> Can the USB2.0 driver use GPIOs for PWEN and OVC?
> 
> > So, I think we can use both usb 2.0 ch3 and ADV7482 if we don't change the SW31 from initial setting.
> 
> OK. But in that case, DT should not configure pinctrl for "usb2_ch3",
> but instead
> describe that it uses GP6_04 and GP6_21.

I'm afraid but I completely misunderstood the pin configurations on the Salvator-XS.
I wrote right things below:
- USB2.0 ch3 pins are GP6_30 and GP6_31.
 - So, initial setting of the board will be conflict with the adv7482's interrupt pins.
- If we change the SW31 as alternative setting for usb, the adv7482's interrupt pins connect
  to GP6_04 and GP6_21.

And, I wrote my opinions below:
- The adv7482 point of view:
 - On both Salvator-X and Salvator-XS, adv7482's interrupt pins connected to GP6_30 and GP6_31.
  - On Salvator-XS, adv7482's interrupt pins can be changed by SW31 and they connected to GP6_04 and GP6_21.
- The usb2 ch3 point of view:
 - It can be used on H3 ES2.0 only. (In other words, M3 cannot use the usb2 ch3.)
- Summary
 - We describe the adv7482's interrupt pins into both salvator-x.dtsi and salvator-xs.dtsi.
  - In salvator-x.dtsi, adv7482 node uses GP6_30 and GP6_31.
  - In salvator-xs.dtsi, adv7482 node uses GP6_04 and GP6_21.
 - We describe the usb2 ch3 pfc node into r8a7795-salvator-xs.dtsi with "usb2_ch3" groups/function and
    describe the SW31 configuration.

What do you think?

Best regards,
Yoshihiro Shimoda





[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