[lkml]   [2011]   [May]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v5 4/8] arm/dt: probe for platforms via the device tree
    On Mon, May 2, 2011 at 9:15 AM, Tony Lindgren <> wrote:
    > * Grant Likely <> [110429 00:13]:
    >> +
    >> +/**
    >> + * setup_machine_fdt - Machine setup when an dtb was passed to the kernel
    >> + * @dt_phys: physical address of dt blob
    >> + *
    >> + * If a dtb was passed to the kernel in r2, then use it to choose the
    >> + * correct machine_desc and to setup the system.
    >> + */
    >> +struct machine_desc * __init setup_machine_fdt(unsigned int dt_phys)
    >> +{
    > ...
    >> +
    >> +     /* Retrieve various information from the /chosen node */
    >> +     of_scan_flat_dt(early_init_dt_scan_chosen, boot_command_line);
    > Just to make sure we're considering properly what this change means..
    > Does this patch require using CONFIG_CMDLINE_FORCE for older bootloaders
    > with the device tree append patch?

    With the current patch? Yes, but there is a follow-on patch that was
    posted a while ago[1] that will copy ATAG data into the device tree in
    the bootloader. I've not been pushing to include that change since it
    adds a lot of code to the wrapper and I'm not sure how deeply we want
    to support that use case. Basically I want to have a better idea what
    the supported use cases are before pushing something like that.

    Adding a default command line to the .dts file instead of using
    CONFIG_CMDLINE_FORCE also works.


    > Or should we still somehow get ATAG_CMDLINE let's say in the uncompress
    > code and use that if nothing else is found?
    > Otherwise booting old boards with the same kernel will be painful as
    > it requires a kernel recompile to set the cmdline..
    > Also, not counting the separate DT append issues, I got things booting
    > with these patches (and the dts files) on omaps. So for patches 1 - 5,
    > and patch 8:
    > Tested-by: Tony Lindgren <>
    > I don't have versatile or tegra, so can't test those :)

    Awesome, thanks.



    Grant Likely, B.Sc., P.Eng.
    Secret Lab Technologies Ltd.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2011-05-02 17:37    [W:0.042 / U:3.424 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site