Re: [PATCH v3 8/8] hw/fsi: Documentation and testing

2023-09-08 Thread Ninad Palsule
Hello Thomas, On 8/30/23 02:05, Thomas Huth wrote: On 30/08/2023 04.26, Ninad Palsule wrote: Added FSI document Added basic qtests for FSI model. Added MAINITAINER for FSI Replaced some qemu logs to traces. Sorry, that's not how we rework patch series in the QEMU development process. Please

[PATCH v4 01/10] hw/fsi: Introduce IBM's Local bus

2023-09-08 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments

[PATCH v4 09/10] hw/fsi: Added FSI documentation

2023-09-08 Thread Ninad Palsule
Documentation for IBM FSI model. Signed-off-by: Ninad Palsule --- v4: - Added separate commit for documentation --- docs/specs/fsi.rst | 141 + 1 file changed, 141 insertions(+) create mode 100644 docs/specs/fsi.rst diff --git a/docs/specs/fsi.rst

[PATCH v4 07/10] hw/arm: Hook up FSI module in AST2600

2023-09-08 Thread Ninad Palsule
es of GPIO IRQs into FSI symbols, and calling the associated symbol method on the slave to map the access onto the CFAM. Testing: Tested by reading cfam config address 0 on rainier machine type. root@p10bmc:~# pdbg -a getcfam 0x0 p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew

[PATCH v4 00/10] Introduce model for IBM's FSI

2023-09-08 Thread Ninad Palsule
Hello, Please review the patch-set version 4. I have incorporated review comments from Cedric and Thomas. Ninad Palsule (10): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce IBM's cfam,fsi-slave hw/fsi: Introduce IBM's FSI hw/fsi: IBM's On-chip

[PATCH v4 08/10] hw/fsi: Added qtest

2023-09-08 Thread Ninad Palsule
Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3: - Added new qtest as per Cedric's comment. V4: - Remove MAINTAINER and documentation changes from this commit --- tests/qtest/fsi-test.c | 210 tests/qtest/meson.build | 2 + 2

[PATCH v4 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-09-08 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated review comment by Joel. --- hw/fsi/Kconfig | 4 + hw

Re: [PATCH v3 8/8] hw/fsi: Documentation and testing

2023-09-08 Thread Ninad Palsule
Hello Cedric, On 8/30/23 02:13, Cédric Le Goater wrote: On 8/30/23 09:05, Thomas Huth wrote: On 30/08/2023 04.26, Ninad Palsule wrote: Added FSI document Added basic qtests for FSI model. Added MAINITAINER for FSI Replaced some qemu logs to traces. Sorry, that's not how we rework patch

[PATCH v4 10/10] hw/fsi: Update MAINTAINER list

2023-09-08 Thread Ninad Palsule
Added maintainer for IBM FSI model Signed-off-by: Ninad Palsule --- V4: - Added separate commit for MAINTAINER change. --- MAINTAINERS | 22 ++ 1 file changed, 22 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6111b6b4d9..285f3a3bc9 100644 --- a/MAINTAINERS

[PATCH v4 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-09-08 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v3: - Incorporated Thomas Huth's review comments. --- hw/fsi/Kconfig | 9 ++ hw/fsi/cfam.c

[PATCH v4 04/10] hw/fsi: Introduce IBM's FSI

2023-09-08 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule Reviewed-by: Joel

[PATCH v4 02/10] hw/fsi: Introduce IBM's scratchpad

2023-09-08 Thread Ninad Palsule
. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. --- hw/fsi/Kconfig | 4 ++ hw/fsi/engine-scratchpad.c | 100 + hw/fsi/meson.build

[PATCH v4 06/10] hw/fsi: Aspeed APB2OPB interface

2023-09-08 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated review comments by Joel v3: - Incorporated review comments by Thomas Huth v4: - Compile FSI with ASPEED_SOC only. --- hw/arm/Kconfig | 1 + hw/fsi/Kconfig

[PATCH v3 6/8] hw/fsi: Aspeed APB2OPB interface

2023-08-29 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated review comments by Joel v3: - Incorporated review comments by Thomas Huth --- hw/arm/Kconfig | 1 + hw/fsi/Kconfig | 20 +- hw/fsi/aspeed-apb2opb.c

[PATCH v3 2/8] hw/fsi: Introduce IBM's scratchpad

2023-08-29 Thread Ninad Palsule
. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. --- hw/fsi/Kconfig | 4 ++ hw/fsi/engine-scratchpad.c | 100 + hw/fsi/meson.build

Re: [PATCH v1 0/7] Introduce model for IBM's FSP

2023-08-29 Thread Ninad Palsule
Hello Cedric, On 8/28/23 03:49, Cédric Le Goater wrote: Hello Ninad, On 8/25/23 22:30, Ninad Palsule wrote: Hello, Please review the patch-set. This is a first step towards introducing model for IBM's Flexible Service Interface. The full functionality will be implemented over the time

[PATCH v3 0/8] Introduce model for IBM's FSI

2023-08-29 Thread Ninad Palsule
Hello, Please review the patch-set version 3. I have incorporated review comments from Cedric and Thomas. Ninad Palsule (8): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce IBM's cfam,fsi-slave hw/fsi: Introduce IBM's FSI hw/fsi: IBM's On-chip

[PATCH v3 5/8] hw/fsi: IBM's On-chip Peripheral Bus

2023-08-29 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated review comment by Joel. --- hw/fsi/Kconfig | 4 + hw/fsi/fsi-master.c | 6

[PATCH v3 7/8] hw/arm: Hook up FSI module in AST2600

2023-08-29 Thread Ninad Palsule
es of GPIO IRQs into FSI symbols, and calling the associated symbol method on the slave to map the access onto the CFAM. Testing: Tested by reading cfam config address 0 on rainier machine type. root@p10bmc:~# pdbg -a getcfam 0x0 p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew

[PATCH v3 8/8] hw/fsi: Documentation and testing

2023-08-29 Thread Ninad Palsule
Added FSI document Added basic qtests for FSI model. Added MAINITAINER for FSI Replaced some qemu logs to traces. Signed-off-by: Ninad Palsule --- v3: - Incorporated Cedric's review comments. --- MAINTAINERS | 20 docs/specs/fsi.rst | 141 +++ hw

[PATCH v3 1/8] hw/fsi: Introduce IBM's Local bus

2023-08-29 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments

[PATCH v3 4/8] hw/fsi: Introduce IBM's FSI

2023-08-29 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2

[PATCH v3 3/8] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-08-29 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v3: - Incorporated Thomas Huth's review comments. --- hw/fsi/Kconfig | 9 ++ hw/fsi/cfam.c

Re: [PATCH v1 3/7] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-08-29 Thread Ninad Palsule
Hello Cedric, On 8/29/23 08:43, Cédric Le Goater wrote: On 8/29/23 15:39, Ninad Palsule wrote: Hello Thomas, On 8/28/23 21:03, Thomas Huth wrote: On 25/08/2023 22.30, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The Common FRU Access

[PATCH v5 00/10] Introduce model for IBM's FSI

2023-10-11 Thread Ninad Palsule
Hello, Please review the patch-set version 5. I have incorporated review comments from Cedric. Ninad Palsule (10): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce IBM's cfam,fsi-slave hw/fsi: Introduce IBM's FSI hw/fsi: IBM's On-chip Peripheral

[PATCH v5 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-11 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated review comments by Joel v3: - Incorporated review comments by Thomas Huth v4: - Compile FSI with ASPEED_SOC only. v5: - Incorporated review comments by Cedric. --- meson.build

[PATCH v5 04/10] hw/fsi: Introduce IBM's FSI

2023-10-11 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule Reviewed

[PATCH v5 08/10] hw/fsi: Added qtest

2023-10-11 Thread Ninad Palsule
Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3: - Added new qtest as per Cedric's comment. V4: - Remove MAINTAINER and documentation changes from this commit --- tests/qtest/fsi-test.c | 210 tests/qtest/meson.build | 2 + 2

[PATCH v5 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-11 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated review comment by Joel. v5: - Incorporated review comments

Re: [PATCH v5 08/10] hw/fsi: Added qtest

2023-10-11 Thread Ninad Palsule
Hello Thomas, Thanks for the review. On 10/11/23 10:35, Thomas Huth wrote: On 11/10/2023 17.13, Ninad Palsule wrote: Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3:   - Added new qtest as per Cedric's comment. V4:   - Remove MAINTAINER and documentation changes from

[PATCH v5 10/10] hw/fsi: Update MAINTAINER list

2023-10-11 Thread Ninad Palsule
Added maintainer for IBM FSI model Signed-off-by: Ninad Palsule --- V4: - Added separate commit for MAINTAINER change. V5: - Use * instead of listing all files in dir --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6111b6b4d9

[PATCH v5 07/10] hw/arm: Hook up FSI module in AST2600

2023-10-11 Thread Ninad Palsule
es of GPIO IRQs into FSI symbols, and calling the associated symbol method on the slave to map the access onto the CFAM. Testing: Tested by reading cfam config address 0 on rainier machine type. root@p10bmc:~# pdbg -a getcfam 0x0 p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew Jeffery Sig

[PATCH v5 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-11 Thread Ninad Palsule
, The LBUS device is parent for the scratchpad. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v5: - Incorporated review comments by Cedric. --- include/hw/fsi/engine-scratchpad.h | 33 ++ hw/fsi/engine

[PATCH v5 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-11 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v3: - Incorporated Thomas Huth's review comments. v5: - Incorporated review comments by Cedric. --- include/hw/fsi/cfam.h

[PATCH v5 09/10] hw/fsi: Added FSI documentation

2023-10-11 Thread Ninad Palsule
Documentation for IBM FSI model. Signed-off-by: Ninad Palsule --- v4: - Added separate commit for documentation --- docs/specs/fsi.rst | 141 + 1 file changed, 141 insertions(+) create mode 100644 docs/specs/fsi.rst diff --git a/docs/specs/fsi.rst

[PATCH v5 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-11 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments

Re: [PATCH v6 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-24 Thread Ninad Palsule
Hello Cedric & Andrew, On 10/24/23 10:21, Cédric Le Goater wrote: On 10/24/23 17:00, Ninad Palsule wrote: Hello Cedric, On 10/24/23 02:46, Cédric Le Goater wrote: On 10/21/23 23:17, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introd

Re: [PATCH v6 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-26 Thread Ninad Palsule
Hello Cedric, On 10/24/23 10:21, Cédric Le Goater wrote: On 10/24/23 17:00, Ninad Palsule wrote: Hello Cedric, On 10/24/23 02:46, Cédric Le Goater wrote: On 10/21/23 23:17, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. An APB-to-OPB

Re: [PATCH v6 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-26 Thread Ninad Palsule
On 10/24/23 02:08, Philippe Mathieu-Daudé wrote: On 23/10/23 19:08, Ninad Palsule wrote: Hello Philippe, On 10/23/23 10:00, Philippe Mathieu-Daudé wrote: On 21/10/23 23:17, Ninad Palsule wrote: This is a part of patchset where scratchpad is introduced. The scratchpad provides a set of non

Re: [PATCH v6 08/10] hw/fsi: Added qtest

2023-10-26 Thread Ninad Palsule
Hello Cedric, On 10/24/23 02:34, Cédric Le Goater wrote: On 10/21/23 23:17, Ninad Palsule wrote: Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3:   - Added new qtest as per Cedric's comment. V4:   - Remove MAINTAINER and documentation changes from this commit v6

Re: [PATCH v6 09/10] hw/fsi: Added FSI documentation

2023-10-26 Thread Ninad Palsule
Hello Cedric, On 10/24/23 02:37, Cédric Le Goater wrote: On 10/21/23 23:17, Ninad Palsule wrote: Documentation for IBM FSI model. Signed-off-by: Ninad Palsule --- v4:    - Added separate commit for documentation ---   docs/specs/fsi.rst | 141

[PATCH v7 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-26 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated review comment by Joel. v5: - Incorporated review comments by Cedric. v6: - Incorporated review

[PATCH v7 09/10] hw/fsi: Added FSI documentation

2023-10-26 Thread Ninad Palsule
Documentation for IBM FSI model. Signed-off-by: Ninad Palsule --- v4: - Added separate commit for documentation v7: - Incorporated review comments by Cedric. --- docs/specs/fsi.rst | 138 +++ docs/specs/index.rst | 1 + 2 files changed, 139

[PATCH v7 00/10] Introduce model for IBM's FSI

2023-10-26 Thread Ninad Palsule
Hello, Please review the patch-set version 7. I have incorporated review comments from Cedric, Philippe and Thomas. Ninad Palsule (10): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce IBM's cfam,fsi-slave hw/fsi: Introduce IBM's FSI hw/fsi: IBM's

[PATCH v7 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-26 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated review comments by Joel v3: - Incorporated review comments by Thomas Huth v4: - Compile FSI with ASPEED_SOC only. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Cedric. v7: - Incorporated

[PATCH v7 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-26 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v5: - Incorporated review

[PATCH v7 10/10] hw/fsi: Update MAINTAINER list

2023-10-26 Thread Ninad Palsule
Added maintainer for IBM FSI model Signed-off-by: Ninad Palsule --- V4: - Added separate commit for MAINTAINER change. V5: - Use * instead of listing all files in dir --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d36aa44661

[PATCH v7 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-26 Thread Ninad Palsule
, The LBUS device is parent for the scratchpad. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Daniel. v7: - Incorporated review comments by Philippe. - Cleaned

[PATCH v7 08/10] hw/fsi: Added qtest

2023-10-26 Thread Ninad Palsule
Added basic qtests for FSI model. Signed-off-by: Ninad Palsule Acked-by: Thomas Huth --- v3: - Added new qtest as per Cedric's comment. V4: - Remove MAINTAINER and documentation changes from this commit v6: - Incorporated review comments by Thomas Huth. v7: - Incorporated review comments

[PATCH v7 07/10] hw/arm: Hook up FSI module in AST2600

2023-10-26 Thread Ninad Palsule
es of GPIO IRQs into FSI symbols, and calling the associated symbol method on the slave to map the access onto the CFAM. Testing: Tested by reading cfam config address 0 on rainier machine type. root@p10bmc:~# pdbg -a getcfam 0x0 p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew Jeffery Sig

[PATCH v7 04/10] hw/fsi: Introduce IBM's FSI

2023-10-26 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated

[PATCH v7 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-26 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v3: - Incorporated Thomas Huth's review comments. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Cedric & Daniel -

Re: [PATCH v5 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-19 Thread Ninad Palsule
Hello Daniel, On 10/19/23 03:14, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:30AM -0500, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS is modelled to maintain the qdev bus hierarchy and to take advantage

Re: [PATCH v4 10/10] hw/fsi: Update MAINTAINER list

2023-10-09 Thread Ninad Palsule
Hello Cedric, On 9/11/23 07:33, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: Added maintainer for IBM FSI model Signed-off-by: Ninad Palsule --- V4:    - Added separate commit for MAINTAINER change. ---   MAINTAINERS | 22 ++   1 file changed, 22

Re: [PATCH v4 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-09 Thread Ninad Palsule
Hello Cederic, On 9/9/23 03:25, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS is modelled to maintain the qdev bus hierarchy and to take advantage of the object model to automatically

Re: [PATCH v4 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-09 Thread Ninad Palsule
Hello Cedric, On 9/9/23 03:37, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS device is embeded inside the scratchpad. The scratchpad embedded Fixed provides a non-functional

Re: [PATCH v4 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-09 Thread Ninad Palsule
Hello Cedric, On 9/11/23 07:19, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS device is embeded inside the scratchpad. The scratchpad provides a non-functional registers. There is a 1-1

Re: [PATCH v4 04/10] hw/fsi: Introduce IBM's FSI

2023-10-10 Thread Ninad Palsule
Hello Cedirc, Thanks for the review. On 9/11/23 07:26, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI bus. CFAM is hanging out of FSI bus. The bus is model such a way

Re: [PATCH v4 07/10] hw/arm: Hook up FSI module in AST2600

2023-10-10 Thread Ninad Palsule
Hello Cedric, Thanks for the review. On 9/11/23 07:43, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This patchset introduces IBM's Flexible Service Interface(FSI). Time for some fun with inter-processor buses. FSI allows a service processor access to the internal buses

Re: [PATCH v4 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-10 Thread Ninad Palsule
Hello Cedric, Thanks for the review. On 9/11/23 07:41, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. An APB-to-OPB bridge enabling access to the OPB from the ARM core in the AST2600. Hardware

Re: [PATCH v4 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-10 Thread Ninad Palsule
Hello Cedric, Thanks for the review. On 9/11/23 07:29, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The On-Chip Peripheral Bus (OPB): A low-speed bus typically found in POWER processors

Re: [PATCH v5 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-21 Thread Ninad Palsule
Hello Daniel, On 10/19/23 03:20, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:31AM -0500, Ninad Palsule wrote: This is a part of patchset where scratchpad is introduced. The scratchpad provides a set of non-functional registers. The firmware is free to use them, hardware does

Re: [PATCH v5 04/10] hw/fsi: Introduce IBM's FSI

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 02:44, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI bus. CFAM is hanging out of FSI bus. The bus is model such a way that it is embedded

Re: [PATCH v5 04/10] hw/fsi: Introduce IBM's FSI

2023-10-21 Thread Ninad Palsule
Hello Daniel, On 10/19/23 03:28, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:33AM -0500, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI bus. CFAM is hanging out of FSI bus. The bus is model

[PATCH v6 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-21 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated review comments by Joel v3: - Incorporated review comments by Thomas Huth v4: - Compile FSI with ASPEED_SOC only. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Cedric. --- include/hw

[PATCH v6 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-21 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated review comment by Joel. v5: - Incorporated review comments by Cedric. v6: - Incorporated review

Re: [PATCH v5 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 03:00, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The Common FRU Access Macro (CFAM), an address space containing various "engines" that drive accesses

Re: [PATCH v5 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 02:26, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The On-Chip Peripheral Bus (OPB): A low-speed bus typically found in POWER processors. This now makes

Re: [PATCH v5 00/10] Introduce model for IBM's FSI

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 03:16, Cédric Le Goater wrote: Hello Ninad, On 10/11/23 17:13, Ninad Palsule wrote: Hello, Please review the patch-set version 5. I have incorporated review comments from Cedric. Ninad Palsule (10):    hw/fsi: Introduce IBM's Local bus    hw/fsi: Introduce IBM's

Re: [PATCH v5 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 03:03, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS is modelled to maintain the qdev bus hierarchy and to take advantage of the object model to automatically

Re: [PATCH v5 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 03:08, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS is modelled to maintain the qdev bus hierarchy and to take advantage of the object model to automatically

Re: [PATCH v5 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-21 Thread Ninad Palsule
Hello Daniel, On 10/19/23 11:09, Daniel P. Berrangé wrote: On Thu, Oct 19, 2023 at 10:34:52AM -0500, Ninad Palsule wrote: Hello Daniel, On 10/19/23 03:14, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:30AM -0500, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible

Re: [PATCH v5 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 12:28, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The Common FRU Access Macro (CFAM), an address space containing various "engines" that drive accesses

Re: [PATCH v5 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-21 Thread Ninad Palsule
Hello Daniel, On 10/19/23 03:26, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:32AM -0500, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The Common FRU Access Macro (CFAM), an address space containing various "en

[PATCH v6 07/10] hw/arm: Hook up FSI module in AST2600

2023-10-21 Thread Ninad Palsule
es of GPIO IRQs into FSI symbols, and calling the associated symbol method on the slave to map the access onto the CFAM. Testing: Tested by reading cfam config address 0 on rainier machine type. root@p10bmc:~# pdbg -a getcfam 0x0 p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew Jeffe

[PATCH v6 00/10] Introduce model for IBM's FSI

2023-10-21 Thread Ninad Palsule
Hello, Please review the patch-set version 6. I have incorporated review comments from Cedric, Daniel and Thomas. Ninad Palsule (10): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce IBM's cfam,fsi-slave hw/fsi: Introduce IBM's FSI hw/fsi: IBM's

[PATCH v6 09/10] hw/fsi: Added FSI documentation

2023-10-21 Thread Ninad Palsule
Documentation for IBM FSI model. Signed-off-by: Ninad Palsule --- v4: - Added separate commit for documentation --- docs/specs/fsi.rst | 141 + 1 file changed, 141 insertions(+) create mode 100644 docs/specs/fsi.rst diff --git a/docs/specs/fsi.rst

[PATCH v6 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-21 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v3: - Incorporated Thomas Huth's review comments. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Cedric & Daniel -

[PATCH v6 08/10] hw/fsi: Added qtest

2023-10-21 Thread Ninad Palsule
Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3: - Added new qtest as per Cedric's comment. V4: - Remove MAINTAINER and documentation changes from this commit v6: - Incorporated review comments by Thomas Huth. --- tests/qtest/fsi-test.c | 207

[PATCH v6 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-21 Thread Ninad Palsule
, The LBUS device is parent for the scratchpad. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v5: - Incorporated review comments by Cedric. v6: - Incorporated review comments by Daniel. --- meson.build| 1 + hw/fsi

[PATCH v6 10/10] hw/fsi: Update MAINTAINER list

2023-10-21 Thread Ninad Palsule
Added maintainer for IBM FSI model Signed-off-by: Ninad Palsule --- V4: - Added separate commit for MAINTAINER change. V5: - Use * instead of listing all files in dir --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7f9912baa0

Re: [PATCH v5 05/10] hw/fsi: IBM's On-chip Peripheral Bus

2023-10-21 Thread Ninad Palsule
Hello Daniel, On 10/19/23 03:30, Daniel P. Berrangé wrote: On Wed, Oct 11, 2023 at 10:13:34AM -0500, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The On-Chip Peripheral Bus (OPB): A low-speed bus typically found in POWER processors

Re: [PATCH v5 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-21 Thread Ninad Palsule
Hello Cedric, On 10/19/23 02:16, Cédric Le Goater wrote: On 10/11/23 17:13, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. An APB-to-OPB bridge enabling access to the OPB from the ARM core in the AST2600. Hardware limitations prevent

[PATCH v6 04/10] hw/fsi: Introduce IBM's FSI

2023-10-21 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule Reviewed-by: Joel Stanley --- v2: - Incorporated

[PATCH v6 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-21 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Ninad Palsule --- v2: - Incorporated Joel's review comments. v5: - Incorporated review

Re: [PATCH v6 08/10] hw/fsi: Added qtest

2023-10-23 Thread Ninad Palsule
Hello Thomas, On 10/23/23 01:51, Thomas Huth wrote: On 21/10/2023 23.17, Ninad Palsule wrote: Added basic qtests for FSI model. Signed-off-by: Ninad Palsule --- v3:   - Added new qtest as per Cedric's comment. V4:   - Remove MAINTAINER and documentation changes from this commit v6

Re: [PATCH v6 02/10] hw/fsi: Introduce IBM's scratchpad

2023-10-23 Thread Ninad Palsule
Hello Philippe, On 10/23/23 10:00, Philippe Mathieu-Daudé wrote: On 21/10/23 23:17, Ninad Palsule wrote: This is a part of patchset where scratchpad is introduced. The scratchpad provides a set of non-functional registers. The firmware is free to use them, hardware does not support any

Re: [PATCH v6 01/10] hw/fsi: Introduce IBM's Local bus

2023-10-23 Thread Ninad Palsule
Hello Philippe, On 10/23/23 09:58, Philippe Mathieu-Daudé wrote: Hi Ninad and Andrew, On 21/10/23 23:17, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The LBUS is modelled to maintain the qdev bus hierarchy and to take advantage

Re: [PATCH v4 03/10] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-10-09 Thread Ninad Palsule
Hello Cedric, Thanks for the review. On 9/11/23 07:19, Cédric Le Goater wrote: On 9/9/23 00:28, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. The Common FRU Access Macro (CFAM), an address space containing various "engines"

Re: [PATCH v6 06/10] hw/fsi: Aspeed APB2OPB interface

2023-10-24 Thread Ninad Palsule
Hello Cedric, On 10/24/23 02:46, Cédric Le Goater wrote: On 10/21/23 23:17, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. An APB-to-OPB bridge enabling access to the OPB from the ARM core in the AST2600. Hardware limitations prevent

[PATCH v1 2/7] hw/fsi: Introduce IBM's scratchpad

2023-08-25 Thread Ninad Palsule
. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/fsi/Kconfig | 4 ++ hw/fsi/engine-scratchpad.c | 100 + hw/fsi/meson.build | 1 + include/hw/fsi/engine-scratchpad.h | 32

[PATCH v1 4/7] hw/fsi: Introduce IBM's FSI

2023-08-25 Thread Ninad Palsule
processor (e.g. BMC) driving CFAM engine accesses into the POWER chip. At the hardware level FSI is a bit-based protocol supporting synchronous and DMA-driven accesses of engines in a CFAM. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/fsi

[PATCH v1 3/7] hw/fsi: Introduce IBM's cfam,fsi-slave

2023-08-25 Thread Ninad Palsule
f the CFAM to which it is attached. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/fsi/Kconfig | 9 ++ hw/fsi/cfam.c | 235 + hw/fsi/fsi-slave.c | 109 +++

[PATCH v1 5/7] hw/fsi: IBM's On-chip Peripheral Bus

2023-08-25 Thread Ninad Palsule
of an MMIO-mapping of the CFAM address straight onto a sub-region of the OPB address space. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/fsi/Kconfig | 4 + hw/fsi/fsi-master.c | 3 +- hw/fsi/meson.build | 1 + hw/fsi/opb.c

[PATCH v1 7/7] hw/arm: Hook up FSI module in AST2600

2023-08-25 Thread Ninad Palsule
lib/phosphor-software-manager/pnor/rw/DEVTREE' p0: 0x0 = 0xc0022d15 Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/arm/aspeed_ast2600.c | 15 +++ include/hw/arm/aspeed_soc.h | 4 2 files changed, 19 insertions(+) d

[PATCH v1 0/7] Introduce model for IBM's FSP

2023-08-25 Thread Ninad Palsule
Hello, Please review the patch-set. This is a first step towards introducing model for IBM's Flexible Service Interface. The full functionality will be implemented over the time. Ninad Palsule (7): hw/fsi: Introduce IBM's Local bus hw/fsi: Introduce IBM's scratchpad hw/fsi: Introduce

[PATCH v1 6/7] hw/fsi: Aspeed APB2OPB interface

2023-08-25 Thread Ninad Palsule
-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/arm/Kconfig | 1 + hw/fsi/Kconfig | 4 + hw/fsi/aspeed-apb2opb.c | 346 hw/fsi/meson.build | 1 + hw/fsi/trace

[PATCH v1 1/7] hw/fsi: Introduce IBM's Local bus

2023-08-25 Thread Ninad Palsule
they are attached to the CFAM's LBUS. Engine implementations should subclass the LBusDevice and set the 'config' member of LBusDeviceClass to match the engine's type. Signed-off-by: Andrew Jeffery Signed-off-by: Cédric Le Goater Signed-off-by: Ninad Palsule --- hw/Kconfig| 1 + hw/fsi

Re: [PATCH v1 6/7] hw/fsi: Aspeed APB2OPB interface

2023-08-28 Thread Ninad Palsule
Hi Joel, On 8/28/23 23:55, Joel Stanley wrote: On Fri, 25 Aug 2023 at 20:31, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. An APB-to-OPB bridge enabling access to the OPB from the ARM core in the AST2600. Hardware limitations prevent

Re: [PATCH v1 4/7] hw/fsi: Introduce IBM's FSI

2023-08-28 Thread Ninad Palsule
Thanks for the review, Joel. On 8/28/23 23:57, Joel Stanley wrote: On Fri, 25 Aug 2023 at 20:44, Ninad Palsule wrote: This is a part of patchset where IBM's Flexible Service Interface is introduced. This commit models the FSI bus. CFAM is hanging out of FSI bus. The bus is model such a way

  1   2   3   4   >