Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1246447 > unrolled thread

Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-14 11:30 +0200
    Re: [kbuild-all] [PATCH] bcma: mips: Allow build if COMPILE_TEST is  enabled Fengguang Wu <lkp@intel.com> - 2015-10-14 11:50 +0200
    Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled Kalle Valo <kvalo@codeaurora.org> - 2015-10-14 13:00 +0200
      Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-14 13:00 +0200
        Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-14 13:10 +0200
        Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled Kalle Valo <kvalo@codeaurora.org> - 2015-10-14 13:10 +0200

#1246447 — Re: [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled

Hello,

On 10/13/2015 05:25 PM, kbuild test robot wrote:
> Hi Javier,
> 
> [auto build test ERROR on v4.3-rc5 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
> 
> url:    https://github.com/0day-ci/linux/commits/Javier-Martinez-Canillas/bcma-mips-Allow-build-if-COMPILE_TEST-is-enabled/20151013-214630
> config: i386-allmodconfig (attached as .config)
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=i386 
> 
> All errors (new ones prefixed by >>):
> 
>    drivers/mtd/devices/bcm47xxsflash.c: In function 'bcm47xxsflash_read':
>>> drivers/mtd/devices/bcm47xxsflash.c:112:37: error: implicit declaration of function 'KSEG0ADDR' [-Werror=implicit-function-declaration]
>      memcpy_fromio(buf, (void __iomem *)KSEG0ADDR(b47s->window + from),
>                    

Thanks for reporting this issue. I only did a partial build with
make M=drivers/bcma but didn't think about implicit dependencies
on other drivers.

I've posted this patch that should avoid this issue:
https://patchwork.kernel.org/patch/7391551/

BTW, 0-day bot used to reports from email address fengguang.wu@intel.com
but now reports from lkp@intel.com. I still added a Reported-by tag from
Fengguang Wu <fengguang.wu@intel.com> but I wonder if I should change it
to kbuild test robot <lkp@intel.com> for future patches?

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1246467 — Re: [kbuild-all] [PATCH] bcma: mips: Allow build if COMPILE_TEST is enabled

Hi Javier,

> BTW, 0-day bot used to reports from email address fengguang.wu@intel.com
> but now reports from lkp@intel.com. I still added a Reported-by tag from
> Fengguang Wu <fengguang.wu@intel.com> but I wonder if I should change it
> to kbuild test robot <lkp@intel.com> for future patches?

Either is fine. lkp is my lab account. I have to use it when replying
to mailing list emails because my account does not subscribe to so many
lists.

Thanks,
Fengguang
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1246513

Javier Martinez Canillas <javier@osg.samsung.com> writes:

> Hello,
>
> On 10/13/2015 05:25 PM, kbuild test robot wrote:
>> Hi Javier,
>> 
>> [auto build test ERROR on v4.3-rc5 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
>> 
>> url:    https://github.com/0day-ci/linux/commits/Javier-Martinez-Canillas/bcma-mips-Allow-build-if-COMPILE_TEST-is-enabled/20151013-214630
>> config: i386-allmodconfig (attached as .config)
>> reproduce:
>>         # save the attached .config to linux build tree
>>         make ARCH=i386 
>> 
>> All errors (new ones prefixed by >>):
>> 
>>    drivers/mtd/devices/bcm47xxsflash.c: In function 'bcm47xxsflash_read':
>>>> drivers/mtd/devices/bcm47xxsflash.c:112:37: error: implicit declaration of function 'KSEG0ADDR' [-Werror=implicit-function-declaration]
>>      memcpy_fromio(buf, (void __iomem *)KSEG0ADDR(b47s->window + from),
>>                    
>
> Thanks for reporting this issue. I only did a partial build with
> make M=drivers/bcma but didn't think about implicit dependencies
> on other drivers.
>
> I've posted this patch that should avoid this issue:
> https://patchwork.kernel.org/patch/7391551/

So this patch ("bcma: mips: Allow build if COMPILE_TEST is enabled")
depend on that patch ("mtd: Make MTD_BCM47XXSFLASH to depend on MIPS"),
right? So I cannot apply the bcma patch until the mtd patch is in my
tree.

-- 
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1246515

Hello Kalle,

On 10/14/2015 12:52 PM, Kalle Valo wrote:
> Javier Martinez Canillas <javier@osg.samsung.com> writes:
> 
>> Hello,
>>
>> On 10/13/2015 05:25 PM, kbuild test robot wrote:
>>> Hi Javier,
>>>
>>> [auto build test ERROR on v4.3-rc5 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
>>>
>>> url:    https://github.com/0day-ci/linux/commits/Javier-Martinez-Canillas/bcma-mips-Allow-build-if-COMPILE_TEST-is-enabled/20151013-214630
>>> config: i386-allmodconfig (attached as .config)
>>> reproduce:
>>>         # save the attached .config to linux build tree
>>>         make ARCH=i386 
>>>
>>> All errors (new ones prefixed by >>):
>>>
>>>    drivers/mtd/devices/bcm47xxsflash.c: In function 'bcm47xxsflash_read':
>>>>> drivers/mtd/devices/bcm47xxsflash.c:112:37: error: implicit declaration of function 'KSEG0ADDR' [-Werror=implicit-function-declaration]
>>>      memcpy_fromio(buf, (void __iomem *)KSEG0ADDR(b47s->window + from),
>>>                    
>>
>> Thanks for reporting this issue. I only did a partial build with
>> make M=drivers/bcma but didn't think about implicit dependencies
>> on other drivers.
>>
>> I've posted this patch that should avoid this issue:
>> https://patchwork.kernel.org/patch/7391551/
> 
> So this patch ("bcma: mips: Allow build if COMPILE_TEST is enabled")
> depend on that patch ("mtd: Make MTD_BCM47XXSFLASH to depend on MIPS"),
> right? So I cannot apply the bcma patch until the mtd patch is in my
> tree.
>

That's correct, sorry for not stating it explicitly.

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1246522

Hello Kale,

On 10/14/2015 01:04 PM, Kalle Valo wrote:
> Javier Martinez Canillas <javier@osg.samsung.com> writes:
> 
>> On 10/14/2015 12:52 PM, Kalle Valo wrote:
>>> Javier Martinez Canillas <javier@osg.samsung.com> writes:
>>>
>>>> Thanks for reporting this issue. I only did a partial build with
>>>> make M=drivers/bcma but didn't think about implicit dependencies
>>>> on other drivers.
>>>>
>>>> I've posted this patch that should avoid this issue:
>>>> https://patchwork.kernel.org/patch/7391551/
>>>
>>> So this patch ("bcma: mips: Allow build if COMPILE_TEST is enabled")
>>> depend on that patch ("mtd: Make MTD_BCM47XXSFLASH to depend on MIPS"),
>>> right? So I cannot apply the bcma patch until the mtd patch is in my
>>> tree.
>>
>> That's correct, sorry for not stating it explicitly.
> 
> Ok, I put the patch now to "Awaiting Upstream" state in patchwork and
> will apply it once I have the mtd patch.
>

Great, thanks a lot for your help!
 
Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1246523

Javier Martinez Canillas <javier@osg.samsung.com> writes:

> On 10/14/2015 12:52 PM, Kalle Valo wrote:
>> Javier Martinez Canillas <javier@osg.samsung.com> writes:
>> 
>>> Thanks for reporting this issue. I only did a partial build with
>>> make M=drivers/bcma but didn't think about implicit dependencies
>>> on other drivers.
>>>
>>> I've posted this patch that should avoid this issue:
>>> https://patchwork.kernel.org/patch/7391551/
>> 
>> So this patch ("bcma: mips: Allow build if COMPILE_TEST is enabled")
>> depend on that patch ("mtd: Make MTD_BCM47XXSFLASH to depend on MIPS"),
>> right? So I cannot apply the bcma patch until the mtd patch is in my
>> tree.
>
> That's correct, sorry for not stating it explicitly.

Ok, I put the patch now to "Awaiting Upstream" state in patchwork and
will apply it once I have the mtd patch.

-- 
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web