[v2] target/avr: Increase page size

[PATCH v2 05/11] target/avr: Remove NUMBER_OF_IO_REGISTERS

Posted by Richard Henderson 10 months, 3 weeks ago

This define isn't used.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/avr/cpu.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/target/avr/cpu.h b/target/avr/cpu.h
index 84a8f5cc8c..b49e7a7056 100644
--- a/target/avr/cpu.h
+++ b/target/avr/cpu.h
@@ -44,8 +44,6 @@
 
 /* Number of CPU registers */
 #define NUMBER_OF_CPU_REGISTERS 32
-/* Number of IO registers accessible by ld/st/in/out */
-#define NUMBER_OF_IO_REGISTERS 64
 
 /*
  * Offsets of AVR memory regions in host memory space.
-- 
2.43.0

Re: [PATCH v2 05/11] target/avr: Remove NUMBER_OF_IO_REGISTERS

Posted by Pierrick Bouvier 10 months, 3 weeks ago

On 3/25/25 15:43, Richard Henderson wrote:
> This define isn't used.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/avr/cpu.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/target/avr/cpu.h b/target/avr/cpu.h
> index 84a8f5cc8c..b49e7a7056 100644
> --- a/target/avr/cpu.h
> +++ b/target/avr/cpu.h
> @@ -44,8 +44,6 @@
>   
>   /* Number of CPU registers */
>   #define NUMBER_OF_CPU_REGISTERS 32
> -/* Number of IO registers accessible by ld/st/in/out */
> -#define NUMBER_OF_IO_REGISTERS 64
>   
>   /*
>    * Offsets of AVR memory regions in host memory space.

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Re: [PATCH v2 05/11] target/avr: Remove NUMBER_OF_IO_REGISTERS

Posted by Philippe Mathieu-Daudé 10 months, 3 weeks ago

On 25/3/25 23:43, Richard Henderson wrote:
> This define isn't used.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/avr/cpu.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/target/avr/cpu.h b/target/avr/cpu.h
> index 84a8f5cc8c..b49e7a7056 100644
> --- a/target/avr/cpu.h
> +++ b/target/avr/cpu.h
> @@ -44,8 +44,6 @@
>   
>   /* Number of CPU registers */
>   #define NUMBER_OF_CPU_REGISTERS 32
> -/* Number of IO registers accessible by ld/st/in/out */
> -#define NUMBER_OF_IO_REGISTERS 64

It looks used until patch #7 of this series which remove
the uses.

Re: [PATCH v2 05/11] target/avr: Remove NUMBER_OF_IO_REGISTERS

Posted by Richard Henderson 10 months, 3 weeks ago

On 3/25/25 16:03, Philippe Mathieu-Daudé wrote:
> On 25/3/25 23:43, Richard Henderson wrote:
>> This define isn't used.
>>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>>   target/avr/cpu.h | 2 --
>>   1 file changed, 2 deletions(-)
>>
>> diff --git a/target/avr/cpu.h b/target/avr/cpu.h
>> index 84a8f5cc8c..b49e7a7056 100644
>> --- a/target/avr/cpu.h
>> +++ b/target/avr/cpu.h
>> @@ -44,8 +44,6 @@
>>   /* Number of CPU registers */
>>   #define NUMBER_OF_CPU_REGISTERS 32
>> -/* Number of IO registers accessible by ld/st/in/out */
>> -#define NUMBER_OF_IO_REGISTERS 64
> 
> It looks used until patch #7 of this series which remove
> the uses.

Oops, I must have re-ordered the patches.

r~