| Message ID | 1380542526-2943-1-git-send-email-sachin.kamat@linaro.org |
|---|---|
| State | Accepted |
| Headers | show |
On Mon, Sep 30, 2013 at 05:32:06PM +0530, Sachin Kamat wrote: > The data structure of_match_ptr() protects is always compiled in. > Hence of_match_ptr() is not needed. > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Applied to for-next, thanks! I hope you checked if you missed other occurences, too. My pointing to mv64xxx was an example, not a complete list.
On 2 October 2013 03:50, Wolfram Sang <wsa@the-dreams.de> wrote: > On Mon, Sep 30, 2013 at 05:32:06PM +0530, Sachin Kamat wrote: >> The data structure of_match_ptr() protects is always compiled in. >> Hence of_match_ptr() is not needed. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > > Applied to for-next, thanks! Thanks. > I hope you checked if you missed other > occurences, too. My pointing to mv64xxx was an example, not a complete > list. I checked all the files in the sub-system manually now and found one other similar instance. Have sent out a patch for it.
diff --git a/drivers/i2c/busses/i2c-mv64xxx.c b/drivers/i2c/busses/i2c-mv64xxx.c index 7f3a474..f42d88a 100644 --- a/drivers/i2c/busses/i2c-mv64xxx.c +++ b/drivers/i2c/busses/i2c-mv64xxx.c @@ -909,7 +909,7 @@ static struct platform_driver mv64xxx_i2c_driver = { .driver = { .owner = THIS_MODULE, .name = MV64XXX_I2C_CTLR_NAME, - .of_match_table = of_match_ptr(mv64xxx_i2c_of_match_table), + .of_match_table = mv64xxx_i2c_of_match_table, }, };
The data structure of_match_ptr() protects is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> --- drivers/i2c/busses/i2c-mv64xxx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)