Re: [RFC PATCH v2 08/20] powerpc/8xx: Simplify struct mmu_psize_def

2024-05-24 Thread Oscar Salvador
On Fri, May 17, 2024 at 09:00:02PM +0200, Christophe Leroy wrote:
> On 8xx, only the shift field is used in struct mmu_psize_def
> 
> Remove other fields and related macros.
> 
> Signed-off-by: Christophe Leroy 

Reviewed-by: Oscar Salvador 


-- 
Oscar Salvador
SUSE Labs


[RFC PATCH v2 08/20] powerpc/8xx: Simplify struct mmu_psize_def

2024-05-17 Thread Christophe Leroy
On 8xx, only the shift field is used in struct mmu_psize_def

Remove other fields and related macros.

Signed-off-by: Christophe Leroy 
---
 arch/powerpc/include/asm/nohash/32/mmu-8xx.h | 9 ++---
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/arch/powerpc/include/asm/nohash/32/mmu-8xx.h 
b/arch/powerpc/include/asm/nohash/32/mmu-8xx.h
index 141d82e249a8..a756a1e59c54 100644
--- a/arch/powerpc/include/asm/nohash/32/mmu-8xx.h
+++ b/arch/powerpc/include/asm/nohash/32/mmu-8xx.h
@@ -189,19 +189,14 @@ typedef struct {
 
 #define PHYS_IMMR_BASE (mfspr(SPRN_IMMR) & 0xfff8)
 
-/* Page size definitions, common between 32 and 64-bit
+/*
+ * Page size definitions for 8xx
  *
  *shift : is the "PAGE_SHIFT" value for that page size
- *penc  : is the pte encoding mask
  *
  */
 struct mmu_psize_def {
unsigned intshift;  /* number of bits */
-   unsigned intenc;/* PTE encoding */
-   unsigned intind;/* Corresponding indirect page size shift */
-   unsigned intflags;
-#define MMU_PAGE_SIZE_DIRECT   0x1 /* Supported as a direct size */
-#define MMU_PAGE_SIZE_INDIRECT 0x2 /* Supported as an indirect size */
 };
 
 extern struct mmu_psize_def mmu_psize_defs[MMU_PAGE_COUNT];
-- 
2.44.0