* [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition
@ 2017-08-17 7:55 Andy Yan
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Andy Yan @ 2017-08-17 7:55 UTC (permalink / raw)
To: u-boot
Macro VA_BITS and PTE_BLOCK_BITS are not used
in the code, so remove them.
Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
---
Changes in v2:
- remove CONFIG_SYS_PTL2_BITS from the whitelist
arch/arm/include/asm/armv8/mmu.h | 9 ---------
scripts/config_whitelist.txt | 2 --
2 files changed, 11 deletions(-)
diff --git a/arch/arm/include/asm/armv8/mmu.h b/arch/arm/include/asm/armv8/mmu.h
index a349903..7752741 100644
--- a/arch/arm/include/asm/armv8/mmu.h
+++ b/arch/arm/include/asm/armv8/mmu.h
@@ -8,15 +8,6 @@
#ifndef _ASM_ARMV8_MMU_H_
#define _ASM_ARMV8_MMU_H_
-/***************************************************************/
-/*
- * The following definitions are related each other, shoud be
- * calculated specifically.
- */
-
-#define VA_BITS CONFIG_SYS_VA_BITS
-#define PTE_BLOCK_BITS CONFIG_SYS_PTL2_BITS
-
/*
* block/section address mask and size definitions.
*/
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index db28457..63b8b25 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -4440,7 +4440,6 @@ CONFIG_SYS_PSDPAR
CONFIG_SYS_PSSR_VAL
CONFIG_SYS_PTCPAR
CONFIG_SYS_PTDPAR
-CONFIG_SYS_PTL2_BITS
CONFIG_SYS_PTV
CONFIG_SYS_PUAPAR
CONFIG_SYS_QE_FMAN_FW_IN_MMC
@@ -4815,7 +4814,6 @@ CONFIG_SYS_USE_SERIALFLASH
CONFIG_SYS_USE_SPIFLASH
CONFIG_SYS_USR_EXCEP
CONFIG_SYS_U_BOOT_OFFS
-CONFIG_SYS_VA_BITS
CONFIG_SYS_VCXK_ACKNOWLEDGE_DDR
CONFIG_SYS_VCXK_ACKNOWLEDGE_PIN
CONFIG_SYS_VCXK_ACKNOWLEDGE_PORT
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread* [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator
2017-08-17 7:55 [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
@ 2017-08-17 7:55 ` Andy Yan
2017-08-18 13:01 ` [U-Boot] [U-Boot,v2,2/2] " Philipp Tomsich
` (2 more replies)
2017-08-31 11:52 ` [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
` (2 subsequent siblings)
3 siblings, 3 replies; 9+ messages in thread
From: Andy Yan @ 2017-08-17 7:55 UTC (permalink / raw)
To: u-boot
Add space around operator "+", make it
match the coding style.
Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
---
Changes in v2: None
arch/arm/include/asm/armv8/mmu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/include/asm/armv8/mmu.h b/arch/arm/include/asm/armv8/mmu.h
index 7752741..6121aab 100644
--- a/arch/arm/include/asm/armv8/mmu.h
+++ b/arch/arm/include/asm/armv8/mmu.h
@@ -16,7 +16,7 @@
#undef PAGE_SIZE
#define PAGE_SHIFT 12
#define PAGE_SIZE (1 << PAGE_SHIFT)
-#define PAGE_MASK (~(PAGE_SIZE-1))
+#define PAGE_MASK (~(PAGE_SIZE - 1))
/***************************************************************/
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread* [U-Boot] [U-Boot,v2,2/2] armv8: mmu: add space around operator
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
@ 2017-08-18 13:01 ` Philipp Tomsich
2017-08-31 11:51 ` Andy Yan
2017-08-31 14:54 ` [U-Boot] [PATCH v2 2/2] " York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot,v2,2/2] " Tom Rini
2 siblings, 1 reply; 9+ messages in thread
From: Philipp Tomsich @ 2017-08-18 13:01 UTC (permalink / raw)
To: u-boot
> Add space around operator "+", make it
> match the coding style.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> ---
>
> Changes in v2: None
>
> arch/arm/include/asm/armv8/mmu.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [U-Boot,v2,2/2] armv8: mmu: add space around operator
2017-08-18 13:01 ` [U-Boot] [U-Boot,v2,2/2] " Philipp Tomsich
@ 2017-08-31 11:51 ` Andy Yan
0 siblings, 0 replies; 9+ messages in thread
From: Andy Yan @ 2017-08-31 11:51 UTC (permalink / raw)
To: u-boot
ping
On 2017年08月18日 21:01, Philipp Tomsich wrote:
>> Add space around operator "+", make it
>> match the coding style.
>>
>> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
>> ---
>>
>> Changes in v2: None
>>
>> arch/arm/include/asm/armv8/mmu.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
2017-08-18 13:01 ` [U-Boot] [U-Boot,v2,2/2] " Philipp Tomsich
@ 2017-08-31 14:54 ` York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot,v2,2/2] " Tom Rini
2 siblings, 0 replies; 9+ messages in thread
From: York Sun @ 2017-08-31 14:54 UTC (permalink / raw)
To: u-boot
On 08/17/2017 12:56 AM, Andy Yan wrote:
> Add space around operator "+", make it
> match the coding style.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> ---
>
> Changes in v2: None
Reviewed-by: York Sun <york.sun@nxp.com>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [U-Boot,v2,2/2] armv8: mmu: add space around operator
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
2017-08-18 13:01 ` [U-Boot] [U-Boot,v2,2/2] " Philipp Tomsich
2017-08-31 14:54 ` [U-Boot] [PATCH v2 2/2] " York Sun
@ 2017-09-13 2:29 ` Tom Rini
2 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2017-09-13 2:29 UTC (permalink / raw)
To: u-boot
On Thu, Aug 17, 2017 at 03:55:50PM +0800, Andy Yan wrote:
> Add space around operator "+", make it
> match the coding style.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Reviewed-by: York Sun <york.sun@nxp.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170912/01f1bb7e/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition
2017-08-17 7:55 [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
@ 2017-08-31 11:52 ` Andy Yan
2017-08-31 14:53 ` York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot, v2, " Tom Rini
3 siblings, 0 replies; 9+ messages in thread
From: Andy Yan @ 2017-08-31 11:52 UTC (permalink / raw)
To: u-boot
ping
On 2017年08月17日 15:55, Andy Yan wrote:
> Macro VA_BITS and PTE_BLOCK_BITS are not used
> in the code, so remove them.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
>
> ---
>
> Changes in v2:
> - remove CONFIG_SYS_PTL2_BITS from the whitelist
>
> arch/arm/include/asm/armv8/mmu.h | 9 ---------
> scripts/config_whitelist.txt | 2 --
> 2 files changed, 11 deletions(-)
>
> diff --git a/arch/arm/include/asm/armv8/mmu.h b/arch/arm/include/asm/armv8/mmu.h
> index a349903..7752741 100644
> --- a/arch/arm/include/asm/armv8/mmu.h
> +++ b/arch/arm/include/asm/armv8/mmu.h
> @@ -8,15 +8,6 @@
> #ifndef _ASM_ARMV8_MMU_H_
> #define _ASM_ARMV8_MMU_H_
>
> -/***************************************************************/
> -/*
> - * The following definitions are related each other, shoud be
> - * calculated specifically.
> - */
> -
> -#define VA_BITS CONFIG_SYS_VA_BITS
> -#define PTE_BLOCK_BITS CONFIG_SYS_PTL2_BITS
> -
> /*
> * block/section address mask and size definitions.
> */
> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
> index db28457..63b8b25 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -4440,7 +4440,6 @@ CONFIG_SYS_PSDPAR
> CONFIG_SYS_PSSR_VAL
> CONFIG_SYS_PTCPAR
> CONFIG_SYS_PTDPAR
> -CONFIG_SYS_PTL2_BITS
> CONFIG_SYS_PTV
> CONFIG_SYS_PUAPAR
> CONFIG_SYS_QE_FMAN_FW_IN_MMC
> @@ -4815,7 +4814,6 @@ CONFIG_SYS_USE_SERIALFLASH
> CONFIG_SYS_USE_SPIFLASH
> CONFIG_SYS_USR_EXCEP
> CONFIG_SYS_U_BOOT_OFFS
> -CONFIG_SYS_VA_BITS
> CONFIG_SYS_VCXK_ACKNOWLEDGE_DDR
> CONFIG_SYS_VCXK_ACKNOWLEDGE_PIN
> CONFIG_SYS_VCXK_ACKNOWLEDGE_PORT
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition
2017-08-17 7:55 [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
2017-08-31 11:52 ` [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
@ 2017-08-31 14:53 ` York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot, v2, " Tom Rini
3 siblings, 0 replies; 9+ messages in thread
From: York Sun @ 2017-08-31 14:53 UTC (permalink / raw)
To: u-boot
On 08/17/2017 12:55 AM, Andy Yan wrote:
> Macro VA_BITS and PTE_BLOCK_BITS are not used
> in the code, so remove them.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
>
> ---
>
> Changes in v2:
> - remove CONFIG_SYS_PTL2_BITS from the whitelist
>
> arch/arm/include/asm/armv8/mmu.h | 9 ---------
> scripts/config_whitelist.txt | 2 --
> 2 files changed, 11 deletions(-)
>
Reviewed-by: York Sun <york.sun@nxp.com>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [U-Boot, v2, 1/2] armv8: mmu: remove unused macro definition
2017-08-17 7:55 [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
` (2 preceding siblings ...)
2017-08-31 14:53 ` York Sun
@ 2017-09-13 2:29 ` Tom Rini
3 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2017-09-13 2:29 UTC (permalink / raw)
To: u-boot
On Thu, Aug 17, 2017 at 03:55:01PM +0800, Andy Yan wrote:
> Macro VA_BITS and PTE_BLOCK_BITS are not used
> in the code, so remove them.
>
> Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Reviewed-by: York Sun <york.sun@nxp.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170912/6a6c6957/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-09-13 2:29 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-17 7:55 [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
2017-08-17 7:55 ` [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator Andy Yan
2017-08-18 13:01 ` [U-Boot] [U-Boot,v2,2/2] " Philipp Tomsich
2017-08-31 11:51 ` Andy Yan
2017-08-31 14:54 ` [U-Boot] [PATCH v2 2/2] " York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot,v2,2/2] " Tom Rini
2017-08-31 11:52 ` [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition Andy Yan
2017-08-31 14:53 ` York Sun
2017-09-13 2:29 ` [U-Boot] [U-Boot, v2, " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox