From: Rahul Sharma <rahul.sharma@samsung.com> To: Tomasz Figa <tomasz.figa@gmail.com> Cc: "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, linux-samsung-soc <linux-samsung-soc@vger.kernel.org>, Kukjin Kim <kgene.kim@samsung.com>, Inki Dae <inki.dae@samsung.com>, sunil joshi <joshi@samsung.com> Subject: Re: [PATCH v4 1/5] ARM: dts: enable hdmi for exynos5250 based snow board Date: Thu, 15 May 2014 10:52:42 +0530 [thread overview] Message-ID: <CAPdUM4Ndn9Y7=SF0BG-j16vCHt_4EDOZSuHEV36NTUovM7FcNQ@mail.gmail.com> (raw) In-Reply-To: <5373C6D7.8010802@gmail.com> Tomasz, On 15 May 2014 01:11, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Rahul, > > On 14.05.2014 04:45, Rahul Sharma wrote: >> From: Rahul Sharma <Rahul.Sharma@samsung.com> >> >> Enable support for HDMI for exynos5250 based Snow board. >> >> Signed-off-by: Rahul Sharma <Rahul.Sharma@samsung.com> >> --- >> arch/arm/boot/dts/exynos5250-cros-common.dtsi | 6 +++++- >> arch/arm/boot/dts/exynos5250-pinctrl.dtsi | 7 +++++++ >> arch/arm/boot/dts/exynos5250-snow.dts | 7 +++++++ >> 3 files changed, 19 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250-cros-common.dtsi b/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> index 2c1560d..89ac90f 100644 >> --- a/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> +++ b/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> @@ -240,7 +240,7 @@ >> samsung,i2c-sda-delay = <100>; >> samsung,i2c-max-bus-freq = <378000>; >> >> - hdmiphy@38 { >> + hdmiphy: hdmiphy@38 { >> compatible = "samsung,exynos4212-hdmiphy"; >> reg = <0x38>; >> }; >> @@ -304,6 +304,10 @@ >> >> hdmi { >> hpd-gpio = <&gpx3 7 0>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&hdmi_hpd_irq>; >> + phy = <&hdmiphy>; >> + ddc = <&i2c_2>; >> }; >> >> gpio-keys { >> diff --git a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> index 9a49e68..da3ae66 100644 >> --- a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> +++ b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> @@ -560,6 +560,13 @@ >> samsung,pin-pud = <0>; >> samsung,pin-drv = <0>; >> }; >> + >> + hdmi_hpd_irq: hdmi-hpd-irq { >> + samsung,pins = "gpx3-7"; >> + samsung,pin-function = <0>; >> + samsung,pin-pud = <1>; >> + samsung,pin-drv = <0>; >> + }; > > Based on the fact that this is just being configured as input, I don't > think this is a generic special function, but rather a board-specific > GPIO, which should be defined in dts of respective board. > Ok. >> }; >> >> pinctrl@13400000 { >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> index 1ce1088..696850a 100644 >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> @@ -206,4 +206,11 @@ >> clock-frequency = <24000000>; >> }; >> }; >> + >> + hdmi { >> + hdmi-en-supply = <&tps65090_fet7>; >> + vdd-supply = <&ldo8_reg>; >> + vdd_osc-supply = <&ldo10_reg>; >> + vdd_pll-supply = <&ldo8_reg>; > > Hmm, those regulators don't seem to be listed in > Documentation/devicetree/bindings/video/exynos_hdmi.txt, at least in > today's linux-next. The driver seems to use them, though. > > This isn't a problem with this patch, but if you're working on this, > could you also fix the documentation in separate patch? > I will post another patch for documentation update. Regards, Rahul Sharma. > Best regards, > Tomasz
WARNING: multiple messages have this Message-ID (diff)
From: rahul.sharma@samsung.com (Rahul Sharma) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 1/5] ARM: dts: enable hdmi for exynos5250 based snow board Date: Thu, 15 May 2014 10:52:42 +0530 [thread overview] Message-ID: <CAPdUM4Ndn9Y7=SF0BG-j16vCHt_4EDOZSuHEV36NTUovM7FcNQ@mail.gmail.com> (raw) In-Reply-To: <5373C6D7.8010802@gmail.com> Tomasz, On 15 May 2014 01:11, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Rahul, > > On 14.05.2014 04:45, Rahul Sharma wrote: >> From: Rahul Sharma <Rahul.Sharma@samsung.com> >> >> Enable support for HDMI for exynos5250 based Snow board. >> >> Signed-off-by: Rahul Sharma <Rahul.Sharma@samsung.com> >> --- >> arch/arm/boot/dts/exynos5250-cros-common.dtsi | 6 +++++- >> arch/arm/boot/dts/exynos5250-pinctrl.dtsi | 7 +++++++ >> arch/arm/boot/dts/exynos5250-snow.dts | 7 +++++++ >> 3 files changed, 19 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250-cros-common.dtsi b/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> index 2c1560d..89ac90f 100644 >> --- a/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> +++ b/arch/arm/boot/dts/exynos5250-cros-common.dtsi >> @@ -240,7 +240,7 @@ >> samsung,i2c-sda-delay = <100>; >> samsung,i2c-max-bus-freq = <378000>; >> >> - hdmiphy at 38 { >> + hdmiphy: hdmiphy at 38 { >> compatible = "samsung,exynos4212-hdmiphy"; >> reg = <0x38>; >> }; >> @@ -304,6 +304,10 @@ >> >> hdmi { >> hpd-gpio = <&gpx3 7 0>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&hdmi_hpd_irq>; >> + phy = <&hdmiphy>; >> + ddc = <&i2c_2>; >> }; >> >> gpio-keys { >> diff --git a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> index 9a49e68..da3ae66 100644 >> --- a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> +++ b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi >> @@ -560,6 +560,13 @@ >> samsung,pin-pud = <0>; >> samsung,pin-drv = <0>; >> }; >> + >> + hdmi_hpd_irq: hdmi-hpd-irq { >> + samsung,pins = "gpx3-7"; >> + samsung,pin-function = <0>; >> + samsung,pin-pud = <1>; >> + samsung,pin-drv = <0>; >> + }; > > Based on the fact that this is just being configured as input, I don't > think this is a generic special function, but rather a board-specific > GPIO, which should be defined in dts of respective board. > Ok. >> }; >> >> pinctrl at 13400000 { >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> index 1ce1088..696850a 100644 >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> @@ -206,4 +206,11 @@ >> clock-frequency = <24000000>; >> }; >> }; >> + >> + hdmi { >> + hdmi-en-supply = <&tps65090_fet7>; >> + vdd-supply = <&ldo8_reg>; >> + vdd_osc-supply = <&ldo10_reg>; >> + vdd_pll-supply = <&ldo8_reg>; > > Hmm, those regulators don't seem to be listed in > Documentation/devicetree/bindings/video/exynos_hdmi.txt, at least in > today's linux-next. The driver seems to use them, though. > > This isn't a problem with this patch, but if you're working on this, > could you also fix the documentation in separate patch? > I will post another patch for documentation update. Regards, Rahul Sharma. > Best regards, > Tomasz
next prev parent reply other threads:[~2014-05-15 5:22 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-05-14 2:45 [PATCH v4 0/5] ARM: dts: enable hdmi for exynos5 based peach and snow boards Rahul Sharma 2014-05-14 2:45 ` Rahul Sharma 2014-05-14 2:45 ` [PATCH v4 1/5] ARM: dts: enable hdmi for exynos5250 based snow board Rahul Sharma 2014-05-14 2:45 ` Rahul Sharma 2014-05-14 19:41 ` Tomasz Figa 2014-05-14 19:41 ` Tomasz Figa 2014-05-15 5:22 ` Rahul Sharma [this message] 2014-05-15 5:22 ` Rahul Sharma 2014-05-14 2:45 ` [PATCH v4 2/5] ARM: dts: change to correct compatible string for exynos5420 hdmi Rahul Sharma 2014-05-14 2:45 ` Rahul Sharma 2014-05-14 19:44 ` Tomasz Figa 2014-05-14 19:44 ` Tomasz Figa 2014-05-15 5:25 ` Rahul Sharma 2014-05-15 5:25 ` Rahul Sharma 2014-05-14 2:46 ` [PATCH v4 3/5] ARM: dts: enable hdmi for exynos5420 based peach-pit board Rahul Sharma 2014-05-14 2:46 ` Rahul Sharma 2014-05-14 19:50 ` Tomasz Figa 2014-05-14 19:50 ` Tomasz Figa 2014-05-15 5:46 ` Rahul Sharma 2014-05-15 5:46 ` Rahul Sharma 2014-05-14 2:46 ` [PATCH v4 4/5] ARM: dts: remove chip specific hdmi hpd pin from board Rahul Sharma 2014-05-14 2:46 ` Rahul Sharma 2014-05-14 18:45 ` Kukjin Kim 2014-05-14 18:45 ` Kukjin Kim 2014-05-14 19:23 ` Rahul Sharma 2014-05-14 19:23 ` Rahul Sharma 2014-05-14 19:51 ` Tomasz Figa 2014-05-14 19:51 ` Tomasz Figa 2014-05-15 5:47 ` Rahul Sharma 2014-05-15 5:47 ` Rahul Sharma 2014-05-14 2:46 ` [PATCH v4 5/5] ARM: dts: enable hdmi for exynos5800 based peach-pi board Rahul Sharma 2014-05-14 2:46 ` Rahul Sharma 2014-05-14 19:56 ` Tomasz Figa 2014-05-14 19:56 ` Tomasz Figa 2014-05-14 19:57 ` Tomasz Figa 2014-05-14 19:57 ` Tomasz Figa 2014-05-15 8:27 ` Rahul Sharma 2014-05-15 8:27 ` Rahul Sharma
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAPdUM4Ndn9Y7=SF0BG-j16vCHt_4EDOZSuHEV36NTUovM7FcNQ@mail.gmail.com' \ --to=rahul.sharma@samsung.com \ --cc=devicetree@vger.kernel.org \ --cc=inki.dae@samsung.com \ --cc=joshi@samsung.com \ --cc=kgene.kim@samsung.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=tomasz.figa@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.