| Message ID | 20161125074900.7172-1-Nicolae_Rosia@mentor.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Nov 25, 2016 at 09:49:00AM +0200, Nicolae Rosia wrote: > We can safely drop of_match_ptr since the driver > depends on CONFIG_OF symbol As with the previous patch I see no benefit here. What's the reason for doing this?
diff --git a/drivers/regulator/twl-regulator.c b/drivers/regulator/twl-regulator.c index 6c9ec84..48c060f 100644 --- a/drivers/regulator/twl-regulator.c +++ b/drivers/regulator/twl-regulator.c @@ -654,7 +654,7 @@ static struct platform_driver twlreg_driver = { */ .driver = { .name = "twl4030_reg", - .of_match_table = of_match_ptr(twl_of_match), + .of_match_table = twl_of_match, }, };
We can safely drop of_match_ptr since the driver depends on CONFIG_OF symbol Signed-off-by: Nicolae Rosia <Nicolae_Rosia@mentor.com> --- Depends on "regulator: twl6030: add dependency on OF". After sending that one I realised that twl-regulator can also have that change. You can squash these two if you like. drivers/regulator/twl-regulator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)