Re: [PATCH] ARM: shmobile: Stop using __raw_*() I/O accessors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

Thanks for your work.

On 2020-11-17 15:24:47 +0100, Geert Uytterhoeven wrote:
> There is no reason to keep on using the __raw_{read,write}l() I/O
> accessors in Renesas ARM platform code.  Switch to using the plain
> {read,write}l() I/O accessors, to have a chance that this works on
> big-endian.
> 
> Suggested-by: Arnd Bergmann <arnd@xxxxxxxxxx>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>

> ---
> To be queued in renesas-devel for v5.11.
> 
>  arch/arm/mach-shmobile/platsmp-scu.c   |  2 +-
>  arch/arm/mach-shmobile/setup-r8a7778.c |  8 ++++----
>  arch/arm/mach-shmobile/setup-r8a7779.c | 14 +++++++-------
>  arch/arm/mach-shmobile/smp-r8a7779.c   |  2 +-
>  arch/arm/mach-shmobile/smp-sh73a0.c    | 10 +++++-----
>  5 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/arch/arm/mach-shmobile/platsmp-scu.c b/arch/arm/mach-shmobile/platsmp-scu.c
> index fcfcef1d1ae4f81f..3849f71e6e12eba6 100644
> --- a/arch/arm/mach-shmobile/platsmp-scu.c
> +++ b/arch/arm/mach-shmobile/platsmp-scu.c
> @@ -64,7 +64,7 @@ static int shmobile_smp_scu_psr_core_disabled(int cpu)
>  {
>  	unsigned long mask = SCU_PM_POWEROFF << (cpu * 8);
>  
> -	if ((__raw_readl(shmobile_scu_base + 8) & mask) == mask)
> +	if ((readl(shmobile_scu_base + 8) & mask) == mask)
>  		return 1;
>  
>  	return 0;
> diff --git a/arch/arm/mach-shmobile/setup-r8a7778.c b/arch/arm/mach-shmobile/setup-r8a7778.c
> index 819dbda47298ae88..02cda9cada4c4546 100644
> --- a/arch/arm/mach-shmobile/setup-r8a7778.c
> +++ b/arch/arm/mach-shmobile/setup-r8a7778.c
> @@ -31,12 +31,12 @@ static void __init r8a7778_init_irq_dt(void)
>  	irqchip_init();
>  
>  	/* route all interrupts to ARM */
> -	__raw_writel(0x73ffffff, base + INT2NTSR0);
> -	__raw_writel(0xffffffff, base + INT2NTSR1);
> +	writel(0x73ffffff, base + INT2NTSR0);
> +	writel(0xffffffff, base + INT2NTSR1);
>  
>  	/* unmask all known interrupts in INTCS2 */
> -	__raw_writel(0x08330773, base + INT2SMSKCR0);
> -	__raw_writel(0x00311110, base + INT2SMSKCR1);
> +	writel(0x08330773, base + INT2SMSKCR0);
> +	writel(0x00311110, base + INT2SMSKCR1);
>  
>  	iounmap(base);
>  }
> diff --git a/arch/arm/mach-shmobile/setup-r8a7779.c b/arch/arm/mach-shmobile/setup-r8a7779.c
> index 446d40b50b7b784b..b6e282116d664705 100644
> --- a/arch/arm/mach-shmobile/setup-r8a7779.c
> +++ b/arch/arm/mach-shmobile/setup-r8a7779.c
> @@ -34,15 +34,15 @@ static void __init r8a7779_init_irq_dt(void)
>  	irqchip_init();
>  
>  	/* route all interrupts to ARM */
> -	__raw_writel(0xffffffff, base + INT2NTSR0);
> -	__raw_writel(0x3fffffff, base + INT2NTSR1);
> +	writel(0xffffffff, base + INT2NTSR0);
> +	writel(0x3fffffff, base + INT2NTSR1);
>  
>  	/* unmask all known interrupts in INTCS2 */
> -	__raw_writel(0xfffffff0, base + INT2SMSKCR0);
> -	__raw_writel(0xfff7ffff, base + INT2SMSKCR1);
> -	__raw_writel(0xfffbffdf, base + INT2SMSKCR2);
> -	__raw_writel(0xbffffffc, base + INT2SMSKCR3);
> -	__raw_writel(0x003fee3f, base + INT2SMSKCR4);
> +	writel(0xfffffff0, base + INT2SMSKCR0);
> +	writel(0xfff7ffff, base + INT2SMSKCR1);
> +	writel(0xfffbffdf, base + INT2SMSKCR2);
> +	writel(0xbffffffc, base + INT2SMSKCR3);
> +	writel(0x003fee3f, base + INT2SMSKCR4);
>  
>  	iounmap(base);
>  }
> diff --git a/arch/arm/mach-shmobile/smp-r8a7779.c b/arch/arm/mach-shmobile/smp-r8a7779.c
> index 80fef14a47ba6b7e..51163854d9a14d17 100644
> --- a/arch/arm/mach-shmobile/smp-r8a7779.c
> +++ b/arch/arm/mach-shmobile/smp-r8a7779.c
> @@ -45,7 +45,7 @@ static void __init r8a7779_smp_prepare_cpus(unsigned int max_cpus)
>  	void __iomem *base = ioremap(HPBREG_BASE, 0x1000);
>  
>  	/* Map the reset vector (in headsmp-scu.S, headsmp.S) */
> -	__raw_writel(__pa(shmobile_boot_vector), base + AVECR);
> +	writel(__pa(shmobile_boot_vector), base + AVECR);
>  
>  	/* setup r8a7779 specific SCU bits */
>  	shmobile_smp_scu_prepare_cpus(R8A7779_SCU_BASE, max_cpus);
> diff --git a/arch/arm/mach-shmobile/smp-sh73a0.c b/arch/arm/mach-shmobile/smp-sh73a0.c
> index 916f6367bc282eb4..62f4d2501e174a78 100644
> --- a/arch/arm/mach-shmobile/smp-sh73a0.c
> +++ b/arch/arm/mach-shmobile/smp-sh73a0.c
> @@ -39,10 +39,10 @@ static int sh73a0_boot_secondary(unsigned int cpu, struct task_struct *idle)
>  	unsigned int lcpu = cpu_logical_map(cpu);
>  	void __iomem *cpg2 = ioremap(CPG_BASE2, PAGE_SIZE);
>  
> -	if (((__raw_readl(cpg2 + PSTR) >> (4 * lcpu)) & 3) == 3)
> -		__raw_writel(1 << lcpu, cpg2 + WUPCR);	/* wake up */
> +	if (((readl(cpg2 + PSTR) >> (4 * lcpu)) & 3) == 3)
> +		writel(1 << lcpu, cpg2 + WUPCR);	/* wake up */
>  	else
> -		__raw_writel(1 << lcpu, cpg2 + SRESCR);	/* reset */
> +		writel(1 << lcpu, cpg2 + SRESCR);	/* reset */
>  	iounmap(cpg2);
>  	return 0;
>  }
> @@ -53,8 +53,8 @@ static void __init sh73a0_smp_prepare_cpus(unsigned int max_cpus)
>  	void __iomem *sysc = ioremap(SYSC_BASE, PAGE_SIZE);
>  
>  	/* Map the reset vector (in headsmp.S) */
> -	__raw_writel(0, ap + APARMBAREA);      /* 4k */
> -	__raw_writel(__pa(shmobile_boot_vector), sysc + SBAR);
> +	writel(0, ap + APARMBAREA);      /* 4k */
> +	writel(__pa(shmobile_boot_vector), sysc + SBAR);
>  	iounmap(sysc);
>  	iounmap(ap);
>  
> -- 
> 2.25.1
> 

-- 
Regards,
Niklas Söderlund



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux