From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 19 Jan 2018 10:41:15 -0500 Subject: [U-Boot] [U-Boot, 1/4] serial: ns16550: da8xx (freon/primus) is not omap-like In-Reply-To: References: <20180108125059.GA14025@squirrel.local> <20180119152650.GA23609@bill-the-cat> Message-ID: <20180119154115.GV4660@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 Fri, Jan 19, 2018 at 04:39:15PM +0100, Matthijs van Duin wrote: > On 19 January 2018 at 16:26, Tom Rini wrote: > > This introduces a warning on all SOC_DA8XX systems because they do not > > set CONFIG_SYS_NS16550_MEM32 so serial_out is writeb > > Well that's a mistake. Its uart uses 32-bit registers, and 8-bit > access is not documented to be acceptable. > > Apologies for forgetting to at least compile-test this for all > affected targets, I really should have done that. > > > Are you able to test SOC_DA8XX platforms? > > No, this was based on my knowledge that freon/primus SoCs > (OMAP-L1xx/AM1xxx/TMS320C674x/DA8xx) are architecturally members of > the TI C6x family and not remotely related to the OMAP (despite the > part code "OMAP-L1xx"), and I double-checked the technical reference > manual: their UART is identical to that on Keystone. OK. For v2, if you can convert CONFIG_SYS_NS16550_MEM32 to Kconfig as well I'd appreciate it. I'll see about digging out and setting up one of my da8xx platforms for a quick runtime test too. Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: