On Wed, Nov 21, 2018 at 4:38 AM Manivannan Sadhasivam <manivannan.sadhasi...@linaro.org> wrote: > + aliases { > + serial0 = &uart0; > + serial1 = &uart1; > + serial2 = &uart2; > + }; > + > +&uart2 { > + status = "okay"; > + clocks = <&uart2_clk>; > +};
This is clearly mismatched here: you mark only one uart as 'enabled, but list three of them as aliases. Having 'serial0' point to a disabled uart may easily break applications that expect the first one to be the console. Best make that serial0 = &uart2; and drop the other ones if only one of them is exposed on the board. If all three are usable, you should enable them all here, and make sure that the numbering of the aliases matches the labels on the board or its documentation. Arnd