* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles [not found] ` <52EFC573.2050302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> @ 2014-02-03 19:45 ` Maxime Ripard 2014-02-03 20:11 ` Rob Herring 2014-02-03 20:17 ` Daniel Lezcano 0 siblings, 2 replies; 6+ messages in thread From: Maxime Ripard @ 2014-02-03 19:45 UTC (permalink / raw) To: Daniel Lezcano Cc: tglx-hfZtesqFncYOwBW4kG4KsQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 3170 bytes --] Hi Daniel, (Adding DT mailing-list in CC) On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: > On 02/02/2014 02:37 PM, Maxime Ripard wrote: > >The Allwinner A10 compatibles were following a slightly different compatible > >patterns than the rest of the SoCs for historical reasons. Add compatibles > >matching the other pattern to the timer driver for consistency, and keep the > >older one for backward compatibility. > > Hi Maxime, > > is it really needed to keep the old pattern ? We agreed during the ARM Kernel Summit to consider the DT as a stable ABI. While I'd be ok with removing the older ones, that also means that we would break the boot of newer kernels with older DT, so yes, we actually need to keep the old compatibles. > >Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> > >--- > > Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt | 5 +++-- > > drivers/clocksource/sun4i_timer.c | 4 ++++ > > 2 files changed, 7 insertions(+), 2 deletions(-) > > > >diff --git a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt > >index 48aeb78..d9e35ae 100644 > >--- a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt > >+++ b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt > >@@ -2,7 +2,8 @@ Allwinner A1X SoCs Timer Controller > > > > Required properties: > > > >-- compatible : should be "allwinner,sun4i-timer" > >+- compatible : should be "allwinner,sun4i-a10-timer" > >+ (Deprecated "allwinner,sun4i-timer") > > - reg : Specifies base physical address and size of the registers. > > - interrupts : The interrupt of the first timer > > - clocks: phandle to the source clock (usually a 24 MHz fixed clock) > >@@ -10,7 +11,7 @@ Required properties: > > Example: > > > > timer { > >- compatible = "allwinner,sun4i-timer"; > >+ compatible = "allwinner,sun4i-a10-timer"; > > reg = <0x01c20c00 0x400>; > > interrupts = <22>; > > clocks = <&osc>; > >diff --git a/drivers/clocksource/sun4i_timer.c b/drivers/clocksource/sun4i_timer.c > >index bf497af..de03895 100644 > >--- a/drivers/clocksource/sun4i_timer.c > >+++ b/drivers/clocksource/sun4i_timer.c > >@@ -196,5 +196,9 @@ static void __init sun4i_timer_init(struct device_node *node) > > clockevents_config_and_register(&sun4i_clockevent, rate, > > TIMER_SYNC_TICKS, 0xffffffff); > > } > >+CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-a10-timer", > >+ sun4i_timer_init); > >+ > >+/* Deprecated */ > > CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-timer", > > sun4i_timer_init); > > > > > -- > <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs > > Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | > <http://twitter.com/#!/linaroorg> Twitter | > <http://www.linaro.org/linaro-blog/> Blog > -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles 2014-02-03 19:45 ` [PATCH 1/2] clocksource: sunxi: Add new compatibles Maxime Ripard @ 2014-02-03 20:11 ` Rob Herring [not found] ` <CAL_JsqJuSS1Hh8Fb26PMKBB_uKmLuiqnZJjz0zvt1Ux7P8cCzw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-02-03 20:17 ` Daniel Lezcano 1 sibling, 1 reply; 6+ messages in thread From: Rob Herring @ 2014-02-03 20:11 UTC (permalink / raw) To: Maxime Ripard Cc: Daniel Lezcano, Thomas Gleixner, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On Mon, Feb 3, 2014 at 1:45 PM, Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote: > Hi Daniel, > > (Adding DT mailing-list in CC) > > On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: >> On 02/02/2014 02:37 PM, Maxime Ripard wrote: >> >The Allwinner A10 compatibles were following a slightly different compatible >> >patterns than the rest of the SoCs for historical reasons. Add compatibles >> >matching the other pattern to the timer driver for consistency, and keep the >> >older one for backward compatibility. >> >> Hi Maxime, >> >> is it really needed to keep the old pattern ? > > We agreed during the ARM Kernel Summit to consider the DT as a stable > ABI. > > While I'd be ok with removing the older ones, that also means that we > would break the boot of newer kernels with older DT, so yes, we > actually need to keep the old compatibles. It all depends if that would really cause problems for a given platform. So if Allwinner DT support is a moving target, then changing is probably okay. For example, if anyone using the platform is going to need to update their DTB to add more nodes to get various features anyway, then breaking it is not all that important. Rob -- 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CAL_JsqJuSS1Hh8Fb26PMKBB_uKmLuiqnZJjz0zvt1Ux7P8cCzw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles [not found] ` <CAL_JsqJuSS1Hh8Fb26PMKBB_uKmLuiqnZJjz0zvt1Ux7P8cCzw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2014-02-04 8:41 ` Maxime Ripard 0 siblings, 0 replies; 6+ messages in thread From: Maxime Ripard @ 2014-02-04 8:41 UTC (permalink / raw) To: Rob Herring Cc: Daniel Lezcano, Thomas Gleixner, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [-- Attachment #1: Type: text/plain, Size: 1627 bytes --] Hi Rob, On Mon, Feb 03, 2014 at 02:11:28PM -0600, Rob Herring wrote: > On Mon, Feb 3, 2014 at 1:45 PM, Maxime Ripard > <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote: > > Hi Daniel, > > > > (Adding DT mailing-list in CC) > > > > On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: > >> On 02/02/2014 02:37 PM, Maxime Ripard wrote: > >> >The Allwinner A10 compatibles were following a slightly different compatible > >> >patterns than the rest of the SoCs for historical reasons. Add compatibles > >> >matching the other pattern to the timer driver for consistency, and keep the > >> >older one for backward compatibility. > >> > >> Hi Maxime, > >> > >> is it really needed to keep the old pattern ? > > > > We agreed during the ARM Kernel Summit to consider the DT as a stable > > ABI. > > > > While I'd be ok with removing the older ones, that also means that we > > would break the boot of newer kernels with older DT, so yes, we > > actually need to keep the old compatibles. > > It all depends if that would really cause problems for a given > platform. So if Allwinner DT support is a moving target, then changing > is probably okay. For example, if anyone using the platform is going > to need to update their DTB to add more nodes to get various features > anyway, then breaking it is not all that important. We keep adding new stuff to the DT, yes, so I guess we can be considered a moving target. Thanks for your input! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles 2014-02-03 19:45 ` [PATCH 1/2] clocksource: sunxi: Add new compatibles Maxime Ripard 2014-02-03 20:11 ` Rob Herring @ 2014-02-03 20:17 ` Daniel Lezcano [not found] ` <52EFF96A.1020302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 1 sibling, 1 reply; 6+ messages in thread From: Daniel Lezcano @ 2014-02-03 20:17 UTC (permalink / raw) To: Maxime Ripard Cc: tglx-hfZtesqFncYOwBW4kG4KsQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA On 02/03/2014 08:45 PM, Maxime Ripard wrote: > Hi Daniel, > > (Adding DT mailing-list in CC) > > On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: >> On 02/02/2014 02:37 PM, Maxime Ripard wrote: >>> The Allwinner A10 compatibles were following a slightly different compatible >>> patterns than the rest of the SoCs for historical reasons. Add compatibles >>> matching the other pattern to the timer driver for consistency, and keep the >>> older one for backward compatibility. >> >> Hi Maxime, >> >> is it really needed to keep the old pattern ? > > We agreed during the ARM Kernel Summit to consider the DT as a stable > ABI. > > While I'd be ok with removing the older ones, that also means that we > would break the boot of newer kernels with older DT, so yes, we > actually need to keep the old compatibles. Thanks for the clarification. So these old compatibles will stay there 'ad vitam aeternam', right ? >>> Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> >>> --- >>> Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt | 5 +++-- >>> drivers/clocksource/sun4i_timer.c | 4 ++++ >>> 2 files changed, 7 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>> index 48aeb78..d9e35ae 100644 >>> --- a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>> +++ b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>> @@ -2,7 +2,8 @@ Allwinner A1X SoCs Timer Controller >>> >>> Required properties: >>> >>> -- compatible : should be "allwinner,sun4i-timer" >>> +- compatible : should be "allwinner,sun4i-a10-timer" >>> + (Deprecated "allwinner,sun4i-timer") >>> - reg : Specifies base physical address and size of the registers. >>> - interrupts : The interrupt of the first timer >>> - clocks: phandle to the source clock (usually a 24 MHz fixed clock) >>> @@ -10,7 +11,7 @@ Required properties: >>> Example: >>> >>> timer { >>> - compatible = "allwinner,sun4i-timer"; >>> + compatible = "allwinner,sun4i-a10-timer"; >>> reg = <0x01c20c00 0x400>; >>> interrupts = <22>; >>> clocks = <&osc>; >>> diff --git a/drivers/clocksource/sun4i_timer.c b/drivers/clocksource/sun4i_timer.c >>> index bf497af..de03895 100644 >>> --- a/drivers/clocksource/sun4i_timer.c >>> +++ b/drivers/clocksource/sun4i_timer.c >>> @@ -196,5 +196,9 @@ static void __init sun4i_timer_init(struct device_node *node) >>> clockevents_config_and_register(&sun4i_clockevent, rate, >>> TIMER_SYNC_TICKS, 0xffffffff); >>> } >>> +CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-a10-timer", >>> + sun4i_timer_init); >>> + >>> +/* Deprecated */ >>> CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-timer", >>> sun4i_timer_init); >>> >> >> >> -- >> <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs >> >> Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | >> <http://twitter.com/#!/linaroorg> Twitter | >> <http://www.linaro.org/linaro-blog/> Blog >> > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog -- 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <52EFF96A.1020302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles [not found] ` <52EFF96A.1020302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> @ 2014-02-04 8:13 ` Gregory CLEMENT 2014-02-04 8:45 ` Maxime Ripard 1 sibling, 0 replies; 6+ messages in thread From: Gregory CLEMENT @ 2014-02-04 8:13 UTC (permalink / raw) To: Daniel Lezcano Cc: Maxime Ripard, devicetree-u79uwXL29TY76Z2rM5mHXA, tglx-hfZtesqFncYOwBW4kG4KsQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Daniel, On 03/02/2014 21:17, Daniel Lezcano wrote: > On 02/03/2014 08:45 PM, Maxime Ripard wrote: >> Hi Daniel, >> >> (Adding DT mailing-list in CC) >> >> On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: >>> On 02/02/2014 02:37 PM, Maxime Ripard wrote: >>>> The Allwinner A10 compatibles were following a slightly different compatible >>>> patterns than the rest of the SoCs for historical reasons. Add compatibles >>>> matching the other pattern to the timer driver for consistency, and keep the >>>> older one for backward compatibility. >>> >>> Hi Maxime, >>> >>> is it really needed to keep the old pattern ? >> >> We agreed during the ARM Kernel Summit to consider the DT as a stable >> ABI. >> >> While I'd be ok with removing the older ones, that also means that we >> would break the boot of newer kernels with older DT, so yes, we >> actually need to keep the old compatibles. > > Thanks for the clarification. > > So these old compatibles will stay there 'ad vitam aeternam', right ? From what I have understood during the ARM Kernel Summit, it was acceptable to remove them after a few release. Gregory > > >>>> Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> >>>> --- >>>> Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt | 5 +++-- >>>> drivers/clocksource/sun4i_timer.c | 4 ++++ >>>> 2 files changed, 7 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>>> index 48aeb78..d9e35ae 100644 >>>> --- a/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>>> +++ b/Documentation/devicetree/bindings/timer/allwinner,sun4i-timer.txt >>>> @@ -2,7 +2,8 @@ Allwinner A1X SoCs Timer Controller >>>> >>>> Required properties: >>>> >>>> -- compatible : should be "allwinner,sun4i-timer" >>>> +- compatible : should be "allwinner,sun4i-a10-timer" >>>> + (Deprecated "allwinner,sun4i-timer") >>>> - reg : Specifies base physical address and size of the registers. >>>> - interrupts : The interrupt of the first timer >>>> - clocks: phandle to the source clock (usually a 24 MHz fixed clock) >>>> @@ -10,7 +11,7 @@ Required properties: >>>> Example: >>>> >>>> timer { >>>> - compatible = "allwinner,sun4i-timer"; >>>> + compatible = "allwinner,sun4i-a10-timer"; >>>> reg = <0x01c20c00 0x400>; >>>> interrupts = <22>; >>>> clocks = <&osc>; >>>> diff --git a/drivers/clocksource/sun4i_timer.c b/drivers/clocksource/sun4i_timer.c >>>> index bf497af..de03895 100644 >>>> --- a/drivers/clocksource/sun4i_timer.c >>>> +++ b/drivers/clocksource/sun4i_timer.c >>>> @@ -196,5 +196,9 @@ static void __init sun4i_timer_init(struct device_node *node) >>>> clockevents_config_and_register(&sun4i_clockevent, rate, >>>> TIMER_SYNC_TICKS, 0xffffffff); >>>> } >>>> +CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-a10-timer", >>>> + sun4i_timer_init); >>>> + >>>> +/* Deprecated */ >>>> CLOCKSOURCE_OF_DECLARE(sun4i, "allwinner,sun4i-timer", >>>> sun4i_timer_init); >>>> >>> >>> >>> -- >>> <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs >>> >>> Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | >>> <http://twitter.com/#!/linaroorg> Twitter | >>> <http://www.linaro.org/linaro-blog/> Blog >>> >> > > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] clocksource: sunxi: Add new compatibles [not found] ` <52EFF96A.1020302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2014-02-04 8:13 ` Gregory CLEMENT @ 2014-02-04 8:45 ` Maxime Ripard 1 sibling, 0 replies; 6+ messages in thread From: Maxime Ripard @ 2014-02-04 8:45 UTC (permalink / raw) To: Daniel Lezcano Cc: tglx-hfZtesqFncYOwBW4kG4KsQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 1317 bytes --] Hi, On Mon, Feb 03, 2014 at 09:17:46PM +0100, Daniel Lezcano wrote: > On 02/03/2014 08:45 PM, Maxime Ripard wrote: > >Hi Daniel, > > > >(Adding DT mailing-list in CC) > > > >On Mon, Feb 03, 2014 at 05:36:03PM +0100, Daniel Lezcano wrote: > >>On 02/02/2014 02:37 PM, Maxime Ripard wrote: > >>>The Allwinner A10 compatibles were following a slightly different compatible > >>>patterns than the rest of the SoCs for historical reasons. Add compatibles > >>>matching the other pattern to the timer driver for consistency, and keep the > >>>older one for backward compatibility. > >> > >>Hi Maxime, > >> > >>is it really needed to keep the old pattern ? > > > >We agreed during the ARM Kernel Summit to consider the DT as a stable > >ABI. > > > >While I'd be ok with removing the older ones, that also means that we > >would break the boot of newer kernels with older DT, so yes, we > >actually need to keep the old compatibles. > > Thanks for the clarification. > > So these old compatibles will stay there 'ad vitam aeternam', right ? Except for what Rob told, yep, that was my feeling, but Gregory and I seem to have a different interpretation of this rule :) Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-02-04 8:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1391348280-9484-1-git-send-email-maxime.ripard@free-electrons.com>
[not found] ` <52EFC573.2050302@linaro.org>
[not found] ` <52EFC573.2050302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2014-02-03 19:45 ` [PATCH 1/2] clocksource: sunxi: Add new compatibles Maxime Ripard
2014-02-03 20:11 ` Rob Herring
[not found] ` <CAL_JsqJuSS1Hh8Fb26PMKBB_uKmLuiqnZJjz0zvt1Ux7P8cCzw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-02-04 8:41 ` Maxime Ripard
2014-02-03 20:17 ` Daniel Lezcano
[not found] ` <52EFF96A.1020302-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2014-02-04 8:13 ` Gregory CLEMENT
2014-02-04 8:45 ` Maxime Ripard
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).