Re: [PATCH 15/19] ppc/ppc405: QOM'ify PLB

[BALATON Zoltan]

On Wed, 3 Aug 2022, Daniel Henrique Barboza wrote:
> On 8/1/22 10:10, Cédric Le Goater wrote:
> > Signed-off-by: Cédric Le Goater <clg@kaod.org>
> > ---
> >   hw/ppc/ppc405.h    | 14 ++++++++++
> >   hw/ppc/ppc405_uc.c | 67 +++++++++++++++++++++++++++++++++-------------
> >   2 files changed, 62 insertions(+), 19 deletions(-)
> >
> > diff --git a/hw/ppc/ppc405.h b/hw/ppc/ppc405.h
> > index d39d65cc86e4..4ff5cdcf5c65 100644
> > --- a/hw/ppc/ppc405.h
> > +++ b/hw/ppc/ppc405.h
> > @@ -65,6 +65,19 @@ struct ppc4xx_bd_info_t {
> >     typedef struct Ppc405SoCState Ppc405SoCState;
> >   +/* Peripheral local bus arbitrer */
>
> I wasn't aware that arbitrer is a word. Google says that 'arbitrer' is an
> old form of 'arbitrator'.
>
> I looked it up because I thought you misspelled 'arbiter', which is the name
> of a Protoss combat unit in Starcraft. And it happens to be a synonym of
> 'arbitrator' as well.
>
> 'arbitrer' is fine, don't worry about it.

I think that's what the chip docs call it so it comes from there.

Regards,
BALATON Zoltan