On Wed, Jan 06, 2021 at 11:10:38AM -0800, Joe Perches wrote: > On Tue, 2021-01-05 at 15:02 +0100, Thomas Bogendoerfer wrote: > > Signed-off-by: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx> > [] > > diff --git a/drivers/dma/txx9dmac.h b/drivers/dma/txx9dmac.h > [] > > @@ -26,11 +26,6 @@ > > * DMA channel. > > */ > > > > > > -#ifdef CONFIG_MACH_TX49XX > > -static inline bool txx9_dma_have_SMPCHN(void) > > -{ > > - return true; > > -} > > #define TXX9_DMA_USE_SIMPLE_CHAIN > > #else > > static inline bool txx9_dma_have_SMPCHN(void) > > This doesn't look like it compiles as there's now an #else > without an #if you are right, no idea what I had in mind while doing that. Vinod, as this patch series found a still active user of the platform, could you drop the patch from your tree, or do you want a revert from me ? Thomas. -- Crap can work. Given enough thrust pigs will fly, but it's not necessarily a good idea. [ RFC1925, 2.3 ]