Em Tue, 24 Jul 2018 10:37:56 -0700 Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> escreveu: > On Mon, Jul 23, 2018 at 8:41 PM Mauro Carvalho Chehab > <mchehab+samsung@xxxxxxxxxx> wrote: > > > > While I won't be against merging it, IMHO a better fix would be to > > add the includes asm/cacheflush.h needs inside it, e. g. something > > like adding: > > No. The <asm/*> includes really should come after <linux/*>. > > This is a media driver bug, plain and simple. Works for me. Do you intend to apply it directly? Otherwise I'll add on my tree - and likely send you during the merge window - this is just Kconfig random COMPILE_TEST build noise, as this driver is ARM-only (for an OMAP4 specific IP block). So, probably not worth sending a pull request just due to that. > We should strive to avoid adding more header includes, it's one of the > major causes of kernel build slowdown. Yeah, some time ago mailing lists got flooded with some janitorial's patchset adding includes (some claiming to be needed on some archs or under some random Kconfigs)... Compile-test ended by adding more such stuff (for a good reason, IMHO). I wonder if are there a better way to handle includes without slowing builds. Thanks, Mauro _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
