From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: linux-next: build failure after merge of the tty tree Date: Thu, 31 Oct 2013 06:24:25 -0700 Message-ID: <20131031132425.GA2913@kroah.com> References: <73605653.13929.1383224991870.JavaMail.root@mail> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from out4-smtp.messagingengine.com ([66.111.4.28]:48077 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753751Ab3JaNWY (ORCPT ); Thu, 31 Oct 2013 09:22:24 -0400 Content-Disposition: inline In-Reply-To: <73605653.13929.1383224991870.JavaMail.root@mail> Sender: linux-serial-owner@vger.kernel.org List-Id: linux-serial@vger.kernel.org To: Philippe Proulx Cc: sedat dilek , linux-next , LKML , linux-serial@vger.kernel.org, Stephen Rothwell On Thu, Oct 31, 2013 at 09:09:51AM -0400, Philippe Proulx wrote: > ----- Original Message ----- > > From: "Sedat Dilek" > > To: "Stephen Rothwell" , "Philippe Proulx" > > Cc: "Greg KH" , "linux-next" , "LKML" , > > linux-serial@vger.kernel.org > > Sent: Thursday, 31 October, 2013 6:35:23 AM > > Subject: Re: linux-next: build failure after merge of the tty tree > > > > On Thu, Oct 31, 2013 at 8:40 AM, Stephen Rothwell > > wrote: > > > Hi Greg, > > > > > > After merging the tty tree, today's linux-next build (arm > > > multi_v7_defconfig) failed like this: > > > > > > drivers/tty/serial/omap-serial.c: In function 'serial_omap_probe': > > > drivers/tty/serial/omap-serial.c:1724:22: error: expected ')' > > > before numeric constant > > > drivers/tty/serial/omap-serial.c:1724:22: warning: format '%d' > > > expects a matching 'int' argument [-Wformat] > > > > > > Caused by commit e5f9bf72efbc ("serial: omap: fix a few checkpatch > > > warnings"). There is a missing ',' in the dev_warn() ... > > > > > > I have used the version of the tty tree from next-20131030 for > > > today. > > > > > > > [ Add CC to linux-serial ML ] > > > > Happy HelloWien and WelcomeBackStephen! > > > > That was only checkpatch-tested, eh :-)? > > > > @@ -1724,8 +1722,9 @@ static int serial_omap_probe(struct > > platform_device *pdev) > > up->port.uartclk = omap_up_info->uartclk; > > if (!up->port.uartclk) { > > up->port.uartclk = DEFAULT_CLK_SPEED; > > - dev_warn(&pdev->dev, "No clock speed specified: using default:" > > - "%d\n", DEFAULT_CLK_SPEED); > > + dev_warn(&pdev->dev, > > + "No clock speed specified: using default: %d\n" <--- Comma missing > > here! > > Oops, something obviously went wrong in my process. Sorry about this. > > Do you want me to fix this commit? Please send me a patch that I can apply. greg k-h