Re: [PATCH v8 05/10] hw/fsi: Introduce IBM's FSI master

2024-01-09 Thread Ninad Palsule
Hello Cedric, +static uint64_t fsi_master_read(void *opaque, hwaddr addr, unsigned size) +{ +    FSIMasterState *s = FSI_MASTER(opaque); + +    trace_fsi_master_read(addr, size); + +    if (addr + size > sizeof(s->regs)) { See comment on patch 3 I fixed it. +   

Re: [PATCH v8 05/10] hw/fsi: Introduce IBM's FSI master

2023-12-12 Thread Cédric Le Goater
On 11/29/23 00:56, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI master. CFAM is hanging out of FSI master which is a bus controller. The FSI master: A controller in the platform service processor (e.g. BMC)

[PATCH v8 05/10] hw/fsi: Introduce IBM's FSI master

2023-11-28 Thread Ninad Palsule
This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI master. CFAM is hanging out of FSI master which is a bus controller. The FSI master: A controller in the platform service processor (e.g. BMC) driving CFAM engine accesses into the POWER