mbox series

[v7,0/5] Add support for the OST in Ingenic X1000.

Message ID 20200717165947.56158-1-zhouyanjie@wanyeetech.com
Headers show
Series Add support for the OST in Ingenic X1000. | expand

Message

Zhou Yanjie July 17, 2020, 4:59 p.m. UTC
v6->v7:
1.Remove "default MACH_INGENIC" and make option silent.
2.Enable the corresponding driver in the platform's Kconfig.
3.Update DT of X1000 and X1830, use SYSOST instead of TCU
  to provide clocksource and clockevent.

周琰杰 (Zhou Yanjie) (5):
  dt-bindings: timer: Add Ingenic X1000 OST bindings.
  clocksource: Ingenic: Add support for the Ingenic X1000 OST.
  MIPS: Ingenic: Let the Kconfig of platform enable the clocksource
    driver.
  MIPS: X1000: Use SYSOST instead of TCU to provide clocksource.
  MIPS: X1830: Use SYSOST instead of TCU to provide clocksource.

 .../devicetree/bindings/timer/ingenic,sysost.yaml  |  63 +++
 arch/mips/boot/dts/ingenic/cu1000-neo.dts          |   9 +-
 arch/mips/boot/dts/ingenic/cu1830-neo.dts          |   9 +-
 arch/mips/boot/dts/ingenic/x1000.dtsi              |  16 +
 arch/mips/boot/dts/ingenic/x1830.dtsi              |  16 +
 arch/mips/jz4740/Kconfig                           |   7 +
 drivers/clocksource/Kconfig                        |  15 +-
 drivers/clocksource/Makefile                       |   1 +
 drivers/clocksource/ingenic-sysost.c               | 539 +++++++++++++++++++++
 include/dt-bindings/clock/ingenic,sysost.h         |  12 +
 10 files changed, 672 insertions(+), 15 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/timer/ingenic,sysost.yaml
 create mode 100644 drivers/clocksource/ingenic-sysost.c
 create mode 100644 include/dt-bindings/clock/ingenic,sysost.h

Comments

Sergei Shtylyov July 18, 2020, 8:39 a.m. UTC | #1
Hello!

On 17.07.2020 19:59, 周琰杰 (Zhou Yanjie) wrote:

> The previous clocksource patch in this series ([2/3]) has remove

   Removed.

> "default MACH_INGENIC" and make option silent, so we need to

   Made?

> enable the corresponding driver in the platform's Kconfig.
> 
> Suggested-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Tested-by: 周正 (Zhou Zheng) <sernia.zhou@foxmail.com>
> Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
> ---
> 
> Notes:
>      v7:
>      New patch.
> 
>   arch/mips/jz4740/Kconfig | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/arch/mips/jz4740/Kconfig b/arch/mips/jz4740/Kconfig
> index 6c065dcaeff8..5ad60998702e 100644
> --- a/arch/mips/jz4740/Kconfig
> +++ b/arch/mips/jz4740/Kconfig
> @@ -36,27 +36,34 @@ endchoice
>   config MACH_JZ4740
>   	bool
>   	select SYS_HAS_CPU_MIPS32_R1
> +    select INGENIC_TIMER

    Please use tab instead of spaces here an below.

[...]

MBR, Sergei
Zhou Yanjie July 18, 2020, 10:10 a.m. UTC | #2
Hi Sergei,

在 2020/7/18 下午4:39, Sergei Shtylyov 写道:
> Hello!
>
> On 17.07.2020 19:59, 周琰杰 (Zhou Yanjie) wrote:
>
>> The previous clocksource patch in this series ([2/3]) has remove
>
>   Removed.
>
>> "default MACH_INGENIC" and make option silent, so we need to
>
>   Made?
>
>> enable the corresponding driver in the platform's Kconfig.
>>
>> Suggested-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>> Tested-by: 周正 (Zhou Zheng) <sernia.zhou@foxmail.com>
>> Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
>> ---
>>
>> Notes:
>>      v7:
>>      New patch.
>>
>>   arch/mips/jz4740/Kconfig | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/arch/mips/jz4740/Kconfig b/arch/mips/jz4740/Kconfig
>> index 6c065dcaeff8..5ad60998702e 100644
>> --- a/arch/mips/jz4740/Kconfig
>> +++ b/arch/mips/jz4740/Kconfig
>> @@ -36,27 +36,34 @@ endchoice
>>   config MACH_JZ4740
>>       bool
>>       select SYS_HAS_CPU_MIPS32_R1
>> +    select INGENIC_TIMER
>
>    Please use tab instead of spaces here an below.
>

Sure, I will do it in the next version.

Thanks and best regards!


> [...]
>
> MBR, Sergei