From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 25 May 2017 16:31:00 -0400 Subject: [U-Boot] [PATCHv4 3/3] ARM64: poplar: hi3798cv200: u-boot support for Poplar 96Boards In-Reply-To: <7fd042e6-fb1f-a4ee-0a54-c39a0e25ef40@linaro.org> References: <20170510144927.GF12511@bill-the-cat> <20170510174555.GH12511@bill-the-cat> <20170511123547.GC5701@bill-the-cat> <20170517220623.GL4631@bill-the-cat> <7fd042e6-fb1f-a4ee-0a54-c39a0e25ef40@linaro.org> Message-ID: <20170525203100.GE10782@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 Thu, May 25, 2017 at 08:38:47PM +0200, Jorge Ramirez wrote: > On 05/18/2017 12:06 AM, Tom Rini wrote: > >>>>having platform data. > >>>No, I think we're going for overkill here by not doing serial_pl01x.c as > >>>platform data. ns16550 does platform data for this already. This > >>>sounds like the lowest overhead way to get the clock populated and not > >>>have some DT data that's not going to be accepted upstream. > >>> > >>ummmm I am a bit lost at this point, could we recap please? > >Lets update the recap: > >- Please re-submit the dts file, now with whatever form is in v4.12-rc1, > > saying as such in the commit (if it's just the commit message that > > changes, that's fine and great). > > The DTS file in v4.12-rc2 still does NOT contain the usb node. > > ==> Should I then not use the DM on USB so I can avoid DTS changes? Well, you can either put it in the -u-boot.dtsi file for the board, and remove that later once it's upstream. > >- Please update serial_pl01x.c to be able to get the clock via platform > > data, update and test your board to confirm it works. > > um, It gets tricky; > I can not use platform_data since I can not use SERIAL_DM because > the device tree does have a UART node which gets picked up. How about disabling the node in -u-boot.dtsi for the board and then you can use platform data, I think... But Rob, this loops us back around to the first problem too, kind of. Can we really just not make use of clock-frequency and the kernel just not use it? -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: