From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Martin Date: Thu, 25 Oct 2012 15:50:57 -0700 Subject: [U-Boot] [PATCH] serial: add environment control for SERIAL_MULTI In-Reply-To: References: <1351202390-11415-1-git-send-email-amartin@nvidia.com> <5089C0A9.4060608@wwwdotorg.org> Message-ID: <20121025225057.GD12183@badger> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, Oct 25, 2012 at 03:46:01PM -0700, Joe Hershberger wrote: > Hi Stephen, > > On Thu, Oct 25, 2012 at 5:43 PM, Stephen Warren wrote: > > On 10/25/2012 03:59 PM, Allen Martin wrote: > >> Add a new special environment variable "serial" that allows selection > >> of serial device when CONFIG_SERIAL_MULTI is defined. This replaces > >> the existing calls to serial_assign() from cmd_nvedit.c and iomux.c > >> that were not doing anything. > > > > So I think this requires (for example) the following environment variables: > > > > stdout=serial > > serial=eserial0 > > > > Is it possible to allow the following instead: > > > > stdout=eserial0 > > This is precisely what the patch I had pre-Marek serial did. In your patch would "stdout=serial" still work for case where there is only one serial port? I think it's important to try to preserve that to no be too disruptive. > > > That way, one could presumably set: > > > > stdout=eserial0,eserial3 > > Though it didn't allow this. > Shouldn't that be (nearly) transparent through iomux? -Allen -- nvpublic