mbox series

[0/2] clocksource: move System Tick Counter from 'arch/mips/ralink'

Message ID 20240920075306.704665-1-sergio.paracuellos@gmail.com (mailing list archive)
Headers show
Series clocksource: move System Tick Counter from 'arch/mips/ralink' | expand

Message

Sergio Paracuellos Sept. 20, 2024, 7:53 a.m. UTC 
Hi Daniel,

System Tick Counter is present in RT3352 and MT7620 Ralink SoCs. This driver has 
been in 'arch/mips/ralink' from the beggining and can be easily moved into a more
accurate place in 'drivers/clocksource' folder. This makes easier to enable it
for compile test targets as well as reduce LOC in architecture specific folders.
Bindings are already mainlined and can be located here [0].

Thanks in advance for your time.

Best regards,
   Sergio Paracuellos

[0]: https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/timer/ralink,cevt-systick.yaml

Sergio Paracuellos (2):
  clocksource: Add Ralink System Tick Counter driver
  MIPS: ralink: remove System Tick Counter driver

 arch/mips/ralink/Kconfig                              |  7 -------
 arch/mips/ralink/Makefile                             |  2 --
 drivers/clocksource/Kconfig                           | 10 ++++++++++
 drivers/clocksource/Makefile                          |  1 +
 .../clocksource/timer-ralink.c                        | 11 ++++-------
 5 files changed, 15 insertions(+), 16 deletions(-)
 rename arch/mips/ralink/cevt-rt3352.c => drivers/clocksource/timer-ralink.c (91%)

Comments

Sergio Paracuellos Oct. 28, 2024, 2:14 p.m. UTC | #1 
On Fri, Sep 20, 2024 at 9:53 AM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> Hi Daniel,
>
> System Tick Counter is present in RT3352 and MT7620 Ralink SoCs. This driver has
> been in 'arch/mips/ralink' from the beggining and can be easily moved into a more
> accurate place in 'drivers/clocksource' folder. This makes easier to enable it
> for compile test targets as well as reduce LOC in architecture specific folders.
> Bindings are already mainlined and can be located here [0].
>
> Thanks in advance for your time.
>
> Best regards,
>    Sergio Paracuellos
>
> [0]: https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/timer/ralink,cevt-systick.yaml
>
> Sergio Paracuellos (2):
>   clocksource: Add Ralink System Tick Counter driver
>   MIPS: ralink: remove System Tick Counter driver
>
>  arch/mips/ralink/Kconfig                              |  7 -------
>  arch/mips/ralink/Makefile                             |  2 --
>  drivers/clocksource/Kconfig                           | 10 ++++++++++
>  drivers/clocksource/Makefile                          |  1 +
>  .../clocksource/timer-ralink.c                        | 11 ++++-------
>  5 files changed, 15 insertions(+), 16 deletions(-)
>  rename arch/mips/ralink/cevt-rt3352.c => drivers/clocksource/timer-ralink.c (91%)

Gentle ping on this patch series :-)

Best regards,
    Sergio Paracuellos
>
> --
> 2.25.1
>
Daniel Lezcano Oct. 28, 2024, 2:16 p.m. UTC | #2 
On 28/10/2024 15:14, Sergio Paracuellos wrote:
> On Fri, Sep 20, 2024 at 9:53 AM Sergio Paracuellos
> <sergio.paracuellos@gmail.com> wrote:
>>
>> Hi Daniel,
>>
>> System Tick Counter is present in RT3352 and MT7620 Ralink SoCs. This driver has
>> been in 'arch/mips/ralink' from the beggining and can be easily moved into a more
>> accurate place in 'drivers/clocksource' folder. This makes easier to enable it
>> for compile test targets as well as reduce LOC in architecture specific folders.
>> Bindings are already mainlined and can be located here [0].
>>
>> Thanks in advance for your time.
>>
>> Best regards,
>>     Sergio Paracuellos
>>
>> [0]: https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/timer/ralink,cevt-systick.yaml
>>
>> Sergio Paracuellos (2):
>>    clocksource: Add Ralink System Tick Counter driver
>>    MIPS: ralink: remove System Tick Counter driver
>>
>>   arch/mips/ralink/Kconfig                              |  7 -------
>>   arch/mips/ralink/Makefile                             |  2 --
>>   drivers/clocksource/Kconfig                           | 10 ++++++++++
>>   drivers/clocksource/Makefile                          |  1 +
>>   .../clocksource/timer-ralink.c                        | 11 ++++-------
>>   5 files changed, 15 insertions(+), 16 deletions(-)
>>   rename arch/mips/ralink/cevt-rt3352.c => drivers/clocksource/timer-ralink.c (91%)
> 
> Gentle ping on this patch series :-)

I'm on it
Sergio Paracuellos Oct. 28, 2024, 2:18 p.m. UTC | #3 
On Mon, Oct 28, 2024 at 3:16 PM Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:
>
> On 28/10/2024 15:14, Sergio Paracuellos wrote:
> > On Fri, Sep 20, 2024 at 9:53 AM Sergio Paracuellos
> > <sergio.paracuellos@gmail.com> wrote:
> >>
> >> Hi Daniel,
> >>
> >> System Tick Counter is present in RT3352 and MT7620 Ralink SoCs. This driver has
> >> been in 'arch/mips/ralink' from the beggining and can be easily moved into a more
> >> accurate place in 'drivers/clocksource' folder. This makes easier to enable it
> >> for compile test targets as well as reduce LOC in architecture specific folders.
> >> Bindings are already mainlined and can be located here [0].
> >>
> >> Thanks in advance for your time.
> >>
> >> Best regards,
> >>     Sergio Paracuellos
> >>
> >> [0]: https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/timer/ralink,cevt-systick.yaml
> >>
> >> Sergio Paracuellos (2):
> >>    clocksource: Add Ralink System Tick Counter driver
> >>    MIPS: ralink: remove System Tick Counter driver
> >>
> >>   arch/mips/ralink/Kconfig                              |  7 -------
> >>   arch/mips/ralink/Makefile                             |  2 --
> >>   drivers/clocksource/Kconfig                           | 10 ++++++++++
> >>   drivers/clocksource/Makefile                          |  1 +
> >>   .../clocksource/timer-ralink.c                        | 11 ++++-------
> >>   5 files changed, 15 insertions(+), 16 deletions(-)
> >>   rename arch/mips/ralink/cevt-rt3352.c => drivers/clocksource/timer-ralink.c (91%)
> >
> > Gentle ping on this patch series :-)
>
> I'm on it

Awesome, thanks!

>
>
> --
> <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