* [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common
@ 2013-10-23 13:11 Doug Anderson
2013-10-23 13:31 ` Marc Zyngier
2013-11-25 23:11 ` Doug Anderson
0 siblings, 2 replies; 5+ messages in thread
From: Doug Anderson @ 2013-10-23 13:11 UTC (permalink / raw)
To: Kukjin Kim
Cc: Olof Johansson, marc.zyngier, Doug Anderson, Russell King,
Rahul Sharma, Tomasz Figa, linux-arm-kernel, linux-kernel
Without the interrupt you'll get problems if you enable
CONFIG_RTC_DRV_MAX77686. Setup the interrupt properly in the device
tree.
Signed-off-by: Doug Anderson <dianders@chromium.org>
---
arch/arm/boot/dts/cros5250-common.dtsi | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
index dc259e8b..9b186ac 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -27,6 +27,13 @@
i2c2_bus: i2c2-bus {
samsung,pin-pud = <0>;
};
+
+ max77686_irq: max77686-irq {
+ samsung,pins = "gpx3-2";
+ samsung,pin-function = <0>;
+ samsung,pin-pud = <0>;
+ samsung,pin-drv = <0>;
+ };
};
i2c@12C60000 {
@@ -35,6 +42,11 @@
max77686@09 {
compatible = "maxim,max77686";
+ interrupt-parent = <&gpx3>;
+ interrupts = <2 0>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&max77686_irq>;
+ wakeup-source;
reg = <0x09>;
voltage-regulators {
--
1.8.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common
2013-10-23 13:11 [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common Doug Anderson
@ 2013-10-23 13:31 ` Marc Zyngier
2013-11-25 23:11 ` Doug Anderson
1 sibling, 0 replies; 5+ messages in thread
From: Marc Zyngier @ 2013-10-23 13:31 UTC (permalink / raw)
To: Doug Anderson
Cc: Kukjin Kim, Russell King, Tomasz Figa, linux-kernel,
Olof Johansson, linux-arm-kernel, Rahul Sharma
On 2013-10-23 14:11, Doug Anderson wrote:
> Without the interrupt you'll get problems if you enable
> CONFIG_RTC_DRV_MAX77686. Setup the interrupt properly in the device
> tree.
Awesome! Thanks for fixing this, Doug. My Chromebook is much happier
now!
> Signed-off-by: Doug Anderson <dianders@chromium.org>
Tested-by: Marc Zyngier <marc.zyngier@arm.com>
M.
> ---
> arch/arm/boot/dts/cros5250-common.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
> b/arch/arm/boot/dts/cros5250-common.dtsi
> index dc259e8b..9b186ac 100644
> --- a/arch/arm/boot/dts/cros5250-common.dtsi
> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
> @@ -27,6 +27,13 @@
> i2c2_bus: i2c2-bus {
> samsung,pin-pud = <0>;
> };
> +
> + max77686_irq: max77686-irq {
> + samsung,pins = "gpx3-2";
> + samsung,pin-function = <0>;
> + samsung,pin-pud = <0>;
> + samsung,pin-drv = <0>;
> + };
> };
>
> i2c@12C60000 {
> @@ -35,6 +42,11 @@
>
> max77686@09 {
> compatible = "maxim,max77686";
> + interrupt-parent = <&gpx3>;
> + interrupts = <2 0>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&max77686_irq>;
> + wakeup-source;
> reg = <0x09>;
>
> voltage-regulators {
--
Fast, cheap, reliable. Pick two.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common
2013-10-23 13:11 [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common Doug Anderson
2013-10-23 13:31 ` Marc Zyngier
@ 2013-11-25 23:11 ` Doug Anderson
2013-11-25 23:16 ` Olof Johansson
1 sibling, 1 reply; 5+ messages in thread
From: Doug Anderson @ 2013-11-25 23:11 UTC (permalink / raw)
To: Kukjin Kim, Olof Johansson
Cc: marc.zyngier, Doug Anderson, Russell King, Rahul Sharma,
Tomasz Figa, linux-arm-kernel, linux-kernel
Olof or Kukjin,
On Wed, Oct 23, 2013 at 6:11 AM, Doug Anderson <dianders@chromium.org> wrote:
> Without the interrupt you'll get problems if you enable
> CONFIG_RTC_DRV_MAX77686. Setup the interrupt properly in the device
> tree.
>
> Signed-off-by: Doug Anderson <dianders@chromium.org>
> ---
> arch/arm/boot/dts/cros5250-common.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
Would one of you be interested in applying this to your tree, with
Marc's "Tested-by"? I know there are lots more things to fixup in the
dts, but this is one that I know works. ;)
-Doug
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common
2013-11-25 23:11 ` Doug Anderson
@ 2013-11-25 23:16 ` Olof Johansson
2013-11-25 23:26 ` Kukjin Kim
0 siblings, 1 reply; 5+ messages in thread
From: Olof Johansson @ 2013-11-25 23:16 UTC (permalink / raw)
To: Doug Anderson
Cc: Kukjin Kim, Marc Zyngier, Russell King, Rahul Sharma,
Tomasz Figa, linux-arm-kernel, linux-kernel
On Mon, Nov 25, 2013 at 3:11 PM, Doug Anderson <dianders@chromium.org> wrote:
> Olof or Kukjin,
>
> On Wed, Oct 23, 2013 at 6:11 AM, Doug Anderson <dianders@chromium.org> wrote:
>> Without the interrupt you'll get problems if you enable
>> CONFIG_RTC_DRV_MAX77686. Setup the interrupt properly in the device
>> tree.
>>
>> Signed-off-by: Doug Anderson <dianders@chromium.org>
>> ---
>> arch/arm/boot/dts/cros5250-common.dtsi | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>
> Would one of you be interested in applying this to your tree, with
> Marc's "Tested-by"? I know there are lots more things to fixup in the
> dts, but this is one that I know works. ;)
Yes, I'll apply this to fixes for 3.13, and mark it for stable 3.12 at least.
-Olof
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common
2013-11-25 23:16 ` Olof Johansson
@ 2013-11-25 23:26 ` Kukjin Kim
0 siblings, 0 replies; 5+ messages in thread
From: Kukjin Kim @ 2013-11-25 23:26 UTC (permalink / raw)
To: 'Olof Johansson', 'Doug Anderson'
Cc: 'Marc Zyngier', 'Russell King',
'Rahul Sharma', 'Tomasz Figa',
linux-arm-kernel, linux-kernel
Olof Johansson wrote:
>
> On Mon, Nov 25, 2013 at 3:11 PM, Doug Anderson <dianders@chromium.org>
> wrote:
> > Olof or Kukjin,
> >
> > On Wed, Oct 23, 2013 at 6:11 AM, Doug Anderson <dianders@chromium.org>
> wrote:
> >> Without the interrupt you'll get problems if you enable
> >> CONFIG_RTC_DRV_MAX77686. Setup the interrupt properly in the device
> >> tree.
> >>
> >> Signed-off-by: Doug Anderson <dianders@chromium.org>
> >> ---
> >> arch/arm/boot/dts/cros5250-common.dtsi | 12 ++++++++++++
> >> 1 file changed, 12 insertions(+)
> >
> > Would one of you be interested in applying this to your tree, with
> > Marc's "Tested-by"? I know there are lots more things to fixup in the
> > dts, but this is one that I know works. ;)
>
> Yes, I'll apply this to fixes for 3.13, and mark it for stable 3.12 at
> least.
>
Olof, thanks.
Please add my ack on this if you want.
- Kukjin
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-25 23:26 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-23 13:11 [PATCH] ARM: dts: Add max77686 RTC interrupt to cros5250-common Doug Anderson
2013-10-23 13:31 ` Marc Zyngier
2013-11-25 23:11 ` Doug Anderson
2013-11-25 23:16 ` Olof Johansson
2013-11-25 23:26 ` Kukjin Kim
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).