[PATCH 02/13] gpio/omap: Adapt GPIO driver to DT

Cousson, Benoit b-cousson at ti.com
Wed Sep 28 04:20:51 EDT 2011


I missed one comment...

On 9/27/2011 7:40 AM, Nayak, Rajendra wrote:
> On Monday 26 September 2011 10:20 PM, Benoit Cousson wrote:

[...]

>> +Required properties:
>> +- compatible:
>> +  - "ti,omap2-gpio" for OMAP2 and OMAP3 controllers
>
> Would it be more readable to have
> "ti,omap2-gpio" for OMAP2 controllers and
> "ti,omap3-gpio" for OMAP3 controllers?

The point here is to identify the IP version used in various OMAPs. 
Since OMAP3 and OMAP2 are using the same IP version, there is no point 
to differentiate the OMAP3 version.
What is doable is to put both "ti,omap3-gpio", "ti,omap2-gpio" to avoid 
modifying the driver for no reason and still being able to identify the 
OMAP3 version.
But I'm not sure we should do that if there is not real difference.

regards,
Benoit



More information about the linux-arm-kernel mailing list