On Mon, 7 Oct 2024 at 12:25, Chao Liu <chao....@yeah.net> wrote:
>
> Hi, maintainer,
>
> Following the reference from the chip manualug585-Zynq-7000-TRM manual
> B.3 (Module Summary), placeholders have been added for all unimplemented
> devices, including the AXI and AMBA bus controllers that interact with
> the FPGA.
>
> We can check against the manual by printing the address space of the
> zynq board with the following qemu command:
> ${QEMU_PATH}/qemu-system-aarch64 \
> -M xilinx-zynq-a9 \
> -display none \
> -monitor stdio -s
> (qemu) info mtree -f
>
> The testing methodology previously discussed in earlier email exchanges
> will not be repeated here.
>
> Chao Liu (3):
> xilink_zynq: Add various missing unimplemented devices
> xilink-zynq-devcfg: Fix up for memory address range size not set
> correctly
> xilink-zynq-devcfg: Avoid disabling devcfg memory region during
> initialization
I've left comments for patches 2 and 3. I have taken patch 1
into target-arm.next, with the ignore_memory_transaction_failures
line reinstated.
I'm all in favour of our being able to get rid of that
legacy flag setting for this board, but as I've said on
previous versions of this patchset, we need to have
confidence that it's not going to break existing guest
code, which means the patch removing it needs to come
with a description of the testing that's been done
(which should be more than "Linux still boots").
thanks
-- PMM
