[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AANLkTimayi4tSe=r1HOPgpcSucBWC7SoBV1ocmV5LEs4@mail.gmail.com>
Date: Tue, 1 Mar 2011 21:35:59 +0200
From: Pekka Enberg <penberg@...nel.org>
To: Randy Dunlap <randy.dunlap@...cle.com>
Cc: Stephen Rothwell <sfr@...b.auug.org.au>,
linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
gregkh@...e.de
Subject: Re: linux-next: Tree for March 1 (staging/winbond)
On Tue, Mar 1, 2011 at 9:19 PM, Randy Dunlap <randy.dunlap@...cle.com> wrote:
> On Tue, 1 Mar 2011 21:10:23 +0200 Pekka Enberg wrote:
>
>> On Tue, Mar 1, 2011 at 8:49 PM, Randy Dunlap <randy.dunlap@...cle.com> wrote:
>> > On Tue, 1 Mar 2011 18:11:19 +1100 Stephen Rothwell wrote:
>> >
>> >> Hi all,
>> >>
>> >> Changes since 20110228:
>> >
>> >
>> > commit ddee7e28e7d5e4ba2b8537c6a59b035745c250bb
>> > Author: Pekka Enberg <penberg@...nel.org>
>> > Date: Sun Nov 28 23:00:01 2010 +0200
>> > Staging: w35und: Remove empty sysdef.h header
>> >
>> >
>> > This header should not have been empty, since in
>> >
>> > commit 412dc7f368bf10a8049a8a4c41abbfd0108742e7
>> > Author: Randy Dunlap <randy.dunlap@...cle.com>
>> > Date: Fri Nov 19 09:30:18 2010 -0800
>> > staging: fix winbond build, needs delay.h
>> >
>> > I added delay.h to it. And now we have the same build
>> > errors that I fixed last November:
>> >
>> > drivers/staging/winbond/phy_calibration.c:986: error: implicit declaration of function 'msleep'
>> > drivers/staging/winbond/phy_calibration.c:1555: error: implicit declaration of function 'udelay'
>> > drivers/staging/winbond/reg.c:893: error: implicit declaration of function 'msleep'
>> > drivers/staging/winbond/reg.c:1169: error: implicit declaration of function 'udelay'
>>
>> Strange. I didn't see the problem. We should add delay.h to
>> phy_calibration.c and reg.c instead. Care to make a patch?
>> --
>
> No thanks, I'll let the staging people do that.
Hmm, commit cecf826df8648c843ea8db63b1f82c154a74db36 ("staging:
winbond: needs <linux/delay.h> for msleep and friends") should fix it.
Isn't it in linux-next yet?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists