From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Christophe PLAGNIOL-VILLARD Date: Sat, 6 Sep 2008 20:29:33 +0200 Subject: [U-Boot] [PATCH] Add background serial print and correct search_device function In-Reply-To: <1219251490-26134-1-git-send-email-ryan.chen@st.com> References: <1219251490-26134-1-git-send-email-ryan.chen@st.com> Message-ID: <20080906182933.GG1249@game.jcrosoft.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 12:58 Wed 20 Aug , Ryan CHEN wrote: > The patch introduces a new macro 'CFG_BG_CONSOLE_SERIAL'. > It means print messages through serial port although current console isn't serial port. > It is important for debugging and looks like multi-consoles in Linux but without input. > > Another modification in the patch is that verify if search_device function found device. > If found, return dev. Otherwise, return NULL. > > Signed-off-by: Ryan Chen > > --- > common/console.c | 13 +++++++++++++ > 1 files changed, 13 insertions(+), 0 deletions(-) > > void fprintf (int file, const char *fmt, ...) > @@ -377,6 +385,7 @@ device_t *search_device (int flags, char *name) > { > int i, items; > device_t *dev = NULL; > + int found_flag = 0; > > items = ListNumItems (devlist); > if (name == NULL) > @@ -385,9 +394,13 @@ device_t *search_device (int flags, char *name) > for (i = 1; i <= items; i++) { > dev = ListGetPtrToItem (devlist, i); > if ((dev->flags & flags) && (strcmp (name, dev->name) == 0)) { > + found_flag = 1; > break; > } > } > + > + if(!found_flag) > + return NULL; > return dev; > } This part is no more need, It's already fix by this commit c1de7a6daf9c657484e1c6d433f01fccd49a7f48 Best Regards, J.