From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 27 Apr 2012 07:53:34 -0700 Subject: [U-Boot] [PATCH v2 1/2] CONFIG_SYS_BAUDRATE_TABLE: Add , place there In-Reply-To: <20120427063137.077F6200244@gemini.denx.de> References: <1335480396-29478-1-git-send-email-trini@ti.com> <20120427063137.077F6200244@gemini.denx.de> Message-ID: <4F9AB2EE.3070202@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 On 04/26/2012 11:31 PM, Wolfgang Denk wrote: > Dear Tom Rini, > > In message<1335480396-29478-1-git-send-email-trini@ti.com> you wrote: >> We provide a default table of { 9600, 19200, 38400, 57600, 115200 } >> in which mkconfig places after in >> the generated config file. This is used when a board has not set its >> own table. A number of boards set values below 9600 but it is my belief >> that this was done for completeness rather than as a requirement so they >> have opted to the new default table. > > Argh... But you cannot change this without explicit ACK from the > respective board maintainers. Then we're really SOL here. Even if we limit it to "I dropped 300/600/900/1200/2400/4800 baud rate support, is that ok". > Technically, I don't like the name "config_fallbacks.h"; how about > "config_defaults.h" instead? Sure. > Finally, your patch does not apply (I guess it is an incremental patch > over obsolete code?). That's odd actually, it was on v2012.04'ish and a pull and rebase -q went quietly. It was build-tested on MAKEALL -a powerpc/arm/x86. -- Tom