Re: [PATCH v2] m25p80: Improve error when the backend file size does not match the device

2022-11-16 Thread Francisco Iglesias
s is confusing for the user. > > Use blk_check_size_and_read_all() instead of blk_pread() to improve > the reported error. > > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesias > --- > hw/block/m25p80.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletio

Re: [PATCH v3 1/8] m25p80: Add basic support for the SFDP command

2022-07-22 Thread Francisco Iglesias
RDSFDP command giving access to > a private SFDP area under the flash. This area now needs to be > populated with the flash device characteristics, using a new > 'sfdp_read' handler under FlashPartInfo. > > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesia

Re: [PATCH 16/21] hw/net/can/versal: Prefer object_initialize_child over object_initialize

2024-02-23 Thread Francisco Iglesias
On 2024-02-16 12:03, Philippe Mathieu-Daudé wrote: When the QOM parent is available, prefer object_initialize_child() over object_initialize(), since it create the parent relationship. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Francisco Iglesias --- hw/net/can/xlnx-versal

Re: [QEMU][PATCH v2] ssi: xilinx_spips: Skip update of cs and fifo releated to spips in gqspi

2019-10-17 Thread Francisco Iglesias
rrects the issue for the zynqmp but not for the other two models (below functions shouldn't be called when writing the mentioned config regs for them either), would it be ok for you to expand to the switch cases you had in v1 (into the switch in this function and return afte

Re: [PATCH v3] ssi: xilinx_spips: Skip spi bus update for few register writes

2019-10-18 Thread Francisco Iglesias
J: > +case R_IOU_TAPDLY_BYPASS: > +case R_DUMMY_CYCLE_EN: > +case R_ECO: Would it be ok for you to move above cases into the switch case above in this same function instead? (And add a reg write before returning) This way all registers are handled at

Re: [QEMU][PATCH v4] ssi: xilinx_spips: Skip spi bus update for a few register writes

2019-10-21 Thread Francisco Iglesias
ize but I forgot to mention that below are not dummy registers (they are configuration registers). Would it be ok for you to remove the last (second) sentence above? The patch looks ok to me after that! (Also tested it!) Best regards, Francisco Iglesias > > Signed-off-by: Sai Pavan Boddu

[PATCH] xilinx_spips: Correct the number of dummy cycles for the FAST_READ_4 cmd

2020-02-18 Thread Francisco Iglesias
From: Francisco Iglesias Correct the number of dummy cycles required by the FAST_READ_4 command (to be eight, one dummy byte). Fixes: ef06ca3946 ("xilinx_spips: Add support for RX discard and RX drain") Suggested-by: Cédric Le Goater Signed-off-by: Francisco Iglesias -

Re: [PATCH 3/3] aspeed/smc: Fix number of dummy cycles for FAST_READ_4 command

2020-02-18 Thread Francisco Iglesias
is needed in the xilinx_spips aswell, I just provided a patch. Thank you for the notification! Best regards, Francisco Iglesias > > Thanks, > > C. > > > > Signed-off-by: Guenter Roeck > > --- > > hw/ssi/aspeed_smc.c | 2 +- > > 1 file changed, 1 insertion

[Qemu-devel] [PATCH] memory: Correct access mask generation in access_with_adjusted_size

2019-08-12 Thread Francisco Iglesias
Also consider the requested transaction size when generating the access mask (so that only the requested bytes are returned when those are less than the memory region's minimum access size). Signed-off-by: Francisco Iglesias --- memory.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[PATCH v1 2/9] hw/arm/xlnx-versal: Connect Versal's PMC SLCR

2021-11-17 Thread Francisco Iglesias
Connect Versal's PMC SLCR (system-level control registers) model. Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal.c | 18 ++ include/hw/arm/xlnx-versal.h | 6 ++ 2 files changed, 24 insertions(+) diff --git a/hw/arm/xlnx-versal.c b/hw/arm/xlnx-ver

[PATCH v1 1/9] hw/misc: Add a model of Versal's PMC SLCR

2021-11-17 Thread Francisco Iglesias
Add a model of Versal's PMC SLCR (system-level control registers). Signed-off-by: Francisco Iglesias Signed-off-by: Edgar E. Iglesias --- hw/misc/meson.build|5 +- hw/misc/xlnx-versal-pmc-iou-slcr.c | 1437 include/hw/misc

[PATCH v1 3/9] include/hw/dma/xlnx_csu_dma: Include ptimer.h and stream.h in the header

2021-11-17 Thread Francisco Iglesias
Include ptimer.h and stream.h in the header for being able to build and reuse the DMA model (the first usage of StreamSink, StreamCanPushNotifyFn and ptimer_state is in the header). Signed-off-by: Francisco Iglesias --- include/hw/dma/xlnx_csu_dma.h | 3 +++ 1 file changed, 3 insertions

[PATCH v1 9/9] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2021-11-17 Thread Francisco Iglesias
Connect Micron Xccela mt35xu01g flashes to the OSPI flash memory controller. Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal-virt.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c index d2f55e29b6

[PATCH v1 4/9] hw/dma: Add the DMA control interface

2021-11-17 Thread Francisco Iglesias
Add an interface for controlling DMA models that are reused with other models. This allows a controlling model to start transfers through the DMA while reusing the DMA's handling of transfer state and completion signaling. Signed-off-by: Francisco Iglesias --- hw/dma/dma-ctrl.c

[PATCH v1 5/9] hw/dma/xlnx_csu_dma: Implement the DMA control interface

2021-11-17 Thread Francisco Iglesias
Implement the DMA control interface for allowing control of DMA operations from inside models that contain instances of (and reuse) the Xilinx CSU DMA. Signed-off-by: Francisco Iglesias --- hw/dma/xlnx_csu_dma.c | 32 include/hw/dma/xlnx_csu_dma.h | 4

[PATCH v1 8/9] hw/block/m25p80: Add support for Micron Xccela flash mt35xu01g

2021-11-17 Thread Francisco Iglesias
Add support for Micron Xccela flash mt35xu01g. Signed-off-by: Francisco Iglesias --- hw/block/m25p80.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index b77503dc84..c6bf3c6bfa 100644 --- a/hw/block/m25p80.c +++ b/hw/block/m25p80.c @@ -255,6 +255,8

[PATCH v1 7/9] hw/arm/xlnx-versal: Connect the OSPI flash memory controller model

2021-11-17 Thread Francisco Iglesias
Connect the OSPI flash memory controller model (including the source and destination DMA). Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal.c | 89 include/hw/arm/xlnx-versal.h | 18 + 2 files changed, 107 insertions(+) diff

[PATCH v1 6/9] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2021-11-17 Thread Francisco Iglesias
Add a model of Xilinx Versal's OSPI flash memory controller. Signed-off-by: Francisco Iglesias --- hw/ssi/meson.build|1 + hw/ssi/xlnx-versal-ospi.c | 1892 + include/hw/ssi/xlnx-versal-ospi.h | 86 ++ 3 files changed,

[PATCH v1 0/9] Xilinx Versal's PMC SLCR and OSPI support

2021-11-17 Thread Francisco Iglesias
a model of Versal's OSPI controller is added and connected to the Versal virt machine. The series then ends with adding initial support for the Micron Xccelera mt35xu01g flash and flashes of this type are connected to the OSPI in the Versal virt machine. Best regards, Francisco Iglesias Franci

Re: [PATCH v1 9/9] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2021-11-23 Thread Francisco Iglesias
Hi Edgar, Thank you for having a look at the series! I made the updates in v2! Best regards, Francisco Iglesias On [2021 Nov 19] Fri 18:16:23, Edgar E. Iglesias wrote: > On Wed, Nov 17, 2021 at 02:18:41PM +0000, Francisco Iglesias wrote: > > Connect Micron Xccela mt35xu01g flashes to

[PATCH v2 05/10] hw/dma/xlnx_csu_dma: Implement the DMA control interface

2021-11-23 Thread Francisco Iglesias
Implement the DMA control interface for allowing control of DMA operations from inside models that contain instances of (and reuse) the Xilinx CSU DMA. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/dma/xlnx_csu_dma.c | 32

[PATCH v2 06/10] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2021-11-23 Thread Francisco Iglesias
Add a model of Xilinx Versal's OSPI flash memory controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/ssi/meson.build|1 + hw/ssi/xlnx-versal-ospi.c | 1892 + include/hw/ssi/xlnx-versal-ospi.h |

[PATCH v2 04/10] hw/dma: Add the DMA control interface

2021-11-23 Thread Francisco Iglesias
Add an interface for controlling DMA models that are reused with other models. This allows a controlling model to start transfers through the DMA while reusing the DMA's handling of transfer state and completion signaling. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --

[PATCH v2 03/10] include/hw/dma/xlnx_csu_dma: Include ptimer.h and stream.h in the header

2021-11-23 Thread Francisco Iglesias
Include ptimer.h and stream.h in the header for being able to build and reuse the DMA model (the first usage of StreamSink, StreamCanPushNotifyFn and ptimer_state is in the header). Signed-off-by: Francisco Iglesias --- include/hw/dma/xlnx_csu_dma.h | 3 +++ 1 file changed, 3 insertions

[PATCH v2 09/10] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2021-11-23 Thread Francisco Iglesias
Connect Micron Xccela mt35xu01g flashes to the OSPI flash memory controller. Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal-virt.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx-versal-virt.c index d2f55e29b6

[PATCH v2 08/10] hw/block/m25p80: Add support for Micron Xccela flash mt35xu01g

2021-11-23 Thread Francisco Iglesias
Add support for Micron Xccela flash mt35xu01g. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/block/m25p80.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index b77503dc84..c6bf3c6bfa 100644 --- a/hw/block/m25p80.c +++ b/hw

[PATCH v2 10/10] MAINTAINERS: Add an entry for Xilinx Versal OSPI

2021-11-23 Thread Francisco Iglesias
List myself as maintainer for the Xilinx Versal OSPI controller. Signed-off-by: Francisco Iglesias --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d3879aa3c1..8c2b01a282 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -963,6 +963,12 @@ F

[PATCH v2 07/10] hw/arm/xlnx-versal: Connect the OSPI flash memory controller model

2021-11-23 Thread Francisco Iglesias
Connect the OSPI flash memory controller model (including the source and destination DMA). Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal.c | 87 include/hw/arm/xlnx-versal.h | 20 ++ 2 files changed, 107 insertions(+) diff

[PATCH v2 00/10] Xilinx Versal's PMC SLCR and OSPI support

2021-11-23 Thread Francisco Iglesias
a model of Versal's OSPI controller is added and connected to the Versal virt machine. The series then ends with adding initial support for the Micron Xccelera mt35xu01g flash and flashes of this type are connected to the OSPI in the Versal virt machine. Best regards, Francisco Iglesias Chan

[PATCH v2 02/10] hw/arm/xlnx-versal: Connect Versal's PMC SLCR

2021-11-23 Thread Francisco Iglesias
Connect Versal's PMC SLCR (system-level control registers) model. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/arm/xlnx-versal.c | 18 ++ include/hw/arm/xlnx-versal.h | 6 ++ 2 files changed, 24 insertions(+) diff --git a/hw/arm

[PATCH v2 01/10] hw/misc: Add a model of Versal's PMC SLCR

2021-11-23 Thread Francisco Iglesias
Add a model of Versal's PMC SLCR (system-level control registers). Signed-off-by: Francisco Iglesias Signed-off-by: Edgar E. Iglesias --- hw/misc/meson.build|5 +- hw/misc/xlnx-versal-pmc-iou-slcr.c | 1445 include/hw/misc

[PATCH v3 01/10] hw/misc: Add a model of Versal's PMC SLCR

2021-11-24 Thread Francisco Iglesias
Add a model of Versal's PMC SLCR (system-level control registers). Signed-off-by: Francisco Iglesias Signed-off-by: Edgar E. Iglesias Acked-by: Edgar E. Iglesias --- hw/misc/meson.build|5 +- hw/misc/xlnx-versal-pmc-iou-slcr.c |

[PATCH v3 04/10] hw/dma: Add the DMA control interface

2021-11-24 Thread Francisco Iglesias
Add an interface for controlling DMA models that are reused with other models. This allows a controlling model to start transfers through the DMA while reusing the DMA's handling of transfer state and completion signaling. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --

[PATCH v3 08/10] hw/block/m25p80: Add support for Micron Xccela flash mt35xu01g

2021-11-24 Thread Francisco Iglesias
Add support for Micron Xccela flash mt35xu01g. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/block/m25p80.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index b77503dc84..c6bf3c6bfa 100644 --- a/hw/block/m25p80.c +++ b/hw

[PATCH v3 05/10] hw/dma/xlnx_csu_dma: Implement the DMA control interface

2021-11-24 Thread Francisco Iglesias
Implement the DMA control interface for allowing control of DMA operations from inside models that contain instances of (and reuse) the Xilinx CSU DMA. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/dma/xlnx_csu_dma.c | 32

[PATCH v3 03/10] include/hw/dma/xlnx_csu_dma: Add in missing includes in the header

2021-11-24 Thread Francisco Iglesias
Add in the missing includes in the header for being able to build the DMA model when reusing it. Signed-off-by: Francisco Iglesias --- include/hw/dma/xlnx_csu_dma.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/hw/dma/xlnx_csu_dma.h b/include/hw/dma/xlnx_csu_dma.h index

[PATCH v3 07/10] hw/arm/xlnx-versal: Connect the OSPI flash memory controller model

2021-11-24 Thread Francisco Iglesias
Connect the OSPI flash memory controller model (including the source and destination DMA). Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/arm/xlnx-versal.c | 87 include/hw/arm/xlnx-versal.h | 20 ++ 2 files

Re: [PATCH v2 03/10] include/hw/dma/xlnx_csu_dma: Include ptimer.h and stream.h in the header

2021-11-24 Thread Francisco Iglesias
Hi Philippe, On [2021 Nov 23] Tue 11:45:45, Philippe Mathieu-Daudé wrote: > On 11/23/21 11:34, Francisco Iglesias wrote: > > Include ptimer.h and stream.h in the header for being able to build and > > reuse the DMA model (the first usage of StreamSink, StreamCanPushNotifyFn >

[PATCH v3 00/10] Xilinx Versal's PMC SLCR and OSPI support

2021-11-24 Thread Francisco Iglesias
a model of Versal's OSPI controller is added and connected to the Versal virt machine. The series then ends with adding initial support for the Micron Xccelera mt35xu01g flash and flashes of this type are connected to the OSPI in the Versal virt machine. Best regards, Francisco Iglesias Chan

[PATCH v3 02/10] hw/arm/xlnx-versal: Connect Versal's PMC SLCR

2021-11-24 Thread Francisco Iglesias
Connect Versal's PMC SLCR (system-level control registers) model. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/arm/xlnx-versal.c | 18 ++ include/hw/arm/xlnx-versal.h | 6 ++ 2 files changed, 24 insertions(+) diff --git a/hw/arm

[PATCH v3 06/10] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2021-11-24 Thread Francisco Iglesias
Add a model of Xilinx Versal's OSPI flash memory controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/ssi/meson.build|1 + hw/ssi/xlnx-versal-ospi.c | 1892 + include/hw/ssi/xlnx-versal-ospi.h |

[PATCH v3 10/10] MAINTAINERS: Add an entry for Xilinx Versal OSPI

2021-11-24 Thread Francisco Iglesias
List myself as maintainer for the Xilinx Versal OSPI controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d3879aa3c1..8c2b01a282 100644 --- a/MAINTAINERS +++ b

[PATCH v3 09/10] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2021-11-24 Thread Francisco Iglesias
Connect Micron Xccela mt35xu01g flashes to the OSPI flash memory controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/arm/xlnx-versal-virt.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/arm/xlnx-versal-virt.c b/hw/arm/xlnx

Re: [PATCH 02/12] aspeed: Introduce a boot_rom region at the machine level

2023-05-08 Thread Francisco Iglesias
On [2023 May 08] Mon 09:58:49, Cédric Le Goater wrote: > This should also avoid Coverity to report a memory leak warning when > the QEMU process exits. See CID 1508061. > > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesias > --- > hw/arm/aspeed.c | 12

Re: [PATCH 01/12] aspeed/hace: Initialize g_autofree pointer

2023-05-08 Thread Francisco Iglesias
essage-Id: <20230421131547.2177449-1-...@kaod.org> > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesias > --- > hw/misc/aspeed_hace.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/misc/aspeed_hace.c b/hw/misc/aspeed_hac

Re: [PATCH] hw/net: Move xilinx_ethlite.c to the target-independent source set

2023-05-08 Thread Francisco Iglesias
Huth Reviewed-by: Francisco Iglesias > --- > hw/net/xilinx_ethlite.c | 2 +- > hw/net/meson.build | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/net/xilinx_ethlite.c b/hw/net/xilinx_ethlite.c > index 99c22819ea..89f4f3b254 100644 > -

Re: [PATCH v4] hw: m25p80: add WP# pin and SRWD bit for write protection

2022-06-22 Thread Francisco Iglesias
On [2022 Jun 21] Tue 13:24:27, Iris Chen wrote: > From: Iris Chen > > Signed-off-by: Iris Chen Reviewed-by: Francisco Iglesias > --- > Fixed .needed for subsection and suggestions from Francisco > > hw/block/m25p80.c | 82 ++--

Re: [PATCH v4] hw: m25p80: add WP# pin and SRWD bit for write protection

2022-06-28 Thread Francisco Iglesias
On [2022 Jun 28] Tue 17:52:50, Cédric Le Goater wrote: > Alistair, Francisco, > > On 6/22/22 11:45, Francisco Iglesias wrote: > > On [2022 Jun 21] Tue 13:24:27, Iris Chen wrote: > > > From: Iris Chen > > > > > > Signed-off-by: Iris Chen > >

Re: [PATCH 1/2] hw: m25p80: Add Block Protect and Top Bottom bits for write protect

2022-07-01 Thread Francisco Iglesias
s->data[0] |= (!!s->top_bottom_bit) << 5; > +} > +if (s->pi->flags & SNOR_F_HAS_SR_BP3_BIT6) { > +s->data[0] |= (!!s->block_protect3) << 6; > +} > > if (get_man(s) == MAN_MACRONIX || get_man(s) == MAN_ISSI) { > s->data[0] |= (!!s->quad_enable) << 6; > @@ -1553,6 +1598,11 @@ static void m25p80_reset(DeviceState *d) > > s->wp_level = true; > s->status_register_write_disabled = false; > +s->block_protect0 = false; > +s->block_protect1 = false; > +s->block_protect2 = false; > +s->block_protect3 = false; > +s->top_bottom_bit = false; We need to place above ones in a subsection in the vmstate (similar to the your previous patch). Looks good to me otherwise! Thanks! Best regards, Francisco Iglesias > > reset_memory(s); > } > -- > 2.30.2 > >

Re: [PATCH 1/2] hw: m25p80: Add Block Protect and Top Bottom bits for write protect

2022-07-01 Thread Francisco Iglesias
On [2022 Jul 01] Fri 14:23:17, Cédric Le Goater wrote: > On 7/1/22 13:40, Francisco Iglesias wrote: > > Hi Iris, > > > > Looks good, a couple of minor comments below! > > > > On [2022 Jun 27] Mon 11:52:33, Iris Chen wrote: > > > Signed-off-by: Iris Chen

Re: [QEMU][PATCH v4 1/4] MAINTAINERS: Include canfd tests under Xilinx CAN

2023-04-25 Thread Francisco Iglesias
On [2023 Apr 24] Mon 23:34:30, Vikram Garhwal wrote: > Signed-off-by: Vikram Garhwal > Reviewed-by: Peter Maydell Reviewed-by: Francisco Iglesias > --- > MAINTAINERS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINE

Re: [QEMU][PATCH v4 3/4] xlnx-versal: Connect Xilinx VERSAL CANFD controllers

2023-04-25 Thread Francisco Iglesias
qemu_fdt_setprop_string(s->fdt, name, "compatible", > +"xlnx,versal-canfd"); And here we can swap above line with (kernel compatible): "xlnx,canfd-2.0"); After changing above the linux

Re: [PATCH] hw/nvram: Avoid unnecessary Xilinx eFuse backstore write

2023-04-26 Thread Francisco Iglesias
igned-off-by: Tong Ho Reviewed-by: Francisco Iglesias > --- > hw/nvram/xlnx-efuse.c | 11 +-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/hw/nvram/xlnx-efuse.c b/hw/nvram/xlnx-efuse.c > index fdfffaab99..655c40b8d1 100644 > --- a/hw/nvram/

Re: [QEMU][PATCH v4 2/4] hw/net/can: Introduce Xilinx Versal CANFD controller

2023-05-02 Thread Francisco Iglesias
Hi Vikram, A few comments below and some suggestions! On [2023 Apr 24] Mon 23:34:31, Vikram Garhwal wrote: > The Xilinx Versal CANFD controller is developed based on SocketCAN, QEMU CAN > bus > implementation. Bus connection and socketCAN connection for each CAN module > can be set through comm

Re: [PATCH 01/12] util/fifo8: Fix typo in fifo8_push_all() description

2023-05-22 Thread Francisco Iglesias
On [2023 May 22] Mon 17:31:33, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Francisco Iglesias > --- > include/qemu/fifo8.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/qemu/fifo8.h b/include/qe

Re: [PATCH 02/12] util/fifo8: Allow fifo8_pop_buf() to not populate popped length

2023-05-22 Thread Francisco Iglesias
ieu-Daudé Reviewed-by: Francisco Iglesias > --- > include/qemu/fifo8.h | 10 +- > util/fifo8.c | 12 > 2 files changed, 13 insertions(+), 9 deletions(-) > > diff --git a/include/qemu/fifo8.h b/include/qemu/fifo8.h > index 16be02f361..d0d02

Re: [PATCH 03/12] util/fifo8: Introduce fifo8_peek_buf()

2023-05-22 Thread Francisco Iglesias
* @numptr: pointer filled with number of bytes returned (can be NULL) > + * > + * Pop a number of elements from the FIFO up to a maximum of max. The buffer s/Pop/Peek into/ > + * containing the popped data is returned. This buffer points directly into s/popped data/data peeked into/ If

Re: [QEMU][PATCH v5 3/4] xlnx-versal: Connect Xilinx VERSAL CANFD controllers

2023-05-22 Thread Francisco Iglesias
On [2023 May 19] Fri 13:36:57, Vikram Garhwal wrote: > Connect CANFD0 and CANFD1 on the Versal-virt machine and update > xlnx-versal-virt > document with CANFD command line examples. > > Signed-off-by: Vikram Garhwal > Reviewed-by: Peter Maydell Reviewed-by: Francisco Iglesi

Re: [QEMU][PATCH v5 2/4] hw/net/can: Introduce Xilinx Versal CANFD controller

2023-05-24 Thread Francisco Iglesias
quot;, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES

Re: [PATCH] hw/arm/xlnx-zynqmp: fix unsigned error when checking the RPUs number

2023-05-24 Thread Francisco Iglesias
he following error afterwards, as Qemu will try to > instantiate some additional RPUs. > | $ qemu-system-aarch64 --smp 1 -M xlnx-zcu102 > | ** > | ERROR:../src/tcg/tcg.c:777:tcg_register_thread: > | assertion failed: (n < tcg_max_ctxs) > > Signed-off-by: Clément Chigot

Re: [PATCH v2 10/10] contrib/gitdm: add group map for AMD

2023-03-13 Thread Francisco Iglesias
On 2023-03-10 19:03, Alex Bennée wrote: AMD recently acquired Xilinx and contributors have been transitioning their emails across. > Signed-off-by: Alex Bennée Reviewed-by: Francisco Iglesias Cc: Vikram Garhwal Cc: Francisco Iglesias Cc: Stefano Stabellini Cc: Sai Pavan Boddu

Re: [PATCH 2/4] aspeed/smc: Dump address offset in trace events

2021-10-05 Thread Francisco Iglesias
On [2021 Oct 04] Mon 17:46:33, Cédric Le Goater wrote: > The register index is currently printed and this is confusing. > > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesias > --- > hw/ssi/aspeed_smc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 dele

Re: [PATCH 1/4] aspeed/wdt: Add trace events

2021-10-05 Thread Francisco Iglesias
On [2021 Oct 04] Mon 17:46:32, Cédric Le Goater wrote: > Signed-off-by: Cédric Le Goater Reviewed-by: Francisco Iglesias > --- > hw/watchdog/wdt_aspeed.c | 5 + > hw/watchdog/trace-events | 4 > 2 files changed, 9 insertions(+) > > diff --git a/hw/watchdog/wdt_a

Re: [PATCH v1 3/6] hw/misc: Add a model of the Xilinx ZynqMP CRF

2022-01-31 Thread Francisco Iglesias
> +.read = register_read_memory, > +.write = register_write_memory, > +.endianness = DEVICE_LITTLE_ENDIAN, > +.valid = { > +.min_access_size = 4, > +.max_access_size = 4, > +}, > +}; > + > +static void crf_init(Object *o

Re: [PATCH v1 4/6] hw/arm/xlnx-zynqmp: Connect the ZynqMP CRF

2022-01-31 Thread Francisco Iglesias
On Mon, Jan 31, 2022 at 12:12:04AM +0100, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Connect the ZynqMP CRF - Clock Reset FPD device. > > Signed-off-by: Edgar E. Iglesias Reviewed-by: Francisco Iglesias > --- > include/hw/arm/xlnx-zynqmp.h

Re: [PATCH v1 5/6] hw/misc: Add a model of the Xilinx ZynqMP APU Control

2022-01-31 Thread Francisco Iglesias
ut is used to connect to PMU GPIs. */ > +qdev_init_gpio_out_named(DEVICE(obj), s->wfi_out, "wfi_out", 4); > +/* CPU_POWER_STATUS is used to connect to INTC redirect. */ > +qdev_init_gpio_out_named(DEVICE(obj), s->cpu_power_status, > + "C

Re: [PATCH v1 1/6] hw/arm/xlnx-zynqmp: Add unimplemented SERDES area

2022-01-31 Thread Francisco Iglesias
On Mon, Jan 31, 2022 at 12:12:01AM +0100, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Add unimplemented SERDES area. > > Signed-off-by: Edgar E. Iglesias Reviewed-by: Francisco Iglesias > --- > include/hw/arm/xlnx-zynqmp.h | 2 +- > hw/arm/x

Re: [PATCH v1 6/6] hw/arm/xlnx-zynqmp: Connect the ZynqMP APU Control

2022-01-31 Thread Francisco Iglesias
On Mon, Jan 31, 2022 at 12:12:06AM +0100, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Connect the ZynqMP APU Control device. > > Signed-off-by: Edgar E. Iglesias Reviewed-by: Francisco Iglesias > --- > include/hw/arm/xlnx-zynqmp.h | 4 +++- >

[PATCH v1] hw/arm/xlnx-zynqmp: 'Or' the QSPI / QSPI DMA IRQs

2022-02-03 Thread Francisco Iglesias
'Or' the IRQs coming from the QSPI and QSPI DMA models. This is done for avoiding the situation where one of the models incorrectly deasserts an interrupt asserted from the other model (which will result in that the IRQ is lost and will not reach guest SW). Signed-off-by: Francisc

Re: [PATCH v5 12/12] docs/devel: Add documentation for the DMA control interface

2022-01-14 Thread Francisco Iglesias
On [2022 Jan 07] Fri 16:07:17, Peter Maydell wrote: > On Tue, 14 Dec 2021 at 11:04, Francisco Iglesias > wrote: > > > > Also, since being the author, list myself as maintainer for the file. > > > > Signed-off-by: Francisco Iglesias > > > > +DmaCtrl

[PATCH v6 01/12] hw/misc: Add a model of Versal's PMC SLCR

2022-01-14 Thread Francisco Iglesias
Add a model of Versal's PMC SLCR (system-level control registers). Signed-off-by: Francisco Iglesias Signed-off-by: Edgar E. Iglesias Reviewed-by: Peter Maydell --- hw/misc/meson.build|5 +- hw/misc/xlnx-versal-pmc-iou-slcr.c |

[PATCH v6 03/12] hw/arm/xlnx-versal: Connect Versal's PMC SLCR

2022-01-14 Thread Francisco Iglesias
Connect Versal's PMC SLCR (system-level control registers) model. Signed-off-by: Francisco Iglesias --- hw/arm/xlnx-versal.c | 71 +++- include/hw/arm/xlnx-versal.h | 5 2 files changed, 75 insertions(+), 1 deletion(-) diff --git a/h

[PATCH v6 02/12] hw/arm/xlnx-versal: 'Or' the interrupts from the BBRAM and RTC models

2022-01-14 Thread Francisco Iglesias
Add an orgate and 'or' the interrupts from the BBRAM and RTC models. Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell --- hw/arm/xlnx-versal-virt.c| 2 +- hw/arm/xlnx-versal.c | 28 ++-- include/hw/arm/xlnx-versal.h | 5 +++-- 3 fil

[PATCH v6 05/12] hw/dma: Add the DMA control interface

2022-01-14 Thread Francisco Iglesias
completion signaling will be read and caught through the DMA engine model's register API and signaling. Signed-off-by: Francisco Iglesias --- hw/dma/dma-ctrl-if.c | 30 +++ hw/dma/meson.build | 1 + include/hw/dma/dma

[PATCH v6 09/12] hw/block/m25p80: Add support for Micron Xccela flash mt35xu01g

2022-01-14 Thread Francisco Iglesias
Add support for Micron Xccela flash mt35xu01g. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/block/m25p80.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index b77503dc84..c6bf3c6bfa 100644 --- a/hw/block/m25p80.c +++ b/hw

[PATCH v6 04/12] include/hw/dma/xlnx_csu_dma: Add in missing includes in the header

2022-01-14 Thread Francisco Iglesias
Add in the missing includes in the header for being able to build the DMA model when reusing it. Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell --- include/hw/dma/xlnx_csu_dma.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/hw/dma/xlnx_csu_dma.h b/include/hw

[PATCH v6 11/12] MAINTAINERS: Add an entry for Xilinx Versal OSPI

2022-01-14 Thread Francisco Iglesias
List myself as maintainer for the Xilinx Versal OSPI controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias Reviewed-by: Peter Maydell --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6ccdec7f02..0e31569d65 100644

[PATCH v6 06/12] hw/dma/xlnx_csu_dma: Implement the DMA control interface

2022-01-14 Thread Francisco Iglesias
Implement the DMA control interface for allowing direct control of DMA operations from inside peripheral models embedding (and reusing) the Xilinx CSU DMA. Signed-off-by: Francisco Iglesias --- hw/dma/xlnx_csu_dma.c | 20 1 file changed, 20 insertions(+) diff --git a/hw

[PATCH v6 07/12] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2022-01-14 Thread Francisco Iglesias
Add a model of Xilinx Versal's OSPI flash memory controller. Signed-off-by: Francisco Iglesias --- hw/ssi/meson.build|1 + hw/ssi/xlnx-versal-ospi.c | 1856 + include/hw/ssi/xlnx-versal-ospi.h | 111 +++ 3 files changed,

[PATCH v6 08/12] hw/arm/xlnx-versal: Connect the OSPI flash memory controller model

2022-01-14 Thread Francisco Iglesias
Connect the OSPI flash memory controller model (including the source and destination DMA). Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell --- hw/arm/xlnx-versal.c | 93 include/hw/arm/xlnx-versal.h | 20 ++ 2 files

[PATCH v6 10/12] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2022-01-14 Thread Francisco Iglesias
Connect Micron Xccela mt35xu01g flashes to the OSPI flash memory controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias Reviewed-by: Peter Maydell --- hw/arm/xlnx-versal-virt.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/arm/xlnx

[PATCH v6 12/12] docs/devel: Add documentation for the DMA control interface

2022-01-14 Thread Francisco Iglesias
Also, since being the author, list myself as maintainer for the file. Signed-off-by: Francisco Iglesias --- MAINTAINERS| 1 + docs/devel/dma-ctrl-if.rst | 243 + docs/devel/index.rst | 1 + 3 files changed, 245 insertions

[PATCH v6 00/12] Xilinx Versal's PMC SLCR and OSPI support

2022-01-14 Thread Francisco Iglesias
a model of Versal's OSPI controller is added and connected to the Versal virt machine. The series then ends with adding initial support for the Micron Xccelera mt35xu01g flash and flashes of this type are connected to the OSPI in the Versal virt machine. Best regards, Francisco Iglesias Chan

Re: [PATCH v6 07/12] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2022-01-21 Thread Francisco Iglesias
Hi Luc, All the suggestions and corrections look good to me so brought them in in v7! Thank you very much reviewing! Best regards, Francisco Iglesias On [2022 Jan 18] Tue 22:46:32, Luc Michel wrote: > Hi Francisco, > > Impressive beast :-) Nicely done. Maybe I would have spli

Re: [PATCH v6 05/12] hw/dma: Add the DMA control interface

2022-01-21 Thread Francisco Iglesias
On [2022 Jan 18] Tue 23:01:42, Luc Michel wrote: > Hi Francisco! > > On 15:28 Fri 14 Jan , Francisco Iglesias wrote: > > An option on real hardware when embedding a DMA engine into a peripheral > > is to make the peripheral control the engine through a custom DMA

[PATCH v7 03/10] hw/arm/xlnx-versal: Connect Versal's PMC SLCR

2022-01-21 Thread Francisco Iglesias
Connect Versal's PMC SLCR (system-level control registers) model. Signed-off-by: Francisco Iglesias Reviewed-by: Luc Michel --- include/hw/arm/xlnx-versal.h | 5 hw/arm/xlnx-versal.c | 71 +++- 2 files changed, 75 insertions(+), 1 del

[PATCH v7 10/10] MAINTAINERS: Add an entry for Xilinx Versal OSPI

2022-01-21 Thread Francisco Iglesias
List myself as maintainer for the Xilinx Versal OSPI controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias Reviewed-by: Peter Maydell --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index e4b3a4bcdf..6797a270e4 100644

[PATCH v7 01/10] hw/misc: Add a model of Versal's PMC SLCR

2022-01-21 Thread Francisco Iglesias
Add a model of Versal's PMC SLCR (system-level control registers). Signed-off-by: Francisco Iglesias Signed-off-by: Edgar E. Iglesias Reviewed-by: Peter Maydell Reviewed-by: Luc Michel --- include/hw/misc/xlnx-versal-pmc-iou-slcr.h | 78 ++ hw/misc/xlnx-versal-pmc-iou-slcr.c |

[PATCH v7 04/10] include/hw/dma/xlnx_csu_dma: Add in missing includes in the header

2022-01-21 Thread Francisco Iglesias
Add in the missing includes in the header for being able to build the DMA model when reusing it. Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell Reviewed-by: Luc Michel --- include/hw/dma/xlnx_csu_dma.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/hw/dma

[PATCH v7 06/10] hw/ssi: Add a model of Xilinx Versal's OSPI flash memory controller

2022-01-21 Thread Francisco Iglesias
Add a model of Xilinx Versal's OSPI flash memory controller. Signed-off-by: Francisco Iglesias --- include/hw/ssi/xlnx-versal-ospi.h | 111 +++ hw/ssi/xlnx-versal-ospi.c | 1853 + hw/ssi/meson.build|1 + 3 files changed,

[PATCH v7 08/10] hw/block/m25p80: Add support for Micron Xccela flash mt35xu01g

2022-01-21 Thread Francisco Iglesias
Add support for Micron Xccela flash mt35xu01g. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias --- hw/block/m25p80.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index b77503dc84..c6bf3c6bfa 100644 --- a/hw/block/m25p80.c +++ b/hw

[PATCH v7 05/10] hw/dma/xlnx_csu_dma: Support starting a read transfer through a class method

2022-01-21 Thread Francisco Iglesias
r API (and signals). This patch adds a class 'read' method for allowing to start read transfers from peripherals embedding and controlling the Xilinx CSU DMA engine as in above scenario. Signed-off-by: Francisco Iglesias --- include/hw/dma/xlnx_csu_dma.h | 19 +-- h

[PATCH v7 07/10] hw/arm/xlnx-versal: Connect the OSPI flash memory controller model

2022-01-21 Thread Francisco Iglesias
Connect the OSPI flash memory controller model (including the source and destination DMA). Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell --- include/hw/arm/xlnx-versal.h | 20 ++ hw/arm/xlnx-versal.c | 93 2 files

[PATCH v7 09/10] hw/arm/xlnx-versal-virt: Connect mt35xu01g flashes to the OSPI

2022-01-21 Thread Francisco Iglesias
Connect Micron Xccela mt35xu01g flashes to the OSPI flash memory controller. Signed-off-by: Francisco Iglesias Reviewed-by: Edgar E. Iglesias Reviewed-by: Peter Maydell --- hw/arm/xlnx-versal-virt.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/arm/xlnx

[PATCH v7 00/10] Xilinx Versal's PMC SLCR and OSPI support

2022-01-21 Thread Francisco Iglesias
a model of Versal's OSPI controller is added and connected to the Versal virt machine. The series then ends with adding initial support for the Micron Xccelera mt35xu01g flash and flashes of this type are connected to the OSPI in the Versal virt machine. Best regards, Francisco Iglesias Chan

[PATCH v7 02/10] hw/arm/xlnx-versal: 'Or' the interrupts from the BBRAM and RTC models

2022-01-21 Thread Francisco Iglesias
Add an orgate and 'or' the interrupts from the BBRAM and RTC models. Signed-off-by: Francisco Iglesias Reviewed-by: Peter Maydell Reviewed-by: Luc Michel --- include/hw/arm/xlnx-versal.h | 5 +++-- hw/arm/xlnx-versal-virt.c| 2 +- hw/arm/xlnx-versal.c

Re: [PATCH v7 00/10] Xilinx Versal's PMC SLCR and OSPI support

2022-01-27 Thread Francisco Iglesias
On Thu, Jan 27, 2022 at 05:27:55PM +, Peter Maydell wrote: > On Fri, 21 Jan 2022 at 16:11, Francisco Iglesias > wrote: > > > > Hi, > > > > This series attempts to add support for Xilinx Versal's PMC SLCR > > (system-level control registers) and O

Re: [PATCH v3] hw: m25p80: allow write_enable latch get/set

2022-05-13 Thread Francisco Iglesias
; +++ b/hw/block/m25p80.c > @@ -1558,6 +1558,7 @@ static int m25p80_pre_save(void *opaque) > > static Property m25p80_properties[] = { > /* This is default value for Micron flash */ > +DEFINE_PROP_BOOL("write-enable", Flash, write_enable, false),

Re: [PATCH] ptimer: Rename PTIMER_POLICY_DEFAULT to PTIMER_POLICY_LEGACY

2022-05-16 Thread Francisco Iglesias
te(s, OFFSET_TCNT >> 2, s->tcnt); > diff --git a/hw/timer/slavio_timer.c b/hw/timer/slavio_timer.c > index 90fdce4c442..8c4f6eb06b6 100644 > --- a/hw/timer/slavio_timer.c > +++ b/hw/timer/slavio_timer.c > @@ -405,7 +405,7 @@ static void slavio_timer_init(Object *obj) > tc->

Re: [PATCH v2 5/5] speed/sdhci: Add trace events

2021-10-20 Thread Francisco Iglesias
Hi Cedric, On the subject s/speed/aspeed/. Otherwise: Reviewed-by: Francisco Iglesias /BR On [2021 Oct 18] Mon 15:26:09, Cédric Le Goater wrote: > Signed-off-by: Cédric Le Goater > --- > hw/sd/aspeed_sdhci.c | 5 + > hw/sd/trace-events | 4 > 2 files change

  1   2   3   4   5   6   >