From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 17 Dec 2012 16:09:11 -0500 Subject: [U-Boot] [PATCH] ns16550: allow UART address to be set dynamically In-Reply-To: <50CBABF6.7040104@wwwdotorg.org> References: <1355354590-10023-1-git-send-email-swarren@wwwdotorg.org> <50C918A5.6090207@wwwdotorg.org> <20121213102950.17EC320120D@gemini.denx.de> <50CA1BB8.4000704@wwwdotorg.org> <20121213203604.1DD9F201213@gemini.denx.de> <50CA3E7A.8020407@wwwdotorg.org> <50CA4056.8060900@ti.com> <50CA437F.7060704@wwwdotorg.org> <50CB8ED1.7020503@ti.com> <50CBA217.3070202@wwwdotorg.org> <50CBABF6.7040104@wwwdotorg.org> Message-ID: <50CF89F7.307@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 12/14/12 17:45, Stephen Warren wrote: > On 12/14/2012 03:22 PM, Simon Glass wrote: >> Hi Stephen, > ... >> Perhaps I can make the point another way. Assuming that the SOC >> in question is ARM-based and has Linux support it either supports >> FDT now or presumably will fairly soon. > > Sure, but I'm *explicitly* avoiding relying on DT for this, > because there are plenty of things that happen before DT can or > should be touched that might warrant serial port output. Also totally true and valid. But what I'd like to see is: parse_odmdata(void *ptr) { if (ptr->console) device_tree_register_uart(... fake it ...); } In other words, if we add the register a port call now, there's a history of adding just one more thing, by someone else (say am335x and our EVMs that differ on where primary UART is, and only need a little different logic to say 'oh! this one.') and making a mess of things. Once we deal with device trees in some manner, then we can just fake their existence at this point and pass in the console information from ODMDATA. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://www.enigmail.net/ iQIcBAEBAgAGBQJQz4n3AAoJENk4IS6UOR1WrqoP/0glYuxrrrn9hgZa4WOhKhCd K14da+7svtAdZLEFNVzbBb67sMsK9f0BH3PiesIxDBU2wAj7CgAZMpkedyAhwPjY /oX8ywaWouckSrtVe1k4FTT7yPJOW6WlljaOFypHvt5VWRfhKJ6kdlMlkAiPPMEY ng8cE1YKcbZZD9RsxhMpW6+OGVN9iRcFDSDW3EvPpvQaIl5hp3LYcKnQvNi7I3kE edGIdzakPjxtJTVv8KiPuIOhYtYlTUvgcaeLgqbm9g/57z01Cz+SrAAaHXR8qxOv pwwR53owgh6Cb033QEClP9mDGrd9QOXs9mkZm/GxhTFI4bWS3qJXvh2I73NUVL3Y OrN0XfoLX5RZzkk/oYUV46OsI6oxPOImH4KEDFm2ST/twni813DIEtPGcotrWZm4 oSKYxONOvofaRalcwH+P89Iq8zLWgjglWwNoOCgItWjhwvY75q/RCQPB2vOkSFtb /Vt4Do5HeqfmR6XPHHvP0dUCZ2IS0/Xh7w+OO44HakDLESiWfnfr/dWSmTiiqBA+ wfWDMhE+pK/fnz3CG4eAOEUQ7Y7Uh3jGo42WpthpCuY0ZltnhyEHwmPJ6MuwAqbs EDUXSbNnpLOC9iIRKnT6J0hkhONp/ip+1Mad2hsVluAMmqYBTdM7rKi/kuV0HdYl LPh6xToy9xyUgkllZZbw =nUfP -----END PGP SIGNATURE-----