Hi Grant,
On 02/16/2011 01:49 PM, Grant Likely wrote:
On Mon, Feb 14, 2011 at 10:30:10AM +0800, Thomas Chou wrote:
From: Albert Herranz<[email protected]>
This patch adds devicetree support to i2c-gpio driver. The allocation
of local data is integrated to a private structure, and use devm_*
helper for easy cleanup.
It is base on an earlier patch for gc-linux from
Albert Herranz<[email protected]>.
Signed-off-by: Thomas Chou<[email protected]>
CC: Albert Herranz<[email protected]>
Acked-by: Haavard Skinnemoen<[email protected]>
One minor nitpick below, but I'd be fine with cleaning that up via a
followup patch. This one looks correct to me.
Acked-by: Grant Likely<[email protected]>
Ben, this one depends on a patch in my devicetree/next tree. How do
you want to handle it?
g.
Thanks a lot.
+#ifdef CONFIG_OF
+#include<linux/of_gpio.h>
Instead of putting the include here, please add #ifdef CONFIG_OF
protection around the contents of linux/of_gpio.h itself. I fully
support that change so that includes can remain at the top where they
belong.
Then, my earlier patch on Jan 31, might be another solution. This way
of_gpio.h can be included at the top, and those of_gpio users will be
optimized out when of or of_gpio is not configured.
[PATCH 2/3] of: of_gpiochip_add is needed only for gpiolib
Cheers,
Thomas
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
