* [PATCH 0/2] s3c RTC fix for Exynos Chromebooks
@ 2014-09-17 11:50 Javier Martinez Canillas
2014-09-17 11:50 ` [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards Javier Martinez Canillas
2014-09-17 11:50 ` [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow Javier Martinez Canillas
0 siblings, 2 replies; 16+ messages in thread
From: Javier Martinez Canillas @ 2014-09-17 11:50 UTC (permalink / raw)
To: Kukjin Kim
Cc: Doug Anderson, Chanwoo Choi, Daniel Drake, linux-samsung-soc,
linux-arm-kernel, linux-kernel, Javier Martinez Canillas
Hello Kukjin,
commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC")
added an "rtc_src" DT property for the Samsung's S3C Real Time Clock
controller that specifies the 32.768 kHz clock that uses the RTC as
its source clock but this breaks the s3c RTC on Exynos Chromebooks
because the rtc device node does not define the "rtc_src" clock.
It was working before only because the source clock was not disabled
by the kernel since the CCF didn't know about it. But once the clock
is known, the CCF disables the clock and does not work so define the
"rtc_src" clock to make the s3c-rtc driver obtain it and kept enabled.
This series makes the s3c RTC to claim its source clock on the Snow
and Peach boards but other Samsung boards will have the same issue.
The following patches are meant to be for the 3.18 rc cycle once
3.18-rc1 is released.
Javier Martinez Canillas (2):
ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards
ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow
arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++-
arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 ++++-
arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 ++++-
3 files changed, 12 insertions(+), 3 deletions(-)
Best regards,
Javier
^ permalink raw reply [flat|nested] 16+ messages in thread* [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards 2014-09-17 11:50 [PATCH 0/2] s3c RTC fix for Exynos Chromebooks Javier Martinez Canillas @ 2014-09-17 11:50 ` Javier Martinez Canillas 2014-09-17 15:24 ` Doug Anderson 2014-09-22 6:41 ` Chanwoo Choi 2014-09-17 11:50 ` [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow Javier Martinez Canillas 1 sibling, 2 replies; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-17 11:50 UTC (permalink / raw) To: Kukjin Kim Cc: Doug Anderson, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel, Javier Martinez Canillas commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") added an "rtc_src" DT property for the Samsung's S3C Real Time Clock controller that specifies the 32.768 kHz clock that uses the RTC as its source clock. In the case of the Peach Pit and Pi machines, the Maxim 77802 32kHz AP clock is used as the source clock. Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> --- arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 ++++- arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 ++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts index 9a23382..bfd189e 100644 --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts @@ -12,6 +12,7 @@ #include <dt-bindings/input/input.h> #include <dt-bindings/gpio/gpio.h> #include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/clock/maxim,max77802.h> #include "exynos5420.dtsi" / { @@ -151,7 +152,7 @@ status = "okay"; clock-frequency = <400000>; - max77802-pmic@9 { + max77802: max77802-pmic@9 { compatible = "maxim,max77802"; interrupt-parent = <&gpx3>; interrupts = <1 IRQ_TYPE_NONE>; @@ -727,6 +728,8 @@ &rtc { status = "okay"; + clocks = <&clock CLK_RTC>, <&max77802 MAX77802_CLK_32K_AP>; + clock-names = "rtc", "rtc_src"; }; &spi_2 { diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts index 1d31c81..84ec1ce 100644 --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts @@ -12,6 +12,7 @@ #include <dt-bindings/input/input.h> #include <dt-bindings/gpio/gpio.h> #include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/clock/maxim,max77802.h> #include "exynos5800.dtsi" / { @@ -150,7 +151,7 @@ status = "okay"; clock-frequency = <400000>; - max77802-pmic@9 { + max77802: max77802-pmic@9 { compatible = "maxim,max77802"; interrupt-parent = <&gpx3>; interrupts = <1 IRQ_TYPE_NONE>; @@ -715,6 +716,8 @@ &rtc { status = "okay"; + clocks = <&clock CLK_RTC>, <&max77802 MAX77802_CLK_32K_AP>; + clock-names = "rtc", "rtc_src"; }; &spi_2 { -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards 2014-09-17 11:50 ` [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards Javier Martinez Canillas @ 2014-09-17 15:24 ` Doug Anderson 2014-09-22 6:41 ` Chanwoo Choi 1 sibling, 0 replies; 16+ messages in thread From: Doug Anderson @ 2014-09-17 15:24 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Kukjin Kim, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel Javier, On Wed, Sep 17, 2014 at 4:50 AM, Javier Martinez Canillas <javier.martinez@collabora.co.uk> wrote: > commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") > added an "rtc_src" DT property for the Samsung's S3C Real Time Clock > controller that specifies the 32.768 kHz clock that uses the RTC as its > source clock. In the case of the Peach Pit and Pi machines, the Maxim > 77802 32kHz AP clock is used as the source clock. > > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > --- > arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 ++++- > arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 ++++- > 2 files changed, 8 insertions(+), 2 deletions(-) Reviewed-by: Doug Anderson <dianders@chromium.org> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards 2014-09-17 11:50 ` [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards Javier Martinez Canillas 2014-09-17 15:24 ` Doug Anderson @ 2014-09-22 6:41 ` Chanwoo Choi 1 sibling, 0 replies; 16+ messages in thread From: Chanwoo Choi @ 2014-09-22 6:41 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Kukjin Kim, Doug Anderson, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel Dear Javier, On 09/17/2014 08:50 PM, Javier Martinez Canillas wrote: > commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") > added an "rtc_src" DT property for the Samsung's S3C Real Time Clock > controller that specifies the 32.768 kHz clock that uses the RTC as its > source clock. In the case of the Peach Pit and Pi machines, the Maxim > 77802 32kHz AP clock is used as the source clock. > > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > --- > arch/arm/boot/dts/exynos5420-peach-pit.dts | 5 ++++- > arch/arm/boot/dts/exynos5800-peach-pi.dts | 5 ++++- > 2 files changed, 8 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts > index 9a23382..bfd189e 100644 > --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts > +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts > @@ -12,6 +12,7 @@ > #include <dt-bindings/input/input.h> > #include <dt-bindings/gpio/gpio.h> > #include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/clock/maxim,max77802.h> > #include "exynos5420.dtsi" > > / { > @@ -151,7 +152,7 @@ > status = "okay"; > clock-frequency = <400000>; > > - max77802-pmic@9 { > + max77802: max77802-pmic@9 { > compatible = "maxim,max77802"; > interrupt-parent = <&gpx3>; > interrupts = <1 IRQ_TYPE_NONE>; > @@ -727,6 +728,8 @@ > > &rtc { > status = "okay"; > + clocks = <&clock CLK_RTC>, <&max77802 MAX77802_CLK_32K_AP>; > + clock-names = "rtc", "rtc_src"; > }; > > &spi_2 { > diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts > index 1d31c81..84ec1ce 100644 > --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts > +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts > @@ -12,6 +12,7 @@ > #include <dt-bindings/input/input.h> > #include <dt-bindings/gpio/gpio.h> > #include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/clock/maxim,max77802.h> > #include "exynos5800.dtsi" > > / { > @@ -150,7 +151,7 @@ > status = "okay"; > clock-frequency = <400000>; > > - max77802-pmic@9 { > + max77802: max77802-pmic@9 { > compatible = "maxim,max77802"; > interrupt-parent = <&gpx3>; > interrupts = <1 IRQ_TYPE_NONE>; > @@ -715,6 +716,8 @@ > > &rtc { > status = "okay"; > + clocks = <&clock CLK_RTC>, <&max77802 MAX77802_CLK_32K_AP>; > + clock-names = "rtc", "rtc_src"; > }; > > &spi_2 { > I'm so sorry for delay reply because I'm out of office for vacation (9/5 ~ 9/20). Looks goot to me for this patch. Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com> Thanks, Chanwoo Choi ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 11:50 [PATCH 0/2] s3c RTC fix for Exynos Chromebooks Javier Martinez Canillas 2014-09-17 11:50 ` [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards Javier Martinez Canillas @ 2014-09-17 11:50 ` Javier Martinez Canillas 2014-09-17 15:47 ` Doug Anderson 2014-09-22 6:44 ` Chanwoo Choi 1 sibling, 2 replies; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-17 11:50 UTC (permalink / raw) To: Kukjin Kim Cc: Doug Anderson, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel, Javier Martinez Canillas commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") added an "rtc_src" DT property for the Samsung's S3C Real Time Clock controller that specifies the 32.768 kHz clock that uses the RTC as its source clock. In the case of the Exynos5250 based Snow board, the Maxim 77686 32kHz AP clock is used as the source clock. Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> --- arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts index e51fcef..56eec56 100644 --- a/arch/arm/boot/dts/exynos5250-snow.dts +++ b/arch/arm/boot/dts/exynos5250-snow.dts @@ -10,6 +10,7 @@ /dts-v1/; #include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/clock/maxim,max77686.h> #include "exynos5250.dtsi" / { @@ -29,6 +30,8 @@ rtc@101E0000 { status = "okay"; + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; + clock-names = "rtc", "rtc_src"; }; pinctrl@11400000 { @@ -350,7 +353,7 @@ samsung,i2c-sda-delay = <100>; samsung,i2c-max-bus-freq = <378000>; - max77686@09 { + max77686: max77686@09 { compatible = "maxim,max77686"; interrupt-parent = <&gpx3>; interrupts = <2 0>; -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 11:50 ` [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow Javier Martinez Canillas @ 2014-09-17 15:47 ` Doug Anderson 2014-09-17 15:51 ` Andreas Färber 2014-09-18 6:48 ` Javier Martinez Canillas 2014-09-22 6:44 ` Chanwoo Choi 1 sibling, 2 replies; 16+ messages in thread From: Doug Anderson @ 2014-09-17 15:47 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Kukjin Kim, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel, Andreas Färber Hi, On Wed, Sep 17, 2014 at 4:50 AM, Javier Martinez Canillas <javier.martinez@collabora.co.uk> wrote: > commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") > added an "rtc_src" DT property for the Samsung's S3C Real Time Clock > controller that specifies the 32.768 kHz clock that uses the RTC as its > source clock. In the case of the Exynos5250 based Snow board, the Maxim > 77686 32kHz AP clock is used as the source clock. > > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > --- > arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > index e51fcef..56eec56 100644 > --- a/arch/arm/boot/dts/exynos5250-snow.dts > +++ b/arch/arm/boot/dts/exynos5250-snow.dts > @@ -10,6 +10,7 @@ > > /dts-v1/; > #include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/clock/maxim,max77686.h> > #include "exynos5250.dtsi" > > / { > @@ -29,6 +30,8 @@ > > rtc@101E0000 { > status = "okay"; > + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; > + clock-names = "rtc", "rtc_src"; Wait, seriously? Snow is still using the "rtc@101E0000" syntax? Whatever happened to the series that Andreas worked so hard on, including <https://patchwork.kernel.org/patch/4664801/>? Kukjin: Andreas's patch series was Reviewed long ago I think and by now I'd imagine it's got some conflicts due to it not having been applied in a timely fashion. Perhaps you could fix it up for Andreas (since he's already rebased it several times) and land it? If I had to guess, outstanding from Andreas's series (patchwork IDs listed): 4751131 New [v7] ARM: dts: Prepare node labels for exynos5250 4664801 New [v6,04/10] ARM: dts: Clean up exynos5250-snow 4664771 New [v6,05/10] ARM: dts: Fill in bootargs for exynos5250-snow 4664731 New [v6,06/10] ARM: dts: Clean up exynos5250-smdk5250 4664751 New [v6,07/10] ARM: dts: Clean up exynos5250-arndale 4664711 New [v6,08/10] ARM: dts: Fix apparent GPIO typo in exynos5250-arndale 4664681 New [v6,09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale 4664691 New [v6,10/10] ARM: dts: Add exynos5250-spring device tree > }; > > pinctrl@11400000 { > @@ -350,7 +353,7 @@ > samsung,i2c-sda-delay = <100>; > samsung,i2c-max-bus-freq = <378000>; > > - max77686@09 { > + max77686: max77686@09 { > compatible = "maxim,max77686"; > interrupt-parent = <&gpx3>; > interrupts = <2 0>; In any case, there's nothing wrong with Javier's patch other than the fact that it will eventually need to get merged with Andreas's, so: Reviewed-by: Doug Anderson <dianders@chromium.org> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 15:47 ` Doug Anderson @ 2014-09-17 15:51 ` Andreas Färber 2014-09-22 12:09 ` Kukjin Kim 2014-09-23 6:17 ` Kukjin Kim 2014-09-18 6:48 ` Javier Martinez Canillas 1 sibling, 2 replies; 16+ messages in thread From: Andreas Färber @ 2014-09-17 15:51 UTC (permalink / raw) To: Doug Anderson Cc: Javier Martinez Canillas, Kukjin Kim, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel Am 17.09.2014 um 17:47 schrieb Doug Anderson: > Hi, > > On Wed, Sep 17, 2014 at 4:50 AM, Javier Martinez Canillas > <javier.martinez@collabora.co.uk> wrote: >> commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") >> added an "rtc_src" DT property for the Samsung's S3C Real Time Clock >> controller that specifies the 32.768 kHz clock that uses the RTC as its >> source clock. In the case of the Exynos5250 based Snow board, the Maxim >> 77686 32kHz AP clock is used as the source clock. >> >> Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> >> --- >> arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> index e51fcef..56eec56 100644 >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> @@ -10,6 +10,7 @@ >> >> /dts-v1/; >> #include <dt-bindings/gpio/gpio.h> >> +#include <dt-bindings/clock/maxim,max77686.h> >> #include "exynos5250.dtsi" >> >> / { >> @@ -29,6 +30,8 @@ >> >> rtc@101E0000 { >> status = "okay"; >> + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; >> + clock-names = "rtc", "rtc_src"; > > Wait, seriously? Snow is still using the "rtc@101E0000" syntax? > Whatever happened to the series that Andreas worked so hard on, > including <https://patchwork.kernel.org/patch/4664801/>? > > Kukjin: Andreas's patch series was Reviewed long ago I think and by > now I'd imagine it's got some conflicts due to it not having been > applied in a timely fashion. Perhaps you could fix it up for Andreas > (since he's already rebased it several times) and land it? > > If I had to guess, outstanding from Andreas's series (patchwork IDs listed): > > 4751131 New [v7] ARM: dts: Prepare node labels for exynos5250 > 4664801 New [v6,04/10] ARM: dts: Clean up exynos5250-snow > 4664771 New [v6,05/10] ARM: dts: Fill in bootargs for exynos5250-snow > 4664731 New [v6,06/10] ARM: dts: Clean up exynos5250-smdk5250 > 4664751 New [v6,07/10] ARM: dts: Clean up exynos5250-arndale > 4664711 New [v6,08/10] ARM: dts: Fix apparent GPIO typo in > exynos5250-arndale > 4664681 New [v6,09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale > 4664691 New [v6,10/10] ARM: dts: Add exynos5250-spring device tree Yes, Kukjin told me not to resend anything, and I haven't heard back from him since his Kernel Summit trip. Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 15:51 ` Andreas Färber @ 2014-09-22 12:09 ` Kukjin Kim 2014-09-23 6:17 ` Kukjin Kim 1 sibling, 0 replies; 16+ messages in thread From: Kukjin Kim @ 2014-09-22 12:09 UTC (permalink / raw) To: 'Andreas Färber', 'Doug Anderson' Cc: 'Javier Martinez Canillas', 'Chanwoo Choi', 'Daniel Drake', 'linux-samsung-soc', linux-arm-kernel, linux-kernel Andreas Färber wrote: > > Am 17.09.2014 um 17:47 schrieb Doug Anderson: > > Hi, > > > > On Wed, Sep 17, 2014 at 4:50 AM, Javier Martinez Canillas > > <javier.martinez@collabora.co.uk> wrote: > >> commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") > >> added an "rtc_src" DT property for the Samsung's S3C Real Time Clock > >> controller that specifies the 32.768 kHz clock that uses the RTC as its > >> source clock. In the case of the Exynos5250 based Snow board, the Maxim > >> 77686 32kHz AP clock is used as the source clock. > >> > >> Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > >> --- > >> arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++- > >> 1 file changed, 4 insertions(+), 1 deletion(-) > >> > >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > >> index e51fcef..56eec56 100644 > >> --- a/arch/arm/boot/dts/exynos5250-snow.dts > >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts > >> @@ -10,6 +10,7 @@ > >> > >> /dts-v1/; > >> #include <dt-bindings/gpio/gpio.h> > >> +#include <dt-bindings/clock/maxim,max77686.h> > >> #include "exynos5250.dtsi" > >> > >> / { > >> @@ -29,6 +30,8 @@ > >> > >> rtc@101E0000 { > >> status = "okay"; > >> + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; > >> + clock-names = "rtc", "rtc_src"; > > > > Wait, seriously? Snow is still using the "rtc@101E0000" syntax? > > Whatever happened to the series that Andreas worked so hard on, > > including <https://patchwork.kernel.org/patch/4664801/>? > > > > Kukjin: Andreas's patch series was Reviewed long ago I think and by > > now I'd imagine it's got some conflicts due to it not having been > > applied in a timely fashion. Perhaps you could fix it up for Andreas > > (since he's already rebased it several times) and land it? > > > > If I had to guess, outstanding from Andreas's series (patchwork IDs listed): > > > > 4751131 New [v7] ARM: dts: Prepare node labels for exynos5250 > > 4664801 New [v6,04/10] ARM: dts: Clean up exynos5250-snow > > 4664771 New [v6,05/10] ARM: dts: Fill in bootargs for exynos5250-snow > > 4664731 New [v6,06/10] ARM: dts: Clean up exynos5250-smdk5250 > > 4664751 New [v6,07/10] ARM: dts: Clean up exynos5250-arndale > > 4664711 New [v6,08/10] ARM: dts: Fix apparent GPIO typo in > > exynos5250-arndale > > 4664681 New [v6,09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale > > 4664691 New [v6,10/10] ARM: dts: Add exynos5250-spring device tree > > Yes, Kukjin told me not to resend anything, and I haven't heard back > from him since his Kernel Summit trip. > Sorry about that and I thought I took them :( I'll sort them out tonight including this series together. - Kukjin ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 15:51 ` Andreas Färber 2014-09-22 12:09 ` Kukjin Kim @ 2014-09-23 6:17 ` Kukjin Kim 2014-09-23 16:00 ` Kukjin Kim 1 sibling, 1 reply; 16+ messages in thread From: Kukjin Kim @ 2014-09-23 6:17 UTC (permalink / raw) To: 'Andreas Färber', 'Doug Anderson' Cc: 'Javier Martinez Canillas', 'Chanwoo Choi', 'Daniel Drake', 'linux-samsung-soc', linux-arm-kernel, linux-kernel Kukjin Kim wrote: > > > Andreas Färber wrote: [...] > > > Kukjin: Andreas's patch series was Reviewed long ago I think and by > > > now I'd imagine it's got some conflicts due to it not having been > > > applied in a timely fashion. Perhaps you could fix it up for Andreas > > > (since he's already rebased it several times) and land it? > > > > > > If I had to guess, outstanding from Andreas's series (patchwork IDs listed): > > > > > > 4751131 New [v7] ARM: dts: Prepare node labels for exynos5250 > > > 4664801 New [v6,04/10] ARM: dts: Clean up exynos5250-snow > > > 4664771 New [v6,05/10] ARM: dts: Fill in bootargs for exynos5250-snow > > > 4664731 New [v6,06/10] ARM: dts: Clean up exynos5250-smdk5250 > > > 4664751 New [v6,07/10] ARM: dts: Clean up exynos5250-arndale > > > 4664711 New [v6,08/10] ARM: dts: Fix apparent GPIO typo in > > > exynos5250-arndale > > > 4664681 New [v6,09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale > > > 4664691 New [v6,10/10] ARM: dts: Add exynos5250-spring device tree > > > > Yes, Kukjin told me not to resend anything, and I haven't heard back > > from him since his Kernel Summit trip. > > > Sorry about that and I thought I took them :( I'll sort them out tonight > including this series together. > Hmm...since some conflicts with dp and mmc_slot related patches, I couldn't, but Andreas' patches will be handled by tonight :( - Kukjin ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-23 6:17 ` Kukjin Kim @ 2014-09-23 16:00 ` Kukjin Kim 2014-09-23 16:18 ` Javier Martinez Canillas 0 siblings, 1 reply; 16+ messages in thread From: Kukjin Kim @ 2014-09-23 16:00 UTC (permalink / raw) To: Kukjin Kim Cc: 'Andreas Färber', 'Doug Anderson', 'linux-samsung-soc', linux-kernel, 'Daniel Drake', 'Chanwoo Choi', 'Javier Martinez Canillas', linux-arm-kernel On 09/23/14 15:17, Kukjin Kim wrote: > Kukjin Kim wrote: >>> >> Andreas Färber wrote: > > [...] > >>>> Kukjin: Andreas's patch series was Reviewed long ago I think and by >>>> now I'd imagine it's got some conflicts due to it not having been >>>> applied in a timely fashion. Perhaps you could fix it up for Andreas >>>> (since he's already rebased it several times) and land it? >>>> >>>> If I had to guess, outstanding from Andreas's series (patchwork IDs listed): >>>> >>>> 4751131 New [v7] ARM: dts: Prepare node labels for exynos5250 >>>> 4664801 New [v6,04/10] ARM: dts: Clean up exynos5250-snow >>>> 4664771 New [v6,05/10] ARM: dts: Fill in bootargs for exynos5250-snow >>>> 4664731 New [v6,06/10] ARM: dts: Clean up exynos5250-smdk5250 >>>> 4664751 New [v6,07/10] ARM: dts: Clean up exynos5250-arndale >>>> 4664711 New [v6,08/10] ARM: dts: Fix apparent GPIO typo in >>>> exynos5250-arndale >>>> 4664681 New [v6,09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale >>>> 4664691 New [v6,10/10] ARM: dts: Add exynos5250-spring device tree >>> >>> Yes, Kukjin told me not to resend anything, and I haven't heard back >>> from him since his Kernel Summit trip. >>> >> Sorry about that and I thought I took them :( I'll sort them out tonight >> including this series together. >> > Hmm...since some conflicts with dp and mmc_slot related patches, I couldn't, > but Andreas' patches will be handled by tonight :( > > - Kukjin I've applied above and this series and please double-check the commits in my tree. If no problems, I will send the branch out for v3.18 soon... Thanks, Kukjin ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-23 16:00 ` Kukjin Kim @ 2014-09-23 16:18 ` Javier Martinez Canillas 2014-09-24 21:41 ` Javier Martinez Canillas 0 siblings, 1 reply; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-23 16:18 UTC (permalink / raw) To: Kukjin Kim, Kukjin Kim Cc: 'Andreas Färber', 'Doug Anderson', 'linux-samsung-soc', linux-kernel, 'Daniel Drake', 'Chanwoo Choi', linux-arm-kernel Hello Kukjin, On 09/23/2014 06:00 PM, Kukjin Kim wrote: > On 09/23/14 15:17, Kukjin Kim wrote: > > I've applied above and this series and please double-check the commits > in my tree. If no problems, I will send the branch out for v3.18 soon... > > Thanks, > Kukjin > I've looked the RTC source clock patches and look good to me. Thanks a lot for your help! Best regards, Javier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-23 16:18 ` Javier Martinez Canillas @ 2014-09-24 21:41 ` Javier Martinez Canillas 2014-09-24 23:27 ` Kukjin Kim 0 siblings, 1 reply; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-24 21:41 UTC (permalink / raw) To: Kukjin Kim, Kukjin Kim Cc: 'Andreas Färber', 'Doug Anderson', 'linux-samsung-soc', linux-kernel, 'Daniel Drake', 'Chanwoo Choi', linux-arm-kernel Hello Kukjin, On 09/23/2014 06:18 PM, Javier Martinez Canillas wrote: > On 09/23/2014 06:00 PM, Kukjin Kim wrote: >> On 09/23/14 15:17, Kukjin Kim wrote: >> >> I've applied above and this series and please double-check the commits >> in my tree. If no problems, I will send the branch out for v3.18 soon... >> >> Thanks, >> Kukjin >> > > I've looked the RTC source clock patches and look good to me. Thanks a lot for your help! > Today I also had time to review the rebased patches from Andreas and found two issues introduced during the merge conflict resolution: 1) The spi_1 dev node on exynos5250-smdk5250.dts now uses the old "cs-gpio" DT property under the controller-data instead of the standard "cs-gpios" under the SPI parent. IOW, is a regression of the bug fixed in commit e138d43 ("ARM: dts: fix the chip select gpios definition in the SPI nodes"). 2) The display-timings child node of the dp-controller was removed in commit a98c3c2 ("ARM: dts: update display related nodes for exynos5250-snow") but was introduced again in the Snow DTS during the merge. I'll post a series that fixes both issues with the following patches: Javier Martinez Canillas (2): ARM: dts: Fix chip select GPIO on smdk5250 ARM: dts: Remove display timings node from exynos5250-snow I see that you already send a pull request to arm-soc maintainers so probably these will be part of the fixes during the 3.18 rc cycle. Best regards, Javier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-24 21:41 ` Javier Martinez Canillas @ 2014-09-24 23:27 ` Kukjin Kim 2014-09-25 7:18 ` Javier Martinez Canillas 0 siblings, 1 reply; 16+ messages in thread From: Kukjin Kim @ 2014-09-24 23:27 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Kukjin Kim, Kukjin Kim, 'Doug Anderson', 'linux-samsung-soc', linux-kernel, 'Daniel Drake', 'Chanwoo Choi', 'Andreas Färber', linux-arm-kernel On 09/25/14 06:41, Javier Martinez Canillas wrote: > Hello Kukjin, > Hi, > On 09/23/2014 06:18 PM, Javier Martinez Canillas wrote: >> On 09/23/2014 06:00 PM, Kukjin Kim wrote: >>> On 09/23/14 15:17, Kukjin Kim wrote: >>> >>> I've applied above and this series and please double-check the commits >>> in my tree. If no problems, I will send the branch out for v3.18 soon... >>> >>> Thanks, >>> Kukjin >>> >> >> I've looked the RTC source clock patches and look good to me. Thanks a lot for your help! >> > > Today I also had time to review the rebased patches from Andreas and found > two issues introduced during the merge conflict resolution: > Oops. Thanks for your quick fix it up. > 1) The spi_1 dev node on exynos5250-smdk5250.dts now uses the old "cs-gpio" > DT property under the controller-data instead of the standard "cs-gpios" > under the SPI parent. IOW, is a regression of the bug fixed in commit > e138d43 ("ARM: dts: fix the chip select gpios definition in the SPI nodes"). > > 2) The display-timings child node of the dp-controller was removed in commit > a98c3c2 ("ARM: dts: update display related nodes for exynos5250-snow") but > was introduced again in the Snow DTS during the merge. > > I'll post a series that fixes both issues with the following patches: > I've applied them. > Javier Martinez Canillas (2): > ARM: dts: Fix chip select GPIO on smdk5250 > ARM: dts: Remove display timings node from exynos5250-snow > > I see that you already send a pull request to arm-soc maintainers so probably > these will be part of the fixes during the 3.18 rc cycle. > I'll queue them in my third pull-request maybe today :) Thanks, Kukjin ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-24 23:27 ` Kukjin Kim @ 2014-09-25 7:18 ` Javier Martinez Canillas 0 siblings, 0 replies; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-25 7:18 UTC (permalink / raw) To: Kukjin Kim Cc: Kukjin Kim, 'Doug Anderson', 'linux-samsung-soc', linux-kernel, 'Daniel Drake', 'Chanwoo Choi', 'Andreas Färber', linux-arm-kernel Hello Kukjin, On 09/25/2014 01:27 AM, Kukjin Kim wrote: >> >> Today I also had time to review the rebased patches from Andreas and found >> two issues introduced during the merge conflict resolution: >> > Oops. Thanks for your quick fix it up. > You are welcome. >> 1) The spi_1 dev node on exynos5250-smdk5250.dts now uses the old "cs-gpio" >> DT property under the controller-data instead of the standard "cs-gpios" >> under the SPI parent. IOW, is a regression of the bug fixed in commit >> e138d43 ("ARM: dts: fix the chip select gpios definition in the SPI nodes"). >> >> 2) The display-timings child node of the dp-controller was removed in commit >> a98c3c2 ("ARM: dts: update display related nodes for exynos5250-snow") but >> was introduced again in the Snow DTS during the merge. >> >> I'll post a series that fixes both issues with the following patches: >> > I've applied them. > >> Javier Martinez Canillas (2): >> ARM: dts: Fix chip select GPIO on smdk5250 >> ARM: dts: Remove display timings node from exynos5250-snow >> >> I see that you already send a pull request to arm-soc maintainers so probably >> these will be part of the fixes during the 3.18 rc cycle. >> > I'll queue them in my third pull-request maybe today :) > Great. It would be awesome also if you can also pick Sjoerd's USB series [0] which solves an issue with USB on certain configurations. I looked at your for-next branch but I didn't see those patches there. > Thanks, > Kukjin > Thanks a lot and best regards, Javier [0]: https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg37045.html ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 15:47 ` Doug Anderson 2014-09-17 15:51 ` Andreas Färber @ 2014-09-18 6:48 ` Javier Martinez Canillas 1 sibling, 0 replies; 16+ messages in thread From: Javier Martinez Canillas @ 2014-09-18 6:48 UTC (permalink / raw) To: Doug Anderson Cc: Kukjin Kim, Chanwoo Choi, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel, Andreas Färber Hello Doug, Andreas, On 09/17/2014 05:47 PM, Doug Anderson wrote: >> >> rtc@101E0000 { >> status = "okay"; >> + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; >> + clock-names = "rtc", "rtc_src"; > > Wait, seriously? Snow is still using the "rtc@101E0000" syntax? > Whatever happened to the series that Andreas worked so hard on, > including <https://patchwork.kernel.org/patch/4664801/>? > Andreas, I completely forgot about your series when writing this patch even though I've on my TO-DO to sync the downstream and upstream Snow DTS once your patches land. Sorry about that. > > In any case, there's nothing wrong with Javier's patch other than the > fact that it will eventually need to get merged with Andreas's, so: > > Reviewed-by: Doug Anderson <dianders@chromium.org> > Thanks, I'll re-spin and post again once Kukjin merges Andreas series. Best regards, Javier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow 2014-09-17 11:50 ` [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow Javier Martinez Canillas 2014-09-17 15:47 ` Doug Anderson @ 2014-09-22 6:44 ` Chanwoo Choi 1 sibling, 0 replies; 16+ messages in thread From: Chanwoo Choi @ 2014-09-22 6:44 UTC (permalink / raw) To: Javier Martinez Canillas Cc: Kukjin Kim, Doug Anderson, Daniel Drake, linux-samsung-soc, linux-arm-kernel, linux-kernel Dear Javier, On 09/17/2014 08:50 PM, Javier Martinez Canillas wrote: > commit 546b117fdf17 ("rtc: s3c: add support for RTC of Exynos3250 SoC") > added an "rtc_src" DT property for the Samsung's S3C Real Time Clock > controller that specifies the 32.768 kHz clock that uses the RTC as its > source clock. In the case of the Exynos5250 based Snow board, the Maxim > 77686 32kHz AP clock is used as the source clock. > > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk> > --- > arch/arm/boot/dts/exynos5250-snow.dts | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > index e51fcef..56eec56 100644 > --- a/arch/arm/boot/dts/exynos5250-snow.dts > +++ b/arch/arm/boot/dts/exynos5250-snow.dts > @@ -10,6 +10,7 @@ > > /dts-v1/; > #include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/clock/maxim,max77686.h> > #include "exynos5250.dtsi" > > / { > @@ -29,6 +30,8 @@ > > rtc@101E0000 { > status = "okay"; > + clocks = <&clock CLK_RTC>, <&max77686 MAX77686_CLK_AP>; > + clock-names = "rtc", "rtc_src"; > }; > > pinctrl@11400000 { > @@ -350,7 +353,7 @@ > samsung,i2c-sda-delay = <100>; > samsung,i2c-max-bus-freq = <378000>; > > - max77686@09 { > + max77686: max77686@09 { > compatible = "maxim,max77686"; > interrupt-parent = <&gpx3>; > interrupts = <2 0>; > Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com> Thanks, Chanwoo Choi ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2014-09-25 7:18 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-09-17 11:50 [PATCH 0/2] s3c RTC fix for Exynos Chromebooks Javier Martinez Canillas 2014-09-17 11:50 ` [PATCH 1/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos Peach boards Javier Martinez Canillas 2014-09-17 15:24 ` Doug Anderson 2014-09-22 6:41 ` Chanwoo Choi 2014-09-17 11:50 ` [PATCH 2/2] ARM: dts: Add rtc_src clk for s3c-rtc on exynos5250-snow Javier Martinez Canillas 2014-09-17 15:47 ` Doug Anderson 2014-09-17 15:51 ` Andreas Färber 2014-09-22 12:09 ` Kukjin Kim 2014-09-23 6:17 ` Kukjin Kim 2014-09-23 16:00 ` Kukjin Kim 2014-09-23 16:18 ` Javier Martinez Canillas 2014-09-24 21:41 ` Javier Martinez Canillas 2014-09-24 23:27 ` Kukjin Kim 2014-09-25 7:18 ` Javier Martinez Canillas 2014-09-18 6:48 ` Javier Martinez Canillas 2014-09-22 6:44 ` Chanwoo Choi
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).