Re: [alsa-devel] [PATCH] ASoC: Intel: Skylake - Add Skylake RT286 I2S machine driver
On Sat, 2015-10-10 at 16:42 +0800, kbuild test robot wrote:
Hi Omair,
[auto build test ERROR on next-20151009 -- if it's inappropriate base, please ignore]
config: arm-allmodconfig (attached as .config) reproduce:
That's nice, thanks. We should add a depends on X86 for this to not compile on non X86 code
Will send updated patch
On Mon, Oct 12, 2015 at 03:12:07AM +0000, Koul, Vinod wrote:
On Sat, 2015-10-10 at 16:42 +0800, kbuild test robot wrote:
[auto build test ERROR on next-20151009 -- if it's inappropriate base, please ignore]
config: arm-allmodconfig (attached as .config) reproduce:
That's nice, thanks. We should add a depends on X86 for this to not compile on non X86 code
Will send updated patch
It looks like the missing functions are just readl() and writel() which means that they're probably only working on x86 through implicit inclusion rather than intentionally - if that's the case adding (IIRC) linux/io.h is a better fix.
On Mon, Oct 12, 2015 at 05:17:36PM +0100, Mark Brown wrote:
On Mon, Oct 12, 2015 at 03:12:07AM +0000, Koul, Vinod wrote:
On Sat, 2015-10-10 at 16:42 +0800, kbuild test robot wrote:
[auto build test ERROR on next-20151009 -- if it's inappropriate base, please ignore]
config: arm-allmodconfig (attached as .config) reproduce:
That's nice, thanks. We should add a depends on X86 for this to not compile on non X86 code
Will send updated patch
It looks like the missing functions are just readl() and writel() which means that they're probably only working on x86 through implicit inclusion rather than intentionally - if that's the case adding (IIRC) linux/io.h is a better fix.
Yes relooking at this, I do agree that seems to be doable. Let me try that and see if we can get this to build :)
On Mon, Oct 12, 2015 at 10:02:25PM +0530, Vinod Koul wrote:
On Mon, Oct 12, 2015 at 05:17:36PM +0100, Mark Brown wrote:
On Mon, Oct 12, 2015 at 03:12:07AM +0000, Koul, Vinod wrote:
On Sat, 2015-10-10 at 16:42 +0800, kbuild test robot wrote:
[auto build test ERROR on next-20151009 -- if it's inappropriate base, please ignore]
config: arm-allmodconfig (attached as .config) reproduce:
That's nice, thanks. We should add a depends on X86 for this to not compile on non X86 code
Will send updated patch
It looks like the missing functions are just readl() and writel() which means that they're probably only working on x86 through implicit inclusion rather than intentionally - if that's the case adding (IIRC) linux/io.h is a better fix.
Yes relooking at this, I do agree that seems to be doable. Let me try that and see if we can get this to build :)
And will need depends on ACPI for acpi_ routines, so need to add that too
participants (3)
-
Koul, Vinod -
Mark Brown -
Vinod Koul