Re: [PATCH] staging: media: atomisp: remove ifdef around HMM_BO_ION

2017-03-27 Thread Alan Cox
> > 2 -- > >  1 file changed, 2 deletions(-) > > Ugh, Alan, what's going on here, I thought you fixed this? I sent you a patch that removed the arrays entirely and turned it into a single string as well as removing the define. Not quite sure what happened but I've resynched to -next and I'll send

Re: [PATCH] staging: media: atomisp: remove ifdef around HMM_BO_ION

2017-03-24 Thread Greg Kroah-Hartman
On Fri, Mar 24, 2017 at 02:20:24PM +0100, Arnd Bergmann wrote: > The revert reintroduced a build failure without CONFIG_ION: > > media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: error: excess elements in array > initializer [-Werror] > media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: note: (near initializati

[PATCH] staging: media: atomisp: remove ifdef around HMM_BO_ION

2017-03-24 Thread Arnd Bergmann
The revert reintroduced a build failure without CONFIG_ION: media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: error: excess elements in array initializer [-Werror] media/atomisp/pci/atomisp2/hmm/hmm.c:52:2: note: (near initialization for 'hmm_bo_type_strings') We should really be able to build in any