On 7/15/2015 2:21 AM, Mark Rutland wrote: > Hi, > >> + chosen { >> + stdout-path = "serial0:115200n8"; >> + bootargs = "earlycon=uart8250,mmio32,0x66130000"; >> + }; > > You shouldn't need those earlycon arguments if your driver has > of_earlycon support. If that's currently missing, could you add that > instead? > > Thanks, > Mark. > I can look into this and add the of_earlycon support for 8250_dw. It looks like it may be only a matter of hooking it up to the setup function in 8250_early.c, which is quite straightforward. So does it make more sense for me to do the following? 1) Drop the earlycon bootargs from ns2-svk.dts and continue the Broadcom iProc arm64 upstreaming process with the current patch series 2) Start a separate, new patch series with OF_EARLYCON support to 8250 DW driver Thanks, Ray -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html