On Fri, Jan 19, 2018 at 04:39:15PM +0100, Matthijs van Duin wrote:
> On 19 January 2018 at 16:26, Tom Rini <tr...@konsulko.com> 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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to