On Wed, 2016-03-02 at 17:25 +0100, Michal Marek wrote: > On 2016-03-02 17:17, James Bottomley wrote: > > On Thu, 2016-02-18 at 00:46 +0100, Michal Marek wrote: > > > On Fri, Feb 12, 2016 at 02:42:26PM -0800, James Bottomley wrote: > > > > I've recently been experimenting with building in emulated > > > > architecture containers which allow me to build natively on my > > > > laptop a kernel for any architecture which qemu will emulate. > > > > To > > > > do this, I've been building into build/$(uname -m) and this > > > > caused > > > > the aicasm stuff to fail to build (using > > > > CONFIG_AIC7XXX_BUILD_FIRMW > > > > ARE=y). I think this patch corrects the problem, but I'm not > > > > hugely familiar with the kbuild infrastructure so I cc'd an > > > > expert > > > > for a second opinion. > > > > > > Hi James, > > > > > > Sorry for the late reply. Letting kbuild handle the aicasm > > > directory > > > is a step in the right direction. However, it still failed for me > > > and > > > instead of trying to understand how the rules work, I removed > > > them > > > and used the existing kbuild infrastructure. Please try the patch > > > below on top of yours. > > > > Sorry for the late testing. This patch causes the build to fail > > again > > for me: > > > > make[5]: *** No rule to make target > > 'drivers/scsi/aic7xxx/aicasm/aicasm_scan.lex.c', needed by > > 'drivers/scsi/aic7xxx/aicasm/aicasm_scan.lex.o'. Stop. > > /home/jejb/git/scsi-misc/drivers/scsi/aic7xxx/Makefile:85: recipe > > for > > target 'drivers/scsi/aic7xxx/aicasm/aicasm' failed > > make[4]: *** [drivers/scsi/aic7xxx/aicasm/aicasm] Error 2 > > /home/jejb/git/scsi-misc/scripts/Makefile.build:407: recipe for > > target > > 'drivers/scsi/aic7xxx' failed > > make[3]: *** [drivers/scsi/aic7xxx] Error 2 > > make[3]: *** Waiting for unfinished jobs.... > > /home/jejb/git/scsi-misc/scripts/Makefile.build:407: recipe for > > target > > 'drivers/scsi' failed > > > > I think the problem is simply that we now have two separate options > > for > > building the firmware: REGENERATE_PARSERS and > > CONFIG_AIC7XXX_BUILD_FIRMWARE and the latter needs to be > > eliminated. > > I'll see if I can work out what's missing. > > Did you run the build with REGENERATE_PARSERS=1? It needs to be done > once and the _shipped files need to be added to git. I did not > include > them in my patch for the sake of brevity. The Build with REGENERATE_PARSERS works. The build with this make line make -j 4 O=build/x86_64 Fails if these config options are set: CONFIG_AIC7XXX_BUILD_FIRMWARE=y CONFIG_AIC79XX_BUILD_FIRMWARE=y I think the fix is just to excise those options from the build system. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html