| Message ID | 20190215125229.30180-5-m.szyprowski@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/5] clocksource: exynos_mct: Remove dead code | expand |
Hi Marek, On 19. 2. 15. 오후 9:52, Marek Szyprowski wrote: > All CortexA7/A15 based Exynos SoCs have ARM architected timers, so enable > support for them directly in the base dtsi for Exynos3250, Exynos5250 > and Exynos54xx SoCs. None of the known firmware properly configures arch > timer registers, so mark them as not-fw-configured and set frequency > to 24MHz, which is the only configuration supported by the remaining > drivers (clocks) so far. > > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- > arch/arm/boot/dts/exynos3250.dtsi | 10 ++++++++++ > arch/arm/boot/dts/exynos5250.dtsi | 1 + > arch/arm/boot/dts/exynos54xx.dtsi | 10 ++++++++++ > 3 files changed, 21 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi > index 608d17454179..3966e7960964 100644 > --- a/arch/arm/boot/dts/exynos3250.dtsi > +++ b/arch/arm/boot/dts/exynos3250.dtsi > @@ -134,6 +134,16 @@ > }; > }; > > + timer { > + compatible = "arm,armv7-timer"; > + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; > + clock-frequency = <24000000>; > + arm,cpu-registers-not-fw-configured; > + }; > + > sysram@2020000 { > compatible = "mmio-sram"; > reg = <0x02020000 0x40000>; > diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi > index 80986b97dfe5..95af4050f69d 100644 > --- a/arch/arm/boot/dts/exynos5250.dtsi > +++ b/arch/arm/boot/dts/exynos5250.dtsi > @@ -239,6 +239,7 @@ > * so we need the value from DT. > */ > clock-frequency = <24000000>; > + arm,cpu-registers-not-fw-configured; > }; > > mct@101c0000 { > diff --git a/arch/arm/boot/dts/exynos54xx.dtsi b/arch/arm/boot/dts/exynos54xx.dtsi > index de26e5ee0d2d..45e06717ee51 100644 > --- a/arch/arm/boot/dts/exynos54xx.dtsi > +++ b/arch/arm/boot/dts/exynos54xx.dtsi > @@ -46,6 +46,16 @@ > status = "disabled"; > }; > > + timer { > + compatible = "arm,armv7-timer"; > + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; > + arm,cpu-registers-not-fw-configured; > + clock-frequency = <24000000>; > + }; > + > sysram@2020000 { > compatible = "mmio-sram"; > reg = <0x02020000 0x54000>; > The exynos54xx.dtsi affects the exynos5410 SoC. When I checked the PPI port number on Exynos5410/Exynos5422 TRM, they have the different PPI port number as following: |Exynos5410 |Exynos5422 | ---------------------------------------------------------------------------------------- ARCH_TIMER_PHYS_SECURE_PPI |1 nCNTPSIRQ |13 nCNTPSIRQ |Secure physical timer ARCH_TIMER_PHYS_NONSECURE_PPI |2 nCNTPNSIRQ |14 nCNTPNSIRQ |Non-secure physical timer ARCH_TIMER_VIRT_PPI |4 nCNTVIRQ |11 nCNTVIRQ |Virtual timer ARCH_TIMER_HYP_PPI |5 nCNTHPIRQ |10 nCNTHPIRQ |Hypervisor timer ---------------------------------------------------------------------------------------- Exynos5410 and Exynos5422 have the different usage of bitfield of nCNTPSIRQ/nCNTPNSIRQ/nCNTVIRQ/nCNTHPIRQ as following: |Exynos5410 |Exynos5422 ------------------------------------------------------------ nCNTPSIRQ[3:0] | KFC | EAGLE nCNTPNSIRQ[3:0] | KFC | EAGLE nCNTVIRQ[3:0] | KFC | EAGLE nCNTHPIRQ[3:0] | KFC | EAGLE ------------------------------------------------------------ nCNTPSIRQ[7:4] | EAGLE | KFC nCNTPNSIRQ[7:4] | EAGLE | KFC nCNTVIRQ[7:4] | EAGLE | KFC nCNTHPIRQ[7:4] | EAGLE | KFC
Hi Chanwoo, On 2019-02-18 12:58, Chanwoo Choi wrote: > On 19. 2. 15. 오후 9:52, Marek Szyprowski wrote: >> All CortexA7/A15 based Exynos SoCs have ARM architected timers, so enable >> support for them directly in the base dtsi for Exynos3250, Exynos5250 >> and Exynos54xx SoCs. None of the known firmware properly configures arch >> timer registers, so mark them as not-fw-configured and set frequency >> to 24MHz, which is the only configuration supported by the remaining >> drivers (clocks) so far. >> >> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> >> --- >> arch/arm/boot/dts/exynos3250.dtsi | 10 ++++++++++ >> arch/arm/boot/dts/exynos5250.dtsi | 1 + >> arch/arm/boot/dts/exynos54xx.dtsi | 10 ++++++++++ >> 3 files changed, 21 insertions(+) >> >> diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi >> index 608d17454179..3966e7960964 100644 >> --- a/arch/arm/boot/dts/exynos3250.dtsi >> +++ b/arch/arm/boot/dts/exynos3250.dtsi >> @@ -134,6 +134,16 @@ >> }; >> }; >> >> + timer { >> + compatible = "arm,armv7-timer"; >> + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; >> + clock-frequency = <24000000>; >> + arm,cpu-registers-not-fw-configured; >> + }; >> + >> sysram@2020000 { >> compatible = "mmio-sram"; >> reg = <0x02020000 0x40000>; >> diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi >> index 80986b97dfe5..95af4050f69d 100644 >> --- a/arch/arm/boot/dts/exynos5250.dtsi >> +++ b/arch/arm/boot/dts/exynos5250.dtsi >> @@ -239,6 +239,7 @@ >> * so we need the value from DT. >> */ >> clock-frequency = <24000000>; >> + arm,cpu-registers-not-fw-configured; >> }; >> >> mct@101c0000 { >> diff --git a/arch/arm/boot/dts/exynos54xx.dtsi b/arch/arm/boot/dts/exynos54xx.dtsi >> index de26e5ee0d2d..45e06717ee51 100644 >> --- a/arch/arm/boot/dts/exynos54xx.dtsi >> +++ b/arch/arm/boot/dts/exynos54xx.dtsi >> @@ -46,6 +46,16 @@ >> status = "disabled"; >> }; >> >> + timer { >> + compatible = "arm,armv7-timer"; >> + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >> + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; >> + arm,cpu-registers-not-fw-configured; >> + clock-frequency = <24000000>; >> + }; >> + >> sysram@2020000 { >> compatible = "mmio-sram"; >> reg = <0x02020000 0x54000>; >> > The exynos54xx.dtsi affects the exynos5410 SoC. > When I checked the PPI port number on Exynos5410/Exynos5422 TRM, > they have the different PPI port number as following: > > |Exynos5410 |Exynos5422 | > ---------------------------------------------------------------------------------------- > ARCH_TIMER_PHYS_SECURE_PPI |1 nCNTPSIRQ |13 nCNTPSIRQ |Secure physical timer > ARCH_TIMER_PHYS_NONSECURE_PPI |2 nCNTPNSIRQ |14 nCNTPNSIRQ |Non-secure physical timer > ARCH_TIMER_VIRT_PPI |4 nCNTVIRQ |11 nCNTVIRQ |Virtual timer > ARCH_TIMER_HYP_PPI |5 nCNTHPIRQ |10 nCNTHPIRQ |Hypervisor timer > ---------------------------------------------------------------------------------------- Sadly, Exynos5410 TRM seems to be wrong. All other Exynos5 TRMs (5250, 5260, 5420, 5422) defines 13,14,11,10 PPI for arch timer. I've checked on the real hardware (Exynos5410-based Odroid XU board) and the values taken from Exynos5422 TRM works fine, while the ones from 5410 TRM don't. > Exynos5410 and Exynos5422 have the different usage of bitfield > of nCNTPSIRQ/nCNTPNSIRQ/nCNTVIRQ/nCNTHPIRQ as following: > > |Exynos5410 |Exynos5422 > ------------------------------------------------------------ > nCNTPSIRQ[3:0] | KFC | EAGLE > nCNTPNSIRQ[3:0] | KFC | EAGLE > nCNTVIRQ[3:0] | KFC | EAGLE > nCNTHPIRQ[3:0] | KFC | EAGLE > ------------------------------------------------------------ > nCNTPSIRQ[7:4] | EAGLE | KFC > nCNTPNSIRQ[7:4] | EAGLE | KFC > nCNTVIRQ[7:4] | EAGLE | KFC > nCNTHPIRQ[7:4] | EAGLE | KFC This is not an issue, this is a result of different order of bit/LITTLE cores. Same for Exynos5420. Best regards
Hi Marek, On 19. 2. 18. 오후 9:24, Marek Szyprowski wrote: > Hi Chanwoo, > > On 2019-02-18 12:58, Chanwoo Choi wrote: >> On 19. 2. 15. 오후 9:52, Marek Szyprowski wrote: >>> All CortexA7/A15 based Exynos SoCs have ARM architected timers, so enable >>> support for them directly in the base dtsi for Exynos3250, Exynos5250 >>> and Exynos54xx SoCs. None of the known firmware properly configures arch >>> timer registers, so mark them as not-fw-configured and set frequency >>> to 24MHz, which is the only configuration supported by the remaining >>> drivers (clocks) so far. >>> >>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> >>> --- >>> arch/arm/boot/dts/exynos3250.dtsi | 10 ++++++++++ >>> arch/arm/boot/dts/exynos5250.dtsi | 1 + >>> arch/arm/boot/dts/exynos54xx.dtsi | 10 ++++++++++ >>> 3 files changed, 21 insertions(+) >>> >>> diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi >>> index 608d17454179..3966e7960964 100644 >>> --- a/arch/arm/boot/dts/exynos3250.dtsi >>> +++ b/arch/arm/boot/dts/exynos3250.dtsi >>> @@ -134,6 +134,16 @@ >>> }; >>> }; >>> >>> + timer { >>> + compatible = "arm,armv7-timer"; >>> + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; >>> + clock-frequency = <24000000>; >>> + arm,cpu-registers-not-fw-configured; >>> + }; >>> + >>> sysram@2020000 { >>> compatible = "mmio-sram"; >>> reg = <0x02020000 0x40000>; >>> diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi >>> index 80986b97dfe5..95af4050f69d 100644 >>> --- a/arch/arm/boot/dts/exynos5250.dtsi >>> +++ b/arch/arm/boot/dts/exynos5250.dtsi >>> @@ -239,6 +239,7 @@ >>> * so we need the value from DT. >>> */ >>> clock-frequency = <24000000>; >>> + arm,cpu-registers-not-fw-configured; >>> }; >>> >>> mct@101c0000 { >>> diff --git a/arch/arm/boot/dts/exynos54xx.dtsi b/arch/arm/boot/dts/exynos54xx.dtsi >>> index de26e5ee0d2d..45e06717ee51 100644 >>> --- a/arch/arm/boot/dts/exynos54xx.dtsi >>> +++ b/arch/arm/boot/dts/exynos54xx.dtsi >>> @@ -46,6 +46,16 @@ >>> status = "disabled"; >>> }; >>> >>> + timer { >>> + compatible = "arm,armv7-timer"; >>> + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, >>> + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; >>> + arm,cpu-registers-not-fw-configured; >>> + clock-frequency = <24000000>; >>> + }; >>> + >>> sysram@2020000 { >>> compatible = "mmio-sram"; >>> reg = <0x02020000 0x54000>; >>> >> The exynos54xx.dtsi affects the exynos5410 SoC. >> When I checked the PPI port number on Exynos5410/Exynos5422 TRM, >> they have the different PPI port number as following: >> >> |Exynos5410 |Exynos5422 | >> ---------------------------------------------------------------------------------------- >> ARCH_TIMER_PHYS_SECURE_PPI |1 nCNTPSIRQ |13 nCNTPSIRQ |Secure physical timer >> ARCH_TIMER_PHYS_NONSECURE_PPI |2 nCNTPNSIRQ |14 nCNTPNSIRQ |Non-secure physical timer >> ARCH_TIMER_VIRT_PPI |4 nCNTVIRQ |11 nCNTVIRQ |Virtual timer >> ARCH_TIMER_HYP_PPI |5 nCNTHPIRQ |10 nCNTHPIRQ |Hypervisor timer >> ---------------------------------------------------------------------------------------- > > Sadly, Exynos5410 TRM seems to be wrong. All other Exynos5 TRMs (5250, > 5260, 5420, 5422) defines 13,14,11,10 PPI for arch timer. I've checked > on the real hardware (Exynos5410-based Odroid XU board) and the values > taken from Exynos5422 TRM works fine, while the ones from 5410 TRM don't. Thanks for sharing the tested result. Reveiwed-by: Chanwoo Choi <cw00.choi@samsung.com>
diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi index 608d17454179..3966e7960964 100644 --- a/arch/arm/boot/dts/exynos3250.dtsi +++ b/arch/arm/boot/dts/exynos3250.dtsi @@ -134,6 +134,16 @@ }; }; + timer { + compatible = "arm,armv7-timer"; + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; + clock-frequency = <24000000>; + arm,cpu-registers-not-fw-configured; + }; + sysram@2020000 { compatible = "mmio-sram"; reg = <0x02020000 0x40000>; diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi index 80986b97dfe5..95af4050f69d 100644 --- a/arch/arm/boot/dts/exynos5250.dtsi +++ b/arch/arm/boot/dts/exynos5250.dtsi @@ -239,6 +239,7 @@ * so we need the value from DT. */ clock-frequency = <24000000>; + arm,cpu-registers-not-fw-configured; }; mct@101c0000 { diff --git a/arch/arm/boot/dts/exynos54xx.dtsi b/arch/arm/boot/dts/exynos54xx.dtsi index de26e5ee0d2d..45e06717ee51 100644 --- a/arch/arm/boot/dts/exynos54xx.dtsi +++ b/arch/arm/boot/dts/exynos54xx.dtsi @@ -46,6 +46,16 @@ status = "disabled"; }; + timer { + compatible = "arm,armv7-timer"; + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; + arm,cpu-registers-not-fw-configured; + clock-frequency = <24000000>; + }; + sysram@2020000 { compatible = "mmio-sram"; reg = <0x02020000 0x54000>;
All CortexA7/A15 based Exynos SoCs have ARM architected timers, so enable support for them directly in the base dtsi for Exynos3250, Exynos5250 and Exynos54xx SoCs. None of the known firmware properly configures arch timer registers, so mark them as not-fw-configured and set frequency to 24MHz, which is the only configuration supported by the remaining drivers (clocks) so far. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- arch/arm/boot/dts/exynos3250.dtsi | 10 ++++++++++ arch/arm/boot/dts/exynos5250.dtsi | 1 + arch/arm/boot/dts/exynos54xx.dtsi | 10 ++++++++++ 3 files changed, 21 insertions(+)