ARM: OMAP2+: PRCM+SCM cleanup against 4.0-rc1
Subsystems (9)
v6 cover ARM: OMAP2+: PRCM+SCM cleanup against 4.0-rc1
Hi,
Decided to post a new version because of the number of minor changes compared to v5, just to avoid any misunderstandings:
- dropped patch #1 ("clk: ti: fix ti_clk_get_reg_addr..."), this has been merged through clock tree now
- fixed scm_conf area size issues on am33xx/am43xx
- fixed scm area size issue on am43xx
- fixed size issues on omap3_pmx_core (was 8 bytes short)
- added warnings on omap2_clk_readl/writel in case legacy access is attempted
- fixed omap3 legacy boot
- fixed boot time warning on omap4+ where scm_conf does not contain clocks node yet (these may be added to DT later if support is required)
- fixed some randconfig warnings
Testing done:
1: am335x-evm : boot 2: am335x-evmsk : boot 3: am3517-evm : boot 4: am43x-epos-evm : boot 5: am437x-gp-evm : boot 6: omap3-beagle-xm : boot 7: omap3-beagle : boot, cpuidle (ret/off), suspend (ret/off), legacy boot 8: am335x-boneblack: boot 9: am335x-bone : boot 10: dra7xx-evm : boot 11: omap3-n900 : boot 12: omap5-uevm : boot 13: omap4-panda-es : boot, cpuidle (ret), suspend (ret) 14: omap4-panda : boot 15: omap2430-sdp : boot 16: omap3430-sdp : boot
17: omap4-sdp-es23plus: boot
New branch also pushed:
tree: https://github.com/t-kristo/linux-pm.git
branch: 4.0-rc1-prcm-cleanup-v6
-Tero
PATCH v6 1/34 ARM: OMAP2+: PRCM: rename of_prcm_init to omap_prcm_init
This avoids conflicts in the global namespace, and is more descriptive of the purpose anyway.
--- arch/arm/mach-omap2/io.c | 2 +- arch/arm/mach-omap2/prm.h | 2 +- arch/arm/mach-omap2/prm_common.c | 8 +++++++- 3 files changed, 9 insertions(+), 3 deletions(-)
arch/arm/mach-omap2/io.c +1 -1
@@ -764,7 +764,7 @@ int __init omap_clk_init(void)ti_clk_init_features ();if (of_have_populated_dt ()) { - ret =of_prcm_init (); + ret =omap_prcm_init ();if (ret)return ret;
arch/arm/mach-omap2/prm.h +1 -1
@@ -19,7 +19,7 @@extern void __iomem *prm_base;extern u16 prm_features;extern void omap2_set_globals_prm (void __iomem *prm); -int of_prcm_init (void ); +int omap_prcm_init (void );void omap3_prcm_legacy_iomaps_init (void );# endif
arch/arm/mach-omap2/prm_common.c +7 -1
@@ -625,7 +625,13 @@ static struct ti_clk_ll_ops omap_clk_ll_ops = { .clk_writel = prm_clk_writel, }; -int __init of_prcm_init (void ) +/** +* omap_prcm_init - low level init for the PRCM drivers +* +* Initializes the low level clock infrastructure for PRCM drivers. +* Returns 0 in success, negative error value in failure. +*/ +int __init omap_prcm_init (void ) {struct device_node *np;void __iomem *mem;
Signature · 1 line
1.7.9.5
PATCH v6 2/34 ARM: OMAP3: PRM: invert the wkst_mask for the prm_clear_mod_irqs
In reply to Tero KristoThis makes the API the same as used with OMAP2, and makes it possible to implement a generic driver API for the functionality.
--- arch/arm/mach-omap2/pm34xx.c | 18 +++++++++--------- arch/arm/mach-omap2/prm3xxx.c | 8 ++++---- arch/arm/mach-omap2/prm3xxx.h | 2 +- 3 files changed, 14 insertions(+), 14 deletions(-)
arch/arm/mach-omap2/pm34xx.c +9 -9
@@ -137,9 +137,8 @@ static irqreturn_t _prcm_int_handle_io(int irq, void *unused) {int c; - c =omap3xxx_prm_clear_mod_irqs (WKUP_MOD ,1 , - ~(OMAP3430_ST_IO_MASK | -OMAP3430_ST_IO_CHAIN_MASK )); + c =omap3xxx_prm_clear_mod_irqs (WKUP_MOD ,1 ,OMAP3430_ST_IO_MASK | +OMAP3430_ST_IO_CHAIN_MASK );return c ?IRQ_HANDLED :IRQ_NONE ; }
@@ -154,13 +153,14 @@ static irqreturn_t _prcm_int_handle_wakeup(int irq, void *unused)* IO events before parsing in mux code */ c =omap3xxx_prm_clear_mod_irqs (WKUP_MOD ,1 , -OMAP3430_ST_IO_MASK | -OMAP3430_ST_IO_CHAIN_MASK ); - c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,1 ,0 ); - c +=omap3xxx_prm_clear_mod_irqs (OMAP3430_PER_MOD ,1 ,0 ); + ~(OMAP3430_ST_IO_MASK | +OMAP3430_ST_IO_CHAIN_MASK )); + c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,1 , ~0 ); + c +=omap3xxx_prm_clear_mod_irqs (OMAP3430_PER_MOD ,1 , ~0 );if (omap_rev () >OMAP3430_REV_ES1_0 ) { - c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,3 ,0 ); - c +=omap3xxx_prm_clear_mod_irqs (OMAP3430ES2_USBHOST_MOD ,1 ,0 ); + c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,3 , ~0 ); + c +=omap3xxx_prm_clear_mod_irqs (OMAP3430ES2_USBHOST_MOD ,1 , + ~0 ); }return c ?IRQ_HANDLED :IRQ_NONE ;
arch/arm/mach-omap2/prm3xxx.c +4 -4
@@ -217,7 +217,7 @@ static void omap3xxx_prm_restore_irqen(u32 *saved_mask)* omap3xxx_prm_clear_mod_irqs - clear wake-up events from PRCM interrupt * @module: PRM module to clear wakeups from * @regs: register set to clear, 1 or 3 -* @ignore_bits: wakeup status bits to ignore +* @wkst_mask: wkst bits to clear * * The purpose of this function is to clear any wake-up events latched * in the PRCM PM_WKST_x registers. It is possible that a wake-up event
@@ -226,7 +226,7 @@ static void omap3xxx_prm_restore_irqen(u32 *saved_mask)* that any peripheral wake-up events occurring while attempting to * clear the PM_WKST_x are detected and cleared. */ -int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 ignore_bits) +int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) {u32 wkst, fclk, iclk, clken;u16 wkst_off = (regs ==3 ) ?OMAP3430ES2_PM_WKST3 :PM_WKST1 ;
@@ -238,7 +238,7 @@ int omap3xxx_prm_clear_mod_irqs(s16 module, u8 regs, u32 ignore_bits) wkst =omap2_prm_read_mod_reg (module, wkst_off); wkst &=omap2_prm_read_mod_reg (module, grpsel_off); - wkst &= ~ignore_bits; + wkst &= wkst_mask;if (wkst) { iclk =omap2_cm_read_mod_reg (module, iclk_off); fclk =omap2_cm_read_mod_reg (module, fclk_off);
@@ -254,7 +254,7 @@ int omap3xxx_prm_clear_mod_irqs(s16 module, u8 regs, u32 ignore_bits)omap2_cm_set_mod_reg_bits (clken, module, fclk_off);omap2_prm_write_mod_reg (wkst, module, wkst_off); wkst =omap2_prm_read_mod_reg (module, wkst_off); - wkst &= ~ignore_bits; + wkst &= wkst_mask; c++; }omap2_cm_write_mod_reg (iclk, module, iclk_off);
arch/arm/mach-omap2/prm3xxx.h +1 -1
@@ -145,7 +145,7 @@ extern void omap3_prm_vcvp_write(u32 val, u8 offset);extern u32 omap3_prm_vcvp_rmw (u32 mask,u32 bits,u8 offset);extern int __init omap3xxx_prm_init (void ); -int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 ignore_bits); +int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask);void omap3xxx_prm_iva_idle (void );void omap3_prm_reset_modem (void );int omap3xxx_prm_clear_global_cold_reset (void );
Signature · 1 line
1.7.9.5
PATCH v6 3/34 ARM: OMAP2+: PRM: add generic API for clear_mod_irqs
In reply to Tero KristoOMAP2/3 now use generic API for the prm_clear_mod_irqs, the SoC specific implementation details are provided through prm_ll_data.
--- arch/arm/mach-omap2/pm24xx.c | 24 +++++++++++------------- arch/arm/mach-omap2/pm34xx.c | 18 ++++++++---------- arch/arm/mach-omap2/prm.h | 2 ++ arch/arm/mach-omap2/prm2xxx.c | 4 +++- arch/arm/mach-omap2/prm2xxx.h | 2 -- arch/arm/mach-omap2/prm3xxx.c | 3 ++- arch/arm/mach-omap2/prm3xxx.h | 1 - arch/arm/mach-omap2/prm_common.c | 21 +++++++++++++++++++++ 8 files changed, 47 insertions(+), 28 deletions(-)
arch/arm/mach-omap2/pm24xx.c +11 -13
@@ -75,9 +75,9 @@ static int omap2_enter_full_retention(void)/* Clear old wake-up events */ /* REVISIT: These write to reserved bits? */ -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 ); -omap2xxx_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 ); +omap_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST , ~0 );pwrdm_set_next_pwrst (core_pwrdm,PWRDM_POWER_RET );pwrdm_set_next_pwrst (mpu_pwrdm,PWRDM_POWER_RET );
@@ -104,18 +104,16 @@ no_sleep:clk_enable (osc_ck);/* clear CORE wake-up events */ -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 );/* wakeup domain events - bit 1: GPT1, bit5 GPIO */ -omap2xxx_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST ,0x4 |0x1 ); +omap_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST ,0x4 |0x1 );/* MPU domain wake events */ -omap2xxx_prm_clear_mod_irqs (OCP_MOD ,OMAP2_PRCM_IRQSTATUS_MPU_OFFSET , -0x1 ); +omap_prm_clear_mod_irqs (OCP_MOD ,OMAP2_PRCM_IRQSTATUS_MPU_OFFSET ,0x1 ); -omap2xxx_prm_clear_mod_irqs (OCP_MOD ,OMAP2_PRCM_IRQSTATUS_MPU_OFFSET , -0x20 ); +omap_prm_clear_mod_irqs (OCP_MOD ,OMAP2_PRCM_IRQSTATUS_MPU_OFFSET ,0x20 );pwrdm_set_next_pwrst (mpu_pwrdm,PWRDM_POWER_ON );pwrdm_set_next_pwrst (core_pwrdm,PWRDM_POWER_ON );
@@ -143,9 +141,9 @@ static void omap2_enter_mpu_retention(void)* it is in retention mode. */ if (omap2_allow_mpu_retention ()) {/* REVISIT: These write to reserved bits? */ -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); -omap2xxx_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 ); -omap2xxx_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,PM_WKST1 , ~0 ); +omap_prm_clear_mod_irqs (CORE_MOD ,OMAP24XX_PM_WKST2 , ~0 ); +omap_prm_clear_mod_irqs (WKUP_MOD ,PM_WKST , ~0 );/* Try to enter MPU retention */ pwrdm_set_next_pwrst (mpu_pwrdm,PWRDM_POWER_RET );
arch/arm/mach-omap2/pm34xx.c +8 -10
@@ -137,8 +137,8 @@ static irqreturn_t _prcm_int_handle_io(int irq, void *unused) {int c; - c =omap3xxx_prm_clear_mod_irqs (WKUP_MOD ,1 ,OMAP3430_ST_IO_MASK | -OMAP3430_ST_IO_CHAIN_MASK ); + c =omap_prm_clear_mod_irqs (WKUP_MOD ,1 ,OMAP3430_ST_IO_MASK | +OMAP3430_ST_IO_CHAIN_MASK );return c ?IRQ_HANDLED :IRQ_NONE ; }
@@ -152,15 +152,13 @@ static irqreturn_t _prcm_int_handle_wakeup(int irq, void *unused)* these are handled in a separate handler to avoid acking * IO events before parsing in mux code */ - c =omap3xxx_prm_clear_mod_irqs (WKUP_MOD ,1 , - ~(OMAP3430_ST_IO_MASK | -OMAP3430_ST_IO_CHAIN_MASK )); - c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,1 , ~0 ); - c +=omap3xxx_prm_clear_mod_irqs (OMAP3430_PER_MOD ,1 , ~0 ); + c =omap_prm_clear_mod_irqs (WKUP_MOD ,1 , ~(OMAP3430_ST_IO_MASK | +OMAP3430_ST_IO_CHAIN_MASK )); + c +=omap_prm_clear_mod_irqs (CORE_MOD ,1 , ~0 ); + c +=omap_prm_clear_mod_irqs (OMAP3430_PER_MOD ,1 , ~0 );if (omap_rev () >OMAP3430_REV_ES1_0 ) { - c +=omap3xxx_prm_clear_mod_irqs (CORE_MOD ,3 , ~0 ); - c +=omap3xxx_prm_clear_mod_irqs (OMAP3430ES2_USBHOST_MOD ,1 , - ~0 ); + c +=omap_prm_clear_mod_irqs (CORE_MOD ,3 , ~0 ); + c +=omap_prm_clear_mod_irqs (OMAP3430ES2_USBHOST_MOD ,1 , ~0 ); }return c ?IRQ_HANDLED :IRQ_NONE ;
arch/arm/mach-omap2/prm.h +2 -0
@@ -146,6 +146,7 @@ struct prm_ll_data {int (*is_hardreset_asserted)(u8 shift,u8 part,s16 prm_mod,u16 offset);void (*reset_system)(void ); +int (*clear_mod_irqs)(s16 module,u8 regs,u32 wkst_mask); };extern int prm_register (struct prm_ll_data *pld);
@@ -161,6 +162,7 @@ extern void prm_clear_context_loss_flags_old(u8 part, s16 inst, u16 idx);void omap_prm_reset_system (void );void omap_prm_reconfigure_io_chain (void ); +int omap_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask);#endif
arch/arm/mach-omap2/prm2xxx.c +3 -1
@@ -123,13 +123,14 @@ static void omap2xxx_prm_dpll_reset(void)* Clears wakeup status bits for a given module, so that the device can * re-enter idle. */ -void omap2xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) +static int omap2xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) {u32 wkst; wkst =omap2_prm_read_mod_reg (module, regs); wkst &= wkst_mask;omap2_prm_write_mod_reg (wkst, module, regs); +return 0 ; }int omap2xxx_clkdm_sleep (struct clockdomain *clkdm)
@@ -216,6 +217,7 @@ static struct prm_ll_data omap2xxx_prm_ll_data = { .deassert_hardreset = &omap2_prm_deassert_hardreset, .is_hardreset_asserted = &omap2_prm_is_hardreset_asserted, .reset_system = &omap2xxx_prm_dpll_reset, + .clear_mod_irqs = &omap2xxx_prm_clear_mod_irqs, };int __init omap2xxx_prm_init (void )
arch/arm/mach-omap2/prm2xxx.h +0 -2
@@ -124,8 +124,6 @@extern int omap2xxx_clkdm_sleep (struct clockdomain *clkdm);extern int omap2xxx_clkdm_wakeup (struct clockdomain *clkdm); -void omap2xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask); -extern int __init omap2xxx_prm_init (void );#endif
arch/arm/mach-omap2/prm3xxx.c +2 -1
@@ -226,7 +226,7 @@ static void omap3xxx_prm_restore_irqen(u32 *saved_mask)* that any peripheral wake-up events occurring while attempting to * clear the PM_WKST_x are detected and cleared. */ -int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) +static int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) {u32 wkst, fclk, iclk, clken;u16 wkst_off = (regs ==3 ) ?OMAP3430ES2_PM_WKST3 :PM_WKST1 ;
@@ -664,6 +664,7 @@ static struct prm_ll_data omap3xxx_prm_ll_data = { .deassert_hardreset = &omap2_prm_deassert_hardreset, .is_hardreset_asserted = &omap2_prm_is_hardreset_asserted, .reset_system = &omap3xxx_prm_dpll3_reset, + .clear_mod_irqs = &omap3xxx_prm_clear_mod_irqs, };int __init omap3xxx_prm_init (void )
arch/arm/mach-omap2/prm3xxx.h +0 -1
@@ -145,7 +145,6 @@ extern void omap3_prm_vcvp_write(u32 val, u8 offset);extern u32 omap3_prm_vcvp_rmw (u32 mask,u32 bits,u8 offset);extern int __init omap3xxx_prm_init (void ); -int omap3xxx_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask);void omap3xxx_prm_iva_idle (void );void omap3_prm_reset_modem (void );int omap3xxx_prm_clear_global_cold_reset (void );
arch/arm/mach-omap2/prm_common.c +21 -0
@@ -534,6 +534,27 @@ void omap_prm_reset_system(void) }/** +* omap_prm_clear_mod_irqs - clear wake-up events from PRCM interrupt +* @module: PRM module to clear wakeups from +* @regs: register to clear +* @wkst_mask: wkst bits to clear +* +* Clears any wakeup events for the module and register set defined. +* Uses SoC specific implementation to do the actual wakeup status +* clearing. +*/ +int omap_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask) +{ +if (!prm_ll_data->clear_mod_irqs) { +WARN_ONCE (1 ,"prm: %s: no mapping function defined\n" , + __func__); +return -EINVAL ; + } + +return prm_ll_data->clear_mod_irqs (module, regs, wkst_mask); +} + +/** * prm_register - register per-SoC low-level data with the PRM * @pld: low-level per-SoC OMAP PRM data & function pointers to register *
Signature · 1 line
1.7.9.5
PATCH v6 4/34 ARM: OMAP3+: PRM: add common APIs for prm_vp_check/clear_txdone
In reply to Tero KristoPRM driver now only exports a generic API for clearing / checking VP txdone status.
--- arch/arm/mach-omap2/prm.h | 14 ++++++++++++++ arch/arm/mach-omap2/prm3xxx.c | 6 ++++-- arch/arm/mach-omap2/prm3xxx.h | 4 ---- arch/arm/mach-omap2/prm44xx.c | 6 ++++-- arch/arm/mach-omap2/prm44xx_54xx.h | 4 ---- arch/arm/mach-omap2/prm_common.c | 34 ++++++++++++++++++++++++++++++++++ arch/arm/mach-omap2/vp.h | 9 --------- arch/arm/mach-omap2/vp3xxx_data.c | 4 ++-- arch/arm/mach-omap2/vp44xx_data.c | 4 ++-- 9 files changed, 60 insertions(+), 25 deletions(-)
arch/arm/mach-omap2/prm.h +14 -0
@@ -147,6 +147,8 @@ struct prm_ll_data { u16 offset);void (*reset_system)(void );int (*clear_mod_irqs)(s16 module,u8 regs,u32 wkst_mask); +u32 (*vp_check_txdone)(u8 vp_id); +void (*vp_clear_txdone)(u8 vp_id); };extern int prm_register (struct prm_ll_data *pld);
@@ -164,6 +166,18 @@ void omap_prm_reset_system(void);void omap_prm_reconfigure_io_chain (void );int omap_prm_clear_mod_irqs (s16 module,u8 regs,u32 wkst_mask); +/* +* Voltage Processor (VP) identifiers +*/ +#define OMAP3_VP_VDD_MPU_ID 0 +#define OMAP3_VP_VDD_CORE_ID 1 +#define OMAP4_VP_VDD_CORE_ID 0 +#define OMAP4_VP_VDD_IVA_ID 1 +#define OMAP4_VP_VDD_MPU_ID 2 + +u32 omap_prm_vp_check_txdone (u8 vp_id); +void omap_prm_vp_clear_txdone (u8 vp_id); +#endif
arch/arm/mach-omap2/prm3xxx.c +4 -2
@@ -96,7 +96,7 @@ static struct omap3_vp omap3_vp[] = {#define MAX_VP_ID ARRAY_SIZE(omap3_vp); -u32 omap3_prm_vp_check_txdone (u8 vp_id) +static u32 omap3_prm_vp_check_txdone (u8 vp_id) {struct omap3_vp *vp = &omap3_vp[vp_id];u32 irqstatus;
@@ -106,7 +106,7 @@ u32 omap3_prm_vp_check_txdone(u8 vp_id)return irqstatus & vp->tranxdone_status; } -void omap3_prm_vp_clear_txdone (u8 vp_id) +static void omap3_prm_vp_clear_txdone (u8 vp_id) {struct omap3_vp *vp = &omap3_vp[vp_id];
@@ -665,6 +665,8 @@ static struct prm_ll_data omap3xxx_prm_ll_data = { .is_hardreset_asserted = &omap2_prm_is_hardreset_asserted, .reset_system = &omap3xxx_prm_dpll3_reset, .clear_mod_irqs = &omap3xxx_prm_clear_mod_irqs, + .vp_check_txdone = &omap3_prm_vp_check_txdone, + .vp_clear_txdone = &omap3_prm_vp_clear_txdone, };int __init omap3xxx_prm_init (void )
arch/arm/mach-omap2/prm3xxx.h +0 -4
@@ -132,10 +132,6 @@#ifndef __ASSEMBLER__ -/* OMAP3-specific VP functions */ -u32 omap3_prm_vp_check_txdone (u8 vp_id); -void omap3_prm_vp_clear_txdone (u8 vp_id); -/* * OMAP3 access functions for voltage controller (VC) and * voltage proccessor (VP) in the PRM.
arch/arm/mach-omap2/prm44xx.c +4 -2
@@ -138,7 +138,7 @@ static struct omap4_vp omap4_vp[] = { }, }; -u32 omap4_prm_vp_check_txdone (u8 vp_id) +static u32 omap4_prm_vp_check_txdone (u8 vp_id) {struct omap4_vp *vp = &omap4_vp[vp_id];u32 irqstatus;
@@ -149,7 +149,7 @@ u32 omap4_prm_vp_check_txdone(u8 vp_id)return irqstatus & vp->tranxdone_status; } -void omap4_prm_vp_clear_txdone (u8 vp_id) +static void omap4_prm_vp_clear_txdone (u8 vp_id) {struct omap4_vp *vp = &omap4_vp[vp_id];
@@ -699,6 +699,8 @@ static struct prm_ll_data omap44xx_prm_ll_data = { .deassert_hardreset = omap4_prminst_deassert_hardreset, .is_hardreset_asserted = omap4_prminst_is_hardreset_asserted, .reset_system = omap4_prminst_global_warm_sw_reset, + .vp_check_txdone = omap4_prm_vp_check_txdone, + .vp_clear_txdone = omap4_prm_vp_clear_txdone, };int __init omap44xx_prm_init (void )
arch/arm/mach-omap2/prm44xx_54xx.h +0 -4
@@ -26,10 +26,6 @@/* Function prototypes */ #ifndef __ASSEMBLER__ -/* OMAP4/OMAP5-specific VP functions */ -u32 omap4_prm_vp_check_txdone (u8 vp_id); -void omap4_prm_vp_clear_txdone (u8 vp_id); -/* * OMAP4/OMAP5 access functions for voltage controller (VC) and * voltage proccessor (VP) in the PRM.
arch/arm/mach-omap2/prm_common.c +34 -0
@@ -555,6 +555,40 @@ int omap_prm_clear_mod_irqs(s16 module, u8 regs, u32 wkst_mask) }/** +* omap_prm_vp_check_txdone - check voltage processor TX done status +* +* Checks if voltage processor transmission has been completed. +* Returns non-zero if a transmission has completed, 0 otherwise. +*/ +u32 omap_prm_vp_check_txdone (u8 vp_id) +{ +if (!prm_ll_data->vp_check_txdone) { +WARN_ONCE (1 ,"prm: %s: no mapping function defined\n" , + __func__); +return 0 ; + } + +return prm_ll_data->vp_check_txdone (vp_id); +} + +/** +* omap_prm_vp_clear_txdone - clears voltage processor TX done status +* +* Clears the status bit for completed voltage processor transmission +* returned by prm_vp_check_txdone. +*/ +void omap_prm_vp_clear_txdone (u8 vp_id) +{ +if (!prm_ll_data->vp_clear_txdone) { +WARN_ONCE (1 ,"prm: %s: no mapping function defined\n" , + __func__); +return ; + } + + prm_ll_data->vp_clear_txdone (vp_id); +} + +/** * prm_register - register per-SoC low-level data with the PRM * @pld: low-level per-SoC OMAP PRM data & function pointers to register *
arch/arm/mach-omap2/vp.h +0 -9
@@ -21,15 +21,6 @@struct voltagedomain ; -/* -* Voltage Processor (VP) identifiers -*/ -#define OMAP3_VP_VDD_MPU_ID 0 -#define OMAP3_VP_VDD_CORE_ID 1 -#define OMAP4_VP_VDD_CORE_ID 0 -#define OMAP4_VP_VDD_IVA_ID 1 -#define OMAP4_VP_VDD_MPU_ID 2 -/* XXX document */ #define VP_IDLE_TIMEOUT 200#define VP_TRANXDONE_TIMEOUT 300
arch/arm/mach-omap2/vp3xxx_data.c +2 -2
@@ -28,8 +28,8 @@#include "prm2xxx_3xxx.h" static const struct omap_vp_ops omap3_vp_ops = { - .check_txdone = omap3_prm_vp_check_txdone, - .clear_txdone = omap3_prm_vp_clear_txdone, + .check_txdone = omap_prm_vp_check_txdone, + .clear_txdone = omap_prm_vp_clear_txdone, };/*
arch/arm/mach-omap2/vp44xx_data.c +2 -2
@@ -28,8 +28,8 @@#include "vp.h" static const struct omap_vp_ops omap4_vp_ops = { - .check_txdone = omap4_prm_vp_check_txdone, - .clear_txdone = omap4_prm_vp_clear_txdone, + .check_txdone = omap_prm_vp_check_txdone, + .clear_txdone = omap_prm_vp_clear_txdone, };/*
Signature · 1 line
1.7.9.5
PATCH v6 5/34 ARM: OMAP4+: PRM: move omap_prm_base_init under OMAP4 PRM driver
In reply to Tero KristoThere is no need to call this separately from io.c, rather this can be done commonly under the PRM driver.
--- arch/arm/mach-omap2/io.c | 4 ---- arch/arm/mach-omap2/prm44xx.c | 2 ++ 2 files changed, 2 insertions(+), 4 deletions(-)
arch/arm/mach-omap2/io.c +0 -4
@@ -623,7 +623,6 @@ void __init am43xx_init_early(void) NULL);omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ));omap2_set_globals_cm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ),NULL ); -omap_prm_base_init ();omap_cm_base_init ();omap3xxx_check_revision ();am33xx_check_features ();
@@ -654,7 +653,6 @@ void __init omap4430_init_early(void)omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP4430_CM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP4430_CM2_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE )); -omap_prm_base_init ();omap_cm_base_init ();omap4xxx_check_revision ();omap4xxx_check_features ();
@@ -690,7 +688,6 @@ void __init omap5_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap_prm_base_init ();omap_cm_base_init ();omap44xx_prm_init ();omap5xxx_check_revision ();
@@ -722,7 +719,6 @@ void __init dra7xx_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap_prm_base_init ();omap_cm_base_init ();omap44xx_prm_init ();dra7xxx_check_revision ();
arch/arm/mach-omap2/prm44xx.c +2 -0
@@ -705,6 +705,8 @@ static struct prm_ll_data omap44xx_prm_ll_data = {int __init omap44xx_prm_init (void ) { +omap_prm_base_init (); +if (cpu_is_omap44xx () ||soc_is_omap54xx () ||soc_is_dra7xx ()) prm_features |=PRM_HAS_IO_WAKEUP ;
Signature · 1 line
1.7.9.5
PATCH v6 6/34 ARM: OMAP4+: CM: move omap_cm_base_init under OMAP4 CM driver
In reply to Tero KristoThere is no need to call this separately from io.c, rather this can be done commonly under the CM driver. Also, this patch makes the API static, as it is no longer used outside the driver file.
--- arch/arm/mach-omap2/cm44xx.h | 1 - arch/arm/mach-omap2/cminst44xx.c | 4 +++- arch/arm/mach-omap2/io.c | 4 ---- 3 files changed, 3 insertions(+), 6 deletions(-)
arch/arm/mach-omap2/cm44xx.h +0 -1
@@ -23,7 +23,6 @@#define OMAP4_CM_CLKSTCTRL 0x0000#define OMAP4_CM_STATICDEP 0x0004 -void omap_cm_base_init (void );int omap4_cm_init (void );#endif
arch/arm/mach-omap2/cminst44xx.c +3 -1
@@ -63,7 +63,7 @@ static void __iomem *_cm_bases[OMAP4_MAX_PRCM_PARTITIONS];* Populates the base addresses of the _cm_bases * array used for read/write of cm module registers. */ -void omap_cm_base_init (void ) +static void omap_cm_base_init (void ) { _cm_bases[OMAP4430_PRM_PARTITION ] = prm_base; _cm_bases[OMAP4430_CM1_PARTITION ] = cm_base;
@@ -516,6 +516,8 @@ static struct cm_ll_data omap4xxx_cm_ll_data = {int __init omap4_cm_init (void ) { +omap_cm_base_init (); +return cm_register (&omap4xxx_cm_ll_data); }
arch/arm/mach-omap2/io.c +0 -4
@@ -623,7 +623,6 @@ void __init am43xx_init_early(void) NULL);omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ));omap2_set_globals_cm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ),NULL ); -omap_cm_base_init ();omap3xxx_check_revision ();am33xx_check_features ();omap44xx_prm_init ();
@@ -653,7 +652,6 @@ void __init omap4430_init_early(void)omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP4430_CM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP4430_CM2_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE )); -omap_cm_base_init ();omap4xxx_check_revision ();omap4xxx_check_features ();omap4_cm_init ();
@@ -688,7 +686,6 @@ void __init omap5_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap_cm_base_init ();omap44xx_prm_init ();omap5xxx_check_revision ();omap4_cm_init ();
@@ -719,7 +716,6 @@ void __init dra7xx_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_CM_CORE_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap_cm_base_init ();omap44xx_prm_init ();dra7xxx_check_revision ();omap4_cm_init ();
Signature · 1 line
1.7.9.5
PATCH v6 7/34 ARM: OMAP4: PRM: move omap4xxx_prm_init earlier in init order
In reply to Tero KristoOMAP4 has different ordering of PRM and CM init calls in the early init. Re-oder these accordingly for OMAP4 also. This is needed so that we can do some optimizations in the following patches for the PRCM init.
--- arch/arm/mach-omap2/io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-omap2/io.c +1 -1
@@ -654,9 +654,9 @@ void __init omap4430_init_early(void)omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE ));omap4xxx_check_revision ();omap4xxx_check_features (); +omap44xx_prm_init ();omap4_cm_init ();omap4_pm_init_early (); -omap44xx_prm_init ();omap44xx_voltagedomains_init ();omap44xx_powerdomains_init ();omap44xx_clockdomains_init ();
Signature · 1 line
1.7.9.5
PATCH v6 8/34 Documentation: DT: document PRCM compatible strings for dm81x SoCs
In reply to Tero KristoThese PRCM nodes were earlier added in patch 7800064ba5 ("ARM: dts: Add basic dm816x device tree configuration"), but the documentation for the same wasn't added. Fix this by adding the missing compatible strings under the generic prcm.txt document.
--- .../devicetree/bindings/arm/omap/prcm.txt | 4 ++++ 1 file changed, 4 insertions(+)
Documentation/devicetree/bindings/arm/omap/prcm.txt +4 -0
@@ -29,6 +29,10 @@ Required properties: "ti,dra7-prm" "ti,dra7-cm-core-aon" "ti,dra7-cm-core" + "ti,dm814-prcm" + "ti,dm814-scrm" + "ti,dm816-prcm" + "ti,dm816-scrm" - reg: Contains PRCM module register address range (base address and length) - clocks: clocks for this module
Signature · 1 line
1.7.9.5
PATCH v6 9/34 ARM: OMAP2+: PRCM: add support for static clock memmap indices
In reply to Tero KristoAll clock provider related drivers will now register their iomaps with a static index. This makes it easier to split up the individual drivers to their own files in subsequent patches.
--- arch/arm/mach-omap2/prcm-common.h | 8 ++++ arch/arm/mach-omap2/prm_common.c | 79 +++++++++++++++++++++++-------------- include/linux/clk/ti.h | 1 + 3 files changed, 59 insertions(+), 29 deletions(-)
arch/arm/mach-omap2/prcm-common.h +8 -0
@@ -518,6 +518,14 @@ struct omap_prcm_irq_setup { .priority = _priority \ } +/** +* struct omap_prcm_init_data - PRCM driver init data +* @index: clock memory mapping index to be used +*/ +struct omap_prcm_init_data { +int index; +}; +extern void omap_prcm_irq_cleanup (void );extern int omap_prcm_register_chain_handler (struct omap_prcm_irq_setup *irq_setup);
arch/arm/mach-omap2/prm_common.c +50 -29
96 lines not on this page · raw message
include/linux/clk/ti.h +1 -0
@@ -221,6 +221,7 @@ struct ti_dt_clk {/* Static memmap indices */ enum {TI_CLKM_CM =0 , +TI_CLKM_CM2 ,TI_CLKM_PRM ,TI_CLKM_SCRM , };
Signature · 1 line
1.7.9.5
PATCH v6 10/34 ARM: OMAP2+: clock: move clock provider infrastructure to clock driver
In reply to Tero KristoSplits the clock provider init out of the PRM driver and moves it to clock driver. This is needed so that once the PRCM drivers are separated, they can logically just access the clock driver not needing to go through common PRM code. This would be wrong in the case of control module for example.
--- arch/arm/mach-omap2/clock.c | 77 +++++++++++++++++++++++++++++++------- arch/arm/mach-omap2/clock.h | 6 ++- arch/arm/mach-omap2/prm_common.c | 36 ++++-------------- 3 files changed, 75 insertions(+), 44 deletions(-)
arch/arm/mach-omap2/clock.c +63 -14
@@ -23,6 +23,7 @@#include <linux/clk-provider.h> #include <linux/io.h> #include <linux/bitops.h> +#include <linux/of_address.h> #include <asm/cpu.h> #include <trace/events/power.h>
@@ -72,30 +73,78 @@ struct ti_clk_features ti_clk_features;static bool clkdm_control = true;static LIST_HEAD (clk_hw_omap_clocks); -void __iomem *clk_memmaps[CLK_MAX_MEMMAPS ]; +static void __iomem *clk_memmaps[CLK_MAX_MEMMAPS ]; + +static void clk_memmap_writel (u32 val,void __iomem *reg) +{ +struct clk_omap_reg *r = (struct clk_omap_reg *)® + +writel_relaxed (val, clk_memmaps[r->index] + r->offset); +} + +static u32 clk_memmap_readl (void __iomem *reg) +{ +struct clk_omap_reg *r = (struct clk_omap_reg *)® + +return readl_relaxed (clk_memmaps[r->index] + r->offset); +}void omap2_clk_writel (u32 val,struct clk_hw_omap *clk,void __iomem *reg) { -if (clk->flags &MEMMAP_ADDRESSING ) { -struct clk_omap_reg *r = (struct clk_omap_reg *)® -writel_relaxed (val, clk_memmaps[r->index] + r->offset); - }else { +if (WARN_ON_ONCE (!(clk->flags &MEMMAP_ADDRESSING )))writel_relaxed (val, reg); - } +else +clk_memmap_writel (val, reg); }u32 omap2_clk_readl (struct clk_hw_omap *clk,void __iomem *reg) { -u32 val; +if (WARN_ON_ONCE (!(clk->flags &MEMMAP_ADDRESSING ))) +return readl_relaxed (reg); +else +return clk_memmap_readl (reg); +} -if (clk->flags &MEMMAP_ADDRESSING ) { -struct clk_omap_reg *r = (struct clk_omap_reg *)® - val =readl_relaxed (clk_memmaps[r->index] + r->offset); - }else { - val =readl_relaxed (reg); - } +static struct ti_clk_ll_ops omap_clk_ll_ops = { + .clk_readl = clk_memmap_readl, + .clk_writel = clk_memmap_writel, +}; + +/** +* omap2_clk_provider_init - initialize a clock provider +* @match_table: DT device table to match for devices to init +* @np: device node pointer for the this clock provider +* @index: index for the clock provider +* @mem: iomem pointer for the clock provider memory area +* +* Initializes a clock provider module (CM/PRM etc.), registering +* the memory mapping at specified index and initializing the +* low level driver infrastructure. Returns 0 in success. +*/ +int __init omap2_clk_provider_init (struct device_node *np,int index, +void __iomem *mem) +{ + ti_clk_ll_ops = &omap_clk_ll_ops; + + clk_memmaps[index] = mem; + +ti_dt_clk_init_provider (np, index); + +return 0 ; +} + +/** +* omap2_clk_legacy_provider_init - initialize a legacy clock provider +* @index: index for the clock provider +* @mem: iomem pointer for the clock provider memory area +* +* Initializes a legacy clock provider memory mapping. +*/ +void __init omap2_clk_legacy_provider_init (int index,void __iomem *mem) +{ + ti_clk_ll_ops = &omap_clk_ll_ops; -return val; + clk_memmaps[index] = mem; }/*
arch/arm/mach-omap2/clock.h +4 -2
@@ -271,10 +271,12 @@ extern const struct clksel_rate div_1_3_rates[];extern const struct clksel_rate div_1_4_rates[];extern const struct clksel_rate div31_1to31_rates[]; -extern void __iomem *clk_memmaps[]; -extern int omap2_clkops_enable_clkdm (struct clk_hw *hw);extern void omap2_clkops_disable_clkdm (struct clk_hw *hw); +int __init omap2_clk_provider_init (struct device_node *np,int index, +void __iomem *mem); +void __init omap2_clk_legacy_provider_init (int index,void __iomem *mem); +void __init ti_clk_init_features (void );#endif
arch/arm/mach-omap2/prm_common.c +8 -28
@@ -677,25 +677,6 @@ static const struct of_device_id omap_prcm_dt_match_table[] = { { } }; -static struct clk_hw_omap memmap_dummy_ck = { - .flags =MEMMAP_ADDRESSING , -}; - -static u32 prm_clk_readl (void __iomem *reg) -{ -return omap2_clk_readl (&memmap_dummy_ck, reg); -} - -static void prm_clk_writel (u32 val,void __iomem *reg) -{ -omap2_clk_writel (val, &memmap_dummy_ck, reg); -} - -static struct ti_clk_ll_ops omap_clk_ll_ops = { - .clk_readl = prm_clk_readl, - .clk_writel = prm_clk_writel, -}; -/** * omap_prcm_init - low level init for the PRCM drivers *
@@ -708,8 +689,7 @@ int __init omap_prcm_init(void)void __iomem *mem;const struct of_device_id *match;const struct omap_prcm_init_data *data; - - ti_clk_ll_ops = &omap_clk_ll_ops; +int ret;for_each_matching_node_and_match (np, omap_prcm_dt_match_table, &match) { data = match->data;
@@ -718,8 +698,9 @@ int __init omap_prcm_init(void)if (!mem)return -ENOMEM ; - clk_memmaps[data->index] = mem; -ti_dt_clk_init_provider (np, data->index); + ret =omap2_clk_provider_init (np, data->index, mem); +if (ret) +return ret; }return 0 ;
@@ -727,11 +708,10 @@ int __init omap_prcm_init(void)void __init omap3_prcm_legacy_iomaps_init (void ) { - ti_clk_ll_ops = &omap_clk_ll_ops; - - clk_memmaps[TI_CLKM_CM ] = cm_base +OMAP3430_IVA2_MOD ; - clk_memmaps[TI_CLKM_PRM ] = prm_base +OMAP3430_IVA2_MOD ; - clk_memmaps[TI_CLKM_SCRM ] =omap_ctrl_base_get (); +omap2_clk_legacy_provider_init (TI_CLKM_CM , cm_base +OMAP3430_IVA2_MOD ); +omap2_clk_legacy_provider_init (TI_CLKM_PRM , + prm_base +OMAP3430_IVA2_MOD ); +omap2_clk_legacy_provider_init (TI_CLKM_SCRM ,omap_ctrl_base_get ()); }static int __init prm_late_init (void )
Signature · 1 line
1.7.9.5
PATCH v6 11/34 ARM: OMAP2+: PRCM: split PRCM module init to their own driver files
In reply to Tero KristoSplits the clock related provider module inits under their own driver files. Previously this was done for all modules under the common PRM driver.
--- arch/arm/mach-omap2/cm.h | 1 + arch/arm/mach-omap2/cm_common.c | 51 ++++++++++++++++++++++++++++++++++++++ arch/arm/mach-omap2/control.c | 47 +++++++++++++++++++++++++++++++++++ arch/arm/mach-omap2/control.h | 1 + arch/arm/mach-omap2/io.c | 4 +++ arch/arm/mach-omap2/prm_common.c | 23 ++--------------- include/linux/clk/ti.h | 5 ++-- 7 files changed, 108 insertions(+), 24 deletions(-)
arch/arm/mach-omap2/cm.h +1 -0
@@ -70,6 +70,7 @@ int omap_cm_module_enable(u8 mode, u8 part, u16 inst, u16 clkctrl_offs);int omap_cm_module_disable (u8 part,u16 inst,u16 clkctrl_offs);extern int cm_register (struct cm_ll_data *cld);extern int cm_unregister (struct cm_ll_data *cld); +int omap_cm_init (void );# endif
arch/arm/mach-omap2/cm_common.c +51 -0
64 lines not on this page · raw message
arch/arm/mach-omap2/control.c +47 -0
62 lines not on this page · raw message
arch/arm/mach-omap2/control.h +1 -0
@@ -464,6 +464,7 @@ extern void omap_ctrl_write_dsp_boot_mode(u8 bootmode);extern void omap3630_ctrl_disable_rta (void );extern int omap3_ctrl_save_padconf (void );void omap3_ctrl_init (void ); +int omap_control_init (void );extern void omap2_set_globals_control (void __iomem *ctrl,void __iomem *ctrl_pad);#else
arch/arm/mach-omap2/io.c +4 -0
@@ -756,6 +756,10 @@ int __init omap_clk_init(void)ti_clk_init_features ();if (of_have_populated_dt ()) { + ret =omap_control_init (); +if (ret) +return ret; + ret =omap_prcm_init ();if (ret)return ret;
arch/arm/mach-omap2/prm_common.c +2 -21
@@ -637,43 +637,22 @@ static struct omap_prcm_init_data prm_data = { .index = TI_CLKM_PRM, }; -static struct omap_prcm_init_data cm_data = { - .index =TI_CLKM_CM , -}; - -static struct omap_prcm_init_data cm2_data = { - .index =TI_CLKM_CM2 , -}; -static struct omap_prcm_init_data scrm_data = { .index =TI_CLKM_SCRM , };static const struct of_device_id omap_prcm_dt_match_table[] = { { .compatible ="ti,am3-prcm" , .data = &prm_data }, - { .compatible ="ti,am3-scrm" , .data = &scrm_data }, { .compatible ="ti,am4-prcm" , .data = &prm_data }, - { .compatible ="ti,am4-scrm" , .data = &scrm_data }, { .compatible ="ti,dm814-prcm" , .data = &prm_data }, - { .compatible ="ti,dm814-scrm" , .data = &scrm_data }, { .compatible ="ti,dm816-prcm" , .data = &prm_data }, - { .compatible ="ti,dm816-scrm" , .data = &scrm_data }, { .compatible ="ti,omap2-prcm" , .data = &prm_data }, - { .compatible ="ti,omap2-scrm" , .data = &scrm_data }, { .compatible ="ti,omap3-prm" , .data = &prm_data }, - { .compatible ="ti,omap3-cm" , .data = &cm_data }, - { .compatible ="ti,omap3-scrm" , .data = &scrm_data }, - { .compatible ="ti,omap4-cm1" , .data = &cm_data }, { .compatible ="ti,omap4-prm" , .data = &prm_data }, - { .compatible ="ti,omap4-cm2" , .data = &cm2_data }, { .compatible ="ti,omap4-scrm" , .data = &scrm_data }, { .compatible ="ti,omap5-prm" , .data = &prm_data }, - { .compatible ="ti,omap5-cm-core-aon" , .data = &cm_data }, { .compatible ="ti,omap5-scrm" , .data = &scrm_data }, - { .compatible ="ti,omap5-cm-core" , .data = &cm2_data }, { .compatible ="ti,dra7-prm" , .data = &prm_data }, - { .compatible ="ti,dra7-cm-core-aon" , .data = &cm_data }, - { .compatible ="ti,dra7-cm-core" , .data = &cm2_data }, { } };
@@ -703,6 +682,8 @@ int __init omap_prcm_init(void)return ret; } +omap_cm_init (); +return 0 ; }
include/linux/clk/ti.h +2 -3
@@ -215,15 +215,14 @@ struct ti_dt_clk { .node_name = name, \ } -/* Maximum number of clock memmaps */ -#define CLK_MAX_MEMMAPS 4 -/* Static memmap indices */ enum {TI_CLKM_CM =0 ,TI_CLKM_CM2 ,TI_CLKM_PRM ,TI_CLKM_SCRM , +TI_CLKM_CTRL , +CLK_MAX_MEMMAPS };typedef void (*ti_of_clk_init_cb_t )(struct clk_hw *,struct device_node *);
Signature · 1 line
1.7.9.5
PATCH v6 12/34 ARM: OMAP2+: CM: determine CM base address from device tree
In reply to Tero KristoThere is no need to provide the CM base address through a low-level API from the low-level IO init, as this information is available through DT. Re-routed the parsing function to be called from the CM drivers also to simplify the implementation under io.c.
--- arch/arm/mach-omap2/cm.h | 1 + arch/arm/mach-omap2/cm2xxx.c | 1 + arch/arm/mach-omap2/cm33xx.c | 1 + arch/arm/mach-omap2/cm3xxx.c | 2 + arch/arm/mach-omap2/cm_common.c | 73 ++++++++++++++++++++++++++++++++++--- arch/arm/mach-omap2/cminst44xx.c | 1 + arch/arm/mach-omap2/io.c | 14 +------ arch/arm/mach-omap2/prcm-common.h | 6 +++ arch/arm/mach-omap2/prm_common.c | 1 - 9 files changed, 81 insertions(+), 19 deletions(-)
arch/arm/mach-omap2/cm.h +1 -0
@@ -71,6 +71,7 @@ int omap_cm_module_disable(u8 part, u16 inst, u16 clkctrl_offs);extern int cm_register (struct cm_ll_data *cld);extern int cm_unregister (struct cm_ll_data *cld);int omap_cm_init (void ); +int omap2_cm_base_init (void );# endif
arch/arm/mach-omap2/cm2xxx.c +1 -0
@@ -395,6 +395,7 @@ static struct cm_ll_data omap2xxx_cm_ll_data = {int __init omap2xxx_cm_init (void ) { +omap2_cm_base_init ();return cm_register (&omap2xxx_cm_ll_data); }
arch/arm/mach-omap2/cm33xx.c +1 -0
@@ -354,6 +354,7 @@ static struct cm_ll_data am33xx_cm_ll_data = {int __init am33xx_cm_init (void ) { +omap2_cm_base_init ();return cm_register (&am33xx_cm_ll_data); }
arch/arm/mach-omap2/cm3xxx.c +2 -0
@@ -673,6 +673,8 @@ static struct cm_ll_data omap3xxx_cm_ll_data = {int __init omap3xxx_cm_init (void ) { +omap2_clk_legacy_provider_init (TI_CLKM_CM , cm_base +OMAP3430_IVA2_MOD ); +omap2_cm_base_init ();return cm_register (&omap3xxx_cm_ll_data); }
arch/arm/mach-omap2/cm_common.c +67 -6
109 lines not on this page · raw message
arch/arm/mach-omap2/cminst44xx.c +1 -0
@@ -516,6 +516,7 @@ static struct cm_ll_data omap4xxx_cm_ll_data = {int __init omap4_cm_init (void ) { +omap2_cm_base_init ();omap_cm_base_init ();return cm_register (&omap4xxx_cm_ll_data);
arch/arm/mach-omap2/io.c +2 -12
@@ -387,7 +387,6 @@ void __init omap2420_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP242X_CTRL_BASE ),NULL );omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP2420_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP2420_CM_BASE ),NULL );omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -417,7 +416,6 @@ void __init omap2430_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP243X_CTRL_BASE ),NULL );omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP2430_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP2430_CM_BASE ),NULL );omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -451,6 +449,8 @@ void __init omap3_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ),NULL );omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP3430_PRM_BASE )); + +/* XXX: remove this once OMAP3 is DT only */ omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP3430_CM_BASE ),NULL );omap3xxx_check_revision ();omap3xxx_check_features ();
@@ -552,7 +552,6 @@ void __init ti814x_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ),NULL );omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE ),NULL );omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -573,7 +572,6 @@ void __init ti816x_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ),NULL );omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE ),NULL );omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -596,7 +594,6 @@ void __init am33xx_init_early(void)omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ),NULL );omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM33XX_PRCM_BASE )); -omap2_set_globals_cm (AM33XX_L4_WK_IO_ADDRESS (AM33XX_PRCM_BASE ),NULL );omap3xxx_check_revision ();am33xx_check_features ();am33xx_prm_init ();
@@ -622,7 +619,6 @@ void __init am43xx_init_early(void)omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ),NULL );omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE )); -omap2_set_globals_cm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ),NULL );omap3xxx_check_revision ();am33xx_check_features ();omap44xx_prm_init ();
@@ -649,8 +645,6 @@ void __init omap4430_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP443X_CTRL_BASE ));omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP4430_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP4430_CM_BASE ), -OMAP2_L4_IO_ADDRESS (OMAP4430_CM2_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE ));omap4xxx_check_revision ();omap4xxx_check_features ();
@@ -682,8 +676,6 @@ void __init omap5_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP54XX_CTRL_BASE ));omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP54XX_CM_CORE_AON_BASE ), -OMAP2_L4_IO_ADDRESS (OMAP54XX_CM_CORE_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap44xx_prm_init ();
@@ -712,8 +704,6 @@ void __init dra7xx_init_early(void)omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ),OMAP2_L4_IO_ADDRESS (DRA7XX_CTRL_BASE ));omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (DRA7XX_CM_CORE_AON_BASE ), -OMAP2_L4_IO_ADDRESS (OMAP54XX_CM_CORE_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap44xx_prm_init ();
arch/arm/mach-omap2/prcm-common.h +6 -0
@@ -521,9 +521,15 @@ struct omap_prcm_irq_setup {/** * struct omap_prcm_init_data - PRCM driver init data * @index: clock memory mapping index to be used +* @mem: IO mem pointer for this module +* @offset: module base address offset from the IO base +* @flags: PRCM module init flags */ struct omap_prcm_init_data {int index; +void __iomem *mem; +s16 offset; +u16 flags; };extern void omap_prcm_irq_cleanup (void );
arch/arm/mach-omap2/prm_common.c +0 -1
@@ -689,7 +689,6 @@ int __init omap_prcm_init(void)void __init omap3_prcm_legacy_iomaps_init (void ) { -omap2_clk_legacy_provider_init (TI_CLKM_CM , cm_base +OMAP3430_IVA2_MOD );omap2_clk_legacy_provider_init (TI_CLKM_PRM , prm_base +OMAP3430_IVA2_MOD );omap2_clk_legacy_provider_init (TI_CLKM_SCRM ,omap_ctrl_base_get ());
Signature · 1 line
1.7.9.5
PATCH v6 13/34 ARM: OMAP2+: PRM: determine PRM base address from device tree
In reply to Tero KristoThere is no need to provide the PRM base address through a low-level API from the low-level IO init, as this information is available through DT. Re-routed the parsing function to be called from the PRM drivers also to simplify the implementation under io.c.
--- arch/arm/mach-omap2/io.c | 12 +-------- arch/arm/mach-omap2/prm.h | 1 + arch/arm/mach-omap2/prm2xxx.c | 1 + arch/arm/mach-omap2/prm33xx.c | 1 + arch/arm/mach-omap2/prm3xxx.c | 5 ++++ arch/arm/mach-omap2/prm44xx.c | 1 + arch/arm/mach-omap2/prm_common.c | 51 +++++++++++++++++++++++++++++++------- 7 files changed, 52 insertions(+), 20 deletions(-)
arch/arm/mach-omap2/io.c +1 -11
@@ -386,7 +386,6 @@ void __init omap2420_init_early(void) OMAP2_L3_IO_ADDRESS(OMAP2420_SMS_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP242X_CTRL_BASE ),NULL ); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP2420_PRM_BASE ));omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -415,7 +414,6 @@ void __init omap2430_init_early(void) OMAP2_L3_IO_ADDRESS(OMAP243X_SMS_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP243X_CTRL_BASE ),NULL ); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP2430_PRM_BASE ));omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -448,9 +446,8 @@ void __init omap3_init_early(void) OMAP2_L3_IO_ADDRESS(OMAP343X_SMS_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ),NULL ); +/* XXX: remove these two once OMAP3 is DT only */ omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP3430_PRM_BASE )); - -/* XXX: remove this once OMAP3 is DT only */ omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP3430_CM_BASE ),NULL );omap3xxx_check_revision ();omap3xxx_check_features ();
@@ -551,7 +548,6 @@ void __init ti814x_init_early(void) OMAP2_L4_IO_ADDRESS(TI81XX_TAP_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ),NULL ); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE ));omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -571,7 +567,6 @@ void __init ti816x_init_early(void) OMAP2_L4_IO_ADDRESS(TI81XX_TAP_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ),NULL ); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (TI81XX_PRCM_BASE ));omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -593,7 +588,6 @@ void __init am33xx_init_early(void) AM33XX_L4_WK_IO_ADDRESS(AM33XX_TAP_BASE));omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ),NULL ); -omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM33XX_PRCM_BASE ));omap3xxx_check_revision ();am33xx_check_features ();am33xx_prm_init ();
@@ -618,7 +612,6 @@ void __init am43xx_init_early(void) AM33XX_L4_WK_IO_ADDRESS(AM33XX_TAP_BASE));omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ),NULL ); -omap2_set_globals_prm (AM33XX_L4_WK_IO_ADDRESS (AM43XX_PRCM_BASE ));omap3xxx_check_revision ();am33xx_check_features ();omap44xx_prm_init ();
@@ -644,7 +637,6 @@ void __init omap4430_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP443X_SCM_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP443X_CTRL_BASE )); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP4430_PRM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE ));omap4xxx_check_revision ();omap4xxx_check_features ();
@@ -675,7 +667,6 @@ void __init omap5_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_SCM_BASE));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ),OMAP2_L4_IO_ADDRESS (OMAP54XX_CTRL_BASE )); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap44xx_prm_init ();
@@ -703,7 +694,6 @@ void __init dra7xx_init_early(void)omap2_set_globals_tap (-1 ,OMAP2_L4_IO_ADDRESS (DRA7XX_TAP_BASE ));omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ),OMAP2_L4_IO_ADDRESS (DRA7XX_CTRL_BASE )); -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap44xx_prm_init ();
arch/arm/mach-omap2/prm.h +1 -0
@@ -20,6 +20,7 @@ extern void __iomem *prm_base;extern u16 prm_features;extern void omap2_set_globals_prm (void __iomem *prm);int omap_prcm_init (void ); +int omap2_prm_base_init (void );void omap3_prcm_legacy_iomaps_init (void );# endif
arch/arm/mach-omap2/prm2xxx.c +1 -0
@@ -222,6 +222,7 @@ static struct prm_ll_data omap2xxx_prm_ll_data = {int __init omap2xxx_prm_init (void ) { +omap2_prm_base_init ();return prm_register (&omap2xxx_prm_ll_data); }
arch/arm/mach-omap2/prm33xx.c +1 -0
@@ -380,6 +380,7 @@ static struct prm_ll_data am33xx_prm_ll_data = {int __init am33xx_prm_init (void ) { +omap2_prm_base_init ();return prm_register (&am33xx_prm_ll_data); }
arch/arm/mach-omap2/prm3xxx.c +5 -0
@@ -29,6 +29,7 @@#include "prm-regbits-34xx.h" #include "cm3xxx.h" #include "cm-regbits-34xx.h" +#include "clock.h" static void omap3xxx_prm_read_pending_irqs (unsigned long *events);static void omap3xxx_prm_ocp_barrier (void );
@@ -671,6 +672,10 @@ static struct prm_ll_data omap3xxx_prm_ll_data = {int __init omap3xxx_prm_init (void ) { +omap2_clk_legacy_provider_init (TI_CLKM_PRM , + prm_base +OMAP3430_IVA2_MOD ); +omap2_prm_base_init (); +if (omap3_has_io_wakeup ()) prm_features |=PRM_HAS_IO_WAKEUP ;
arch/arm/mach-omap2/prm44xx.c +1 -0
@@ -705,6 +705,7 @@ static struct prm_ll_data omap44xx_prm_ll_data = {int __init omap44xx_prm_init (void ) { +omap2_prm_base_init ();omap_prm_base_init ();if (cpu_is_omap44xx () ||soc_is_omap54xx () ||soc_is_dra7xx ())
arch/arm/mach-omap2/prm_common.c +42 -9
87 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH v6 14/34 ARM: OMAP2+: control: determine control module base address from DT
In reply to Tero KristoThere is no need to provide the control module base address through a low-level API from the low-level IO init, as this information is available through DT. This patch adds a new API to initialize the control module though, but mostly makes the old API obsolete. The old API can be completely removed once OMAP3 is made DT only.
--- arch/arm/mach-omap2/control.c | 49 +++++++++++++++++++++++++++++++++----- arch/arm/mach-omap2/control.h | 2 ++ arch/arm/mach-omap2/io.c | 34 +++++++++++++------------- arch/arm/mach-omap2/prm.h | 1 - arch/arm/mach-omap2/prm_common.c | 5 ---- 5 files changed, 61 insertions(+), 30 deletions(-)
arch/arm/mach-omap2/control.c +43 -6
@@ -616,6 +616,7 @@ void __init omap3_ctrl_init(void)struct control_init_data {int index; +void __iomem *mem; };static struct control_init_data ctrl_data = {
@@ -627,10 +628,39 @@ static const struct of_device_id omap_scrm_dt_match_table[] = { { .compatible = "ti,am4-scrm", .data = &ctrl_data }, { .compatible = "ti,omap2-scrm", .data = &ctrl_data }, { .compatible = "ti,omap3-scrm", .data = &ctrl_data }, + { .compatible = "ti,dm816-scrm", .data = &ctrl_data }, { } };/** +* omap2_control_base_init - initialize iomappings for the control driver +* +* Detects and initializes the iomappings for the control driver, based +* on the DT data. Returns 0 in success, negative error value +* otherwise. +*/ +int __init omap2_control_base_init (void ) +{ +struct device_node *np; +const struct of_device_id *match; +struct control_init_data *data; +void __iomem *mem; + +for_each_matching_node_and_match (np, omap_scrm_dt_match_table, &match) { + data = (struct control_init_data *)match->data; + + mem =of_iomap (np,0 ); +if (!mem) +return -ENOMEM ; + + omap2_ctrl_base = mem; + data->mem = mem; + } + +return 0 ; +} + +/** * omap_control_init - low level init for the control driver * * Initializes the low level clock infrastructure for control driver.
@@ -639,7 +669,6 @@ static const struct of_device_id omap_scrm_dt_match_table[] = {int __init omap_control_init (void ) {struct device_node *np; -void __iomem *mem;const struct of_device_id *match;const struct omap_prcm_init_data *data;int ret;
@@ -647,14 +676,22 @@ int __init omap_control_init(void)for_each_matching_node_and_match (np, omap_scrm_dt_match_table, &match) { data = match->data; - mem =of_iomap (np,0 ); -if (!mem) -return -ENOMEM ; - - ret =omap2_clk_provider_init (np, data->index, mem); + ret =omap2_clk_provider_init (np, data->index, data->mem);if (ret)return ret; }return 0 ; } + +/** +* omap3_control_legacy_iomap_init - legacy iomap init for clock providers +* +* Legacy iomap init for clock provider. Needed only by legacy boot mode, +* where the base addresses are not parsed from DT, but still required +* by the clock driver to be setup properly. +*/ +void __init omap3_control_legacy_iomap_init (void ) +{ +omap2_clk_legacy_provider_init (TI_CLKM_SCRM , omap2_ctrl_base); +}
arch/arm/mach-omap2/control.h +2 -0
@@ -464,9 +464,11 @@ extern void omap_ctrl_write_dsp_boot_mode(u8 bootmode);extern void omap3630_ctrl_disable_rta (void );extern int omap3_ctrl_save_padconf (void );void omap3_ctrl_init (void ); +int omap2_control_base_init (void );int omap_control_init (void );extern void omap2_set_globals_control (void __iomem *ctrl,void __iomem *ctrl_pad); +void __init omap3_control_legacy_iomap_init (void );#else #define omap_ctrl_base_get () 0#define omap_ctrl_readb (x) 0
arch/arm/mach-omap2/io.c +16 -18
@@ -384,8 +384,7 @@ void __init omap2420_init_early(void)omap2_set_globals_tap (OMAP242X_CLASS ,OMAP2_L4_IO_ADDRESS (0x48014000 ));omap2_set_globals_sdrc (OMAP2_L3_IO_ADDRESS (OMAP2420_SDRC_BASE ),OMAP2_L3_IO_ADDRESS (OMAP2420_SMS_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP242X_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -412,8 +411,7 @@ void __init omap2430_init_early(void)omap2_set_globals_tap (OMAP243X_CLASS ,OMAP2_L4_IO_ADDRESS (0x4900a000 ));omap2_set_globals_sdrc (OMAP2_L3_IO_ADDRESS (OMAP243X_SDRC_BASE ),OMAP2_L3_IO_ADDRESS (OMAP243X_SMS_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP243X_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap2xxx_check_revision ();omap2xxx_prm_init ();omap2xxx_cm_init ();
@@ -444,11 +442,15 @@ void __init omap3_init_early(void)omap2_set_globals_tap (OMAP343X_CLASS ,OMAP2_L4_IO_ADDRESS (0x4830A000 ));omap2_set_globals_sdrc (OMAP2_L3_IO_ADDRESS (OMAP343X_SDRC_BASE ),OMAP2_L3_IO_ADDRESS (OMAP343X_SMS_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ), -NULL ); -/* XXX: remove these two once OMAP3 is DT only */ -omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP3430_PRM_BASE )); -omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP3430_CM_BASE ),NULL ); +/* XXX: remove these once OMAP3 is DT only */ +if (!of_have_populated_dt ()) { +omap2_set_globals_control ( +OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ),NULL ); +omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP3430_PRM_BASE )); +omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP3430_CM_BASE ), +NULL ); + } +omap2_control_base_init ();omap3xxx_check_revision ();omap3xxx_check_features ();omap3xxx_prm_init ();
@@ -459,7 +461,7 @@ void __init omap3_init_early(void)omap3xxx_hwmod_init ();omap_hwmod_init_postsetup ();if (!of_have_populated_dt ()) { -omap3_prcm_legacy_iomaps_init (); +omap3_control_legacy_iomap_init ();if (soc_is_am35xx ()) omap_clk_soc_init = am35xx_clk_legacy_init;else if (cpu_is_omap3630 ())
@@ -546,8 +548,7 @@ void __init ti814x_init_early(void) {omap2_set_globals_tap (TI814X_CLASS ,OMAP2_L4_IO_ADDRESS (TI81XX_TAP_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -565,8 +566,7 @@ void __init ti816x_init_early(void) {omap2_set_globals_tap (TI816X_CLASS ,OMAP2_L4_IO_ADDRESS (TI81XX_TAP_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (TI81XX_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap3xxx_check_revision ();ti81xx_check_features ();am33xx_prm_init ();
@@ -586,8 +586,7 @@ void __init am33xx_init_early(void) {omap2_set_globals_tap (AM335X_CLASS ,AM33XX_L4_WK_IO_ADDRESS (AM33XX_TAP_BASE )); -omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap3xxx_check_revision ();am33xx_check_features ();am33xx_prm_init ();
@@ -610,8 +609,7 @@ void __init am43xx_init_early(void) {omap2_set_globals_tap (AM335X_CLASS ,AM33XX_L4_WK_IO_ADDRESS (AM33XX_TAP_BASE )); -omap2_set_globals_control (AM33XX_L4_WK_IO_ADDRESS (AM33XX_CTRL_BASE ), -NULL ); +omap2_control_base_init ();omap3xxx_check_revision ();am33xx_check_features ();omap44xx_prm_init ();
arch/arm/mach-omap2/prm.h +0 -1
@@ -21,7 +21,6 @@ extern u16 prm_features;extern void omap2_set_globals_prm (void __iomem *prm);int omap_prcm_init (void );int omap2_prm_base_init (void ); -void omap3_prcm_legacy_iomaps_init (void );# endif /*
arch/arm/mach-omap2/prm_common.c +0 -5
@@ -722,11 +722,6 @@ int __init omap_prcm_init(void)return 0 ; } -void __init omap3_prcm_legacy_iomaps_init (void ) -{ -omap2_clk_legacy_provider_init (TI_CLKM_SCRM ,omap_ctrl_base_get ()); -} -static int __init prm_late_init (void ) {if (prm_ll_data->late_init)
Signature · 1 line
1.7.9.5
PATCH v6 15/34 ARM: OMAP2+: PRM: move SoC specific init calls within a generic API
In reply to Tero KristoThis gets rid of need for some exported driver APIs, and simplifies the initialization of the PRM driver. Done in preparation to make PRM a separate driver. The init data is now also passed to the SoC specific implementations, allowing future expansion to add feature flags etc.
--- arch/arm/mach-omap2/io.c | 21 +++++----- arch/arm/mach-omap2/prcm-common.h | 4 ++ arch/arm/mach-omap2/prm.h | 1 + arch/arm/mach-omap2/prm2xxx.c | 3 +- arch/arm/mach-omap2/prm2xxx.h | 2 +- arch/arm/mach-omap2/prm33xx.c | 3 +- arch/arm/mach-omap2/prm33xx.h | 2 +- arch/arm/mach-omap2/prm3xxx.c | 4 +- arch/arm/mach-omap2/prm3xxx.h | 2 +- arch/arm/mach-omap2/prm44xx.c | 3 +- arch/arm/mach-omap2/prm44xx.h | 1 - arch/arm/mach-omap2/prm44xx_54xx.h | 4 +- arch/arm/mach-omap2/prm54xx.h | 1 - arch/arm/mach-omap2/prm7xx.h | 2 +- arch/arm/mach-omap2/prm_common.c | 76 ++++++++++++++++++++++++++++++------ 15 files changed, 90 insertions(+), 39 deletions(-)
arch/arm/mach-omap2/io.c +11 -10
@@ -386,7 +386,7 @@ void __init omap2420_init_early(void) OMAP2_L3_IO_ADDRESS(OMAP2420_SMS_BASE));omap2_control_base_init ();omap2xxx_check_revision (); -omap2xxx_prm_init (); +omap2_prcm_base_init ();omap2xxx_cm_init ();omap2xxx_voltagedomains_init ();omap242x_powerdomains_init ();
@@ -413,7 +413,7 @@ void __init omap2430_init_early(void) OMAP2_L3_IO_ADDRESS(OMAP243X_SMS_BASE));omap2_control_base_init ();omap2xxx_check_revision (); -omap2xxx_prm_init (); +omap2_prcm_base_init ();omap2xxx_cm_init ();omap2xxx_voltagedomains_init ();omap243x_powerdomains_init ();
@@ -453,7 +453,8 @@ void __init omap3_init_early(void)omap2_control_base_init ();omap3xxx_check_revision ();omap3xxx_check_features (); -omap3xxx_prm_init (); +omap2_prcm_base_init (); +omap3xxx_prm_init (NULL );omap3xxx_cm_init ();omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();
@@ -551,7 +552,7 @@ void __init ti814x_init_early(void)omap2_control_base_init ();omap3xxx_check_revision ();ti81xx_check_features (); -am33xx_prm_init (); +omap2_prcm_base_init ();am33xx_cm_init ();omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();
@@ -569,7 +570,7 @@ void __init ti816x_init_early(void)omap2_control_base_init ();omap3xxx_check_revision ();ti81xx_check_features (); -am33xx_prm_init (); +omap2_prcm_base_init ();am33xx_cm_init ();omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();
@@ -589,7 +590,7 @@ void __init am33xx_init_early(void)omap2_control_base_init ();omap3xxx_check_revision ();am33xx_check_features (); -am33xx_prm_init (); +omap2_prcm_base_init ();am33xx_cm_init ();am33xx_powerdomains_init ();am33xx_clockdomains_init ();
@@ -612,7 +613,7 @@ void __init am43xx_init_early(void)omap2_control_base_init ();omap3xxx_check_revision ();am33xx_check_features (); -omap44xx_prm_init (); +omap2_prcm_base_init ();omap4_cm_init ();am43xx_powerdomains_init ();am43xx_clockdomains_init ();
@@ -638,7 +639,7 @@ void __init omap4430_init_early(void)omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE ));omap4xxx_check_revision ();omap4xxx_check_features (); -omap44xx_prm_init (); +omap2_prcm_base_init ();omap4_cm_init ();omap4_pm_init_early ();omap44xx_voltagedomains_init ();
@@ -667,7 +668,7 @@ void __init omap5_init_early(void) OMAP2_L4_IO_ADDRESS(OMAP54XX_CTRL_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap44xx_prm_init (); +omap2_prcm_base_init ();omap5xxx_check_revision ();omap4_cm_init ();omap54xx_voltagedomains_init ();
@@ -694,7 +695,7 @@ void __init dra7xx_init_early(void) OMAP2_L4_IO_ADDRESS(DRA7XX_CTRL_BASE));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early (); -omap44xx_prm_init (); +omap2_prcm_base_init ();dra7xxx_check_revision ();omap4_cm_init ();dra7xx_powerdomains_init ();
arch/arm/mach-omap2/prcm-common.h +4 -0
@@ -524,12 +524,16 @@ struct omap_prcm_irq_setup {* @mem: IO mem pointer for this module * @offset: module base address offset from the IO base * @flags: PRCM module init flags +* @init: low level PRCM init function for this module +* @np: device node for this PRCM module */ struct omap_prcm_init_data {int index;void __iomem *mem;s16 offset;u16 flags; +int (*init)(const struct omap_prcm_init_data *data); +struct device_node *np; };extern void omap_prcm_irq_cleanup (void );
arch/arm/mach-omap2/prm.h +1 -0
@@ -21,6 +21,7 @@ extern u16 prm_features;extern void omap2_set_globals_prm (void __iomem *prm);int omap_prcm_init (void );int omap2_prm_base_init (void ); +int omap2_prcm_base_init (void );# endif /*
arch/arm/mach-omap2/prm2xxx.c +1 -2
@@ -220,9 +220,8 @@ static struct prm_ll_data omap2xxx_prm_ll_data = { .clear_mod_irqs = &omap2xxx_prm_clear_mod_irqs, }; -int __init omap2xxx_prm_init (void ) +int __init omap2xxx_prm_init (const struct omap_prcm_init_data *data) { -omap2_prm_base_init ();return prm_register (&omap2xxx_prm_ll_data); }
arch/arm/mach-omap2/prm2xxx.h +1 -1
@@ -124,7 +124,7 @@extern int omap2xxx_clkdm_sleep (struct clockdomain *clkdm);extern int omap2xxx_clkdm_wakeup (struct clockdomain *clkdm); -extern int __init omap2xxx_prm_init (void ); +int __init omap2xxx_prm_init (const struct omap_prcm_init_data *data);#endif
arch/arm/mach-omap2/prm33xx.c +1 -2
@@ -378,9 +378,8 @@ static struct prm_ll_data am33xx_prm_ll_data = { .reset_system = am33xx_prm_global_warm_sw_reset, }; -int __init am33xx_prm_init (void ) +int __init am33xx_prm_init (const struct omap_prcm_init_data *data) { -omap2_prm_base_init ();return prm_register (&am33xx_prm_ll_data); }
arch/arm/mach-omap2/prm33xx.h +1 -1
@@ -118,7 +118,7 @@#define AM33XX_PM_CEFUSE_PWRSTST AM33XX_PRM_REGADDR(AM33XX_PRM_CEFUSE_MOD, 0x0004)#ifndef __ASSEMBLER__ -int am33xx_prm_init (void ); +int am33xx_prm_init (const struct omap_prcm_init_data *data);#endif /* ASSEMBLER */ #endif
arch/arm/mach-omap2/prm3xxx.c +1 -3
@@ -670,12 +670,10 @@ static struct prm_ll_data omap3xxx_prm_ll_data = { .vp_clear_txdone = &omap3_prm_vp_clear_txdone, }; -int __init omap3xxx_prm_init (void ) +int __init omap3xxx_prm_init (const struct omap_prcm_init_data *data) {omap2_clk_legacy_provider_init (TI_CLKM_PRM , prm_base +OMAP3430_IVA2_MOD ); -omap2_prm_base_init (); -if (omap3_has_io_wakeup ()) prm_features |=PRM_HAS_IO_WAKEUP ;
arch/arm/mach-omap2/prm3xxx.h +1 -1
@@ -140,7 +140,7 @@ extern u32 omap3_prm_vcvp_read(u8 offset);extern void omap3_prm_vcvp_write (u32 val,u8 offset);extern u32 omap3_prm_vcvp_rmw (u32 mask,u32 bits,u8 offset); -extern int __init omap3xxx_prm_init (void ); +int __init omap3xxx_prm_init (const struct omap_prcm_init_data *data);void omap3xxx_prm_iva_idle (void );void omap3_prm_reset_modem (void );int omap3xxx_prm_clear_global_cold_reset (void );
arch/arm/mach-omap2/prm44xx.c +1 -2
@@ -703,9 +703,8 @@ static struct prm_ll_data omap44xx_prm_ll_data = { .vp_clear_txdone = omap4_prm_vp_clear_txdone, }; -int __init omap44xx_prm_init (void ) +int __init omap44xx_prm_init (const struct omap_prcm_init_data *data) { -omap2_prm_base_init ();omap_prm_base_init ();if (cpu_is_omap44xx () ||soc_is_omap54xx () ||soc_is_dra7xx ())
arch/arm/mach-omap2/prm44xx.h +0 -1
@@ -26,7 +26,6 @@#define __ARCH_ARM_MACH_OMAP2_PRM44XX_H#include "prm44xx_54xx.h" -#include "prcm-common.h" #include "prm.h" #define OMAP4430_PRM_BASE 0x4a306000
arch/arm/mach-omap2/prm44xx_54xx.h +3 -1
@@ -23,6 +23,8 @@#ifndef __ARCH_ARM_MACH_OMAP2_PRM44XX_54XX_H#define __ARCH_ARM_MACH_OMAP2_PRM44XX_54XX_H +#include "prcm-common.h" +/* Function prototypes */ #ifndef __ASSEMBLER__
@@ -34,7 +36,7 @@ extern u32 omap4_prm_vcvp_read(u8 offset);extern void omap4_prm_vcvp_write (u32 val,u8 offset);extern u32 omap4_prm_vcvp_rmw (u32 mask,u32 bits,u8 offset); -extern int __init omap44xx_prm_init (void ); +int __init omap44xx_prm_init (const struct omap_prcm_init_data *data);#endif
arch/arm/mach-omap2/prm54xx.h +0 -1
@@ -22,7 +22,6 @@#define __ARCH_ARM_MACH_OMAP2_PRM54XX_H#include "prm44xx_54xx.h" -#include "prcm-common.h" #include "prm.h" #define OMAP54XX_PRM_BASE 0x4ae06000
arch/arm/mach-omap2/prm7xx.h +1 -1
@@ -22,8 +22,8 @@#ifndef __ARCH_ARM_MACH_OMAP2_PRM7XX_H#define __ARCH_ARM_MACH_OMAP2_PRM7XX_H -#include "prm44xx_54xx.h" #include "prcm-common.h" +#include "prm44xx_54xx.h" #include "prm.h" #define DRA7XX_PRM_BASE 0x4ae06000
arch/arm/mach-omap2/prm_common.c +63 -13
115 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH v6 16/34 ARM: OMAP4+: PRM: determine prm_device_inst based on DT compatibility
In reply to Tero KristoPRM device instance offset is now provided through the prm_init_data. This gets rid of some cpu_is_X / soc_is_X calls from PRM core code, preparing for PRM to be its own separate driver.
--- arch/arm/mach-omap2/prcm-common.h | 2 ++ arch/arm/mach-omap2/prm44xx.c | 2 ++ arch/arm/mach-omap2/prm_common.c | 37 ++++++++++++++++++++++++++++++++----- arch/arm/mach-omap2/prminst44xx.c | 18 +++++------------- arch/arm/mach-omap2/prminst44xx.h | 1 + 5 files changed, 42 insertions(+), 18 deletions(-)
arch/arm/mach-omap2/prcm-common.h +2 -0
@@ -524,6 +524,7 @@ struct omap_prcm_irq_setup {* @mem: IO mem pointer for this module * @offset: module base address offset from the IO base * @flags: PRCM module init flags +* @device_inst_offset: device instance offset within the module address space * @init: low level PRCM init function for this module * @np: device node for this PRCM module */
@@ -532,6 +533,7 @@ struct omap_prcm_init_data {void __iomem *mem;s16 offset;u16 flags; +s32 device_inst_offset;int (*init)(const struct omap_prcm_init_data *data);struct device_node *np; };
arch/arm/mach-omap2/prm44xx.c +2 -0
@@ -713,6 +713,8 @@ int __init omap44xx_prm_init(const struct omap_prcm_init_data *data)if (!soc_is_dra7xx ()) prm_features |=PRM_HAS_VOLTAGE ; +omap4_prminst_set_prm_dev_inst (data->device_inst_offset); +return prm_register (&omap44xx_prm_ll_data); }
arch/arm/mach-omap2/prm_common.c +32 -5
@@ -34,6 +34,9 @@#include "prm3xxx.h" #include "prm33xx.h" #include "prm44xx.h" +#include "prm54xx.h" +#include "prm7xx.h" +#include "prcm43xx.h" #include "common.h" #include "clock.h" #include "cm.h"
@@ -661,11 +664,35 @@ static struct omap_prcm_init_data am3_prm_data __initdata = { };#endif -#if defined(CONFIG_ARCH_OMAP4 ) || defined(CONFIG_SOC_OMAP5 ) || \ - defined(CONFIG_SOC_DRA7XX ) || defined(CONFIG_SOC_AM43XX ) +#ifdef CONFIG_ARCH_OMAP4 static struct omap_prcm_init_data omap4_prm_data__initdata = { .index =TI_CLKM_PRM , .init = omap44xx_prm_init, + .device_inst_offset =OMAP4430_PRM_DEVICE_INST , +}; +#endif + +#ifdef CONFIG_SOC_OMAP5 +static struct omap_prcm_init_data omap5_prm_data__initdata = { + .index =TI_CLKM_PRM , + .init = omap44xx_prm_init, + .device_inst_offset =OMAP54XX_PRM_DEVICE_INST , +}; +#endif + +#ifdef CONFIG_SOC_DRA7XX +static struct omap_prcm_init_data dra7_prm_data__initdata = { + .index =TI_CLKM_PRM , + .init = omap44xx_prm_init, + .device_inst_offset =DRA7XX_PRM_DEVICE_INST , +}; +#endif + +#ifdef CONFIG_SOC_AM43XX +static struct omap_prcm_init_data am4_prm_data__initdata = { + .index =TI_CLKM_PRM , + .init = omap44xx_prm_init, + .device_inst_offset =AM43XX_PRM_DEVICE_INST , };#endif
@@ -680,7 +707,7 @@ static const struct of_device_id omap_prcm_dt_match_table[] __initconst = { { .compatible = "ti,am3-prcm", .data = &am3_prm_data }, #endif #ifdef CONFIG_SOC_AM43XX - { .compatible = "ti,am4-prcm", .data = &omap4_prm_data }, + { .compatible = "ti,am4-prcm", .data = &am4_prm_data }, #endif #ifdef CONFIG_SOC_TI81XX { .compatible = "ti,dm814-prcm", .data = &am3_prm_data },
@@ -697,11 +724,11 @@ static const struct of_device_id omap_prcm_dt_match_table[] __initconst = { { .compatible = "ti,omap4-scrm", .data = &scrm_data }, #endif #ifdef CONFIG_SOC_OMAP5 - { .compatible = "ti,omap5-prm", .data = &omap4_prm_data }, + { .compatible = "ti,omap5-prm", .data = &omap5_prm_data }, { .compatible = "ti,omap5-scrm", .data = &scrm_data }, #endif #ifdef CONFIG_SOC_DRA7XX - { .compatible = "ti,dra7-prm", .data = &omap4_prm_data }, + { .compatible = "ti,dra7-prm", .data = &dra7_prm_data }, #endif { } };
arch/arm/mach-omap2/prminst44xx.c +5 -13
@@ -47,22 +47,14 @@ void omap_prm_base_init(void)s32 omap4_prmst_get_prm_dev_inst (void ) { -if (prm_dev_inst !=PRM_INSTANCE_UNKNOWN ) -return prm_dev_inst; - -/* This cannot be done way early at boot.. as things are not setup */ -if (cpu_is_omap44xx ()) - prm_dev_inst =OMAP4430_PRM_DEVICE_INST ; -else if (soc_is_omap54xx ()) - prm_dev_inst =OMAP54XX_PRM_DEVICE_INST ; -else if (soc_is_dra7xx ()) - prm_dev_inst =DRA7XX_PRM_DEVICE_INST ; -else if (soc_is_am43xx ()) - prm_dev_inst =AM43XX_PRM_DEVICE_INST ; -return prm_dev_inst; } +void omap4_prminst_set_prm_dev_inst (s32 dev_inst) +{ + prm_dev_inst = dev_inst; +} +/* Read a register in a PRM instance */ u32 omap4_prminst_read_inst_reg (u8 part,s16 inst,u16 idx) {
arch/arm/mach-omap2/prminst44xx.h +1 -0
@@ -14,6 +14,7 @@#define PRM_INSTANCE_UNKNOWN -1extern s32 omap4_prmst_get_prm_dev_inst (void ); +void omap4_prminst_set_prm_dev_inst (s32 dev_inst);/* * In an ideal world, we would not export these low-level functions,
Signature · 1 line
1.7.9.5
PATCH v6 17/34 ARM: OMAP2+: CM: move SoC specific init calls within a generic API
In reply to Tero KristoThis gets rid of need for some exported driver APIs, and simplifies the initialization of the CM driver. Done in preparation to make CM a separate driver. The init data is now also passed to the SoC specific implementations, allowing future expansion to add feature flags etc.
--- arch/arm/mach-omap2/cm2xxx.c | 3 +- arch/arm/mach-omap2/cm2xxx.h | 2 +- arch/arm/mach-omap2/cm33xx.c | 3 +- arch/arm/mach-omap2/cm33xx.h | 3 +- arch/arm/mach-omap2/cm3xxx.c | 3 +- arch/arm/mach-omap2/cm3xxx.h | 2 +- arch/arm/mach-omap2/cm44xx.h | 2 +- arch/arm/mach-omap2/cm_common.c | 64 ++++++++++++++++++++++++++++++++------ arch/arm/mach-omap2/cminst44xx.c | 3 +- arch/arm/mach-omap2/io.c | 16 +++------- arch/arm/mach-omap2/prm_common.c | 8 ++++- 11 files changed, 75 insertions(+), 34 deletions(-)
arch/arm/mach-omap2/cm2xxx.c +1 -2
@@ -393,9 +393,8 @@ static struct cm_ll_data omap2xxx_cm_ll_data = { .wait_module_ready = &omap2xxx_cm_wait_module_ready, }; -int __init omap2xxx_cm_init (void ) +int __init omap2xxx_cm_init (const struct omap_prcm_init_data *data) { -omap2_cm_base_init ();return cm_register (&omap2xxx_cm_ll_data); }
arch/arm/mach-omap2/cm2xxx.h +1 -1
@@ -63,7 +63,7 @@ extern u32 omap2xxx_cm_get_core_pll_config(void);extern void omap2xxx_cm_set_mod_dividers (u32 mpu,u32 dsp,u32 gfx,u32 core,u32 mdm); -extern int __init omap2xxx_cm_init (void ); +int __init omap2xxx_cm_init (const struct omap_prcm_init_data *data);#endif
arch/arm/mach-omap2/cm33xx.c +1 -2
@@ -352,9 +352,8 @@ static struct cm_ll_data am33xx_cm_ll_data = { .module_disable = &am33xx_cm_module_disable, }; -int __init am33xx_cm_init (void ) +int __init am33xx_cm_init (const struct omap_prcm_init_data *data) { -omap2_cm_base_init ();return cm_register (&am33xx_cm_ll_data); }
arch/arm/mach-omap2/cm33xx.h +2 -1
@@ -19,6 +19,7 @@#include "cm.h" #include "cm-regbits-33xx.h" +#include "prcm-common.h" /* CM base address */ #define AM33XX_CM_BASE 0x44e00000
@@ -374,6 +375,6 @@#ifndef __ASSEMBLER__ -int am33xx_cm_init (void ); +int am33xx_cm_init (const struct omap_prcm_init_data *data);#endif /* ASSEMBLER */ #endif
arch/arm/mach-omap2/cm3xxx.c +1 -2
@@ -671,10 +671,9 @@ static struct cm_ll_data omap3xxx_cm_ll_data = { .wait_module_ready = &omap3xxx_cm_wait_module_ready, }; -int __init omap3xxx_cm_init (void ) +int __init omap3xxx_cm_init (const struct omap_prcm_init_data *data) {omap2_clk_legacy_provider_init (TI_CLKM_CM , cm_base +OMAP3430_IVA2_MOD ); -omap2_cm_base_init ();return cm_register (&omap3xxx_cm_ll_data); }
arch/arm/mach-omap2/cm3xxx.h +1 -1
@@ -72,7 +72,7 @@ extern void omap3_cm_save_context(void);extern void omap3_cm_restore_context (void );extern void omap3_cm_save_scratchpad_contents (u32 *ptr); -extern int __init omap3xxx_cm_init (void ); +int __init omap3xxx_cm_init (const struct omap_prcm_init_data *data);#endif
arch/arm/mach-omap2/cm44xx.h +1 -1
@@ -23,6 +23,6 @@#define OMAP4_CM_CLKSTCTRL 0x0000#define OMAP4_CM_STATICDEP 0x0004 -int omap4_cm_init (void ); +int omap4_cm_init (const struct omap_prcm_init_data *data);#endif
arch/arm/mach-omap2/cm_common.c +54 -10
121 lines not on this page · raw message
arch/arm/mach-omap2/cminst44xx.c +1 -2
@@ -514,9 +514,8 @@ static struct cm_ll_data omap4xxx_cm_ll_data = { .module_disable = &omap4_cminst_module_disable, }; -int __init omap4_cm_init (void ) +int __init omap4_cm_init (const struct omap_prcm_init_data *data) { -omap2_cm_base_init ();omap_cm_base_init ();return cm_register (&omap4xxx_cm_ll_data);
arch/arm/mach-omap2/io.c +5 -11
@@ -387,7 +387,6 @@ void __init omap2420_init_early(void)omap2_control_base_init ();omap2xxx_check_revision ();omap2_prcm_base_init (); -omap2xxx_cm_init ();omap2xxx_voltagedomains_init ();omap242x_powerdomains_init ();omap242x_clockdomains_init ();
@@ -414,7 +413,6 @@ void __init omap2430_init_early(void)omap2_control_base_init ();omap2xxx_check_revision ();omap2_prcm_base_init (); -omap2xxx_cm_init ();omap2xxx_voltagedomains_init ();omap243x_powerdomains_init ();omap243x_clockdomains_init ();
@@ -454,8 +452,11 @@ void __init omap3_init_early(void)omap3xxx_check_revision ();omap3xxx_check_features ();omap2_prcm_base_init (); -omap3xxx_prm_init (NULL ); -omap3xxx_cm_init (); +/* XXX: remove these once OMAP3 is DT only */ +if (!of_have_populated_dt ()) { +omap3xxx_prm_init (NULL ); +omap3xxx_cm_init (NULL ); + }omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();omap3xxx_clockdomains_init ();
@@ -553,7 +554,6 @@ void __init ti814x_init_early(void)omap3xxx_check_revision ();ti81xx_check_features ();omap2_prcm_base_init (); -am33xx_cm_init ();omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();ti81xx_clockdomains_init ();
@@ -571,7 +571,6 @@ void __init ti816x_init_early(void)omap3xxx_check_revision ();ti81xx_check_features ();omap2_prcm_base_init (); -am33xx_cm_init ();omap3xxx_voltagedomains_init ();omap3xxx_powerdomains_init ();ti81xx_clockdomains_init ();
@@ -591,7 +590,6 @@ void __init am33xx_init_early(void)omap3xxx_check_revision ();am33xx_check_features ();omap2_prcm_base_init (); -am33xx_cm_init ();am33xx_powerdomains_init ();am33xx_clockdomains_init ();am33xx_hwmod_init ();
@@ -614,7 +612,6 @@ void __init am43xx_init_early(void)omap3xxx_check_revision ();am33xx_check_features ();omap2_prcm_base_init (); -omap4_cm_init ();am43xx_powerdomains_init ();am43xx_clockdomains_init ();am43xx_hwmod_init ();
@@ -640,7 +637,6 @@ void __init omap4430_init_early(void)omap4xxx_check_revision ();omap4xxx_check_features ();omap2_prcm_base_init (); -omap4_cm_init ();omap4_pm_init_early ();omap44xx_voltagedomains_init ();omap44xx_powerdomains_init ();
@@ -670,7 +666,6 @@ void __init omap5_init_early(void)omap4_pm_init_early ();omap2_prcm_base_init ();omap5xxx_check_revision (); -omap4_cm_init ();omap54xx_voltagedomains_init ();omap54xx_powerdomains_init ();omap54xx_clockdomains_init ();
@@ -697,7 +692,6 @@ void __init dra7xx_init_early(void)omap4_pm_init_early ();omap2_prcm_base_init ();dra7xxx_check_revision (); -omap4_cm_init ();dra7xx_powerdomains_init ();dra7xx_clockdomains_init ();dra7xx_hwmod_init ();
arch/arm/mach-omap2/prm_common.c +7 -1
@@ -770,7 +770,13 @@ int __init omap2_prm_base_init(void)int __init omap2_prcm_base_init (void ) { -return omap2_prm_base_init (); +int ret; + + ret =omap2_prm_base_init (); +if (ret) +return ret; + +return omap2_cm_base_init (); }/**
Signature · 1 line
1.7.9.5
PATCH v6 18/34 ARM: OMAP4+: PRM: setup prm_features from the PRM init time flags
In reply to Tero KristoCurrently some cpu_is_X checks are used to setup prm_features, however the same can be accomplished by just passing these flags from the PRM init data. This is done in preparation to make PRM a separate driver.
--- arch/arm/mach-omap2/prm44xx.c | 4 ++-- arch/arm/mach-omap2/prm_common.c | 3 +++ 2 files changed, 5 insertions(+), 2 deletions(-)
arch/arm/mach-omap2/prm44xx.c +2 -2
@@ -707,10 +707,10 @@ int __init omap44xx_prm_init(const struct omap_prcm_init_data *data) {omap_prm_base_init (); -if (cpu_is_omap44xx () ||soc_is_omap54xx () ||soc_is_dra7xx ()) +if (data->flags &PRM_HAS_IO_WAKEUP ) prm_features |=PRM_HAS_IO_WAKEUP ; -if (!soc_is_dra7xx ()) +if (data->flags &PRM_HAS_VOLTAGE ) prm_features |=PRM_HAS_VOLTAGE ;omap4_prminst_set_prm_dev_inst (data->device_inst_offset);
arch/arm/mach-omap2/prm_common.c +3 -0
@@ -669,6 +669,7 @@ static struct omap_prcm_init_data omap4_prm_data __initdata = { .index = TI_CLKM_PRM, .init = omap44xx_prm_init, .device_inst_offset = OMAP4430_PRM_DEVICE_INST, + .flags = PRM_HAS_IO_WAKEUP | PRM_HAS_VOLTAGE, };#endif
@@ -677,6 +678,7 @@ static struct omap_prcm_init_data omap5_prm_data __initdata = { .index = TI_CLKM_PRM, .init = omap44xx_prm_init, .device_inst_offset = OMAP54XX_PRM_DEVICE_INST, + .flags = PRM_HAS_IO_WAKEUP | PRM_HAS_VOLTAGE, };#endif
@@ -685,6 +687,7 @@ static struct omap_prcm_init_data dra7_prm_data __initdata = { .index = TI_CLKM_PRM, .init = omap44xx_prm_init, .device_inst_offset = DRA7XX_PRM_DEVICE_INST, + .flags = PRM_HAS_IO_WAKEUP, };#endif
Signature · 1 line
1.7.9.5
PATCH v6 19/34 ARM: OMAP4+: PRM: get rid of cpu_is_omap44xx calls from interrupt init
In reply to Tero KristoThe compatible DT node is now passed with the prm init, so there is no need to do node matching here again. Added a new flag to the init data also, to detect default IRQ support for OMAP4. Also, any booting omap4 DT setup always has a PRM node, so there is no need to check against the special case where it would be missing.
--- arch/arm/mach-omap2/prm.h | 6 +++-- arch/arm/mach-omap2/prm44xx.c | 54 +++++++++++++++----------------------- arch/arm/mach-omap2/prm_common.c | 2 +- 3 files changed, 26 insertions(+), 36 deletions(-)
arch/arm/mach-omap2/prm.h +4 -2
@@ -29,9 +29,11 @@ int omap2_prcm_base_init(void);* * PRM_HAS_IO_WAKEUP: has IO wakeup capability * PRM_HAS_VOLTAGE: has voltage domains +* PRM_IRQ_DEFAULT: use default irq number for PRM irq */ -#define PRM_HAS_IO_WAKEUP (1 << 0) -#define PRM_HAS_VOLTAGE (1 << 1) +#define PRM_HAS_IO_WAKEUP BIT(0) +#define PRM_HAS_VOLTAGE BIT(1) +#define PRM_IRQ_DEFAULT BIT(2)/* * MAX_MODULE_SOFTRESET_WAIT: Maximum microseconds to wait for OMAP
arch/arm/mach-omap2/prm44xx.c +21 -33
78 lines not on this page · raw message
arch/arm/mach-omap2/prm_common.c +1 -1
@@ -669,7 +669,7 @@ static struct omap_prcm_init_data omap4_prm_data __initdata = { .index = TI_CLKM_PRM, .init = omap44xx_prm_init, .device_inst_offset = OMAP4430_PRM_DEVICE_INST, - .flags = PRM_HAS_IO_WAKEUP | PRM_HAS_VOLTAGE, + .flags = PRM_HAS_IO_WAKEUP | PRM_HAS_VOLTAGE | PRM_IRQ_DEFAULT, };#endif
Signature · 1 line
1.7.9.5
PATCH v6 20/34 ARM: OMAP2+: clock: add low-level support for regmap
In reply to Tero KristoSome of the TI clock providers will be converted to use syscon, thus low-level regmap support is needed for the clock drivers also. This patch adds this support, which can be enabled for individual drivers in later patches.
--- arch/arm/mach-omap2/clock.c | 48 ++++++++++++++++++++++++++++++++------ arch/arm/mach-omap2/clock.h | 4 +++- arch/arm/mach-omap2/cm_common.c | 2 +- arch/arm/mach-omap2/control.c | 2 +- arch/arm/mach-omap2/prm_common.c | 2 +- 5 files changed, 47 insertions(+), 11 deletions(-)
arch/arm/mach-omap2/clock.c +41 -7
@@ -23,7 +23,9 @@#include <linux/clk-provider.h> #include <linux/io.h> #include <linux/bitops.h> +#include <linux/regmap.h> #include <linux/of_address.h> +#include <linux/bootmem.h> #include <asm/cpu.h> #include <trace/events/power.h>
@@ -73,20 +75,37 @@ struct ti_clk_features ti_clk_features;static bool clkdm_control = true;static LIST_HEAD (clk_hw_omap_clocks); -static void __iomem *clk_memmaps[CLK_MAX_MEMMAPS ]; + +struct clk_iomap { +struct regmap *regmap; +void __iomem *mem; +}; + +static struct clk_iomap *clk_memmaps[CLK_MAX_MEMMAPS ];static void clk_memmap_writel (u32 val,void __iomem *reg) {struct clk_omap_reg *r = (struct clk_omap_reg *)® +struct clk_iomap *io = clk_memmaps[r->index]; -writel_relaxed (val, clk_memmaps[r->index] + r->offset); +if (io->regmap) +regmap_write (io->regmap, r->offset, val); +else +writel_relaxed (val, io->mem + r->offset); }static u32 clk_memmap_readl (void __iomem *reg) { +u32 val;struct clk_omap_reg *r = (struct clk_omap_reg *)® +struct clk_iomap *io = clk_memmaps[r->index]; -return readl_relaxed (clk_memmaps[r->index] + r->offset); +if (io->regmap) +regmap_read (io->regmap, r->offset, &val); +else + val =readl_relaxed (io->mem + r->offset); + +return val; }void omap2_clk_writel (u32 val,struct clk_hw_omap *clk,void __iomem *reg)
@@ -115,18 +134,27 @@ static struct ti_clk_ll_ops omap_clk_ll_ops = {* @match_table: DT device table to match for devices to init * @np: device node pointer for the this clock provider * @index: index for the clock provider -* @mem: iomem pointer for the clock provider memory area ++ @syscon: syscon regmap pointer +* @mem: iomem pointer for the clock provider memory area, only used if +* syscon is not provided * * Initializes a clock provider module (CM/PRM etc.), registering * the memory mapping at specified index and initializing the * low level driver infrastructure. Returns 0 in success. */ int __init omap2_clk_provider_init (struct device_node *np,int index, -void __iomem *mem) +struct regmap *syscon,void __iomem *mem) { +struct clk_iomap *io; + ti_clk_ll_ops = &omap_clk_ll_ops; - clk_memmaps[index] = mem; + io =kzalloc (sizeof (*io),GFP_KERNEL ); + + io->regmap = syscon; + io->mem = mem; + + clk_memmaps[index] = io;ti_dt_clk_init_provider (np, index);
@@ -142,9 +170,15 @@ int __init omap2_clk_provider_init(struct device_node *np, int index,*/ void __init omap2_clk_legacy_provider_init (int index,void __iomem *mem) { +struct clk_iomap *io; + ti_clk_ll_ops = &omap_clk_ll_ops; - clk_memmaps[index] = mem; + io =memblock_virt_alloc (sizeof (*io),0 ); + + io->mem = mem; + + clk_memmaps[index] = io; }/*
arch/arm/mach-omap2/clock.h +3 -1
@@ -274,8 +274,10 @@ extern const struct clksel_rate div31_1to31_rates[];extern int omap2_clkops_enable_clkdm (struct clk_hw *hw);extern void omap2_clkops_disable_clkdm (struct clk_hw *hw); +struct regmap ; +int __init omap2_clk_provider_init (struct device_node *np,int index, -void __iomem *mem); +struct regmap *syscon,void __iomem *mem);void __init omap2_clk_legacy_provider_init (int index,void __iomem *mem);void __init ti_clk_init_features (void );
arch/arm/mach-omap2/cm_common.c +1 -1
@@ -361,7 +361,7 @@ int __init omap_cm_init(void)if (data->flags &CM_NO_CLOCKS )continue ; - ret =omap2_clk_provider_init (np, data->index, data->mem); + ret =omap2_clk_provider_init (np, data->index,NULL , data->mem);if (ret)return ret; }
arch/arm/mach-omap2/control.c +1 -1
@@ -676,7 +676,7 @@ int __init omap_control_init(void)for_each_matching_node_and_match (np, omap_scrm_dt_match_table, &match) { data = match->data; - ret =omap2_clk_provider_init (np, data->index, data->mem); + ret =omap2_clk_provider_init (np, data->index,NULL , data->mem);if (ret)return ret; }
arch/arm/mach-omap2/prm_common.c +1 -1
@@ -798,7 +798,7 @@ int __init omap_prcm_init(void)for_each_matching_node_and_match (np, omap_prcm_dt_match_table, &match) { data = match->data; - ret =omap2_clk_provider_init (np, data->index, data->mem); + ret =omap2_clk_provider_init (np, data->index,NULL , data->mem);if (ret)return ret; }
Signature · 1 line
1.7.9.5
PATCH v6 21/34 ARM: OMAP2+: control: remove API for getting control module base address
In reply to Tero KristoThis shall not be used anymore, as control module driver is converted into using regmap.
--- arch/arm/mach-omap2/control.c | 5 ----- arch/arm/mach-omap2/control.h | 2 -- 2 files changed, 7 deletions(-)
arch/arm/mach-omap2/control.c +0 -5
@@ -145,11 +145,6 @@ void __init omap2_set_globals_control(void __iomem *ctrl, omap4_ctrl_pad_base = ctrl_pad; } -void __iomem *omap_ctrl_base_get (void ) -{ -return omap2_ctrl_base; -} -u8 omap_ctrl_readb (u16 offset) {return readb_relaxed (OMAP_CTRL_REGADDR (offset));
arch/arm/mach-omap2/control.h +0 -2
@@ -440,7 +440,6 @@#ifndef __ASSEMBLY__#ifdef CONFIG_ARCH_OMAP2PLUS -extern void __iomem *omap_ctrl_base_get (void );extern u8 omap_ctrl_readb (u16 offset);extern u16 omap_ctrl_readw (u16 offset);extern u32 omap_ctrl_readl (u16 offset);
@@ -470,7 +469,6 @@ extern void omap2_set_globals_control(void __iomem *ctrl, void __iomem *ctrl_pad);void __init omap3_control_legacy_iomap_init (void );#else -#define omap_ctrl_base_get () 0#define omap_ctrl_readb (x) 0#define omap_ctrl_readw (x) 0#define omap_ctrl_readl (x) 0
Signature · 1 line
1.7.9.5
PATCH v6 22/34 ARM: OMAP2+: id: cache omap_type value
In reply to Tero KristoThere is no need to read the register with every invocation of the function, as the value is constant. Thus, cache the value in a static variable.
--- arch/arm/mach-omap2/id.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
arch/arm/mach-omap2/id.c +4 -1
@@ -52,7 +52,10 @@ EXPORT_SYMBOL(omap_rev);int omap_type (void ) { -u32 val =0 ; +static u32 val =OMAP2_DEVICETYPE_MASK ; + +if (val <OMAP2_DEVICETYPE_MASK ) +return val;if (cpu_is_omap24xx ()) { val =omap_ctrl_readl (OMAP24XX_CONTROL_STATUS );
Signature · 1 line
1.7.9.5
PATCH v6 23/34 ARM: OMAP2+: control: add syscon support for register accesses
In reply to Tero KristoControl module driver needs to support syscon for register accesses, as the DT hierarchy for control module driver is going to be changed. All the control module related features will be moved under control module node, including clocks, pinctrl, and generic configuration register access. Temporary iomap is still provided very early in the boot for access while syscon is not yet ready.
--- arch/arm/mach-omap2/control.c | 104 ++++++++++++++++++++++++++++++++++------- 1 file changed, 87 insertions(+), 17 deletions(-)
arch/arm/mach-omap2/control.c +87 -17
@@ -15,6 +15,8 @@#include <linux/kernel.h> #include <linux/io.h> #include <linux/of_address.h> +#include <linux/regmap.h> +#include <linux/mfd/syscon.h> #include "soc.h" #include "iomap.h"
@@ -33,7 +35,9 @@#define PADCONF_SAVE_DONE 0x1static void __iomem *omap2_ctrl_base; +static s16 omap2_ctrl_offset;static void __iomem *omap4_ctrl_pad_base; +static struct regmap *omap2_ctrl_syscon;#if defined(CONFIG_ARCH_OMAP3 ) && defined(CONFIG_PM ) struct omap3_scratchpad {
@@ -135,7 +139,6 @@ struct omap3_control_regs {static struct omap3_control_regs control_context;#endif /* CONFIG_ARCH_OMAP3 && CONFIG_PM */ -#define OMAP_CTRL_REGADDR (reg) (omap2_ctrl_base + (reg))#define OMAP4_CTRL_PAD_REGADDR (reg) (omap4_ctrl_pad_base + (reg)) void __init omap2_set_globals_control(void __iomem *ctrl,
@@ -147,32 +150,72 @@ void __init omap2_set_globals_control(void __iomem *ctrl,u8 omap_ctrl_readb (u16 offset) { -return readb_relaxed (OMAP_CTRL_REGADDR (offset)); +u32 val; +u8 byte_offset = offset &0x3 ; + + val =omap_ctrl_readl (offset); + +return (val >> (byte_offset *8 )) &0xff ; }u16 omap_ctrl_readw (u16 offset) { -return readw_relaxed (OMAP_CTRL_REGADDR (offset)); +u32 val; +u16 byte_offset = offset &0x2 ; + + val =omap_ctrl_readl (offset); + +return (val >> (byte_offset *8 )) &0xffff ; }u32 omap_ctrl_readl (u16 offset) { -return readl_relaxed (OMAP_CTRL_REGADDR (offset)); +u32 val; + + offset &=0xfffc ; +if (!omap2_ctrl_syscon) + val =readl_relaxed (omap2_ctrl_base + offset); +else +regmap_read (omap2_ctrl_syscon, omap2_ctrl_offset + offset, + &val); + +return val; }void omap_ctrl_writeb (u8 val,u16 offset) { -writeb_relaxed (val,OMAP_CTRL_REGADDR (offset)); +u32 tmp; +u8 byte_offset = offset &0x3 ; + + tmp =omap_ctrl_readl (offset); + + tmp &=0xffffffff ^ (0xff << (byte_offset *8 )); + tmp |= val << (byte_offset *8 ); + +omap_ctrl_writel (tmp, offset); }void omap_ctrl_writew (u16 val,u16 offset) { -writew_relaxed (val,OMAP_CTRL_REGADDR (offset)); +u32 tmp; +u8 byte_offset = offset &0x2 ; + + tmp =omap_ctrl_readl (offset); + + tmp &=0xffffffff ^ (0xffff << (byte_offset *8 )); + tmp |= val << (byte_offset *8 ); + +omap_ctrl_writel (tmp, offset); }void omap_ctrl_writel (u32 val,u16 offset) { -writel_relaxed (val,OMAP_CTRL_REGADDR (offset)); + offset &=0xfffc ; +if (!omap2_ctrl_syscon) +writel_relaxed (val, omap2_ctrl_base + offset); +else +regmap_write (omap2_ctrl_syscon, omap2_ctrl_offset + offset, + val); }/*
@@ -611,7 +654,7 @@ void __init omap3_ctrl_init(void)struct control_init_data {int index; -void __iomem *mem; +s16 offset; };static struct control_init_data ctrl_data = {
@@ -639,17 +682,15 @@ int __init omap2_control_base_init(void)struct device_node *np;const struct of_device_id *match;struct control_init_data *data; -void __iomem *mem;for_each_matching_node_and_match (np, omap_scrm_dt_match_table, &match) { data = (struct control_init_data *)match->data; - mem =of_iomap (np,0 ); -if (!mem) + omap2_ctrl_base =of_iomap (np,0 ); +if (!omap2_ctrl_base)return -ENOMEM ; - omap2_ctrl_base = mem; - data->mem = mem; + omap2_ctrl_offset = data->offset; }return 0 ;
@@ -663,17 +704,46 @@ int __init omap2_control_base_init(void)*/ int __init omap_control_init (void ) { -struct device_node *np; +struct device_node *np, *scm_conf;const struct of_device_id *match;const struct omap_prcm_init_data *data;int ret; +struct regmap *syscon;for_each_matching_node_and_match (np, omap_scrm_dt_match_table, &match) { data = match->data; - ret =omap2_clk_provider_init (np, data->index,NULL , data->mem); -if (ret) -return ret; +/* +* Check if we have scm_conf node, if yes, use this to +* access clock registers. +*/ + scm_conf =of_get_child_by_name (np,"scm_conf" ); + +if (scm_conf) { + syscon =syscon_node_to_regmap (scm_conf); + +if (IS_ERR (syscon)) +return PTR_ERR (syscon); + + omap2_ctrl_syscon = syscon; + +if (of_get_child_by_name (scm_conf,"clocks" )) { + ret =omap2_clk_provider_init (scm_conf, + data->index, + syscon,NULL ); +if (ret) +return ret; + } + +iounmap (omap2_ctrl_base); + omap2_ctrl_base =NULL ; + }else { +/* No scm_conf found, direct access */ + ret =omap2_clk_provider_init (np, data->index,NULL , + omap2_ctrl_base); +if (ret) +return ret; + } }return 0 ;
Signature · 1 line
1.7.9.5
PATCH v6 24/34 ARM: dts: omap24xx: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates an l4 / l4-wkup interconnects for omap2420 / omap2430 SoCs, and moves some of the generic peripherals under it. System control module nodes are moved under this new interconnect also, and the SCM clock layout is changed to use the new SCM node as the clock provider.
--- Documentation/devicetree/bindings/arm/omap/l4.txt | 17 ++++ .../devicetree/bindings/arm/omap/prcm.txt | 2 +- arch/arm/boot/dts/omap2420.dtsi | 80 +++++++++------ arch/arm/boot/dts/omap2430-clocks.dtsi | 8 +- arch/arm/boot/dts/omap2430.dtsi | 107 +++++++++++--------- arch/arm/boot/dts/omap24xx-clocks.dtsi | 6 +- arch/arm/mach-omap2/control.c | 7 +- 7 files changed, 140 insertions(+), 87 deletions(-) create mode 100644 Documentation/devicetree/bindings/arm/omap/l4.txt
Documentation/devicetree/bindings/arm/omap/l4.txt new file +17 -0
@@ -0,0 +1,17 @@ +L4 interconnect bindings + +These bindings describe the OMAP SoCs L4 interconnect bus. + +Required properties: +- compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus + Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus +- ranges : contains the IO map range for the bus + +Examples: + +l4: l4 at 48000000 { + compatible "ti,omap2-l4", "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x48000000 0x100000>; +};
Documentation/devicetree/bindings/arm/omap/prcm.txt +1 -1
@@ -14,7 +14,7 @@ Required properties: "ti,am4-prcm" "ti,am4-scrm" "ti,omap2-prcm" - "ti,omap2-scrm" + "ti,omap2-scm" "ti,omap3-prm" "ti,omap3-cm" "ti,omap3-scrm"
arch/arm/boot/dts/omap2420.dtsi +49 -31
96 lines not on this page · raw message
arch/arm/boot/dts/omap2430-clocks.dtsi +4 -4
@@ -8,12 +8,12 @@ * published by the Free Software Foundation. */ -&scrm_clocks { +&scm_clocks { mcbsp3_mux_fck: mcbsp3_mux_fck { #clock-cells = <0>; compatible = "ti,composite-mux-clock"; clocks = <&func_96m_ck>, <&mcbsp_clks>; - reg = <0x02e8>; + reg = <0x78>; }; mcbsp3_fck: mcbsp3_fck {
@@ -27,7 +27,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&func_96m_ck>, <&mcbsp_clks>; ti,bit-shift = <2>; - reg = <0x02e8>; + reg = <0x78>; }; mcbsp4_fck: mcbsp4_fck {
@@ -41,7 +41,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&func_96m_ck>, <&mcbsp_clks>; ti,bit-shift = <4>; - reg = <0x02e8>; + reg = <0x78>; }; mcbsp5_fck: mcbsp5_fck {
arch/arm/boot/dts/omap2430.dtsi +60 -47
120 lines not on this page · raw message
arch/arm/boot/dts/omap24xx-clocks.dtsi +3 -3
@@ -7,13 +7,13 @@ * it under the terms of the GNU General Public License version 2 as * published by the Free Software Foundation. */ -&scrm_clocks { +&scm_clocks { mcbsp1_mux_fck: mcbsp1_mux_fck { #clock-cells = <0>; compatible = "ti,composite-mux-clock"; clocks = <&func_96m_ck>, <&mcbsp_clks>; ti,bit-shift = <2>; - reg = <0x0274>; + reg = <0x4>; }; mcbsp1_fck: mcbsp1_fck {
@@ -27,7 +27,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&func_96m_ck>, <&mcbsp_clks>; ti,bit-shift = <6>; - reg = <0x0274>; + reg = <0x4>; }; mcbsp2_fck: mcbsp2_fck {
arch/arm/mach-omap2/control.c +6 -1
@@ -661,10 +661,15 @@ static struct control_init_data ctrl_data = { .index = TI_CLKM_CTRL, }; +static const struct control_init_data omap2_ctrl_data = { + .index =TI_CLKM_CTRL , + .offset = -OMAP2_CONTROL_GENERAL , +}; +static const struct of_device_id omap_scrm_dt_match_table[] = { { .compatible ="ti,am3-scrm" , .data = &ctrl_data }, { .compatible ="ti,am4-scrm" , .data = &ctrl_data }, - { .compatible ="ti,omap2-scrm" , .data = &ctrl_data }, + { .compatible ="ti,omap2-scm" , .data = &omap2_ctrl_data }, { .compatible ="ti,omap3-scrm" , .data = &ctrl_data }, { .compatible ="ti,dm816-scrm" , .data = &ctrl_data }, { }
Signature · 1 line
1.7.9.5
PATCH v6 25/34 ARM: dts: omap3: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates an l4_core interconnect for OMAP3, and moves some of the generic peripherals under it. System control module nodes are moved under this new interconnect also, and the SCM clock layout is changed to use the renamed SCM node as the clock provider.
--- Documentation/devicetree/bindings/arm/omap/l4.txt | 1 + .../devicetree/bindings/arm/omap/prcm.txt | 2 +- arch/arm/boot/dts/am3517.dtsi | 2 +- arch/arm/boot/dts/am35xx-clocks.dtsi | 2 +- arch/arm/boot/dts/omap3.dtsi | 96 +++++++++++--------- arch/arm/boot/dts/omap3xxx-clocks.dtsi | 13 +-- arch/arm/mach-omap2/control.c | 2 +- 7 files changed, 67 insertions(+), 51 deletions(-)
Documentation/devicetree/bindings/arm/omap/l4.txt +1 -0
@@ -5,6 +5,7 @@ These bindings describe the OMAP SoCs L4 interconnect bus. Required properties: - compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus + Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus - ranges : contains the IO map range for the bus Examples:
Documentation/devicetree/bindings/arm/omap/prcm.txt +1 -1
@@ -17,7 +17,7 @@ Required properties: "ti,omap2-scm" "ti,omap3-prm" "ti,omap3-cm" - "ti,omap3-scrm" + "ti,omap3-scm" "ti,omap4-cm1" "ti,omap4-prm" "ti,omap4-cm2"
arch/arm/boot/dts/am3517.dtsi +1 -1
@@ -31,7 +31,7 @@ status = "disabled"; reg = <0x5c000000 0x30000>; interrupts = <67 68 69 70>; - syscon = <&omap3_scm_general>; + syscon = <&scm_conf>; ti,davinci-ctrl-reg-offset = <0x10000>; ti,davinci-ctrl-mod-reg-offset = <0>; ti,davinci-ctrl-ram-offset = <0x20000>;
arch/arm/boot/dts/am35xx-clocks.dtsi +1 -1
@@ -7,7 +7,7 @@ * it under the terms of the GNU General Public License version 2 as * published by the Free Software Foundation. */ -&scrm_clocks { +&scm_clocks { emac_ick: emac_ick { #clock-cells = <0>; compatible = "ti,am35xx-gate-clock";
arch/arm/boot/dts/omap3.dtsi +55 -41
117 lines not on this page · raw message
arch/arm/boot/dts/omap3xxx-clocks.dtsi +7 -6
@@ -79,13 +79,14 @@ clock-div = <1>; }; }; -&scrm_clocks { + +&scm_clocks { mcbsp5_mux_fck: mcbsp5_mux_fck { #clock-cells = <0>; compatible = "ti,composite-mux-clock"; clocks = <&core_96m_fck>, <&mcbsp_clks>; ti,bit-shift = <4>; - reg = <0x02d8>; + reg = <0x68>; }; mcbsp5_fck: mcbsp5_fck {
@@ -99,7 +100,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&core_96m_fck>, <&mcbsp_clks>; ti,bit-shift = <2>; - reg = <0x0274>; + reg = <0x04>; }; mcbsp1_fck: mcbsp1_fck {
@@ -113,7 +114,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&per_96m_fck>, <&mcbsp_clks>; ti,bit-shift = <6>; - reg = <0x0274>; + reg = <0x04>; }; mcbsp2_fck: mcbsp2_fck {
@@ -126,7 +127,7 @@ #clock-cells = <0>; compatible = "ti,composite-mux-clock"; clocks = <&per_96m_fck>, <&mcbsp_clks>; - reg = <0x02d8>; + reg = <0x68>; }; mcbsp3_fck: mcbsp3_fck {
@@ -140,7 +141,7 @@ compatible = "ti,composite-mux-clock"; clocks = <&per_96m_fck>, <&mcbsp_clks>; ti,bit-shift = <2>; - reg = <0x02d8>; + reg = <0x68>; }; mcbsp4_fck: mcbsp4_fck {
arch/arm/mach-omap2/control.c +1 -1
@@ -670,7 +670,7 @@ static const struct of_device_id omap_scrm_dt_match_table[] = { { .compatible = "ti,am3-scrm", .data = &ctrl_data }, { .compatible = "ti,am4-scrm", .data = &ctrl_data }, { .compatible = "ti,omap2-scm", .data = &omap2_ctrl_data }, - { .compatible = "ti,omap3-scrm", .data = &ctrl_data }, + { .compatible = "ti,omap3-scm", .data = &omap2_ctrl_data }, { .compatible = "ti,dm816-scrm", .data = &ctrl_data }, { } };
Signature · 1 line
1.7.9.5
PATCH v6 26/34 ARM: dts: am33xx: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates an l4_wkup interconnect for AM33xx, and moves some of the generic peripherals under it. System control module nodes are moved under this new interconnect also, and the SCM clock layout is changed to use the renamed SCM node as the clock provider.
--- Documentation/devicetree/bindings/arm/omap/l4.txt | 1 + .../devicetree/bindings/arm/omap/prcm.txt | 2 +- arch/arm/boot/dts/am33xx-clocks.dtsi | 2 +- arch/arm/boot/dts/am33xx.dtsi | 87 +++++++++++--------- arch/arm/mach-omap2/control.c | 2 +- 5 files changed, 51 insertions(+), 43 deletions(-)
Documentation/devicetree/bindings/arm/omap/l4.txt +1 -0
@@ -6,6 +6,7 @@ Required properties: - compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus + Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus - ranges : contains the IO map range for the bus Examples:
Documentation/devicetree/bindings/arm/omap/prcm.txt +1 -1
@@ -10,7 +10,7 @@ documentation about the individual clock/clockdomain nodes. Required properties: - compatible: Must be one of: "ti,am3-prcm" - "ti,am3-scrm" + "ti,am3-scm" "ti,am4-prcm" "ti,am4-scrm" "ti,omap2-prcm"
arch/arm/boot/dts/am33xx-clocks.dtsi +1 -1
@@ -7,7 +7,7 @@ * it under the terms of the GNU General Public License version 2 as * published by the Free Software Foundation. */ -&scrm_clocks { +&scm_clocks { sys_clkin_ck: sys_clkin_ck { #clock-cells = <0>; compatible = "ti,mux-clock";
arch/arm/boot/dts/am33xx.dtsi +47 -40
125 lines not on this page · raw message
arch/arm/mach-omap2/control.c +1 -1
@@ -667,7 +667,7 @@ static const struct control_init_data omap2_ctrl_data = { };static const struct of_device_id omap_scrm_dt_match_table[] = { - { .compatible ="ti,am3-scrm" , .data = &ctrl_data }, + { .compatible ="ti,am3-scm" , .data = &ctrl_data }, { .compatible ="ti,am4-scrm" , .data = &ctrl_data }, { .compatible ="ti,omap2-scm" , .data = &omap2_ctrl_data }, { .compatible ="ti,omap3-scm" , .data = &omap2_ctrl_data },
Signature · 1 line
1.7.9.5
PATCH v6 27/34 ARM: dts: am43xx-epos-evm: fix pinmux node layout
In reply to Tero KristoPinmux node should be a reference to the base one, not a complete re-write of it. Having the node like this also prevents modifying the node layout in the base am4372.dtsi file, which is needed for control module changes.
--- arch/arm/boot/dts/am43x-epos-evm.dts | 84 +++++++++++++++++----------------- 1 file changed, 42 insertions(+), 42 deletions(-)
arch/arm/boot/dts/am43x-epos-evm.dts +42 -42
@@ -69,7 +69,48 @@ }; }; - am43xx_pinmux: pinmux at 44e10800 { + matrix_keypad: matrix_keypad at 0 { + compatible = "gpio-matrix-keypad"; + debounce-delay-ms = <5>; + col-scan-delay-us = <2>; + + row-gpios = <&gpio0 12 GPIO_ACTIVE_HIGH /* Bank0, pin12 */ + &gpio0 13 GPIO_ACTIVE_HIGH /* Bank0, pin13 */ + &gpio0 14 GPIO_ACTIVE_HIGH /* Bank0, pin14 */ + &gpio0 15 GPIO_ACTIVE_HIGH>; /* Bank0, pin15 */ + + col-gpios = <&gpio3 9 GPIO_ACTIVE_HIGH /* Bank3, pin9 */ + &gpio3 10 GPIO_ACTIVE_HIGH /* Bank3, pin10 */ + &gpio2 18 GPIO_ACTIVE_HIGH /* Bank2, pin18 */ + &gpio2 19 GPIO_ACTIVE_HIGH>; /* Bank2, pin19 */ + + linux,keymap = <0x00000201 /* P1 */ + 0x01000204 /* P4 */ + 0x02000207 /* P7 */ + 0x0300020a /* NUMERIC_STAR */ + 0x00010202 /* P2 */ + 0x01010205 /* P5 */ + 0x02010208 /* P8 */ + 0x03010200 /* P0 */ + 0x00020203 /* P3 */ + 0x01020206 /* P6 */ + 0x02020209 /* P9 */ + 0x0302020b /* NUMERIC_POUND */ + 0x00030067 /* UP */ + 0x0103006a /* RIGHT */ + 0x0203006c /* DOWN */ + 0x03030069>; /* LEFT */ + }; + + backlight { + compatible = "pwm-backlight"; + pwms = <&ecap0 0 50000 PWM_POLARITY_INVERTED>; + brightness-levels = <0 51 53 56 62 75 101 152 255>; + default-brightness-level = <8>; + }; +}; + +&am43xx_pinmux { cpsw_default: cpsw_default { pinctrl-single,pins = < /* Slave 1 */
@@ -279,47 +320,6 @@ 0x204 (DS0_PULL_UP_DOWN_EN | INPUT_EN | MUX_MODE7) >; }; - }; - - matrix_keypad: matrix_keypad at 0 { - compatible = "gpio-matrix-keypad"; - debounce-delay-ms = <5>; - col-scan-delay-us = <2>; - - row-gpios = <&gpio0 12 GPIO_ACTIVE_HIGH /* Bank0, pin12 */ - &gpio0 13 GPIO_ACTIVE_HIGH /* Bank0, pin13 */ - &gpio0 14 GPIO_ACTIVE_HIGH /* Bank0, pin14 */ - &gpio0 15 GPIO_ACTIVE_HIGH>; /* Bank0, pin15 */ - - col-gpios = <&gpio3 9 GPIO_ACTIVE_HIGH /* Bank3, pin9 */ - &gpio3 10 GPIO_ACTIVE_HIGH /* Bank3, pin10 */ - &gpio2 18 GPIO_ACTIVE_HIGH /* Bank2, pin18 */ - &gpio2 19 GPIO_ACTIVE_HIGH>; /* Bank2, pin19 */ - - linux,keymap = <0x00000201 /* P1 */ - 0x01000204 /* P4 */ - 0x02000207 /* P7 */ - 0x0300020a /* NUMERIC_STAR */ - 0x00010202 /* P2 */ - 0x01010205 /* P5 */ - 0x02010208 /* P8 */ - 0x03010200 /* P0 */ - 0x00020203 /* P3 */ - 0x01020206 /* P6 */ - 0x02020209 /* P9 */ - 0x0302020b /* NUMERIC_POUND */ - 0x00030067 /* UP */ - 0x0103006a /* RIGHT */ - 0x0203006c /* DOWN */ - 0x03030069>; /* LEFT */ - }; - - backlight { - compatible = "pwm-backlight"; - pwms = <&ecap0 0 50000 PWM_POLARITY_INVERTED>; - brightness-levels = <0 51 53 56 62 75 101 152 255>; - default-brightness-level = <8>; - }; }; &mmc1 {
Signature · 1 line
1.7.9.5
PATCH v6 28/34 ARM: dts: am4372: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates an l4_wkup interconnect for AM43xx, and moves some of the generic peripherals under it. System control module nodes are moved under this new interconnect also, and the SCM clock layout is changed to use the renamed SCM nodea as the clock provider.
--- Documentation/devicetree/bindings/arm/omap/l4.txt | 1 + .../devicetree/bindings/arm/omap/prcm.txt | 2 +- arch/arm/boot/dts/am4372.dtsi | 85 +++++++++++--------- arch/arm/boot/dts/am43xx-clocks.dtsi | 2 +- arch/arm/mach-omap2/control.c | 2 +- 5 files changed, 53 insertions(+), 39 deletions(-)
Documentation/devicetree/bindings/arm/omap/l4.txt +1 -0
@@ -7,6 +7,7 @@ Required properties: Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus + Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus - ranges : contains the IO map range for the bus Examples:
Documentation/devicetree/bindings/arm/omap/prcm.txt +1 -1
@@ -12,7 +12,7 @@ Required properties: "ti,am3-prcm" "ti,am3-scm" "ti,am4-prcm" - "ti,am4-scrm" + "ti,am4-scm" "ti,omap2-prcm" "ti,omap2-scm" "ti,omap3-prm"
arch/arm/boot/dts/am4372.dtsi +49 -36
114 lines not on this page · raw message
arch/arm/boot/dts/am43xx-clocks.dtsi +1 -1
@@ -7,7 +7,7 @@ * it under the terms of the GNU General Public License version 2 as * published by the Free Software Foundation. */ -&scrm_clocks { +&scm_clocks { sys_clkin_ck: sys_clkin_ck { #clock-cells = <0>; compatible = "ti,mux-clock";
arch/arm/mach-omap2/control.c +1 -1
@@ -668,7 +668,7 @@ static const struct control_init_data omap2_ctrl_data = {static const struct of_device_id omap_scrm_dt_match_table[] = { { .compatible ="ti,am3-scm" , .data = &ctrl_data }, - { .compatible ="ti,am4-scrm" , .data = &ctrl_data }, + { .compatible ="ti,am4-scm" , .data = &ctrl_data }, { .compatible ="ti,omap2-scm" , .data = &omap2_ctrl_data }, { .compatible ="ti,omap3-scm" , .data = &omap2_ctrl_data }, { .compatible ="ti,dm816-scrm" , .data = &ctrl_data },
Signature · 1 line
1.7.9.5
PATCH v6 29/34 ARM: dts: omap4: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates the l4_cfg and l4_wkup interconnects for OMAP4, and moves some of the generic peripherals under it. System control module support is added to the device tree also, and the existing SCM related functionality is moved under it.
--- .../devicetree/bindings/arm/omap/ctrl.txt | 76 ++++++++ Documentation/devicetree/bindings/arm/omap/l4.txt | 2 + .../devicetree/bindings/arm/omap/prcm.txt | 6 - arch/arm/boot/dts/omap4.dtsi | 200 ++++++++++++-------- 4 files changed, 199 insertions(+), 85 deletions(-) create mode 100644 Documentation/devicetree/bindings/arm/omap/ctrl.txt
Documentation/devicetree/bindings/arm/omap/ctrl.txt new file +76 -0
@@ -0,0 +1,76 @@ +OMAP Control Module bindings + +Control Module contains miscellaneous features under it based on SoC type. +Pincontrol is one common feature, and it has a specialized support +described in [1]. Typically some clock nodes are also under control module. +Syscon is used to share register level access to drivers external to +control module driver itself. + +See [2] for documentation about clock/clockdomain nodes. + +[1] Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt +[2] Documentation/devicetree/bindings/clock/ti/* + +Required properties: +- compatible: Must be one of: + "ti,am3-scm" + "ti,am4-scm" + "ti,dm814-scrm" + "ti,dm816-scrm" + "ti,omap2-scm" + "ti,omap3-scm" + "ti,omap4-scm-core" + "ti,omap4-scm-padconf-core" +- reg: Contains Control Module register address range + (base address and length) + +Optional properties: +- clocks: clocks for this module +- clockdomains: clockdomains for this module + +Examples: + +scm: scm at 2000 { + compatible = "ti,omap3-scm", "simple-bus"; + reg = <0x2000 0x2000>; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0x2000 0x2000>; + + omap3_pmx_core: pinmux at 30 { + compatible = "ti,omap3-padconf", + "pinctrl-single"; + reg = <0x30 0x230>; + #address-cells = <1>; + #size-cells = <0>; + #interrupt-cells = <1>; + interrupt-controller; + pinctrl-single,register-width = <16>; + pinctrl-single,function-mask = <0xff1f>; + }; + + scm_conf: scm_conf at 270 { + compatible = "syscon"; + reg = <0x270 0x330>; + #address-cells = <1>; + #size-cells = <1>; + + scm_clocks: clocks { + #address-cells = <1>; + #size-cells = <0>; + }; + }; + + scm_clockdomains: clockdomains { + }; +} + +&scm_clocks { + mcbsp5_mux_fck: mcbsp5_mux_fck { + #clock-cells = <0>; + compatible = "ti,composite-mux-clock"; + clocks = <&core_96m_fck>, <&mcbsp_clks>; + ti,bit-shift = <4>; + reg = <0x02d8>; + }; +};
Documentation/devicetree/bindings/arm/omap/l4.txt +2 -0
@@ -6,6 +6,8 @@ Required properties: - compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus + Should be "ti,omap4-l4-cfg" for OMAP4 family l4 cfg bus + Should be "ti,omap4-l4-wkup" for OMAP4 family l4 wkup bus Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus - ranges : contains the IO map range for the bus
Documentation/devicetree/bindings/arm/omap/prcm.txt +0 -6
@@ -10,14 +10,10 @@ documentation about the individual clock/clockdomain nodes. Required properties: - compatible: Must be one of: "ti,am3-prcm" - "ti,am3-scm" "ti,am4-prcm" - "ti,am4-scm" "ti,omap2-prcm" - "ti,omap2-scm" "ti,omap3-prm" "ti,omap3-cm" - "ti,omap3-scm" "ti,omap4-cm1" "ti,omap4-prm" "ti,omap4-cm2"
@@ -30,9 +26,7 @@ Required properties: "ti,dra7-cm-core-aon" "ti,dra7-cm-core" "ti,dm814-prcm" - "ti,dm814-scrm" "ti,dm816-prcm" - "ti,dm816-scrm" - reg: Contains PRCM module register address range (base address and length) - clocks: clocks for this module
arch/arm/boot/dts/omap4.dtsi +121 -79
220 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH v6 30/34 ARM: OMAP4: display: convert display to use syscon for dsi muxing
In reply to Tero KristoThe legacy control module APIs will be gone, thus convert the display driver to use syscon. This change should eventually be moved to display driver from the board directory.
--- arch/arm/mach-omap2/display.c | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-)
arch/arm/mach-omap2/display.c +13 -2
@@ -26,6 +26,8 @@#include <linux/of.h> #include <linux/of_platform.h> #include <linux/slab.h> +#include <linux/mfd/syscon.h> +#include <linux/regmap.h> #include <video/omapdss.h> #include "omap_hwmod.h"
@@ -104,6 +106,10 @@ static const struct omap_dss_hwmod_data omap4_dss_hwmod_data[] __initconst = { { "dss_hdmi", "omapdss_hdmi", -1 }, }; +#define OMAP4_DSIPHY_SYSCON_OFFSET 0x78 + +static struct regmap *omap4_dsi_mux_syscon; +static int omap4_dsi_mux_pads (int dsi_id,unsigned lanes) {u32 enable_mask, enable_shift;
@@ -124,7 +130,7 @@ static int omap4_dsi_mux_pads(int dsi_id, unsigned lanes)return -ENODEV ; } - reg =omap4_ctrl_pad_readl (OMAP4_CTRL_MODULE_PAD_CORE_CONTROL_DSIPHY ); +regmap_read (omap4_dsi_mux_syscon,OMAP4_DSIPHY_SYSCON_OFFSET , ®); reg &= ~enable_mask; reg &= ~pipd_mask;
@@ -132,7 +138,7 @@ static int omap4_dsi_mux_pads(int dsi_id, unsigned lanes) reg |= (lanes << enable_shift) & enable_mask; reg |= (lanes << pipd_shift) & pipd_mask; -omap4_ctrl_pad_writel (reg,OMAP4_CTRL_MODULE_PAD_CORE_CONTROL_DSIPHY ); +regmap_write (omap4_dsi_mux_syscon,OMAP4_DSIPHY_SYSCON_OFFSET , reg);return 0 ; }
@@ -665,5 +671,10 @@ int __init omapdss_init_of(void)return r; } +/* add DSI info for omap4 */ + node =of_find_node_by_name (NULL ,"omap4_padconf_global" ); +if (node) + omap4_dsi_mux_syscon =syscon_node_to_regmap (node); +return 0 ; }
Signature · 1 line
1.7.9.5
PATCH v6 31/34 ARM: OMAP4+: control: remove support for legacy pad read/write
In reply to Tero Kristoomap4_ctrl_pad_readl/writel are no longer used by anybody, so remove these. Syscon / pinctrl should be used to access the padconf area instead.
--- arch/arm/mach-omap2/control.c | 24 +----------------------- arch/arm/mach-omap2/control.h | 5 +---- arch/arm/mach-omap2/io.c | 11 ++++------- 3 files changed, 6 insertions(+), 34 deletions(-)
arch/arm/mach-omap2/control.c +1 -23
@@ -36,7 +36,6 @@static void __iomem *omap2_ctrl_base;static s16 omap2_ctrl_offset; -static void __iomem *omap4_ctrl_pad_base;static struct regmap *omap2_ctrl_syscon;#if defined(CONFIG_ARCH_OMAP3 ) && defined(CONFIG_PM )
@@ -139,13 +138,9 @@ struct omap3_control_regs {static struct omap3_control_regs control_context;#endif /* CONFIG_ARCH_OMAP3 && CONFIG_PM */ -#define OMAP4_CTRL_PAD_REGADDR (reg) (omap4_ctrl_pad_base + (reg)) - -void __init omap2_set_globals_control (void __iomem *ctrl, -void __iomem *ctrl_pad) +void __init omap2_set_globals_control (void __iomem *ctrl) { omap2_ctrl_base = ctrl; - omap4_ctrl_pad_base = ctrl_pad; }u8 omap_ctrl_readb (u16 offset)
@@ -218,23 +213,6 @@ void omap_ctrl_writel(u32 val, u16 offset) val); } -/* -* On OMAP4 control pad are not addressable from control -* core base. So the common omap_ctrl_read/write APIs breaks -* Hence export separate APIs to manage the omap4 pad control -* registers. This APIs will work only for OMAP4 -*/ - -u32 omap4_ctrl_pad_readl (u16 offset) -{ -return readl_relaxed (OMAP4_CTRL_PAD_REGADDR (offset)); -} - -void omap4_ctrl_pad_writel (u32 val,u16 offset) -{ -writel_relaxed (val,OMAP4_CTRL_PAD_REGADDR (offset)); -} -#ifdef CONFIG_ARCH_OMAP3 /**
arch/arm/mach-omap2/control.h +1 -4
@@ -443,11 +443,9 @@extern u8 omap_ctrl_readb (u16 offset);extern u16 omap_ctrl_readw (u16 offset);extern u32 omap_ctrl_readl (u16 offset); -extern u32 omap4_ctrl_pad_readl (u16 offset);extern void omap_ctrl_writeb (u8 val,u16 offset);extern void omap_ctrl_writew (u16 val,u16 offset);extern void omap_ctrl_writel (u32 val,u16 offset); -extern void omap4_ctrl_pad_writel (u32 val,u16 offset);extern void omap3_save_scratchpad_contents (void );extern void omap3_clear_scratchpad_contents (void );
@@ -465,8 +463,7 @@ extern int omap3_ctrl_save_padconf(void);void omap3_ctrl_init (void );int omap2_control_base_init (void );int omap_control_init (void ); -extern void omap2_set_globals_control (void __iomem *ctrl, -void __iomem *ctrl_pad); +void omap2_set_globals_control (void __iomem *ctrl);void __init omap3_control_legacy_iomap_init (void );#else #define omap_ctrl_readb (x) 0
arch/arm/mach-omap2/io.c +4 -7
@@ -443,7 +443,7 @@ void __init omap3_init_early(void)/* XXX: remove these once OMAP3 is DT only */ if (!of_have_populated_dt ()) {omap2_set_globals_control ( -OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ),NULL ); +OMAP2_L4_IO_ADDRESS (OMAP343X_CTRL_BASE ));omap2_set_globals_prm (OMAP2_L4_IO_ADDRESS (OMAP3430_PRM_BASE ));omap2_set_globals_cm (OMAP2_L4_IO_ADDRESS (OMAP3430_CM_BASE ),NULL );
@@ -631,8 +631,7 @@ void __init omap4430_init_early(void) {omap2_set_globals_tap (OMAP443X_CLASS ,OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE ), -OMAP2_L4_IO_ADDRESS (OMAP443X_CTRL_BASE )); +omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE ));omap4xxx_check_revision ();omap4xxx_check_features ();
@@ -660,8 +659,7 @@ void __init omap5_init_early(void) {omap2_set_globals_tap (OMAP54XX_CLASS ,OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ), -OMAP2_L4_IO_ADDRESS (OMAP54XX_CTRL_BASE )); +omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap2_prcm_base_init ();
@@ -686,8 +684,7 @@ void __init omap5_init_late(void)void __init dra7xx_init_early (void ) {omap2_set_globals_tap (-1 ,OMAP2_L4_IO_ADDRESS (DRA7XX_TAP_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ), -OMAP2_L4_IO_ADDRESS (DRA7XX_CTRL_BASE )); +omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE ));omap4_pm_init_early ();omap2_prcm_base_init ();
Signature · 1 line
1.7.9.5
PATCH v6 32/34 ARM: dts: omap5: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates the l4_cfg and l4_wkup interconnects for OMAP5, and moves some of the generic peripherals under it. System control module support is added to the device tree also, and the existing SCM related functionality is moved under it.
--- .../devicetree/bindings/arm/omap/ctrl.txt | 2 + Documentation/devicetree/bindings/arm/omap/l4.txt | 2 + arch/arm/boot/dts/omap5.dtsi | 182 ++++++++++++-------- 3 files changed, 116 insertions(+), 70 deletions(-)
Documentation/devicetree/bindings/arm/omap/ctrl.txt +2 -0
@@ -21,6 +21,8 @@ Required properties: "ti,omap3-scm" "ti,omap4-scm-core" "ti,omap4-scm-padconf-core" + "ti,omap5-scm-core" + "ti,omap5-scm-padconf-core" - reg: Contains Control Module register address range (base address and length)
Documentation/devicetree/bindings/arm/omap/l4.txt +2 -0
@@ -8,6 +8,8 @@ Required properties: Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus Should be "ti,omap4-l4-cfg" for OMAP4 family l4 cfg bus Should be "ti,omap4-l4-wkup" for OMAP4 family l4 wkup bus + Should be "ti,omap5-l4-cfg" for OMAP5 family l4 cfg bus + Should be "ti,omap5-l4-wkup" for OMAP5 family l4 wkup bus Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus - ranges : contains the IO map range for the bus
arch/arm/boot/dts/omap5.dtsi +112 -70
211 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH v6 33/34 ARM: dts: dra7: add minimal l4 bus layout with control module support
In reply to Tero KristoThis patch creates the l4_cfg and l4_wkup interconnects for DRA7, and moves some of the generic peripherals under it. System control module support is added to the device tree also, and the existing SCM related functionality is moved under it.
--- .../devicetree/bindings/arm/omap/ctrl.txt | 1 + Documentation/devicetree/bindings/arm/omap/l4.txt | 2 + arch/arm/boot/dts/dra7.dtsi | 166 +++++++++++--------- 3 files changed, 96 insertions(+), 73 deletions(-)
Documentation/devicetree/bindings/arm/omap/ctrl.txt +1 -0
@@ -23,6 +23,7 @@ Required properties: "ti,omap4-scm-padconf-core" "ti,omap5-scm-core" "ti,omap5-scm-padconf-core" + "ti,dra7-scm-core" - reg: Contains Control Module register address range (base address and length)
Documentation/devicetree/bindings/arm/omap/l4.txt +2 -0
@@ -10,6 +10,8 @@ Required properties: Should be "ti,omap4-l4-wkup" for OMAP4 family l4 wkup bus Should be "ti,omap5-l4-cfg" for OMAP5 family l4 cfg bus Should be "ti,omap5-l4-wkup" for OMAP5 family l4 wkup bus + Should be "ti,dra7-l4-cfg" for DRA7 family l4 cfg bus + Should be "ti,dra7-l4-wkup" for DRA7 family l4 wkup bus Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus - ranges : contains the IO map range for the bus
arch/arm/boot/dts/dra7.dtsi +93 -73
194 lines not on this page · raw message
Signature · 1 line
1.7.9.5
PATCH v6 34/34 ARM: OMAP4+: control: add support for initializing control module via DT
In reply to Tero KristoOMAP4, OMAP5 and DRA7 now parse DT entries for control module address spaces, and set up syscon mappings appropriately. Low level IO init is updated to remove the legacy control module mappings for these devices also.
--- arch/arm/mach-omap2/control.c | 3 +++ arch/arm/mach-omap2/io.c | 6 +++--- 2 files changed, 6 insertions(+), 3 deletions(-)
arch/arm/mach-omap2/control.c +3 -0
@@ -650,6 +650,9 @@ static const struct of_device_id omap_scrm_dt_match_table[] = { { .compatible = "ti,omap2-scm", .data = &omap2_ctrl_data }, { .compatible = "ti,omap3-scm", .data = &omap2_ctrl_data }, { .compatible = "ti,dm816-scrm", .data = &ctrl_data }, + { .compatible = "ti,omap4-scm-core", .data = &ctrl_data }, + { .compatible = "ti,omap5-scm-core", .data = &ctrl_data }, + { .compatible = "ti,dra7-scm-core", .data = &ctrl_data }, { } };
arch/arm/mach-omap2/io.c +3 -3
@@ -631,8 +631,8 @@ void __init omap4430_init_early(void) {omap2_set_globals_tap (OMAP443X_CLASS ,OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP443X_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP4430_PRCM_MPU_BASE )); +omap2_control_base_init ();omap4xxx_check_revision ();omap4xxx_check_features ();omap2_prcm_base_init ();
@@ -659,8 +659,8 @@ void __init omap5_init_early(void) {omap2_set_globals_tap (OMAP54XX_CLASS ,OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE )); +omap2_control_base_init ();omap4_pm_init_early ();omap2_prcm_base_init ();omap5xxx_check_revision ();
@@ -684,8 +684,8 @@ void __init omap5_init_late(void)void __init dra7xx_init_early (void ) {omap2_set_globals_tap (-1 ,OMAP2_L4_IO_ADDRESS (DRA7XX_TAP_BASE )); -omap2_set_globals_control (OMAP2_L4_IO_ADDRESS (OMAP54XX_SCM_BASE ));omap2_set_globals_prcm_mpu (OMAP2_L4_IO_ADDRESS (OMAP54XX_PRCM_MPU_BASE )); +omap2_control_base_init ();omap4_pm_init_early ();omap2_prcm_base_init ();dra7xxx_check_revision ();
Signature · 1 line
1.7.9.5
Tony Lindgren
maintainerv6 cover ARM: OMAP2+: PRCM+SCM cleanup against 4.0-rc1
In reply to Tero KristoQuoted 33 lines · Tero Kristo
Hi,
Decided to post a new version because of the number of minor changes compared to v5, just to avoid any misunderstandings:
- dropped patch #1 ("clk: ti: fix
ti_clk_get_reg_addr..."), this has been merged through clock tree now
- fixedscm_confarea size issues on am33xx/am43xx
- fixed scm area size issue on am43xx
- fixed size issues onomap3_pmx_core(was 8 bytes short)
- added warnings on omap2_clk_readl/writel in case legacy access is attempted
- fixed omap3 legacy boot
- fixed boot time warning on omap4+ wherescm_confdoes not contain clocks node yet (these may be added to DT later if support is required)
- fixed some randconfig warningsTesting done:
1: am335x-evm : boot 2: am335x-evmsk : boot 3: am3517-evm : boot 4: am43x-epos-evm : boot 5: am437x-gp-evm : boot 6: omap3-beagle-xm : boot 7: omap3-beagle : boot, cpuidle (ret/off), suspend (ret/off), legacy boot 8: am335x-boneblack: boot 9: am335x-bone : boot 10: dra7xx-evm : boot 11: omap3-n900 : boot 12: omap5-uevm : boot 13: omap4-panda-es : boot, cpuidle (ret), suspend (ret) 14: omap4-panda : boot 15: omap2430-sdp : boot 16: omap3430-sdp : boot
Quoted 5 lines
17: omap4-sdp-es23plus: boot
New branch also pushed:
tree: https://github.com/t-kristo/linux-pm.git
branch: 4.0-rc1-prcm-cleanup-v6
OK great, this version seems to work for me. I'll merge this into omap-for-v4.1/prcm-cleanup and will send a pull request probably on Thursday or Friday to arm-soc. Note that it's getting late for v4.1 (except for fixes), so no guarantees this will still get in.
Regards,
Tony