Russell King - ARM Linux <li...@arm.linux.org.uk> writes:

> On Mon, Jul 06, 2009 at 02:14:36PM -0700, Kevin Hilman wrote:
>> From: Sudhakar Rajashekhara <sudhakar....@ti.com>
>> 
>> Define a structure to store EDMA channel controller based information.
>> Use platform_device.id to find out the instance being configured in
>> probe function.
>
> Why all this complexity?  Resources are specific to each platform
> device, so just provide the _specific_ resources for each platform
> device.

Agreed.  I've asked Sudhakar to take a look at this and rework.

Thanks for the review.

Kevin

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to