From: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
To: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org>
Cc: "Peter Ujfalusi" <peter.ujfalusi-l0cyMroinI0@public.gmane.org>,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
"Enric Balletbo i Serra"
<eballetbo-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org>,
"Agustí Fontquerni" <af-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org>,
"Eduard Gavin" <egavin-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [PATCH] ARM: dts: omap5-igep0050: Correct hdmi regulator
Date: Thu, 5 May 2016 11:46:09 -0700 [thread overview]
Message-ID: <20160505184609.GB5995@atomide.com> (raw)
In-Reply-To: <572A38FD.9050801-l0cyMroinI0@public.gmane.org>
* Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> [160504 11:03]:
> On 05/04/2016 12:26 PM, Tony Lindgren wrote:
> > Hi,
> >
> > Adding Nishanth to Cc.
> >
> > * Peter Ujfalusi <peter.ujfalusi-l0cyMroinI0@public.gmane.org> [160429 05:15]:
> >> ldo7_reg, obviously ;)
> >>
> >>> status = "okay";
> >>> regulator-min-microvolt = <1800000>;
> >>> regulator-max-microvolt = <1800000>;
> >>> };
> >
> > It really seems to be ldo7, otherwise there's no HDMI output. I don't
> > have the schematics either.
> >
> > But it seems we have at least two other regressions in Linux next that
> > prevent me from testing this properly:
> >
> > 1. On igepv5, I get this for the MMC regulator
> >
> > ldo9: bypassed regulator has no supply!
> > ldo9: failed to get the current voltage(-517)
> > palmas-pmic 48070000.i2c:palmas@48:palmas_pmic: failed to register 48070000.i2c:palmar
The patch I posted fixes this. Then for the HDMI see below.
I've kept it at 2V like we have in omap5-board-common.dtsi,
I wonder if it really should be 1.8V though for ldo7 in this
case?
Regards,
Tony
8< -----------------
From: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
Date: Thu, 5 May 2016 11:29:23 -0700
Subject: [PATCH] ARM: dts: Fix ldo7 source for HDMI on igepv5
Fix ldo7 source for HDMI on igepv5.
Suggested-by: Peter Ujfalusi <peter.ujfalusi-l0cyMroinI0@public.gmane.org>
Signed-off-by: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
diff --git a/arch/arm/boot/dts/omap5-igep0050.dts b/arch/arm/boot/dts/omap5-igep0050.dts
index 46ecb1d..72ff6a4 100644
--- a/arch/arm/boot/dts/omap5-igep0050.dts
+++ b/arch/arm/boot/dts/omap5-igep0050.dts
@@ -35,6 +35,12 @@
};
};
+&ldo7_reg {
+ status = "okay";
+ regulator-min-microvolt = <2000000>;
+ regulator-max-microvolt = <2000000>;
+};
+
&omap5_pmx_core {
i2c4_pins: pinmux_i2c4_pins {
pinctrl-single,pins = <
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: omap5-igep0050: Correct hdmi regulator
Date: Thu, 5 May 2016 11:46:09 -0700 [thread overview]
Message-ID: <20160505184609.GB5995@atomide.com> (raw)
In-Reply-To: <572A38FD.9050801@ti.com>
* Nishanth Menon <nm@ti.com> [160504 11:03]:
> On 05/04/2016 12:26 PM, Tony Lindgren wrote:
> > Hi,
> >
> > Adding Nishanth to Cc.
> >
> > * Peter Ujfalusi <peter.ujfalusi@ti.com> [160429 05:15]:
> >> ldo7_reg, obviously ;)
> >>
> >>> status = "okay";
> >>> regulator-min-microvolt = <1800000>;
> >>> regulator-max-microvolt = <1800000>;
> >>> };
> >
> > It really seems to be ldo7, otherwise there's no HDMI output. I don't
> > have the schematics either.
> >
> > But it seems we have at least two other regressions in Linux next that
> > prevent me from testing this properly:
> >
> > 1. On igepv5, I get this for the MMC regulator
> >
> > ldo9: bypassed regulator has no supply!
> > ldo9: failed to get the current voltage(-517)
> > palmas-pmic 48070000.i2c:palmas at 48:palmas_pmic: failed to register 48070000.i2c:palmar
The patch I posted fixes this. Then for the HDMI see below.
I've kept it at 2V like we have in omap5-board-common.dtsi,
I wonder if it really should be 1.8V though for ldo7 in this
case?
Regards,
Tony
8< -----------------
From: Tony Lindgren <tony@atomide.com>
Date: Thu, 5 May 2016 11:29:23 -0700
Subject: [PATCH] ARM: dts: Fix ldo7 source for HDMI on igepv5
Fix ldo7 source for HDMI on igepv5.
Suggested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
diff --git a/arch/arm/boot/dts/omap5-igep0050.dts b/arch/arm/boot/dts/omap5-igep0050.dts
index 46ecb1d..72ff6a4 100644
--- a/arch/arm/boot/dts/omap5-igep0050.dts
+++ b/arch/arm/boot/dts/omap5-igep0050.dts
@@ -35,6 +35,12 @@
};
};
+&ldo7_reg {
+ status = "okay";
+ regulator-min-microvolt = <2000000>;
+ regulator-max-microvolt = <2000000>;
+};
+
&omap5_pmx_core {
i2c4_pins: pinmux_i2c4_pins {
pinctrl-single,pins = <
WARNING: multiple messages have this Message-ID (diff)
From: Tony Lindgren <tony@atomide.com>
To: Nishanth Menon <nm@ti.com>
Cc: "Peter Ujfalusi" <peter.ujfalusi@ti.com>,
linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
"Enric Balletbo i Serra" <eballetbo@iseebcn.com>,
"Agustí Fontquerni" <af@iseebcn.com>,
"Eduard Gavin" <egavin@iseebcn.com>
Subject: Re: [PATCH] ARM: dts: omap5-igep0050: Correct hdmi regulator
Date: Thu, 5 May 2016 11:46:09 -0700 [thread overview]
Message-ID: <20160505184609.GB5995@atomide.com> (raw)
In-Reply-To: <572A38FD.9050801@ti.com>
* Nishanth Menon <nm@ti.com> [160504 11:03]:
> On 05/04/2016 12:26 PM, Tony Lindgren wrote:
> > Hi,
> >
> > Adding Nishanth to Cc.
> >
> > * Peter Ujfalusi <peter.ujfalusi@ti.com> [160429 05:15]:
> >> ldo7_reg, obviously ;)
> >>
> >>> status = "okay";
> >>> regulator-min-microvolt = <1800000>;
> >>> regulator-max-microvolt = <1800000>;
> >>> };
> >
> > It really seems to be ldo7, otherwise there's no HDMI output. I don't
> > have the schematics either.
> >
> > But it seems we have at least two other regressions in Linux next that
> > prevent me from testing this properly:
> >
> > 1. On igepv5, I get this for the MMC regulator
> >
> > ldo9: bypassed regulator has no supply!
> > ldo9: failed to get the current voltage(-517)
> > palmas-pmic 48070000.i2c:palmas@48:palmas_pmic: failed to register 48070000.i2c:palmar
The patch I posted fixes this. Then for the HDMI see below.
I've kept it at 2V like we have in omap5-board-common.dtsi,
I wonder if it really should be 1.8V though for ldo7 in this
case?
Regards,
Tony
8< -----------------
From: Tony Lindgren <tony@atomide.com>
Date: Thu, 5 May 2016 11:29:23 -0700
Subject: [PATCH] ARM: dts: Fix ldo7 source for HDMI on igepv5
Fix ldo7 source for HDMI on igepv5.
Suggested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
diff --git a/arch/arm/boot/dts/omap5-igep0050.dts b/arch/arm/boot/dts/omap5-igep0050.dts
index 46ecb1d..72ff6a4 100644
--- a/arch/arm/boot/dts/omap5-igep0050.dts
+++ b/arch/arm/boot/dts/omap5-igep0050.dts
@@ -35,6 +35,12 @@
};
};
+&ldo7_reg {
+ status = "okay";
+ regulator-min-microvolt = <2000000>;
+ regulator-max-microvolt = <2000000>;
+};
+
&omap5_pmx_core {
i2c4_pins: pinmux_i2c4_pins {
pinctrl-single,pins = <
next prev parent reply other threads:[~2016-05-05 18:46 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-29 10:21 [PATCH] ARM: dts: omap5-igep0050: Correct hdmi regulator Peter Ujfalusi
2016-04-29 10:21 ` Peter Ujfalusi
2016-04-29 10:21 ` Peter Ujfalusi
[not found] ` <1461925297-9734-1-git-send-email-peter.ujfalusi-l0cyMroinI0@public.gmane.org>
2016-04-29 10:22 ` Peter Ujfalusi
2016-04-29 10:22 ` Peter Ujfalusi
2016-04-29 10:22 ` Peter Ujfalusi
[not found] ` <6ee9d67a-e907-715a-f3f5-d90a470314c3-l0cyMroinI0@public.gmane.org>
2016-05-04 17:26 ` Tony Lindgren
2016-05-04 17:26 ` Tony Lindgren
2016-05-04 17:26 ` Tony Lindgren
2016-05-04 18:01 ` Nishanth Menon
2016-05-04 18:01 ` Nishanth Menon
2016-05-04 18:01 ` Nishanth Menon
[not found] ` <572A38FD.9050801-l0cyMroinI0@public.gmane.org>
2016-05-05 18:46 ` Tony Lindgren [this message]
2016-05-05 18:46 ` Tony Lindgren
2016-05-05 18:46 ` Tony Lindgren
2016-05-09 8:25 ` Peter Ujfalusi
2016-05-09 8:25 ` Peter Ujfalusi
2016-05-09 8:34 ` Tomi Valkeinen
2016-05-09 8:34 ` Tomi Valkeinen
[not found] ` <2822D1C2B9B19F4A8261F756A13373F801BEF7B4@DFLE11.ent.ti.com>
[not found] ` <CAPjucKb+4vd8Ar3MATCuTx+9R36PeB0hbvwDaAD1KTZFCiVJEA@mail.gmail.com>
[not found] ` <2822D1C2B9B19F4A8261F756A13373F801BF86D2@DFLE11.ent.ti.com>
[not found] ` <2822D1C2B9B19F4A8261F756A13373F801BF8703@DFLE11.ent.ti.com>
[not found] ` <CAPjucKb2PLSZcD3YZ9xQk1F62ugFaGAkBBqMp5GmoWFd0H8Vfg@mail.gmail.com>
2016-05-12 16:57 ` Tony Lindgren
2016-05-12 16:57 ` Tony Lindgren
2016-05-12 19:19 ` Nishanth Menon
2016-05-12 19:19 ` Nishanth Menon
2016-05-12 20:26 ` Tony Lindgren
2016-05-12 20:26 ` Tony Lindgren
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=20160505184609.GB5995@atomide.com \
--to=tony-4v6ys6ai5vpbdgjk7y7tuq@public.gmane.org \
--cc=af-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=eballetbo-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org \
--cc=egavin-VIneJrwqLopBDgjK7y7TUQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=nm-l0cyMroinI0@public.gmane.org \
--cc=peter.ujfalusi-l0cyMroinI0@public.gmane.org \
/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: link
Be 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.