On Fri May 3, 2024 at 3:44 PM AEST, Cédric Le Goater wrote: > On 5/3/24 06:51, Nicholas Piggin wrote: > > On Thu May 2, 2024 at 6:47 PM AEST, Cédric Le Goater wrote: > >> On 5/1/24 14:39, Nicholas Piggin wrote: > >>> On Wed Apr 17, 2024 at 9:25 PM AEST, Cédric Le Goater wrote: > >>>> Hello Nick, > >>>> > >>>> On 4/17/24 13:02, Nicholas Piggin wrote: > >>>>> This implements a framework for an ADU unit model. > >>>>> > >>>>> The ADU unit actually implements XSCOM, which is the bridge between MMIO > >>>>> and PIB. However it also includes control and status registers and other > >>>>> functions that are exposed as PIB (xscom) registers. > >>>>> > >>>>> To keep things simple, pnv_xscom.c remains the XSCOM bridge > >>>>> implementation, and pnv_adu.c implements the ADU registers and other > >>>>> functions. > >>>>> > >>>>> So far, just the ADU no-op registers in the pnv_xscom.c default handler > >>>>> are moved over to the adu model. > >>>>> > >>>>> Signed-off-by: Nicholas Piggin <npig...@gmail.com> > >>>>> --- > >>>>> include/hw/ppc/pnv_adu.h | 34 ++++++++++++ > >>>>> include/hw/ppc/pnv_chip.h | 3 + > >>>>> include/hw/ppc/pnv_xscom.h | 6 ++ > >>>>> hw/ppc/pnv.c | 16 ++++++ > >>>>> hw/ppc/pnv_adu.c | 111 > >>>>> +++++++++++++++++++++++++++++++++++++ > >>>>> hw/ppc/pnv_xscom.c | 9 --- > >>>>> hw/ppc/meson.build | 1 + > >>>>> hw/ppc/trace-events | 4 ++ > >>>>> 8 files changed, 175 insertions(+), 9 deletions(-) > >>>>> create mode 100644 include/hw/ppc/pnv_adu.h > >>>>> create mode 100644 hw/ppc/pnv_adu.c > >>>>> > >>>>> diff --git a/include/hw/ppc/pnv_adu.h b/include/hw/ppc/pnv_adu.h > >>>>> new file mode 100644 > >>>>> index 0000000000..9dc91857a9 > >>>>> --- /dev/null > >>>>> +++ b/include/hw/ppc/pnv_adu.h > >>>>> @@ -0,0 +1,34 @@ > >>>>> +/* > >>>>> + * QEMU PowerPC PowerNV Emulation of some ADU behaviour > >>>>> + * > >>>>> + * Copyright (c) 2024, IBM Corporation. > >>>>> + * > >>>>> + * SPDX-License-Identifier: LGPL-2.1-or-later > >>>> > >>>> > >>>> Did you mean GPL-2.0-or-later ? > >>> > >>> Hey Cedric, > >>> > >>> Thanks for reviewing, I've been away so sorry for the late reply. > >>> > >>> It just came from one of the headers I copied which was LGPL. But > >>> there's really nothing much in it and could find a GPL header to > >>> copy. Is GPL-2.0-or-later preferred? > >> > >> I would since all pnv models are GPL. > > > > Some of pnv is actually LGPL. > > I was grepping for 'LGPL' and not 'Lesser' ... Indeed you are right. > Most files miss an SPDX-License-Identifier tag also. > > > That's okay I'll change to GPL. > > LGPL is more relaxed if the code needs to be used in libraries, but > I am not sure it applies to the PNV models. What would you prefer ?
GPL seems to be more common and I don't see a need for LGPL here, so maybe GPL? We could probably switch all LGPL pnv over to GPL if we wanted to. I think LGPL permits such relicensing. Will leave this discussion for another time though. Thanks, Nick