[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] arm: add device tree support
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH] arm: add device tree support |
Date: |
Fri, 27 Jan 2012 22:34:01 +0000 |
User-agent: |
KMail/1.13.7 (Linux/3.1.0-1-amd64; KDE/4.6.5; x86_64; ; ) |
> If compiled with CONFIG_FDT, allow user to specify a device tree file using
> the -dtb argument. If the machine supports it then the dtb will be loaded
> into memory and passed to the kernel on boot.
Adding annother machine feels wrong. Why does the board specific code need to
know about this at all? You already going it via a global variable, so can't
this be entirely contained within arm_boot.c? If the board file is involved,
why is it asking the user?
> + versatile_init(ram_size,
> + boot_device,
> + kernel_filename, kernel_cmdline,
> + initrd_filename, cpu_model, 0xffffffff);
This only works because we're currently too dumb to emulate the differences
between the two board variants.
What we probably want to be doing is shipping/constructing device trees for
the boards we implement, with an option to turn this on/off. Requiring a user
to invent their own seems deeply sub-optimal given we know exactly what
hardware we're emulating. A user that needs to provide their own FDT seems
like a fairly rare corner case.
This gets slightly more interesting when you have custom machine variants
(i.e. once we fix the object model, and have proper dynamic machine
construction). Even then I'd expect the FDT to be derived from/specificed by
the machine description, not a separate option.
Paul
- [Qemu-devel] [PATCH] arm: add device tree support, Grant Likely, 2012/01/27
- Re: [Qemu-devel] [PATCH] arm: add device tree support,
Paul Brook <=
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Grant Likely, 2012/01/28
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Paul Brook, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Grant Likely, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Andreas Färber, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Peter Maydell, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Grant Likely, 2012/01/30
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Peter Maydell, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, John Williams, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Grant Likely, 2012/01/29
- Re: [Qemu-devel] [PATCH] arm: add device tree support, Andreas Färber, 2012/01/30