On 03/19/2012 04:59 PM, Simon Glass wrote: > Hi Stephen, > > On Mon, Mar 19, 2012 at 2:18 PM, Stephen Warren <swar...@wwwdotorg.org> wrote: >> On 03/19/2012 02:27 PM, Simon Glass wrote: >>> We enable this feature on all UARTs for Seaboard. This ensures that a >>> message is printed if CONFIG_OF_CONTROL is in use and a value device tree >>> is not available. >> >> Why not just enabled this on UARTD, since that's what Seaboard uses? >> >> I guess some derivatives do use UARTB too, which makes things quite >> painful. Perhaps at least limit this to UARTB + UARTD, and not all the >> others? > > At the moment we can use Seaboard as a generic Tegra2 board, so we > want the widest possible select of UARTs. I think there is one board > that uses A? > > Really I would prefer that we explicitly create a generic Tegra2 > board, once the fdt stuff is bedded in.
Well, one of Wolfgang's main objections was blasting the panic message through all possible UARTs, which might send junk to something other than a debug UART (e.g. machinery and life support systems were mentioned). This change doesn't seem to solve that. For low-level debug like this, shouldn't we just route it to one single UART that the config file selects? We can certainly think about refactoring things into a unified board file, but that seems like something unrelated to do later... _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot