Re: [PATCH 22/28] usb: musb: introduce is_inventra_dma helper

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/12/13 Felipe Balbi <balbi@xxxxxx>:
> On Mon, Dec 13, 2010 at 01:02:16AM +0800, tom.leiming@xxxxxxxxx wrote:
>>
>> From: Ming Lei <tom.leiming@xxxxxxxxx>
>>
>> This patch introduces is_inventra_dma helper, so that
>> we can remove dependency on CONFIG_USB_INVENTRA_DMA
>> in source file of musb core driver in later patches.
>>
>> Signed-off-by: Ming Lei <tom.leiming@xxxxxxxxx>
>> ---
>> drivers/usb/musb/musb_dma.h |    6 ++++++
>> 1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/usb/musb/musb_dma.h b/drivers/usb/musb/musb_dma.h
>> index 70a64a2..b4271b8 100644
>> --- a/drivers/usb/musb/musb_dma.h
>> +++ b/drivers/usb/musb/musb_dma.h
>> @@ -68,6 +68,12 @@ struct musb_hw_ep;
>> #define is_dma_capable()        (0)
>> #endif
>>
>> +#ifdef CONFIG_USB_INVENTRA_DMA
>> +#define        is_inventra_dma()       1
>> +#else
>> +#define        is_inventra_dma()       0
>> +#endif
>
> we actually want to get rid of that, all DMA-specific crap should move
> to dma layer, MUSB should not care about which DMA it's using.

Yes, the helpers above will be removed finally. I think it will be close to
the final goal to convert 'ifdef CONFIG_USB_*_DMA' into is_*_dma first.

Once the conversion is finished, it will be very easier to remove
this helpers and move all dma specific code into its dma controller driver,
then musb core(musb_hdrc) will be much more generic than now.

thanks,
-- 
Lei Ming
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux