On Mon, 26 Feb 2024 at 00:05, Sergey Kambalin <serg.o...@gmail.com> wrote: > > Signed-off-by: Sergey Kambalin <sergey.kamba...@auriga.com> > --- > tests/avocado/boot_linux_console.py | 92 +++++++++++++++++++++++++++++ > 1 file changed, 92 insertions(+)
I think it would be good to get the base rpi4b support upstream now (all those parts are reviewed), and then land the ethernet and PCI support afterwards, rather than holding up most of the patchset while we work on the PCI/ethernet patches. That will mean we can definitely get something at least into QEMU 9.0 (softfreeze is 12 March), will reduce the size of the patchset a lot, and get the refactoring changes in the early patches upstream, reducing the potential for conflicts on rebase for you. To that end, my proposal is to take from this series patches 1-12, 18, and 30-41, with the following minor changes to this patch which are necessary to get it to boot on the "no PCI" version of the machine: --- a/tests/avocado/boot_linux_console.py +++ b/tests/avocado/boot_linux_console.py @@ -530,13 +530,17 @@ def test_arm_raspi4(self): 'dwc_otg.fiq_fsm_enable=0') self.vm.add_args('-kernel', kernel_path, '-dtb', dtb_path, - '-append', kernel_command_line, - '-device', 'qemu-xhci,bus=pcie.1,id=xhci', - '-device', 'usb-kbd,bus=xhci.0') + '-append', kernel_command_line) + # When PCI is supported we can add a USB controller: + # '-device', 'qemu-xhci,bus=pcie.1,id=xhci', + # '-device', 'usb-kbd,bus=xhci.0', self.vm.launch() console_pattern = 'Kernel command line: %s' % kernel_command_line self.wait_for_console_pattern(console_pattern) - console_pattern = 'Product: QEMU USB Keyboard' + # When USB is enabled we can look for this + # console_pattern = 'Product: QEMU USB Keyboard' + # self.wait_for_console_pattern(console_pattern) + console_pattern = 'Waiting for root device' self.wait_for_console_pattern(console_pattern) @@ -578,9 +582,10 @@ def test_arm_raspi4_initrd(self): '-dtb', dtb_path, '-initrd', initrd_path, '-append', kernel_command_line, - '-device', 'qemu-xhci,bus=pcie.1,id=xhci', - '-device', 'usb-kbd,bus=xhci.0', '-no-reboot') + # When PCI is supported we can add a USB controller: + # '-device', 'qemu-xhci,bus=pcie.1,id=xhci', + # '-device', 'usb-kbd,bus=xhci.0', self.vm.launch() self.wait_for_console_pattern('Boot successful.') I'll also tweak patch 41 to not list PCI and ethernet as supported yet. We can then undo these changes when we land PCI and ethernet. thanks -- PMM