From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 5 Apr 2012 21:19:11 +0200 Subject: [U-Boot] [PATCH] usb: use noinline define In-Reply-To: <201204051342.47692.vapier@gentoo.org> References: <1333601093-21481-1-git-send-email-vapier@gentoo.org> <201204051213.21759.marek.vasut@gmail.com> <201204051342.47692.vapier@gentoo.org> Message-ID: <201204052119.11557.marek.vasut@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Mike Frysinger, > On Thursday 05 April 2012 06:13:21 Marek Vasut wrote: > > Dear Mike Frysinger, > > > > > Building usb for Blackfin boards fails as we get linux/compiler.h > > > included which expands the "noinline" inside of the attribute and > > > we get attribute(attribute(noinline)). > > > > > > Explicitly use the helper define to avoid this. > > > > Ain't compiler.h broken then? > > no ... why would it be ? Because it colides with gcc stuff? > > Btw. is this a fix I should push to .04 release? > > i looked closer at how exactly it's failing, and i think waiting for the > next merge window is OK if you want I'm fine either way as I don't observe this. On the other hand, I'd be inclined to push this into current release if you're 100% sure about the fix and it fixes real issue for some people. > -mike Best regards, Marek Vasut