ARM: integrator: check PL010 base address from resource
Subsystems (2)
PATCH v3 2/6 ARM: integrator: check PL010 base address from resource
In the PL010 UART callback a comparison against the location of the statically allocated PL010 device is done to figure out which UART is doing the callback. This does not play well with dynamic devices such as in device tree, so let's check the base address of the memory resource inside the amba_device instead.
---
ChangeLog v2->v3:
- Check for the base address of the physical memory resource instead of checking for the device name (which will also be dynamic when we use device tree).
--- arch/arm/mach-integrator/core.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
arch/arm/mach-integrator/core.c +4 -1
@@ -99,11 +99,14 @@ int __init integrator_init(bool is_cp)static void integrator_uart_set_mctrl (struct amba_device *dev,void __iomem *base,unsigned int mctrl) {unsigned int ctrls =0 , ctrlc =0 , rts_mask, dtr_mask; +u32 phybase = dev->res.start; -if (dev == &uart0_device) { +if (phybase ==INTEGRATOR_UART0_BASE ) { +/* UART0 */ rts_mask =1 <<4 ; dtr_mask =1 <<5 ; }else { +/* UART1 */ rts_mask =1 <<6 ; dtr_mask =1 <<7 ; }
Signature · 1 line
1.7.11.4
PATCH v3 2/6 ARM: integrator: check PL010 base address from resource
Quoted 6 lines · Linus Walleij
In the PL010 UART callback a comparison against the location of the statically allocated PL010 device is done to figure out which UART is doing the callback. This does not play well with dynamic devices such as in device tree, so let's check the base address of the memory resource inside the
amba_deviceinstead.
Quoted 6 lines
Signed-off-by Linus Walleij <[email protected]>---
ChangeLogv2->v3:
- Check for the base address of the physical memory resource instead of checking for the device name (which will also be dynamic when we use device tree).
Ok, looks good to me.