From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 29 May 2017 07:57:00 -0400 Subject: [U-Boot] [PATCHv4 3/3] ARM64: poplar: hi3798cv200: u-boot support for Poplar 96Boards In-Reply-To: <20507012-1989-607a-dcf1-ddfa5837f033@linaro.org> References: <21be0cad-5ac4-63c2-851a-fd28c4dc2a35@linaro.org> <20170525211205.GF10782@bill-the-cat> <20170525220853.GG10782@bill-the-cat> <29be56cb-767b-2992-1d4a-9149ddac5092@linaro.org> <20170526124630.GM10782@bill-the-cat> <20170526160937.GP10782@bill-the-cat> <20507012-1989-607a-dcf1-ddfa5837f033@linaro.org> Message-ID: <20170529115700.GJ10782@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, May 29, 2017 at 11:00:32AM +0200, Jorge Ramirez wrote: > On 05/26/2017 06:09 PM, Tom Rini wrote: > >>>So to sum up: > >>> > >>>In order to get the platform data for pl01x I have to either disable > >>>OF (so I lose the USB node as I said earlier) or*completely* remove > >>>the UART0 node from from the kernel dts. > >>>I personally would rather not modify the kernel's DTS trees that I > >>>am importing into uboot but I am really confused about the policy > >>>now. > >>> > >>>please could you clarify? > >>> > >>>I still think what I proposed when we started is the better way to > >>>go: a uboot specific hi3798cv200-u-boot.dtsifile that contains the > >>>two nodes that are giving trouble. > >>I don't understand what we're not understanding, yes, you should be > >>using a -u-boot.dtsi file to mark uart0 as disabled and not have to > >>modify the kernel dts file at all. > >> > >> > >> > >>This the bit that is NOT possible. Doing that is not enough. > >To be clear, are you trying this on current mainline? Simon reminded me > >that if you don't have 7452946e7f37 in your tree, the -u-boot.dtsi file > >cannot disable a node. > > yes I have that commit (thanks Tom for checking this) > > The issue is actually with serial-uclass.c when the kernel dts > contains a chosen node that contains the stdout-path. > chosen { > stdout-path = "serial0:115200n8"; > }; > > Disabling uart0 (ie serial0) in u-boot.dtsi loses the console > instead of probing the pl01x for the platform_data. > > is there a pre-defined way to work around this? Provide a new stdout-path in the -u-boot.dtsi too? Any changes you could make to the main dts file so that this would work should be able to be done in the -u-boot.dtsi too. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: