| Message ID | 20240126034026.31068-4-ninad@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | Introduce model for IBM's FSI | expand |
On 1/26/24 04:40, Ninad Palsule wrote: > This is a part of patchset where FSI bus is introduced. > > The FSI bus is a simple bus where FSI master is attached. > > Signed-off-by: Andrew Jeffery <andrew@aj.id.au> > [ clg: - removed include/hw/fsi/engine-scratchpad.h and > hw/fsi/engine-scratchpad.c > - dropped FSI_SCRATCHPAD > - included FSIBus definition > - dropped hw/fsi/trace-events changes ] > Signed-off-by: Cédric Le Goater <clg@kaod.org> > Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> Reviewed-by: Cédric Le Goater <clg@kaod.org> Thanks, C. > --- > v11: > - Split the patch. > --- > include/hw/fsi/fsi.h | 19 +++++++++++++++++++ > hw/fsi/fsi.c | 22 ++++++++++++++++++++++ > hw/fsi/meson.build | 2 +- > 3 files changed, 42 insertions(+), 1 deletion(-) > create mode 100644 include/hw/fsi/fsi.h > create mode 100644 hw/fsi/fsi.c > > diff --git a/include/hw/fsi/fsi.h b/include/hw/fsi/fsi.h > new file mode 100644 > index 0000000000..50e8f5c888 > --- /dev/null > +++ b/include/hw/fsi/fsi.h > @@ -0,0 +1,19 @@ > +/* > + * SPDX-License-Identifier: GPL-2.0-or-later > + * Copyright (C) 2024 IBM Corp. > + * > + * IBM Flexible Service Interface > + */ > +#ifndef FSI_FSI_H > +#define FSI_FSI_H > + > +#include "hw/qdev-core.h" > + > +#define TYPE_FSI_BUS "fsi.bus" > +OBJECT_DECLARE_SIMPLE_TYPE(FSIBus, FSI_BUS) > + > +typedef struct FSIBus { > + BusState bus; > +} FSIBus; > + > +#endif /* FSI_FSI_H */ > diff --git a/hw/fsi/fsi.c b/hw/fsi/fsi.c > new file mode 100644 > index 0000000000..60cb03f7a2 > --- /dev/null > +++ b/hw/fsi/fsi.c > @@ -0,0 +1,22 @@ > +/* > + * SPDX-License-Identifier: GPL-2.0-or-later > + * Copyright (C) 2024 IBM Corp. > + * > + * IBM Flexible Service Interface > + */ > +#include "qemu/osdep.h" > + > +#include "hw/fsi/fsi.h" > + > +static const TypeInfo fsi_bus_info = { > + .name = TYPE_FSI_BUS, > + .parent = TYPE_BUS, > + .instance_size = sizeof(FSIBus), > +}; > + > +static void fsi_bus_register_types(void) > +{ > + type_register_static(&fsi_bus_info); > +} > + > +type_init(fsi_bus_register_types); > diff --git a/hw/fsi/meson.build b/hw/fsi/meson.build > index 93ba19dd04..574f5f9289 100644 > --- a/hw/fsi/meson.build > +++ b/hw/fsi/meson.build > @@ -1 +1 @@ > -system_ss.add(when: 'CONFIG_FSI', if_true: files('lbus.c')) > +system_ss.add(when: 'CONFIG_FSI', if_true: files('lbus.c','fsi.c'))
Hello Cedric, >> Signed-off-by: Cédric Le Goater <clg@kaod.org> >> Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> > > > Reviewed-by: Cédric Le Goater <clg@kaod.org> Added tag. Thanks for the review. Regards, Ninad
diff --git a/include/hw/fsi/fsi.h b/include/hw/fsi/fsi.h new file mode 100644 index 0000000000..50e8f5c888 --- /dev/null +++ b/include/hw/fsi/fsi.h @@ -0,0 +1,19 @@ +/* + * SPDX-License-Identifier: GPL-2.0-or-later + * Copyright (C) 2024 IBM Corp. + * + * IBM Flexible Service Interface + */ +#ifndef FSI_FSI_H +#define FSI_FSI_H + +#include "hw/qdev-core.h" + +#define TYPE_FSI_BUS "fsi.bus" +OBJECT_DECLARE_SIMPLE_TYPE(FSIBus, FSI_BUS) + +typedef struct FSIBus { + BusState bus; +} FSIBus; + +#endif /* FSI_FSI_H */ diff --git a/hw/fsi/fsi.c b/hw/fsi/fsi.c new file mode 100644 index 0000000000..60cb03f7a2 --- /dev/null +++ b/hw/fsi/fsi.c @@ -0,0 +1,22 @@ +/* + * SPDX-License-Identifier: GPL-2.0-or-later + * Copyright (C) 2024 IBM Corp. + * + * IBM Flexible Service Interface + */ +#include "qemu/osdep.h" + +#include "hw/fsi/fsi.h" + +static const TypeInfo fsi_bus_info = { + .name = TYPE_FSI_BUS, + .parent = TYPE_BUS, + .instance_size = sizeof(FSIBus), +}; + +static void fsi_bus_register_types(void) +{ + type_register_static(&fsi_bus_info); +} + +type_init(fsi_bus_register_types); diff --git a/hw/fsi/meson.build b/hw/fsi/meson.build index 93ba19dd04..574f5f9289 100644 --- a/hw/fsi/meson.build +++ b/hw/fsi/meson.build @@ -1 +1 @@ -system_ss.add(when: 'CONFIG_FSI', if_true: files('lbus.c')) +system_ss.add(when: 'CONFIG_FSI', if_true: files('lbus.c','fsi.c'))