mbox series

[RESEND,00/10] ARM: davinci: use the new clocksource driver

Message ID 20190722131748.30319-1-brgl@bgdev.pl (mailing list archive)
Headers show
Series ARM: davinci: use the new clocksource driver | expand

Message

Bartosz Golaszewski July 22, 2019, 1:17 p.m. UTC
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>

Sekhar,

the following patches switch DaVinci to using the new clocksource driver which
is now upstream. They are rebased on top of v5.3-rc1. Additionally the
following two patches were reverted locally due to a regression in v5.3-rc1
about which the relevant maintainers have been already notified:

  2eef1399a866 modules: fix BUG when load module with rodata=n
  93651f80dcb6 modules: fix compile error if don't have strict module rwx

Bartosz Golaszewski (10):
  ARM: davinci: enable the clocksource driver for DT mode
  ARM: davinci: WARN_ON() if clk_get() fails
  ARM: davinci: da850: switch to using the clocksource driver
  ARM: davinci: da830: switch to using the clocksource driver
  ARM: davinci: move timer definitions to davinci.h
  ARM: davinci: dm355: switch to using the clocksource driver
  ARM: davinci: dm365: switch to using the clocksource driver
  ARM: davinci: dm644x: switch to using the clocksource driver
  ARM: davinci: dm646x: switch to using the clocksource driver
  ARM: davinci: remove legacy timer support

 arch/arm/Kconfig                            |   1 +
 arch/arm/mach-davinci/Makefile              |   3 +-
 arch/arm/mach-davinci/da830.c               |  45 +--
 arch/arm/mach-davinci/da850.c               |  50 +--
 arch/arm/mach-davinci/davinci.h             |   3 +
 arch/arm/mach-davinci/devices-da8xx.c       |   1 -
 arch/arm/mach-davinci/devices.c             |  19 -
 arch/arm/mach-davinci/dm355.c               |  28 +-
 arch/arm/mach-davinci/dm365.c               |  26 +-
 arch/arm/mach-davinci/dm644x.c              |  28 +-
 arch/arm/mach-davinci/dm646x.c              |  28 +-
 arch/arm/mach-davinci/include/mach/common.h |  17 -
 arch/arm/mach-davinci/include/mach/time.h   |  35 --
 arch/arm/mach-davinci/time.c                | 414 --------------------
 14 files changed, 110 insertions(+), 588 deletions(-)
 delete mode 100644 arch/arm/mach-davinci/include/mach/time.h
 delete mode 100644 arch/arm/mach-davinci/time.c

Comments

Bartosz Golaszewski Aug. 5, 2019, 8:29 a.m. UTC | #1
pon., 22 lip 2019 o 15:17 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Sekhar,
>
> the following patches switch DaVinci to using the new clocksource driver which
> is now upstream. They are rebased on top of v5.3-rc1. Additionally the
> following two patches were reverted locally due to a regression in v5.3-rc1
> about which the relevant maintainers have been already notified:
>
>   2eef1399a866 modules: fix BUG when load module with rodata=n
>   93651f80dcb6 modules: fix compile error if don't have strict module rwx
>
> Bartosz Golaszewski (10):
>   ARM: davinci: enable the clocksource driver for DT mode
>   ARM: davinci: WARN_ON() if clk_get() fails
>   ARM: davinci: da850: switch to using the clocksource driver
>   ARM: davinci: da830: switch to using the clocksource driver
>   ARM: davinci: move timer definitions to davinci.h
>   ARM: davinci: dm355: switch to using the clocksource driver
>   ARM: davinci: dm365: switch to using the clocksource driver
>   ARM: davinci: dm644x: switch to using the clocksource driver
>   ARM: davinci: dm646x: switch to using the clocksource driver
>   ARM: davinci: remove legacy timer support
>
>  arch/arm/Kconfig                            |   1 +
>  arch/arm/mach-davinci/Makefile              |   3 +-
>  arch/arm/mach-davinci/da830.c               |  45 +--
>  arch/arm/mach-davinci/da850.c               |  50 +--
>  arch/arm/mach-davinci/davinci.h             |   3 +
>  arch/arm/mach-davinci/devices-da8xx.c       |   1 -
>  arch/arm/mach-davinci/devices.c             |  19 -
>  arch/arm/mach-davinci/dm355.c               |  28 +-
>  arch/arm/mach-davinci/dm365.c               |  26 +-
>  arch/arm/mach-davinci/dm644x.c              |  28 +-
>  arch/arm/mach-davinci/dm646x.c              |  28 +-
>  arch/arm/mach-davinci/include/mach/common.h |  17 -
>  arch/arm/mach-davinci/include/mach/time.h   |  35 --
>  arch/arm/mach-davinci/time.c                | 414 --------------------
>  14 files changed, 110 insertions(+), 588 deletions(-)
>  delete mode 100644 arch/arm/mach-davinci/include/mach/time.h
>  delete mode 100644 arch/arm/mach-davinci/time.c
>
> --
> 2.21.0
>

Hi Sekhar,

a gentle ping. Is this series good to go in for v5.4?

Bart
Sekhar Nori Aug. 7, 2019, 7:28 p.m. UTC | #2
On 05/08/19 1:59 PM, Bartosz Golaszewski wrote:
> pon., 22 lip 2019 o 15:17 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>>
>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> Sekhar,
>>
>> the following patches switch DaVinci to using the new clocksource driver which
>> is now upstream. They are rebased on top of v5.3-rc1. Additionally the
>> following two patches were reverted locally due to a regression in v5.3-rc1
>> about which the relevant maintainers have been already notified:
>>
>>   2eef1399a866 modules: fix BUG when load module with rodata=n
>>   93651f80dcb6 modules: fix compile error if don't have strict module rwx
>>
>> Bartosz Golaszewski (10):
>>   ARM: davinci: enable the clocksource driver for DT mode
>>   ARM: davinci: WARN_ON() if clk_get() fails
>>   ARM: davinci: da850: switch to using the clocksource driver
>>   ARM: davinci: da830: switch to using the clocksource driver
>>   ARM: davinci: move timer definitions to davinci.h
>>   ARM: davinci: dm355: switch to using the clocksource driver
>>   ARM: davinci: dm365: switch to using the clocksource driver
>>   ARM: davinci: dm644x: switch to using the clocksource driver
>>   ARM: davinci: dm646x: switch to using the clocksource driver
>>   ARM: davinci: remove legacy timer support
>>
>>  arch/arm/Kconfig                            |   1 +
>>  arch/arm/mach-davinci/Makefile              |   3 +-
>>  arch/arm/mach-davinci/da830.c               |  45 +--
>>  arch/arm/mach-davinci/da850.c               |  50 +--
>>  arch/arm/mach-davinci/davinci.h             |   3 +
>>  arch/arm/mach-davinci/devices-da8xx.c       |   1 -
>>  arch/arm/mach-davinci/devices.c             |  19 -
>>  arch/arm/mach-davinci/dm355.c               |  28 +-
>>  arch/arm/mach-davinci/dm365.c               |  26 +-
>>  arch/arm/mach-davinci/dm644x.c              |  28 +-
>>  arch/arm/mach-davinci/dm646x.c              |  28 +-
>>  arch/arm/mach-davinci/include/mach/common.h |  17 -
>>  arch/arm/mach-davinci/include/mach/time.h   |  35 --
>>  arch/arm/mach-davinci/time.c                | 414 --------------------
>>  14 files changed, 110 insertions(+), 588 deletions(-)
>>  delete mode 100644 arch/arm/mach-davinci/include/mach/time.h
>>  delete mode 100644 arch/arm/mach-davinci/time.c
>>
>> --
>> 2.21.0
>>
> 
> Hi Sekhar,
> 
> a gentle ping. Is this series good to go in for v5.4?

Hi Bartosz, a quick test shows that DM365 fails to boot after this. Can
you please see if there is anything obviously wrong for that SoC. Rest
seems to be okay.

Thanks,
Sekhar
Bartosz Golaszewski Aug. 8, 2019, 7:41 a.m. UTC | #3
śr., 7 sie 2019 o 21:28 Sekhar Nori <nsekhar@ti.com> napisał(a):
>
> On 05/08/19 1:59 PM, Bartosz Golaszewski wrote:
> > pon., 22 lip 2019 o 15:17 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
> >>
> >> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> >>
> >> Sekhar,
> >>
> >> the following patches switch DaVinci to using the new clocksource driver which
> >> is now upstream. They are rebased on top of v5.3-rc1. Additionally the
> >> following two patches were reverted locally due to a regression in v5.3-rc1
> >> about which the relevant maintainers have been already notified:
> >>
> >>   2eef1399a866 modules: fix BUG when load module with rodata=n
> >>   93651f80dcb6 modules: fix compile error if don't have strict module rwx
> >>
> >> Bartosz Golaszewski (10):
> >>   ARM: davinci: enable the clocksource driver for DT mode
> >>   ARM: davinci: WARN_ON() if clk_get() fails
> >>   ARM: davinci: da850: switch to using the clocksource driver
> >>   ARM: davinci: da830: switch to using the clocksource driver
> >>   ARM: davinci: move timer definitions to davinci.h
> >>   ARM: davinci: dm355: switch to using the clocksource driver
> >>   ARM: davinci: dm365: switch to using the clocksource driver
> >>   ARM: davinci: dm644x: switch to using the clocksource driver
> >>   ARM: davinci: dm646x: switch to using the clocksource driver
> >>   ARM: davinci: remove legacy timer support
> >>
> >>  arch/arm/Kconfig                            |   1 +
> >>  arch/arm/mach-davinci/Makefile              |   3 +-
> >>  arch/arm/mach-davinci/da830.c               |  45 +--
> >>  arch/arm/mach-davinci/da850.c               |  50 +--
> >>  arch/arm/mach-davinci/davinci.h             |   3 +
> >>  arch/arm/mach-davinci/devices-da8xx.c       |   1 -
> >>  arch/arm/mach-davinci/devices.c             |  19 -
> >>  arch/arm/mach-davinci/dm355.c               |  28 +-
> >>  arch/arm/mach-davinci/dm365.c               |  26 +-
> >>  arch/arm/mach-davinci/dm644x.c              |  28 +-
> >>  arch/arm/mach-davinci/dm646x.c              |  28 +-
> >>  arch/arm/mach-davinci/include/mach/common.h |  17 -
> >>  arch/arm/mach-davinci/include/mach/time.h   |  35 --
> >>  arch/arm/mach-davinci/time.c                | 414 --------------------
> >>  14 files changed, 110 insertions(+), 588 deletions(-)
> >>  delete mode 100644 arch/arm/mach-davinci/include/mach/time.h
> >>  delete mode 100644 arch/arm/mach-davinci/time.c
> >>
> >> --
> >> 2.21.0
> >>
> >
> > Hi Sekhar,
> >
> > a gentle ping. Is this series good to go in for v5.4?
>
> Hi Bartosz, a quick test shows that DM365 fails to boot after this. Can
> you please see if there is anything obviously wrong for that SoC. Rest
> seems to be okay.
>
> Thanks,
> Sekhar

Hi Sekhar,

just verified on Kevin's dm365-evm rebased on top of v5.3-rc3 and it
boots fine. I know that davinci failed to boot at v5.3-rc1.

Let me know if I can help with debugging.

Bart
Sekhar Nori Aug. 26, 2019, 1:33 p.m. UTC | #4
On 08/08/19 1:11 PM, Bartosz Golaszewski wrote:
> śr., 7 sie 2019 o 21:28 Sekhar Nori <nsekhar@ti.com> napisał(a):
>>
>> On 05/08/19 1:59 PM, Bartosz Golaszewski wrote:
>>> pon., 22 lip 2019 o 15:17 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>>>>
>>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>>
>>>> Sekhar,
>>>>
>>>> the following patches switch DaVinci to using the new clocksource driver which
>>>> is now upstream. They are rebased on top of v5.3-rc1. Additionally the
>>>> following two patches were reverted locally due to a regression in v5.3-rc1
>>>> about which the relevant maintainers have been already notified:
>>>>
>>>>   2eef1399a866 modules: fix BUG when load module with rodata=n
>>>>   93651f80dcb6 modules: fix compile error if don't have strict module rwx
>>>>
>>>> Bartosz Golaszewski (10):
>>>>   ARM: davinci: enable the clocksource driver for DT mode
>>>>   ARM: davinci: WARN_ON() if clk_get() fails
>>>>   ARM: davinci: da850: switch to using the clocksource driver
>>>>   ARM: davinci: da830: switch to using the clocksource driver
>>>>   ARM: davinci: move timer definitions to davinci.h
>>>>   ARM: davinci: dm355: switch to using the clocksource driver
>>>>   ARM: davinci: dm365: switch to using the clocksource driver
>>>>   ARM: davinci: dm644x: switch to using the clocksource driver
>>>>   ARM: davinci: dm646x: switch to using the clocksource driver
>>>>   ARM: davinci: remove legacy timer support
>>>>
>>>>  arch/arm/Kconfig                            |   1 +
>>>>  arch/arm/mach-davinci/Makefile              |   3 +-
>>>>  arch/arm/mach-davinci/da830.c               |  45 +--
>>>>  arch/arm/mach-davinci/da850.c               |  50 +--
>>>>  arch/arm/mach-davinci/davinci.h             |   3 +
>>>>  arch/arm/mach-davinci/devices-da8xx.c       |   1 -
>>>>  arch/arm/mach-davinci/devices.c             |  19 -
>>>>  arch/arm/mach-davinci/dm355.c               |  28 +-
>>>>  arch/arm/mach-davinci/dm365.c               |  26 +-
>>>>  arch/arm/mach-davinci/dm644x.c              |  28 +-
>>>>  arch/arm/mach-davinci/dm646x.c              |  28 +-
>>>>  arch/arm/mach-davinci/include/mach/common.h |  17 -
>>>>  arch/arm/mach-davinci/include/mach/time.h   |  35 --
>>>>  arch/arm/mach-davinci/time.c                | 414 --------------------
>>>>  14 files changed, 110 insertions(+), 588 deletions(-)
>>>>  delete mode 100644 arch/arm/mach-davinci/include/mach/time.h
>>>>  delete mode 100644 arch/arm/mach-davinci/time.c
>>>>
>>>> --
>>>> 2.21.0
>>>>
>>>
>>> Hi Sekhar,
>>>
>>> a gentle ping. Is this series good to go in for v5.4?
>>
>> Hi Bartosz, a quick test shows that DM365 fails to boot after this. Can
>> you please see if there is anything obviously wrong for that SoC. Rest
>> seems to be okay.
>>
>> Thanks,
>> Sekhar
> 
> Hi Sekhar,
> 
> just verified on Kevin's dm365-evm rebased on top of v5.3-rc3 and it
> boots fine. I know that davinci failed to boot at v5.3-rc1.
> 
> Let me know if I can help with debugging.

Added series except 7/10 and 10/10 to v5.4/soc. Debug of DM365 issue is
still going on offline. DM365 migration is postponed pending conclusion
of that debug.

Thanks,
Sekhar