<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 22, 2015 at 4:45 PM, Felix Fietkau <span dir="ltr"><<a href="mailto:nbd@openwrt.org" target="_blank">nbd@openwrt.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 2015-09-22 15:11, Alexandru Ardelean wrote:<br>
><br>
><br>
> On Tue, Sep 22, 2015 at 4:05 PM, Matthias Schiffer<br>
</span>> <<a href="mailto:mschiffer@universe-factory.net" target="_blank">mschiffer@universe-factory.net</a> <mailto:<a href="mailto:mschiffer@universe-factory.net" target="_blank">mschiffer@universe-factory.net</a>>><br>
<span>> wrote:<br>
><br>
>     On 09/22/2015 06:25 PM, Alexandru Ardelean wrote:<br>
</span>>     > From: Alexandru Ardelean <<a href="mailto:aa@ocedo.com" target="_blank">aa@ocedo.com</a> <mailto:<a href="mailto:aa@ocedo.com" target="_blank">aa@ocedo.com</a>>><br>
>     ><br>
>     > Signed-off-by: Alexandru Ardelean <<a href="mailto:ardeleanalex@gmail.com" target="_blank">ardeleanalex@gmail.com</a><br>
>     <mailto:<a href="mailto:ardeleanalex@gmail.com" target="_blank">ardeleanalex@gmail.com</a>>><br>
<span>>     > ---<br>
>     >  target/linux/mpc85xx/Makefile | 5 ++++-<br>
>     >  1 file changed, 4 insertions(+), 1 deletion(-)<br>
>     ><br>
>     > diff --git a/target/linux/mpc85xx/Makefile<br>
>     b/target/linux/mpc85xx/Makefile<br>
>     > index e43327a..bfc2c8d 100644<br>
>     > --- a/target/linux/mpc85xx/Makefile<br>
>     > +++ b/target/linux/mpc85xx/Makefile<br>
>     > @@ -16,7 +16,10 @@ SUBTARGETS=generic p1020<br>
>     ><br>
>     >  KERNEL_PATCHVER:=3.18<br>
>     ><br>
>     > -KERNEL_IMAGES := zImage cuImage.tl-wdr4900-v1<br>
>     > +KERNEL_IMAGES := zImage<br>
>     > +ifeq ($(CONFIG_TL_WDR4900_V1),y)<br>
>     > +KERNEL_IMAGES += cuImage.tl-wdr4900-v1<br>
>     > +endif<br>
>     ><br>
</span>>     >  include $(INCLUDE_DIR)/<a href="http://target.mk" rel="noreferrer" target="_blank">target.mk</a> <<a href="http://target.mk" rel="noreferrer" target="_blank">http://target.mk</a>><br>
<span>>     ><br>
>     ><br>
><br>
>     This doesn't work, CONFIG_TL_WDR4900_V1 is a kernel .config option, not<br>
>     a OpenWrt .config option, and thus isn't available in OpenWrt Makefiles.<br>
>     What is this supposed to fix?<br>
><br>
><br>
> Seems I'm confused here with the makefiles.<br>
><br>
> For our case, removing that kernel image file is needed, because we<br>
> build our own mpc85xx image that is different from wdr4900-v1 ; so no<br>
> file named cuImage.tl-wdr4900-v1.<br>
> Ideally, I'd like to put it under a config option that disables/enables<br>
> this ; I will admit that I haven't tried the case where this is on.<br>
><br>
> I'll try to find another option that disables this, that can also enable it.<br>
> Any suggestions ?<br>
</span>Is this about non-upstream changes to the mpc85xx target? Or are you<br>
using a different subtarget?<br>
Either way, we should probably make the check for the 'generic' subtarget...<br>
<span><font color="#888888"><br>
- Felix<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">We use a different subtarget, and we're trying to find a solution for this that is also upstream-able, in the idea that it makes sync-ing with upstream easier.<br></div><div class="gmail_extra">Checking for the Generic subtarget sounds good.<br><br></div><div class="gmail_extra">Should I send a patch for that ?</div></div>