[RFC PATCH] SDHCI: S3C: Add support for retrieving memory and irq resource information from device tree.

Grant Likely grant.likely at secretlab.ca
Tue Feb 1 05:00:26 EST 2011


On Mon, Jan 31, 2011 at 10:00 AM, Rob Herring <robherring2 at gmail.com> wrote:
> Thomas,
>
> On 01/31/2011 10:28 AM, thomas.abraham at linaro.org wrote:
>>
>> From: Thomas Abraham<thomas.abraham at linaro.com>
>>
>> Add support for retrieving memory and irq resource information
>> from device tree for Samsung's SDHCI controller driver.
>>
>> Signed-off-by: Thomas Abraham<thomas.abraham at linaro.org>
>> ---
>>
>> The modification will be made more generic to support both
>> DT and non-DT versions of the driver without the #ifdef's.
>> For now, this patch is for review and to understand if the
>> approach adopted to obtain resource information from the
>> device tree is appropriate.
>>

> You are missing an of_match_table. This patch could not work without it.

Actually, the way he is implementing it the of_match_table would have
absolutely no effect.  However, you are right, it *should* be
implemented so that it binds via the of_match_table.  I suspect Thomas
did it this way because he doesn't yet have an easy way to get his
platform devices registered from the device tree.

g.


More information about the devicetree-discuss mailing list