clocksource: sysctr: add nxp,no-divider property

[PATCH 0/2] clocksource: sysctr: add nxp,no-divider property

Posted by Peng Fan (OSS) 3 years, 7 months ago

From: Peng Fan <peng.fan@nxp.com>

Legacy design used in i.MX8M* has a internal divider, but new design
used in i.MX9* not has that divider. So add a property to indicate
the divider exists or not.

I thought to extend the binding such as nxp,imx93-sysctr-timer, and
update driver to support i.MX93, then no need to add nxp,no-divider.
But this means we need to extend the binding again when add new SoC,
saying imx9[x].

Not very sure which is better, so just post the NXP downstream way.

Peng Fan (2):
  dt-bindings: timer: nxp,sysctr-timer: add nxp,no-divider property
  clocksource/drivers/imx-sysctr: handle nxp,no-divider property

 .../devicetree/bindings/timer/nxp,sysctr-timer.yaml         | 4 ++++
 drivers/clocksource/timer-imx-sysctr.c                      | 6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

-- 
2.37.1

Re: [PATCH 0/2] clocksource: sysctr: add nxp,no-divider property

Posted by Daniel Lezcano 3 years, 6 months ago

On 02/09/2022 13:12, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> Legacy design used in i.MX8M* has a internal divider, but new design
> used in i.MX9* not has that divider. So add a property to indicate
> the divider exists or not.
> 
> I thought to extend the binding such as nxp,imx93-sysctr-timer, and
> update driver to support i.MX93, then no need to add nxp,no-divider.
> But this means we need to extend the binding again when add new SoC,
> saying imx9[x].
> 
> Not very sure which is better, so just post the NXP downstream way.
> 

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