* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-05 16:36 ` Nicolas Ferre
0 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-05 16:36 UTC (permalink / raw)
To: Arnd Bergmann, Olof Johansson, arm
Cc: Nicolas Ferre, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
Arnd, Olof, Kevin,
This pull-request is focused on the work that Maxime did for migrating our timer
(PIT) to the clocksource sub-system. A big cleanup happened which allows us to
be even closer to the point when we have only the bare minimum in our formerly
crowded mach-at91 directory.
This pull-request goes on top of the "drivers" one already sent to you on Sept.
01st.
Thanks, best regards,
The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
are available in the git repository at:
git://github.com/at91linux/linux-at91.git tags/at91-drivers2
for you to fetch changes up to 5c1406f573f0f13b85c738a21030aadc9bb36bdd:
ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-03 10:55:46 +0200)
----------------------------------------------------------------
Second drivers series for AT91/3.18:
- move of the PIT (basic timer) from mach-at91 to its
proper location: drivers/clocksource
- big cleanup of this driver along the way
----------------------------------------------------------------
Maxime Ripard (14):
ARM: at91: PIT: Follow the general coding rules
ARM: at91: generic.h: Add include safe guards
ARM: at91: PIT: Use DIV_ROUND_CLOSEST to compute the cycles
ARM: at91: PIT: Use of_have_populated_dt instead of CONFIG_OF
ARM: at91: PIT: Rework probe functions
ARM: at91: dt: Remove init_time definitions
ARM: at91: PIT: Use consistent exit path in probe
ARM: at91: PIT: Use pr_fmt
ARM: at91: PIT: use request_irq instead of setup_irq
ARM: at91: PIT: (Almost) remove the global variables
ARM: at91: soc: Add init_time callback
ARM: at91: Convert the boards to the init_time callback
ARM: at91: PIT: Convert to an early_platform_device
ARM: at91: PIT: Move the driver to drivers/clocksource
arch/arm/mach-at91/Kconfig | 5 -
arch/arm/mach-at91/Makefile | 1 -
arch/arm/mach-at91/at91sam9260.c | 34 +++-
arch/arm/mach-at91/at91sam9261.c | 34 +++-
arch/arm/mach-at91/at91sam9263.c | 34 +++-
arch/arm/mach-at91/at91sam926x_time.c | 294 ------------------------------
arch/arm/mach-at91/at91sam9g45.c | 34 +++-
arch/arm/mach-at91/at91sam9rl.c | 34 +++-
arch/arm/mach-at91/board-afeb-9260v1.c | 2 +-
arch/arm/mach-at91/board-cam60.c | 2 +-
arch/arm/mach-at91/board-cpu9krea.c | 2 +-
arch/arm/mach-at91/board-dt-sam9.c | 10 --
arch/arm/mach-at91/board-dt-sama5.c | 9 -
arch/arm/mach-at91/board-flexibity.c | 2 +-
arch/arm/mach-at91/board-foxg20.c | 2 +-
arch/arm/mach-at91/board-gsia18s.c | 2 +-
arch/arm/mach-at91/board-pcontrol-g20.c | 2 +-
arch/arm/mach-at91/board-sam9-l9260.c | 2 +-
arch/arm/mach-at91/board-sam9260ek.c | 2 +-
arch/arm/mach-at91/board-sam9261ek.c | 4 +-
arch/arm/mach-at91/board-sam9263ek.c | 2 +-
arch/arm/mach-at91/board-sam9g20ek.c | 4 +-
arch/arm/mach-at91/board-sam9m10g45ek.c | 2 +-
arch/arm/mach-at91/board-sam9rlek.c | 2 +-
arch/arm/mach-at91/board-snapper9260.c | 2 +-
arch/arm/mach-at91/board-stamp9g20.c | 4 +-
arch/arm/mach-at91/generic.h | 8 +-
arch/arm/mach-at91/setup.c | 5 +
arch/arm/mach-at91/soc.h | 1 +
drivers/clocksource/Kconfig | 4 +
drivers/clocksource/Makefile | 1 +
drivers/clocksource/timer-atmel-pit.c | 304 ++++++++++++++++++++++++++++++++
32 files changed, 505 insertions(+), 345 deletions(-)
delete mode 100644 arch/arm/mach-at91/at91sam926x_time.c
create mode 100644 drivers/clocksource/timer-atmel-pit.c
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-05 16:36 ` Nicolas Ferre
0 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-05 16:36 UTC (permalink / raw)
To: linux-arm-kernel
Arnd, Olof, Kevin,
This pull-request is focused on the work that Maxime did for migrating our timer
(PIT) to the clocksource sub-system. A big cleanup happened which allows us to
be even closer to the point when we have only the bare minimum in our formerly
crowded mach-at91 directory.
This pull-request goes on top of the "drivers" one already sent to you on Sept.
01st.
Thanks, best regards,
The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
are available in the git repository at:
git://github.com/at91linux/linux-at91.git tags/at91-drivers2
for you to fetch changes up to 5c1406f573f0f13b85c738a21030aadc9bb36bdd:
ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-03 10:55:46 +0200)
----------------------------------------------------------------
Second drivers series for AT91/3.18:
- move of the PIT (basic timer) from mach-at91 to its
proper location: drivers/clocksource
- big cleanup of this driver along the way
----------------------------------------------------------------
Maxime Ripard (14):
ARM: at91: PIT: Follow the general coding rules
ARM: at91: generic.h: Add include safe guards
ARM: at91: PIT: Use DIV_ROUND_CLOSEST to compute the cycles
ARM: at91: PIT: Use of_have_populated_dt instead of CONFIG_OF
ARM: at91: PIT: Rework probe functions
ARM: at91: dt: Remove init_time definitions
ARM: at91: PIT: Use consistent exit path in probe
ARM: at91: PIT: Use pr_fmt
ARM: at91: PIT: use request_irq instead of setup_irq
ARM: at91: PIT: (Almost) remove the global variables
ARM: at91: soc: Add init_time callback
ARM: at91: Convert the boards to the init_time callback
ARM: at91: PIT: Convert to an early_platform_device
ARM: at91: PIT: Move the driver to drivers/clocksource
arch/arm/mach-at91/Kconfig | 5 -
arch/arm/mach-at91/Makefile | 1 -
arch/arm/mach-at91/at91sam9260.c | 34 +++-
arch/arm/mach-at91/at91sam9261.c | 34 +++-
arch/arm/mach-at91/at91sam9263.c | 34 +++-
arch/arm/mach-at91/at91sam926x_time.c | 294 ------------------------------
arch/arm/mach-at91/at91sam9g45.c | 34 +++-
arch/arm/mach-at91/at91sam9rl.c | 34 +++-
arch/arm/mach-at91/board-afeb-9260v1.c | 2 +-
arch/arm/mach-at91/board-cam60.c | 2 +-
arch/arm/mach-at91/board-cpu9krea.c | 2 +-
arch/arm/mach-at91/board-dt-sam9.c | 10 --
arch/arm/mach-at91/board-dt-sama5.c | 9 -
arch/arm/mach-at91/board-flexibity.c | 2 +-
arch/arm/mach-at91/board-foxg20.c | 2 +-
arch/arm/mach-at91/board-gsia18s.c | 2 +-
arch/arm/mach-at91/board-pcontrol-g20.c | 2 +-
arch/arm/mach-at91/board-sam9-l9260.c | 2 +-
arch/arm/mach-at91/board-sam9260ek.c | 2 +-
arch/arm/mach-at91/board-sam9261ek.c | 4 +-
arch/arm/mach-at91/board-sam9263ek.c | 2 +-
arch/arm/mach-at91/board-sam9g20ek.c | 4 +-
arch/arm/mach-at91/board-sam9m10g45ek.c | 2 +-
arch/arm/mach-at91/board-sam9rlek.c | 2 +-
arch/arm/mach-at91/board-snapper9260.c | 2 +-
arch/arm/mach-at91/board-stamp9g20.c | 4 +-
arch/arm/mach-at91/generic.h | 8 +-
arch/arm/mach-at91/setup.c | 5 +
arch/arm/mach-at91/soc.h | 1 +
drivers/clocksource/Kconfig | 4 +
drivers/clocksource/Makefile | 1 +
drivers/clocksource/timer-atmel-pit.c | 304 ++++++++++++++++++++++++++++++++
32 files changed, 505 insertions(+), 345 deletions(-)
delete mode 100644 arch/arm/mach-at91/at91sam926x_time.c
create mode 100644 drivers/clocksource/timer-atmel-pit.c
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-05 16:36 ` Nicolas Ferre
@ 2014-09-05 21:25 ` Arnd Bergmann
-1 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-05 21:25 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Olof Johansson, arm, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Friday 05 September 2014, Nicolas Ferre wrote:
> Arnd, Olof, Kevin,
>
> This pull-request is focused on the work that Maxime did for migrating our timer
> (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> be even closer to the point when we have only the bare minimum in our formerly
> crowded mach-at91 directory.
>
> This pull-request goes on top of the "drivers" one already sent to you on Sept.
> 01st.
Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
we could kill that off in the long run. This is only used for the legacy
board files, not for DT, right?
Do you have a timeline for getting rid of the board files completely?
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-05 21:25 ` Arnd Bergmann
0 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-05 21:25 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 05 September 2014, Nicolas Ferre wrote:
> Arnd, Olof, Kevin,
>
> This pull-request is focused on the work that Maxime did for migrating our timer
> (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> be even closer to the point when we have only the bare minimum in our formerly
> crowded mach-at91 directory.
>
> This pull-request goes on top of the "drivers" one already sent to you on Sept.
> 01st.
Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
we could kill that off in the long run. This is only used for the legacy
board files, not for DT, right?
Do you have a timeline for getting rid of the board files completely?
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-05 21:25 ` Arnd Bergmann
@ 2014-09-05 21:39 ` Arnd Bergmann
-1 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-05 21:39 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Olof Johansson, arm, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Friday 05 September 2014 23:25:11 Arnd Bergmann wrote:
> On Friday 05 September 2014, Nicolas Ferre wrote:
> > Arnd, Olof, Kevin,
> >
> > This pull-request is focused on the work that Maxime did for migrating our timer
> > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > be even closer to the point when we have only the bare minimum in our formerly
> > crowded mach-at91 directory.
> >
> > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > 01st.
>
> Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> we could kill that off in the long run. This is only used for the legacy
> board files, not for DT, right?
>
> Do you have a timeline for getting rid of the board files completely?
>
I've looked more closely, and I think a good way to solve this would be
to revert or remove "ARM: at91: PIT: Convert to an early_platform_device",
and add declarations for the two global functions to
include/clocksource/at91.h and passing the interrupt number as an argument
to at91sam926x_pit_init.
I understand it's not the goal you had in mind here, but it's far simpler
and it gives me hope that we can eventually kill that early platform
code.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-05 21:39 ` Arnd Bergmann
0 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-05 21:39 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 05 September 2014 23:25:11 Arnd Bergmann wrote:
> On Friday 05 September 2014, Nicolas Ferre wrote:
> > Arnd, Olof, Kevin,
> >
> > This pull-request is focused on the work that Maxime did for migrating our timer
> > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > be even closer to the point when we have only the bare minimum in our formerly
> > crowded mach-at91 directory.
> >
> > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > 01st.
>
> Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> we could kill that off in the long run. This is only used for the legacy
> board files, not for DT, right?
>
> Do you have a timeline for getting rid of the board files completely?
>
I've looked more closely, and I think a good way to solve this would be
to revert or remove "ARM: at91: PIT: Convert to an early_platform_device",
and add declarations for the two global functions to
include/clocksource/at91.h and passing the interrupt number as an argument
to at91sam926x_pit_init.
I understand it's not the goal you had in mind here, but it's far simpler
and it gives me hope that we can eventually kill that early platform
code.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-05 21:25 ` Arnd Bergmann
@ 2014-09-08 9:26 ` Maxime Ripard
-1 siblings, 0 replies; 22+ messages in thread
From: Maxime Ripard @ 2014-09-08 9:26 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Nicolas Ferre, Olof Johansson, arm, Linux Kernel list,
linux-arm-kernel, Alexandre Belloni, Boris BREZILLON,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
[-- Attachment #1: Type: text/plain, Size: 1255 bytes --]
Hi Arnd,
On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> On Friday 05 September 2014, Nicolas Ferre wrote:
> > Arnd, Olof, Kevin,
> >
> > This pull-request is focused on the work that Maxime did for migrating our timer
> > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > be even closer to the point when we have only the bare minimum in our formerly
> > crowded mach-at91 directory.
> >
> > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > 01st.
>
> Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> we could kill that off in the long run. This is only used for the legacy
> board files, not for DT, right?
Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
I wasn't aware that early_platform drivers were in the killzone, but
I'm definitely aware that global custom exported functions are, hence
why I went this way.
> Do you have a timeline for getting rid of the board files completely?
I don't, but it would be great if we could kill these in the next
couple releases.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-08 9:26 ` Maxime Ripard
0 siblings, 0 replies; 22+ messages in thread
From: Maxime Ripard @ 2014-09-08 9:26 UTC (permalink / raw)
To: linux-arm-kernel
Hi Arnd,
On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> On Friday 05 September 2014, Nicolas Ferre wrote:
> > Arnd, Olof, Kevin,
> >
> > This pull-request is focused on the work that Maxime did for migrating our timer
> > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > be even closer to the point when we have only the bare minimum in our formerly
> > crowded mach-at91 directory.
> >
> > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > 01st.
>
> Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> we could kill that off in the long run. This is only used for the legacy
> board files, not for DT, right?
Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
I wasn't aware that early_platform drivers were in the killzone, but
I'm definitely aware that global custom exported functions are, hence
why I went this way.
> Do you have a timeline for getting rid of the board files completely?
I don't, but it would be great if we could kill these in the next
couple releases.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140908/5a2806c9/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-08 9:26 ` Maxime Ripard
@ 2014-09-08 9:52 ` Arnd Bergmann
-1 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-08 9:52 UTC (permalink / raw)
To: Maxime Ripard
Cc: Nicolas Ferre, Olof Johansson, arm, Linux Kernel list,
linux-arm-kernel, Alexandre Belloni, Boris BREZILLON,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Monday 08 September 2014 11:26:42 Maxime Ripard wrote:
> On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> > On Friday 05 September 2014, Nicolas Ferre wrote:
> > > Arnd, Olof, Kevin,
> > >
> > > This pull-request is focused on the work that Maxime did for migrating our timer
> > > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > > be even closer to the point when we have only the bare minimum in our formerly
> > > crowded mach-at91 directory.
> > >
> > > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > > 01st.
> >
> > Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> > we could kill that off in the long run. This is only used for the legacy
> > board files, not for DT, right?
>
> Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
>
> I wasn't aware that early_platform drivers were in the killzone, but
> I'm definitely aware that global custom exported functions are, hence
> why I went this way.
I don't think it has been discussed much on the mailing list or IRC.
The early platform devices have not been used much outside of arch/sh
and arch/arm/mach-shmobile, and those only use it for clocksource and
serial.
Now we have a new method for both of these, at least with DT, so
my impression is that we won't need the early_platform support in
the future. One of the problems with the current interface is that
it requires statically declaring platform_device structures, which is
something that has been on Greg's list of device model antipatterns
for a long time.
> > Do you have a timeline for getting rid of the board files completely?
>
> I don't, but it would be great if we could kill these in the next
> couple releases.
Ok, good.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-08 9:52 ` Arnd Bergmann
0 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-08 9:52 UTC (permalink / raw)
To: linux-arm-kernel
On Monday 08 September 2014 11:26:42 Maxime Ripard wrote:
> On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> > On Friday 05 September 2014, Nicolas Ferre wrote:
> > > Arnd, Olof, Kevin,
> > >
> > > This pull-request is focused on the work that Maxime did for migrating our timer
> > > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > > be even closer to the point when we have only the bare minimum in our formerly
> > > crowded mach-at91 directory.
> > >
> > > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > > 01st.
> >
> > Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> > we could kill that off in the long run. This is only used for the legacy
> > board files, not for DT, right?
>
> Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
>
> I wasn't aware that early_platform drivers were in the killzone, but
> I'm definitely aware that global custom exported functions are, hence
> why I went this way.
I don't think it has been discussed much on the mailing list or IRC.
The early platform devices have not been used much outside of arch/sh
and arch/arm/mach-shmobile, and those only use it for clocksource and
serial.
Now we have a new method for both of these, at least with DT, so
my impression is that we won't need the early_platform support in
the future. One of the problems with the current interface is that
it requires statically declaring platform_device structures, which is
something that has been on Greg's list of device model antipatterns
for a long time.
> > Do you have a timeline for getting rid of the board files completely?
>
> I don't, but it would be great if we could kill these in the next
> couple releases.
Ok, good.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-08 9:52 ` Arnd Bergmann
@ 2014-09-09 9:15 ` Maxime Ripard
-1 siblings, 0 replies; 22+ messages in thread
From: Maxime Ripard @ 2014-09-09 9:15 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Nicolas Ferre, Olof Johansson, arm, Linux Kernel list,
linux-arm-kernel, Alexandre Belloni, Boris BREZILLON,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
[-- Attachment #1: Type: text/plain, Size: 2212 bytes --]
On Mon, Sep 08, 2014 at 11:52:35AM +0200, Arnd Bergmann wrote:
> On Monday 08 September 2014 11:26:42 Maxime Ripard wrote:
> > On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> > > On Friday 05 September 2014, Nicolas Ferre wrote:
> > > > Arnd, Olof, Kevin,
> > > >
> > > > This pull-request is focused on the work that Maxime did for migrating our timer
> > > > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > > > be even closer to the point when we have only the bare minimum in our formerly
> > > > crowded mach-at91 directory.
> > > >
> > > > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > > > 01st.
> > >
> > > Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> > > we could kill that off in the long run. This is only used for the legacy
> > > board files, not for DT, right?
> >
> > Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
> >
> > I wasn't aware that early_platform drivers were in the killzone, but
> > I'm definitely aware that global custom exported functions are, hence
> > why I went this way.
>
> I don't think it has been discussed much on the mailing list or IRC.
> The early platform devices have not been used much outside of arch/sh
> and arch/arm/mach-shmobile, and those only use it for clocksource and
> serial.
That's what I found, yes.
> Now we have a new method for both of these, at least with DT, so
> my impression is that we won't need the early_platform support in
> the future.
I agree with that. I only see the early platform stuff as a temporary
measure for board files, before they're removed.
> One of the problems with the current interface is that it requires
> statically declaring platform_device structures, which is something
> that has been on Greg's list of device model antipatterns for a long
> time.
I didn't find any difference with how you declare platform_devices
compared to the old-usual way in board files, or was it something on
the list too ? :)
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-09 9:15 ` Maxime Ripard
0 siblings, 0 replies; 22+ messages in thread
From: Maxime Ripard @ 2014-09-09 9:15 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Sep 08, 2014 at 11:52:35AM +0200, Arnd Bergmann wrote:
> On Monday 08 September 2014 11:26:42 Maxime Ripard wrote:
> > On Fri, Sep 05, 2014 at 11:25:11PM +0200, Arnd Bergmann wrote:
> > > On Friday 05 September 2014, Nicolas Ferre wrote:
> > > > Arnd, Olof, Kevin,
> > > >
> > > > This pull-request is focused on the work that Maxime did for migrating our timer
> > > > (PIT) to the clocksource sub-system. A big cleanup happened which allows us to
> > > > be even closer to the point when we have only the bare minimum in our formerly
> > > > crowded mach-at91 directory.
> > > >
> > > > This pull-request goes on top of the "drivers" one already sent to you on Sept.
> > > > 01st.
> > >
> > > Hmm, I'm not too happy to see more uses of early_platform_*, I was hoping
> > > we could kill that off in the long run. This is only used for the legacy
> > > board files, not for DT, right?
> >
> > Yes, the DT uses the usual CLOCKSOURCE_OF_DECLARE mechanism.
> >
> > I wasn't aware that early_platform drivers were in the killzone, but
> > I'm definitely aware that global custom exported functions are, hence
> > why I went this way.
>
> I don't think it has been discussed much on the mailing list or IRC.
> The early platform devices have not been used much outside of arch/sh
> and arch/arm/mach-shmobile, and those only use it for clocksource and
> serial.
That's what I found, yes.
> Now we have a new method for both of these, at least with DT, so
> my impression is that we won't need the early_platform support in
> the future.
I agree with that. I only see the early platform stuff as a temporary
measure for board files, before they're removed.
> One of the problems with the current interface is that it requires
> statically declaring platform_device structures, which is something
> that has been on Greg's list of device model antipatterns for a long
> time.
I didn't find any difference with how you declare platform_devices
compared to the old-usual way in board files, or was it something on
the list too ? :)
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140909/d719a72e/attachment.sig>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2
2014-09-09 9:15 ` Maxime Ripard
@ 2014-09-09 9:27 ` Arnd Bergmann
-1 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-09 9:27 UTC (permalink / raw)
To: Maxime Ripard
Cc: Nicolas Ferre, Olof Johansson, arm, Linux Kernel list,
linux-arm-kernel, Alexandre Belloni, Boris BREZILLON,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Tuesday 09 September 2014 11:15:20 Maxime Ripard wrote:
>
> > One of the problems with the current interface is that it requires
> > statically declaring platform_device structures, which is something
> > that has been on Greg's list of device model antipatterns for a long
> > time.
>
> I didn't find any difference with how you declare platform_devices
> compared to the old-usual way in board files, or was it something on
> the list too ?
Yes. We never really bothered with cleaning up the existing static
platform_device instances, because the plan is to move away from
board files to DT anyway, but in short, if you ever need to add
a platform device to a legacy board file, do it like
pdev = platform_device_alloc(...);
platform_device_register(dev);
or
platform_device_register_simple(...);
or
static const struct platform_device_info info __initconst = { ... };
platform_device_register_full(&info);
but not
static struct platform_device pdev = { ... };
platform_device_register(&pdev);
Unfortunately, the early platform_device only allows the last type.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2
@ 2014-09-09 9:27 ` Arnd Bergmann
0 siblings, 0 replies; 22+ messages in thread
From: Arnd Bergmann @ 2014-09-09 9:27 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 09 September 2014 11:15:20 Maxime Ripard wrote:
>
> > One of the problems with the current interface is that it requires
> > statically declaring platform_device structures, which is something
> > that has been on Greg's list of device model antipatterns for a long
> > time.
>
> I didn't find any difference with how you declare platform_devices
> compared to the old-usual way in board files, or was it something on
> the list too ?
Yes. We never really bothered with cleaning up the existing static
platform_device instances, because the plan is to move away from
board files to DT anyway, but in short, if you ever need to add
a platform device to a legacy board file, do it like
pdev = platform_device_alloc(...);
platform_device_register(dev);
or
platform_device_register_simple(...);
or
static const struct platform_device_info info __initconst = { ... };
platform_device_register_full(&info);
but not
static struct platform_device pdev = { ... };
platform_device_register(&pdev);
Unfortunately, the early platform_device only allows the last type.
Arnd
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2 (bis)
2014-09-05 16:36 ` Nicolas Ferre
@ 2014-09-15 16:12 ` Nicolas Ferre
-1 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-15 16:12 UTC (permalink / raw)
To: Arnd Bergmann, Olof Johansson, arm
Cc: Nicolas Ferre, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
Arnd, Olof, Kevin,
I re-send this pull-request with the work done by Maxime to take Arnd's
comments into annount: removal of the early_platform devices, and by keeping
the old mechanism with a function exported by the PIT timer and called by the
board files.
Here is the old message that I sent you: just as a reference.
"
This pull-request is focused on the work that Maxime did for migrating our
timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
us to be even closer to the point when we have only the bare minimum in our
formerly crowded mach-at91 directory.
"
This pull-request goes on top of what is already in your at91/drivers branch.
Thanks, best regards,
The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
are available in the git repository at:
git://github.com/at91linux/linux-at91.git tags/at91-drivers2
for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
----------------------------------------------------------------
Second drivers series for AT91/3.18:
- move of the PIT (basic timer) from mach-at91 to its proper location:
drivers/clocksource
- big cleanup of this driver along the way
----------------------------------------------------------------
Maxime Ripard (14):
ARM: at91: PIT: Follow the general coding rules
ARM: at91: generic.h: Add include safe guards
ARM: at91: PIT: Use DIV_ROUND_CLOSEST to compute the cycles
ARM: at91: PIT: Use of_have_populated_dt instead of CONFIG_OF
ARM: at91: PIT: Rework probe functions
ARM: at91: dt: Remove init_time definitions
ARM: at91: PIT: Use consistent exit path in probe
ARM: at91: PIT: Use pr_fmt
ARM: at91: PIT: use request_irq instead of setup_irq
ARM: at91: PIT: (Almost) remove the global variables
ARM: at91: soc: Add init_time callback
ARM: at91: Convert the boards to the init_time callback
ARM: at91: Give the PIT irq as an argument of at91sam926x_pit_init
ARM: at91: PIT: Move the driver to drivers/clocksource
arch/arm/mach-at91/Kconfig | 5 -
arch/arm/mach-at91/Makefile | 1 -
arch/arm/mach-at91/at91sam9260.c | 6 +
arch/arm/mach-at91/at91sam9261.c | 6 +
arch/arm/mach-at91/at91sam9263.c | 6 +
arch/arm/mach-at91/at91sam926x_time.c | 294 -------------------------------
arch/arm/mach-at91/at91sam9g45.c | 6 +
arch/arm/mach-at91/at91sam9rl.c | 6 +
arch/arm/mach-at91/board-afeb-9260v1.c | 2 +-
arch/arm/mach-at91/board-cam60.c | 2 +-
arch/arm/mach-at91/board-cpu9krea.c | 2 +-
arch/arm/mach-at91/board-dt-sam9.c | 10 --
arch/arm/mach-at91/board-dt-sama5.c | 9 -
arch/arm/mach-at91/board-flexibity.c | 2 +-
arch/arm/mach-at91/board-foxg20.c | 2 +-
arch/arm/mach-at91/board-gsia18s.c | 2 +-
arch/arm/mach-at91/board-pcontrol-g20.c | 2 +-
arch/arm/mach-at91/board-sam9-l9260.c | 2 +-
arch/arm/mach-at91/board-sam9260ek.c | 2 +-
arch/arm/mach-at91/board-sam9261ek.c | 4 +-
arch/arm/mach-at91/board-sam9263ek.c | 2 +-
arch/arm/mach-at91/board-sam9g20ek.c | 4 +-
arch/arm/mach-at91/board-sam9m10g45ek.c | 2 +-
arch/arm/mach-at91/board-sam9rlek.c | 2 +-
arch/arm/mach-at91/board-snapper9260.c | 2 +-
arch/arm/mach-at91/board-stamp9g20.c | 4 +-
arch/arm/mach-at91/generic.h | 8 +-
arch/arm/mach-at91/setup.c | 5 +
arch/arm/mach-at91/soc.h | 1 +
drivers/clocksource/Kconfig | 4 +
drivers/clocksource/Makefile | 1 +
drivers/clocksource/timer-atmel-pit.c | 296 ++++++++++++++++++++++++++++++++
32 files changed, 363 insertions(+), 339 deletions(-)
delete mode 100644 arch/arm/mach-at91/at91sam926x_time.c
create mode 100644 drivers/clocksource/timer-atmel-pit.c
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2 (bis)
@ 2014-09-15 16:12 ` Nicolas Ferre
0 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-15 16:12 UTC (permalink / raw)
To: linux-arm-kernel
Arnd, Olof, Kevin,
I re-send this pull-request with the work done by Maxime to take Arnd's
comments into annount: removal of the early_platform devices, and by keeping
the old mechanism with a function exported by the PIT timer and called by the
board files.
Here is the old message that I sent you: just as a reference.
"
This pull-request is focused on the work that Maxime did for migrating our
timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
us to be even closer to the point when we have only the bare minimum in our
formerly crowded mach-at91 directory.
"
This pull-request goes on top of what is already in your at91/drivers branch.
Thanks, best regards,
The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
are available in the git repository at:
git://github.com/at91linux/linux-at91.git tags/at91-drivers2
for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
----------------------------------------------------------------
Second drivers series for AT91/3.18:
- move of the PIT (basic timer) from mach-at91 to its proper location:
drivers/clocksource
- big cleanup of this driver along the way
----------------------------------------------------------------
Maxime Ripard (14):
ARM: at91: PIT: Follow the general coding rules
ARM: at91: generic.h: Add include safe guards
ARM: at91: PIT: Use DIV_ROUND_CLOSEST to compute the cycles
ARM: at91: PIT: Use of_have_populated_dt instead of CONFIG_OF
ARM: at91: PIT: Rework probe functions
ARM: at91: dt: Remove init_time definitions
ARM: at91: PIT: Use consistent exit path in probe
ARM: at91: PIT: Use pr_fmt
ARM: at91: PIT: use request_irq instead of setup_irq
ARM: at91: PIT: (Almost) remove the global variables
ARM: at91: soc: Add init_time callback
ARM: at91: Convert the boards to the init_time callback
ARM: at91: Give the PIT irq as an argument of at91sam926x_pit_init
ARM: at91: PIT: Move the driver to drivers/clocksource
arch/arm/mach-at91/Kconfig | 5 -
arch/arm/mach-at91/Makefile | 1 -
arch/arm/mach-at91/at91sam9260.c | 6 +
arch/arm/mach-at91/at91sam9261.c | 6 +
arch/arm/mach-at91/at91sam9263.c | 6 +
arch/arm/mach-at91/at91sam926x_time.c | 294 -------------------------------
arch/arm/mach-at91/at91sam9g45.c | 6 +
arch/arm/mach-at91/at91sam9rl.c | 6 +
arch/arm/mach-at91/board-afeb-9260v1.c | 2 +-
arch/arm/mach-at91/board-cam60.c | 2 +-
arch/arm/mach-at91/board-cpu9krea.c | 2 +-
arch/arm/mach-at91/board-dt-sam9.c | 10 --
arch/arm/mach-at91/board-dt-sama5.c | 9 -
arch/arm/mach-at91/board-flexibity.c | 2 +-
arch/arm/mach-at91/board-foxg20.c | 2 +-
arch/arm/mach-at91/board-gsia18s.c | 2 +-
arch/arm/mach-at91/board-pcontrol-g20.c | 2 +-
arch/arm/mach-at91/board-sam9-l9260.c | 2 +-
arch/arm/mach-at91/board-sam9260ek.c | 2 +-
arch/arm/mach-at91/board-sam9261ek.c | 4 +-
arch/arm/mach-at91/board-sam9263ek.c | 2 +-
arch/arm/mach-at91/board-sam9g20ek.c | 4 +-
arch/arm/mach-at91/board-sam9m10g45ek.c | 2 +-
arch/arm/mach-at91/board-sam9rlek.c | 2 +-
arch/arm/mach-at91/board-snapper9260.c | 2 +-
arch/arm/mach-at91/board-stamp9g20.c | 4 +-
arch/arm/mach-at91/generic.h | 8 +-
arch/arm/mach-at91/setup.c | 5 +
arch/arm/mach-at91/soc.h | 1 +
drivers/clocksource/Kconfig | 4 +
drivers/clocksource/Makefile | 1 +
drivers/clocksource/timer-atmel-pit.c | 296 ++++++++++++++++++++++++++++++++
32 files changed, 363 insertions(+), 339 deletions(-)
delete mode 100644 arch/arm/mach-at91/at91sam926x_time.c
create mode 100644 drivers/clocksource/timer-atmel-pit.c
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2 (bis)
2014-09-15 16:12 ` Nicolas Ferre
@ 2014-09-24 4:59 ` Olof Johansson
-1 siblings, 0 replies; 22+ messages in thread
From: Olof Johansson @ 2014-09-24 4:59 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Arnd Bergmann, arm, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
> Arnd, Olof, Kevin,
>
> I re-send this pull-request with the work done by Maxime to take Arnd's
> comments into annount: removal of the early_platform devices, and by keeping
> the old mechanism with a function exported by the PIT timer and called by the
> board files.
>
> Here is the old message that I sent you: just as a reference.
> "
> This pull-request is focused on the work that Maxime did for migrating our
> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
> us to be even closer to the point when we have only the bare minimum in our
> formerly crowded mach-at91 directory.
> "
>
> This pull-request goes on top of what is already in your at91/drivers branch.
>
> Thanks, best regards,
>
> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
>
> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
>
> are available in the git repository at:
>
> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
>
> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
>
> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
>
> ----------------------------------------------------------------
> Second drivers series for AT91/3.18:
> - move of the PIT (basic timer) from mach-at91 to its proper location:
> drivers/clocksource
> - big cleanup of this driver along the way
Thanks, merged.
-Olof
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2 (bis)
@ 2014-09-24 4:59 ` Olof Johansson
0 siblings, 0 replies; 22+ messages in thread
From: Olof Johansson @ 2014-09-24 4:59 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
> Arnd, Olof, Kevin,
>
> I re-send this pull-request with the work done by Maxime to take Arnd's
> comments into annount: removal of the early_platform devices, and by keeping
> the old mechanism with a function exported by the PIT timer and called by the
> board files.
>
> Here is the old message that I sent you: just as a reference.
> "
> This pull-request is focused on the work that Maxime did for migrating our
> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
> us to be even closer to the point when we have only the bare minimum in our
> formerly crowded mach-at91 directory.
> "
>
> This pull-request goes on top of what is already in your at91/drivers branch.
>
> Thanks, best regards,
>
> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
>
> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
>
> are available in the git repository at:
>
> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
>
> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
>
> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
>
> ----------------------------------------------------------------
> Second drivers series for AT91/3.18:
> - move of the PIT (basic timer) from mach-at91 to its proper location:
> drivers/clocksource
> - big cleanup of this driver along the way
Thanks, merged.
-Olof
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2 (bis)
2014-09-24 4:59 ` Olof Johansson
@ 2014-09-24 10:50 ` Nicolas Ferre
-1 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-24 10:50 UTC (permalink / raw)
To: Olof Johansson, Arnd Bergmann, arm
Cc: Linux Kernel list, linux-arm-kernel, Alexandre Belloni,
Boris BREZILLON, Maxime Ripard, Daniel Lezcano,
Jean-Christophe PLAGNIOL-VILLARD, Ludovic Desroches
On 24/09/2014 06:59, Olof Johansson :
> On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
>> Arnd, Olof, Kevin,
>>
>> I re-send this pull-request with the work done by Maxime to take Arnd's
>> comments into annount: removal of the early_platform devices, and by keeping
>> the old mechanism with a function exported by the PIT timer and called by the
>> board files.
>>
>> Here is the old message that I sent you: just as a reference.
>> "
>> This pull-request is focused on the work that Maxime did for migrating our
>> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
>> us to be even closer to the point when we have only the bare minimum in our
>> formerly crowded mach-at91 directory.
>> "
>>
>> This pull-request goes on top of what is already in your at91/drivers branch.
>>
>> Thanks, best regards,
>>
>> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
>>
>> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
>>
>> are available in the git repository at:
>>
>> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
>>
>> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
>>
>> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
>>
>> ----------------------------------------------------------------
>> Second drivers series for AT91/3.18:
>> - move of the PIT (basic timer) from mach-at91 to its proper location:
>> drivers/clocksource
>> - big cleanup of this driver along the way
>
> Thanks, merged.
Thanks Olof, but there is a merge issue in the merge commit
58c19114886d5352c019979b84f5375970d81f05 (Merge branch 'next/drivers'
into for-next). In fact the sama5_dt_timer_init() and
ksz9021rn_phy_fixup() functions must be completely removed in this
arch/arm/mach-at91/board-dt-sama5.c file.
This is causing the errors seen in your and Kevin's buildbots:
http://images.armcloud.us/kernel-ci//arm-soc/v3.17-rc4-608-g58bf4da/arm-sama5_defconfig/build.log
and
http://arm-soc.lixom.net/buildlogs/arm-soc/v3.17-rc4-608-g58bf4da/buildall.arm.sama5_defconfig.log.failed
I pushed a branch for conflict resolutions here:
git://github.com/at91linux/linux-at91.git at91-3.18-resolved2
https://github.com/at91linux/linux-at91/commits/at91-3.18-resolved2
Bye,
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2 (bis)
@ 2014-09-24 10:50 ` Nicolas Ferre
0 siblings, 0 replies; 22+ messages in thread
From: Nicolas Ferre @ 2014-09-24 10:50 UTC (permalink / raw)
To: linux-arm-kernel
On 24/09/2014 06:59, Olof Johansson :
> On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
>> Arnd, Olof, Kevin,
>>
>> I re-send this pull-request with the work done by Maxime to take Arnd's
>> comments into annount: removal of the early_platform devices, and by keeping
>> the old mechanism with a function exported by the PIT timer and called by the
>> board files.
>>
>> Here is the old message that I sent you: just as a reference.
>> "
>> This pull-request is focused on the work that Maxime did for migrating our
>> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
>> us to be even closer to the point when we have only the bare minimum in our
>> formerly crowded mach-at91 directory.
>> "
>>
>> This pull-request goes on top of what is already in your at91/drivers branch.
>>
>> Thanks, best regards,
>>
>> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
>>
>> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
>>
>> are available in the git repository at:
>>
>> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
>>
>> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
>>
>> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
>>
>> ----------------------------------------------------------------
>> Second drivers series for AT91/3.18:
>> - move of the PIT (basic timer) from mach-at91 to its proper location:
>> drivers/clocksource
>> - big cleanup of this driver along the way
>
> Thanks, merged.
Thanks Olof, but there is a merge issue in the merge commit
58c19114886d5352c019979b84f5375970d81f05 (Merge branch 'next/drivers'
into for-next). In fact the sama5_dt_timer_init() and
ksz9021rn_phy_fixup() functions must be completely removed in this
arch/arm/mach-at91/board-dt-sama5.c file.
This is causing the errors seen in your and Kevin's buildbots:
http://images.armcloud.us/kernel-ci//arm-soc/v3.17-rc4-608-g58bf4da/arm-sama5_defconfig/build.log
and
http://arm-soc.lixom.net/buildlogs/arm-soc/v3.17-rc4-608-g58bf4da/buildall.arm.sama5_defconfig.log.failed
I pushed a branch for conflict resolutions here:
git://github.com/at91linux/linux-at91.git at91-3.18-resolved2
https://github.com/at91linux/linux-at91/commits/at91-3.18-resolved2
Bye,
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] at91: drivers for 3.18 #2 (bis)
2014-09-24 10:50 ` Nicolas Ferre
@ 2014-09-24 17:34 ` Olof Johansson
-1 siblings, 0 replies; 22+ messages in thread
From: Olof Johansson @ 2014-09-24 17:34 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Arnd Bergmann, arm, Linux Kernel list, linux-arm-kernel,
Alexandre Belloni, Boris BREZILLON, Maxime Ripard,
Daniel Lezcano, Jean-Christophe PLAGNIOL-VILLARD,
Ludovic Desroches
On Wed, Sep 24, 2014 at 12:50:02PM +0200, Nicolas Ferre wrote:
> On 24/09/2014 06:59, Olof Johansson :
> > On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
> >> Arnd, Olof, Kevin,
> >>
> >> I re-send this pull-request with the work done by Maxime to take Arnd's
> >> comments into annount: removal of the early_platform devices, and by keeping
> >> the old mechanism with a function exported by the PIT timer and called by the
> >> board files.
> >>
> >> Here is the old message that I sent you: just as a reference.
> >> "
> >> This pull-request is focused on the work that Maxime did for migrating our
> >> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
> >> us to be even closer to the point when we have only the bare minimum in our
> >> formerly crowded mach-at91 directory.
> >> "
> >>
> >> This pull-request goes on top of what is already in your at91/drivers branch.
> >>
> >> Thanks, best regards,
> >>
> >> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
> >>
> >> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
> >>
> >> are available in the git repository at:
> >>
> >> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
> >>
> >> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
> >>
> >> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
> >>
> >> ----------------------------------------------------------------
> >> Second drivers series for AT91/3.18:
> >> - move of the PIT (basic timer) from mach-at91 to its proper location:
> >> drivers/clocksource
> >> - big cleanup of this driver along the way
> >
> > Thanks, merged.
>
> Thanks Olof, but there is a merge issue in the merge commit
> 58c19114886d5352c019979b84f5375970d81f05 (Merge branch 'next/drivers'
> into for-next). In fact the sama5_dt_timer_init() and
> ksz9021rn_phy_fixup() functions must be completely removed in this
> arch/arm/mach-at91/board-dt-sama5.c file.
Thanks, fixed up.
> This is causing the errors seen in your and Kevin's buildbots:
> http://images.armcloud.us/kernel-ci//arm-soc/v3.17-rc4-608-g58bf4da/arm-sama5_defconfig/build.log
> and
> http://arm-soc.lixom.net/buildlogs/arm-soc/v3.17-rc4-608-g58bf4da/buildall.arm.sama5_defconfig.log.failed
Yeah, it's what I get for doing this when tired. :-) I didn't stay up to check build results.
> I pushed a branch for conflict resolutions here:
> git://github.com/at91linux/linux-at91.git at91-3.18-resolved2
> https://github.com/at91linux/linux-at91/commits/at91-3.18-resolved2
All good, I've fixed it up with an incremental commit on our for-next,
so we'll just keep an eye on it when we send it up.
-Olof
^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] at91: drivers for 3.18 #2 (bis)
@ 2014-09-24 17:34 ` Olof Johansson
0 siblings, 0 replies; 22+ messages in thread
From: Olof Johansson @ 2014-09-24 17:34 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Sep 24, 2014 at 12:50:02PM +0200, Nicolas Ferre wrote:
> On 24/09/2014 06:59, Olof Johansson :
> > On Mon, Sep 15, 2014 at 06:12:43PM +0200, Nicolas Ferre wrote:
> >> Arnd, Olof, Kevin,
> >>
> >> I re-send this pull-request with the work done by Maxime to take Arnd's
> >> comments into annount: removal of the early_platform devices, and by keeping
> >> the old mechanism with a function exported by the PIT timer and called by the
> >> board files.
> >>
> >> Here is the old message that I sent you: just as a reference.
> >> "
> >> This pull-request is focused on the work that Maxime did for migrating our
> >> timer (PIT) to the clocksource sub-system. A big cleanup happened which allows
> >> us to be even closer to the point when we have only the bare minimum in our
> >> formerly crowded mach-at91 directory.
> >> "
> >>
> >> This pull-request goes on top of what is already in your at91/drivers branch.
> >>
> >> Thanks, best regards,
> >>
> >> The following changes since commit 405a72c5e78b5c560c8b2711d4000fa5eb063e1b:
> >>
> >> power: reset: at91-poweroff: fix wakeup status register index (2014-09-01 18:40:44 +0200)
> >>
> >> are available in the git repository at:
> >>
> >> git://github.com/at91linux/linux-at91.git tags/at91-drivers2
> >>
> >> for you to fetch changes up to b052ff30cd450c91a32e8e928979bca021462996:
> >>
> >> ARM: at91: PIT: Move the driver to drivers/clocksource (2014-09-15 17:55:48 +0200)
> >>
> >> ----------------------------------------------------------------
> >> Second drivers series for AT91/3.18:
> >> - move of the PIT (basic timer) from mach-at91 to its proper location:
> >> drivers/clocksource
> >> - big cleanup of this driver along the way
> >
> > Thanks, merged.
>
> Thanks Olof, but there is a merge issue in the merge commit
> 58c19114886d5352c019979b84f5375970d81f05 (Merge branch 'next/drivers'
> into for-next). In fact the sama5_dt_timer_init() and
> ksz9021rn_phy_fixup() functions must be completely removed in this
> arch/arm/mach-at91/board-dt-sama5.c file.
Thanks, fixed up.
> This is causing the errors seen in your and Kevin's buildbots:
> http://images.armcloud.us/kernel-ci//arm-soc/v3.17-rc4-608-g58bf4da/arm-sama5_defconfig/build.log
> and
> http://arm-soc.lixom.net/buildlogs/arm-soc/v3.17-rc4-608-g58bf4da/buildall.arm.sama5_defconfig.log.failed
Yeah, it's what I get for doing this when tired. :-) I didn't stay up to check build results.
> I pushed a branch for conflict resolutions here:
> git://github.com/at91linux/linux-at91.git at91-3.18-resolved2
> https://github.com/at91linux/linux-at91/commits/at91-3.18-resolved2
All good, I've fixed it up with an incremental commit on our for-next,
so we'll just keep an eye on it when we send it up.
-Olof
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2014-09-24 18:56 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-05 16:36 [GIT PULL] at91: drivers for 3.18 #2 Nicolas Ferre
2014-09-05 16:36 ` Nicolas Ferre
2014-09-05 21:25 ` Arnd Bergmann
2014-09-05 21:25 ` Arnd Bergmann
2014-09-05 21:39 ` Arnd Bergmann
2014-09-05 21:39 ` Arnd Bergmann
2014-09-08 9:26 ` Maxime Ripard
2014-09-08 9:26 ` Maxime Ripard
2014-09-08 9:52 ` Arnd Bergmann
2014-09-08 9:52 ` Arnd Bergmann
2014-09-09 9:15 ` Maxime Ripard
2014-09-09 9:15 ` Maxime Ripard
2014-09-09 9:27 ` Arnd Bergmann
2014-09-09 9:27 ` Arnd Bergmann
2014-09-15 16:12 ` [GIT PULL] at91: drivers for 3.18 #2 (bis) Nicolas Ferre
2014-09-15 16:12 ` Nicolas Ferre
2014-09-24 4:59 ` Olof Johansson
2014-09-24 4:59 ` Olof Johansson
2014-09-24 10:50 ` Nicolas Ferre
2014-09-24 10:50 ` Nicolas Ferre
2014-09-24 17:34 ` Olof Johansson
2014-09-24 17:34 ` Olof Johansson
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.