On Tue, Nov 21, 2017 at 7:02 AM, Nathan Rossi <[email protected]> wrote:
> On 21 November 2017 at 07:33, Alistair Francis
> <[email protected]> wrote:
>> Add support for both mainline and Xilinx's QEMU.
>>
>> Signed-off-by: Alistair Francis <[email protected]>
>> ---
>>  conf/machine/zc702-zynq7.conf | 14 ++++++++++++++
>>  1 file changed, 14 insertions(+)
>>
>> diff --git a/conf/machine/zc702-zynq7.conf b/conf/machine/zc702-zynq7.conf
>> index ef42301..d5f641f 100644
>> --- a/conf/machine/zc702-zynq7.conf
>> +++ b/conf/machine/zc702-zynq7.conf
>> @@ -12,6 +12,7 @@
>>  require conf/machine/include/tune-zynq.inc
>>  require conf/machine/include/machine-xilinx-default.inc
>>  require conf/machine/include/machine-xilinx-board.inc
>> +require conf/machine/include/machine-xilinx-qemu.inc
>>
>>  MACHINE_FEATURES = "rtc ext2 ext3 vfat usbhost"
>>
>> @@ -28,3 +29,16 @@ KERNEL_FEATURES += "features/rtc/rtc.scc"
>>
>>  IMAGE_BOOT_FILES += "boot.bin uEnv.txt ${KERNEL_IMAGETYPE}-zynq-zc702.dtb"
>>
>> +# Althogh not fully supported you can run this machin on the mainline QEMU 
>> 'xilinx-zynq-a9' machine
>
> Couple of spelling mistakes here, they got copied into the zc706 patch too.

Fixed in both.

>
>> +IMAGE_CLASSES += "qemuboot"
>> +QB_MEM = "-m 1024"
>> +QB_MACHINE = "-machine xilinx-zynq-a9"
>> +QB_OPT_APPEND = "-nographic -serial null -serial mon:stdio"
>> +QB_NETWORK_DEVICE = "-net nic,netdev=net0,macaddr=@MAC@"
>> +
>> +# Xilinx's fork of QEMU has much better results, so let's default to that
>> +# Use qemu-xilinx instead of mainline
>> +PREFERRED_PROVIDER_qemu-helper-native = "qemu-xilinx-helper-native"
>> +
>> +IMAGE_CLASSES += "qemuboot-xilinx"
>> +QB_MACHINE_qemuboot-xilinx = "-machine arm-generic-fdt-7series"
>
> Setting QB_MACHINE is not needed as qemuboot-xilinx.bbclass sets this
> via a "_arm" override.

Fixed in both.

>
> Also so users can observe the support easily could you update the
> README for these machines, but use "(with QEMU support)" wording
> instead of just "(QEMU support)" (the previous wording is a bit
> ambiguous).

Done for both!

Alistair

>
> Regards,
> Nathan
>
>> --
>> 2.14.1
>>
>> --
>> _______________________________________________
>> meta-xilinx mailing list
>> [email protected]
>> https://lists.yoctoproject.org/listinfo/meta-xilinx
> --
> _______________________________________________
> meta-xilinx mailing list
> [email protected]
> https://lists.yoctoproject.org/listinfo/meta-xilinx
-- 
_______________________________________________
meta-xilinx mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-xilinx

Reply via email to