Re: [PATCH 1/5] ARM: dts: move dp hpd line to the board file for exynos5420

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

 




On Thursday, May 08, 2014 6:08 PM, Rahul Sharma wrote:
> On 8 May 2014 12:02, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote:
> > On Sunday, April 20, 2014 8:49 PM, Rahul Sharma wrote:
> >>
> >> From: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx>
> >>
> >> Display controller HPD Gpio line is board specific. This patch
> >> is moving the pinctrl for hpd gpio line to the respective board
> >> file.
> >>
> >> Signed-off-by: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx>
> >> ---
> >>  arch/arm/boot/dts/exynos5420-pinctrl.dtsi |    7 -------
> >>  arch/arm/boot/dts/exynos5420-smdk5420.dts |    7 +++++++
> >>  2 files changed, 7 insertions(+), 7 deletions(-)
> >>
> >> diff --git a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> >> index e62c8eb..5848c42 100644
> >> --- a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> >> +++ b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> >> @@ -59,13 +59,6 @@
> >>                       interrupt-controller;
> >>                       #interrupt-cells = <2>;
> >>               };
> >> -
> >> -             dp_hpd: dp_hpd {
> >> -                     samsung,pins = "gpx0-7";
> >> -                     samsung,pin-function = <3>;
> >> -                     samsung,pin-pud = <0>;
> >> -                     samsung,pin-drv = <0>;
> >> -             };
> >>       };
> >
> > (+cc Doug Anderson, Tomasz Figa)
> >
> > 'gpx0-7' pin is assigned for standard HPD pin. This is chip specific.
> > So, there should be "gpx0-7" part on chip specific dts file such as
> > 'exynos5420-pinctrl.dtsi'.
> >
> > However, some boards use GPIO as HPD pin, instead of 'gpx0-7' pin.
> > This is board specific. In my opinion, the following would look
> > better. In this way, this patch is not necessary.
> >
> > 1. case: Standard HPD pin is used. (smdk5420)
> > arch/arm/boot/dts/exynos5420-pinctrl.dts
> >         pinctrl@13400000 {
> >                 dp_hpd: dp_hpd {
> >                         samsung,pins = "gpx0-7";
> >                         samsung,pin-function = <3>;
> >                         samsung,pin-pud = <0>;
> >                         samsung,pin-drv = <0>;
> >                 };
> > arch/arm/boot/dts/exynos5420-smdk5420.dts
> >         dp-controller@145B0000 {
> >                 pinctrl-names = "default";
> >                 pinctrl-0 = <&dp_hpd>;
> >                 ...
> >
> >
> > 2. case: non-standard HPD pin such as GPIO is used. (peach pit)
> > arch/arm/boot/dts/exynos5420-peach-pit.dts
> >         pinctrl@13400000 {
> >                 dp_hpd: dp_hpd {
> >                         samsung,pins = "gpx2-6";
> >                         samsung,pin-function = <0>;
> >                         samsung,pin-pud = <3>;
> >                         samsung,pin-drv = <0>;
> >                 };
> >         dp-controller@145B0000 {
> >                 pinctrl-names = "default";
> >                 pinctrl-0 = <&dp_hpd>;
> >                 ...
> >                 samsung,hpd-gpio = <&gpx2 6 0>;
> >
> > Even though, there is 'dp_hpd' in exynos5420-pinctrl.dts,
> > 'smdk5420' and 'peach pit' can use "gpx0-7" and "gpx2-6",
> > respectively.
> >
> > A year ago, Doug Anderson and me agreed that standard HPD pin
> > part should be included to 'exynos5250-pinctrl.dtsi'.
> > (https://lkml.org/lkml/2013/5/11/134)
> >
> 
> I wasn't aware of this discussion. It make sense. I will drop this patch.
> I need to choose some other name for this pin, say "peach_dp_hpd".
> Let me know if any concern.

In my opinion, "dp_hpd_gpio" looks better as below.

arch/arm/boot/dts/exynos5420-peach-pit.dts
	pinctrl@13400000 {
		dp_hpd_gpio: dp_hpd_gpio {
			samsung,pins = "gpx2-6";
			samsung,pin-function = <0>;
			samsung,pin-pud = <3>;
			samsung,pin-drv = <0>;
		};
	dp-controller@145B0000 {
		pinctrl-names = "default";
		pinctrl-0 = <&dp_hpd_gpio>;
		...
		samsung,hpd-gpio = <&gpx2 6 0>;

Best regards,
Jingoo Han

> 
> Regards,
> Rahul Sharma.
> 
> > Best regards,
> > Jingoo Han
> >
> >>
> >>       pinctrl@13410000 {
> >> diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> >> index 6910485..62d38c8 100644
> >> --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> >> +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> >> @@ -138,6 +138,13 @@
> >>                       samsung,pin-pud = <1>;
> >>                       samsung,pin-drv = <0>;
> >>               };
> >> +
> >> +             dp_hpd: dp_hpd {
> >> +                     samsung,pins = "gpx0-7";
> >> +                     samsung,pin-function = <3>;
> >> +                     samsung,pin-pud = <0>;
> >> +                     samsung,pin-drv = <0>;
> >> +             };
> >>       };
> >>
> >>       hdmi@14530000 {
> >> --
> >> 1.7.9.5
> >

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux