2016-11-25 07:49:30

by Rosia, Nicolae

[permalink] [raw]
Subject: [PATCH] regulator: twl: drop of_match_ptr

We can safely drop of_match_ptr since the driver
depends on CONFIG_OF symbol

Signed-off-by: Nicolae Rosia <[email protected]>
---
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(-)

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,
},
};

--
2.9.3


2016-11-25 13:04:41

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] regulator: twl: drop of_match_ptr

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?


Attachments:
(No filename) (230.00 B)
signature.asc (455.00 B)
Download all attachments