2018-12-27 18:15:50

by Peng Hao

Subject: [PATCH] arm/mm/pmsa-v8 : remove unneeded semicolon

Remove unneeded semicolon.

Signed-off-by: Peng Hao <[email protected]>
---
arch/arm/mm/pmsa-v8.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mm/pmsa-v8.c b/arch/arm/mm/pmsa-v8.c
index 617a83d..0d7d5fb 100644
--- a/arch/arm/mm/pmsa-v8.c
+++ b/arch/arm/mm/pmsa-v8.c
@@ -165,7 +165,7 @@ static int __init pmsav8_setup_ram(unsigned int number, phys_addr_t start,phys_a
return -EINVAL;

bar = start;
- lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);;
+ lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);

bar |= PMSAv8_AP_PL1RW_PL0RW | PMSAv8_RGN_SHARED;
lar |= PMSAv8_LAR_IDX(PMSAv8_RGN_NORMAL) | PMSAv8_LAR_EN;
@@ -181,7 +181,7 @@ static int __init pmsav8_setup_io(unsigned int number, phys_addr_t start,phys_ad
return -EINVAL;

bar = start;
- lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);;
+ lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);

bar |= PMSAv8_AP_PL1RW_PL0RW | PMSAv8_RGN_SHARED | PMSAv8_BAR_XN;
lar |= PMSAv8_LAR_IDX(PMSAv8_RGN_DEVICE_nGnRnE) | PMSAv8_LAR_EN;
--
1.8.3.1

2019-01-02 13:15:06

by Vladimir Murzin

Subject: Re: [PATCH] arm/mm/pmsa-v8 : remove unneeded semicolon

On 12/27/18 6:26 AM, Peng Hao wrote:
> Remove unneeded semicolon.
>
> Signed-off-by: Peng Hao <[email protected]>
> ---
> arch/arm/mm/pmsa-v8.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mm/pmsa-v8.c b/arch/arm/mm/pmsa-v8.c
> index 617a83d..0d7d5fb 100644
> --- a/arch/arm/mm/pmsa-v8.c
> +++ b/arch/arm/mm/pmsa-v8.c
> @@ -165,7 +165,7 @@ static int __init pmsav8_setup_ram(unsigned int number, phys_addr_t start,phys_a
> return -EINVAL;
>
> bar = start;
> - lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);;
> + lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);
>
> bar |= PMSAv8_AP_PL1RW_PL0RW | PMSAv8_RGN_SHARED;
> lar |= PMSAv8_LAR_IDX(PMSAv8_RGN_NORMAL) | PMSAv8_LAR_EN;
> @@ -181,7 +181,7 @@ static int __init pmsav8_setup_io(unsigned int number, phys_addr_t start,phys_ad
> return -EINVAL;
>
> bar = start;
> - lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);;
> + lar = (end - 1) & ~(PMSAv8_MINALIGN - 1);
>
> bar |= PMSAv8_AP_PL1RW_PL0RW | PMSAv8_RGN_SHARED | PMSAv8_BAR_XN;
> lar |= PMSAv8_LAR_IDX(PMSAv8_RGN_DEVICE_nGnRnE) | PMSAv8_LAR_EN;
>

Acked-by: Vladimir Murzin <[email protected]>

You need to drop it into Russell's patch system [1]

[1] http://www.armlinux.org.uk/developer/patches/

Cheers
Vladimir