mbox series

[0/2] Consolidate embedded PPC initial mappung functions

Message ID cover.1721131193.git.balaton@eik.bme.hu (mailing list archive)
Headers show
Series Consolidate embedded PPC initial mappung functions | expand

Message

BALATON Zoltan July 16, 2024, 12:07 p.m. UTC 
Embedded PPC has always enabled MMU so it needs initial mappings to
start. This code is duplicated within machines which this small series
aims to consolidate into helper functions to reduce duplicated code
and make the board code simpler.

Regards,
BALATON Zoltan

BALATON Zoltan (2):
  hw/ppc: Consolidate e500 initial mapping creation functions
  hw/ppc: Consolidate ppc440 initial mapping creation functions

 hw/ppc/e500.c          | 41 ++++++++++++++++++--------------------
 hw/ppc/e500.h          |  2 --
 hw/ppc/ppc440_bamboo.c | 28 +++-----------------------
 hw/ppc/ppc_booke.c     | 10 ++++++++++
 hw/ppc/ppce500_spin.c  | 30 +++++++++-------------------
 hw/ppc/sam460ex.c      | 45 ++++++++++--------------------------------
 hw/ppc/virtex_ml507.c  | 28 +++-----------------------
 include/hw/ppc/ppc.h   |  7 +++++++
 8 files changed, 61 insertions(+), 130 deletions(-)

Comments

Edgar E. Iglesias July 16, 2024, 5:07 p.m. UTC | #1 
On Tue, Jul 16, 2024 at 02:07:56PM +0200, BALATON Zoltan wrote:
> Embedded PPC has always enabled MMU so it needs initial mappings to
> start. This code is duplicated within machines which this small series
> aims to consolidate into helper functions to reduce duplicated code
> and make the board code simpler.
> 
> Regards,
> BALATON Zoltan

I tested this with my virtex-ml507 images:

Reviewed-by: Edgar E. Iglesias <edgar.iglesias@amd.com>
Tested-by: Edgar E. Iglesias <edgar.iglesias@amd.com>

Cheers,
Edgar


> 
> BALATON Zoltan (2):
>   hw/ppc: Consolidate e500 initial mapping creation functions
>   hw/ppc: Consolidate ppc440 initial mapping creation functions
> 
>  hw/ppc/e500.c          | 41 ++++++++++++++++++--------------------
>  hw/ppc/e500.h          |  2 --
>  hw/ppc/ppc440_bamboo.c | 28 +++-----------------------
>  hw/ppc/ppc_booke.c     | 10 ++++++++++
>  hw/ppc/ppce500_spin.c  | 30 +++++++++-------------------
>  hw/ppc/sam460ex.c      | 45 ++++++++++--------------------------------
>  hw/ppc/virtex_ml507.c  | 28 +++-----------------------
>  include/hw/ppc/ppc.h   |  7 +++++++
>  8 files changed, 61 insertions(+), 130 deletions(-)
> 
> -- 
> 2.30.9
>
BALATON Zoltan Oct. 8, 2024, 1:32 p.m. UTC | #2 
On Tue, 16 Jul 2024, BALATON Zoltan wrote:
> Embedded PPC has always enabled MMU so it needs initial mappings to
> start. This code is duplicated within machines which this small series
> aims to consolidate into helper functions to reduce duplicated code
> and make the board code simpler.

Ping?

> Regards,
> BALATON Zoltan
>
> BALATON Zoltan (2):
>  hw/ppc: Consolidate e500 initial mapping creation functions
>  hw/ppc: Consolidate ppc440 initial mapping creation functions
>
> hw/ppc/e500.c          | 41 ++++++++++++++++++--------------------
> hw/ppc/e500.h          |  2 --
> hw/ppc/ppc440_bamboo.c | 28 +++-----------------------
> hw/ppc/ppc_booke.c     | 10 ++++++++++
> hw/ppc/ppce500_spin.c  | 30 +++++++++-------------------
> hw/ppc/sam460ex.c      | 45 ++++++++++--------------------------------
> hw/ppc/virtex_ml507.c  | 28 +++-----------------------
> include/hw/ppc/ppc.h   |  7 +++++++
> 8 files changed, 61 insertions(+), 130 deletions(-)
>
>
BALATON Zoltan Nov. 2, 2024, 12:20 p.m. UTC | #3 
On Tue, 8 Oct 2024, BALATON Zoltan wrote:
> On Tue, 16 Jul 2024, BALATON Zoltan wrote:
>> Embedded PPC has always enabled MMU so it needs initial mappings to
>> start. This code is duplicated within machines which this small series
>> aims to consolidate into helper functions to reduce duplicated code
>> and make the board code simpler.
>
> Ping?

Ping^2

>> Regards,
>> BALATON Zoltan
>> 
>> BALATON Zoltan (2):
>>  hw/ppc: Consolidate e500 initial mapping creation functions
>>  hw/ppc: Consolidate ppc440 initial mapping creation functions
>> 
>> hw/ppc/e500.c          | 41 ++++++++++++++++++--------------------
>> hw/ppc/e500.h          |  2 --
>> hw/ppc/ppc440_bamboo.c | 28 +++-----------------------
>> hw/ppc/ppc_booke.c     | 10 ++++++++++
>> hw/ppc/ppce500_spin.c  | 30 +++++++++-------------------
>> hw/ppc/sam460ex.c      | 45 ++++++++++--------------------------------
>> hw/ppc/virtex_ml507.c  | 28 +++-----------------------
>> include/hw/ppc/ppc.h   |  7 +++++++
>> 8 files changed, 61 insertions(+), 130 deletions(-)
>> 
>> 
>
>