From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: linux-next: build failure after merge of the wireless-drivers-next tree Date: Tue, 19 Jul 2016 17:08:59 +0300 Message-ID: <87h9blvhxg.fsf@kamboji.qca.qualcomm.com> References: <20160719160831.79c0303f@canb.auug.org.au> <8760s2w369.fsf@kamboji.qca.qualcomm.com> <871t2qw1cz.fsf@kamboji.qca.qualcomm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: (=?utf-8?Q?=22Rafa=C5=82_Mi=C5=82ecki=22's?= message of "Tue, 19 Jul 2016 09:11:11 +0200") Sender: linux-kernel-owner@vger.kernel.org To: =?utf-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Cc: Brian Norris , "linux-mtd@lists.infradead.org" , Stephen Rothwell , "linux-wireless@vger.kernel.org" , linux-next , Linux Kernel Mailing List List-Id: linux-next.vger.kernel.org Rafa=C5=82 Mi=C5=82ecki writes: > On 19 July 2016 at 09:09, Kalle Valo wrote: >> Rafa=C5=82 Mi=C5=82ecki writes: >> >>> On 19 July 2016 at 08:30, Kalle Valo wrote: >>>> Stephen Rothwell writes: >>>> >>>>> After merging the wireless-drivers-next tree, today's linux-next = build >>>>> (powerpc allyesconfig) failed like this: >>>>> >>>>> drivers/mtd/devices/bcm47xxsflash.c: In function 'bcm47xxsflash_b= cma_probe': >>>>> drivers/mtd/devices/bcm47xxsflash.c:299:17: error: implicit decla= ration of function 'ioremap_cache' [-Werror=3Dimplicit-function-declara= tion] >>>>> b47s->window =3D ioremap_cache(res->start, resource_size(res)); >>>>> ^ >>>>> drivers/mtd/devices/bcm47xxsflash.c:299:15: warning: assignment m= akes pointer from integer without a cast [-Wint-conversion] >>>>> b47s->window =3D ioremap_cache(res->start, resource_size(res)); >>>>> ^ >>>>> >>>>> Caused by commit >>>>> >>>>> 57d8f7dd2132 ("bcma: allow enabling serial flash support on non= -MIPS SoCs") >>>>> >>>>> I have reverted that commit for today. >>>> >>>> Thanks, kbuild bot also notified about this. >>>> >>>> Rafa=C5=82, as the merge window is getting really close I would li= ke to >>>> revert this for now and we can reapply it once we the problem is f= ully >>>> solved. And we can use my pending branch for build testing the pat= ch >>>> with kbuild bot. What do you think? >>> >>> The simplest fix sounds pretty much as simple as the revert. >> >> The problem is that I don't have much time, I need to send the pull >> request to Dave within next few days. The fix might create other >> problems so I'm not enthusiastic about iterating with this patch at = the >> moment. I think it can wait for 4.9. > > I sent a patch seconds ago, you may just take a look at it. If you > still prefer to revert my commit, go ahead. Ok, let's try your fix. We still have few days. I applied the patch[1] to the pending branch this morning and if I don't see any errors I appl= y it later tonight and hopefully it will be in tomorrow's linux-next build. Thanks for the quick fix, very much appreciated. [1] "mtd: add arch dependency for MTD_BCM47XXSFLASH symbol" https://patchwork.kernel.org/patch/9236053/ --=20 Kalle Valo