The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-at91.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index f350fd2..2673ce3 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -1671,7 +1671,7 @@ static struct platform_driver at91_gpio_driver = {
.driver = {
.name = "gpio-at91",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(at91_gpio_of_match),
+ .of_match_table = at91_gpio_of_match,
},
.probe = at91_gpio_probe,
};
@@ -1680,7 +1680,7 @@ static struct platform_driver at91_pinctrl_driver = {
.driver = {
.name = "pinctrl-at91",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(at91_pinctrl_of_match),
+ .of_match_table = at91_pinctrl_of_match,
},
.probe = at91_pinctrl_probe,
.remove = at91_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx35.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx35.c b/drivers/pinctrl/pinctrl-imx35.c
index c454982..278a04a 100644
--- a/drivers/pinctrl/pinctrl-imx35.c
+++ b/drivers/pinctrl/pinctrl-imx35.c
@@ -1019,7 +1019,7 @@ static struct platform_driver imx35_pinctrl_driver = {
.driver = {
.name = "imx35-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx35_pinctrl_of_match),
+ .of_match_table = imx35_pinctrl_of_match,
},
.probe = imx35_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx6q.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6q.c b/drivers/pinctrl/pinctrl-imx6q.c
index 76dd9c4..8eb5ac1 100644
--- a/drivers/pinctrl/pinctrl-imx6q.c
+++ b/drivers/pinctrl/pinctrl-imx6q.c
@@ -480,7 +480,7 @@ static struct platform_driver imx6q_pinctrl_driver = {
.driver = {
.name = "imx6q-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx6q_pinctrl_of_match),
+ .of_match_table = imx6q_pinctrl_of_match,
},
.probe = imx6q_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
Cc: Viresh Kumar <[email protected]>
---
drivers/pinctrl/spear/pinctrl-plgpio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
index 0a7f0bdb..ff2940e 100644
--- a/drivers/pinctrl/spear/pinctrl-plgpio.c
+++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
@@ -735,7 +735,7 @@ static struct platform_driver plgpio_driver = {
.owner = THIS_MODULE,
.name = "spear-plgpio",
.pm = &plgpio_dev_pm_ops,
- .of_match_table = of_match_ptr(plgpio_of_match),
+ .of_match_table = plgpio_of_match,
},
};
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx6dl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6dl.c b/drivers/pinctrl/pinctrl-imx6dl.c
index a76b724..db2a148 100644
--- a/drivers/pinctrl/pinctrl-imx6dl.c
+++ b/drivers/pinctrl/pinctrl-imx6dl.c
@@ -474,7 +474,7 @@ static struct platform_driver imx6dl_pinctrl_driver = {
.driver = {
.name = "imx6dl-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx6dl_pinctrl_of_match),
+ .of_match_table = imx6dl_pinctrl_of_match,
},
.probe = imx6dl_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-vf610.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-vf610.c b/drivers/pinctrl/pinctrl-vf610.c
index 68a970b..bddd913 100644
--- a/drivers/pinctrl/pinctrl-vf610.c
+++ b/drivers/pinctrl/pinctrl-vf610.c
@@ -316,7 +316,7 @@ static struct platform_driver vf610_pinctrl_driver = {
.driver = {
.name = "vf610-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(vf610_pinctrl_of_match),
+ .of_match_table = vf610_pinctrl_of_match,
},
.probe = vf610_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx6sl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx6sl.c b/drivers/pinctrl/pinctrl-imx6sl.c
index 4eb7cca..f21b738 100644
--- a/drivers/pinctrl/pinctrl-imx6sl.c
+++ b/drivers/pinctrl/pinctrl-imx6sl.c
@@ -380,7 +380,7 @@ static struct platform_driver imx6sl_pinctrl_driver = {
.driver = {
.name = "imx6sl-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx6sl_pinctrl_of_match),
+ .of_match_table = imx6sl_pinctrl_of_match,
},
.probe = imx6sl_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-samsung.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-samsung.c b/drivers/pinctrl/pinctrl-samsung.c
index 92a9d6c..47ec2e8 100644
--- a/drivers/pinctrl/pinctrl-samsung.c
+++ b/drivers/pinctrl/pinctrl-samsung.c
@@ -1148,7 +1148,7 @@ static struct platform_driver samsung_pinctrl_driver = {
.driver = {
.name = "samsung-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(samsung_pinctrl_dt_match),
+ .of_match_table = samsung_pinctrl_dt_match,
},
};
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx53.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx53.c b/drivers/pinctrl/pinctrl-imx53.c
index 17562ae..f8d45c4 100644
--- a/drivers/pinctrl/pinctrl-imx53.c
+++ b/drivers/pinctrl/pinctrl-imx53.c
@@ -468,7 +468,7 @@ static struct platform_driver imx53_pinctrl_driver = {
.driver = {
.name = "imx53-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx53_pinctrl_of_match),
+ .of_match_table = imx53_pinctrl_of_match,
},
.probe = imx53_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-imx51.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-imx51.c b/drivers/pinctrl/pinctrl-imx51.c
index db268b9..19ab182 100644
--- a/drivers/pinctrl/pinctrl-imx51.c
+++ b/drivers/pinctrl/pinctrl-imx51.c
@@ -782,7 +782,7 @@ static struct platform_driver imx51_pinctrl_driver = {
.driver = {
.name = "imx51-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(imx51_pinctrl_of_match),
+ .of_match_table = imx51_pinctrl_of_match,
},
.probe = imx51_pinctrl_probe,
.remove = imx_pinctrl_remove,
--
1.7.9.5
The data structure of_match_ptr() protects is always compiled in.
Hence of_match_ptr() is not needed.
Signed-off-by: Sachin Kamat <[email protected]>
---
drivers/pinctrl/pinctrl-exynos5440.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-exynos5440.c b/drivers/pinctrl/pinctrl-exynos5440.c
index 544d469..8fe2ab0 100644
--- a/drivers/pinctrl/pinctrl-exynos5440.c
+++ b/drivers/pinctrl/pinctrl-exynos5440.c
@@ -1048,7 +1048,7 @@ static struct platform_driver exynos5440_pinctrl_driver = {
.driver = {
.name = "exynos5440-pinctrl",
.owner = THIS_MODULE,
- .of_match_table = of_match_ptr(exynos5440_pinctrl_dt_match),
+ .of_match_table = exynos5440_pinctrl_dt_match,
},
};
--
1.7.9.5
On 28 September 2013 17:38, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
of_match_ptr() isn't a data structure.. And also what does "protects" mean here?
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
> Cc: Viresh Kumar <[email protected]>
> ---
> drivers/pinctrl/spear/pinctrl-plgpio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
> index 0a7f0bdb..ff2940e 100644
> --- a/drivers/pinctrl/spear/pinctrl-plgpio.c
> +++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
> @@ -735,7 +735,7 @@ static struct platform_driver plgpio_driver = {
> .owner = THIS_MODULE,
> .name = "spear-plgpio",
> .pm = &plgpio_dev_pm_ops,
> - .of_match_table = of_match_ptr(plgpio_of_match),
> + .of_match_table = plgpio_of_match,
> },
> };
Patch is okay otherwise.
On 3 October 2013 10:07, Viresh Kumar <[email protected]> wrote:
> On 28 September 2013 17:38, Sachin Kamat <[email protected]> wrote:
>> The data structure of_match_ptr() protects is always compiled in.
>
> of_match_ptr() isn't a data structure..
Yes you are right. of_match_ptr() is a macro. The data structure here
refers to its argument and not of_match_ptr() itself.
>And also what does "protects" mean here?
It just means that of_match_ptr() avoids an undefined reference error
had CONFIG_OF been used around the data structures definition by
making the pointer NULL.
>
>> Hence of_match_ptr() is not needed.
>>
>> Signed-off-by: Sachin Kamat <[email protected]>
>> Cc: Viresh Kumar <[email protected]>
>> ---
>> drivers/pinctrl/spear/pinctrl-plgpio.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
>> index 0a7f0bdb..ff2940e 100644
>> --- a/drivers/pinctrl/spear/pinctrl-plgpio.c
>> +++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
>> @@ -735,7 +735,7 @@ static struct platform_driver plgpio_driver = {
>> .owner = THIS_MODULE,
>> .name = "spear-plgpio",
>> .pm = &plgpio_dev_pm_ops,
>> - .of_match_table = of_match_ptr(plgpio_of_match),
>> + .of_match_table = plgpio_of_match,
>> },
>> };
>
> Patch is okay otherwise.
Hope to have your ack on this :)
Thanks for reviewing.
--
With warm regards,
Sachin
On 3 October 2013 12:36, Sachin Kamat <[email protected]> wrote:
> On 3 October 2013 10:07, Viresh Kumar <[email protected]> wrote:
>> On 28 September 2013 17:38, Sachin Kamat <[email protected]> wrote:
>>> The data structure of_match_ptr() protects is always compiled in.
>>
>> of_match_ptr() isn't a data structure..
>
> Yes you are right. of_match_ptr() is a macro. The data structure here
> refers to its argument and not of_match_ptr() itself.
>
>>And also what does "protects" mean here?
>
> It just means that of_match_ptr() avoids an undefined reference error
> had CONFIG_OF been used around the data structures definition by
> making the pointer NULL.
>
>>
>>> Hence of_match_ptr() is not needed.
>>>
>>> Signed-off-by: Sachin Kamat <[email protected]>
>>> Cc: Viresh Kumar <[email protected]>
>>> ---
>>> drivers/pinctrl/spear/pinctrl-plgpio.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
>>> index 0a7f0bdb..ff2940e 100644
>>> --- a/drivers/pinctrl/spear/pinctrl-plgpio.c
>>> +++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
>>> @@ -735,7 +735,7 @@ static struct platform_driver plgpio_driver = {
>>> .owner = THIS_MODULE,
>>> .name = "spear-plgpio",
>>> .pm = &plgpio_dev_pm_ops,
>>> - .of_match_table = of_match_ptr(plgpio_of_match),
>>> + .of_match_table = plgpio_of_match,
>>> },
>>> };
>>
>> Patch is okay otherwise.
>
> Hope to have your ack on this :)
> Thanks for reviewing.
Yeah I will Ack this.. but was waiting for your next version with fixed log. :)
On 3 October 2013 12:37, Viresh Kumar <[email protected]> wrote:
> On 3 October 2013 12:36, Sachin Kamat <[email protected]> wrote:
>> On 3 October 2013 10:07, Viresh Kumar <[email protected]> wrote:
>>> On 28 September 2013 17:38, Sachin Kamat <[email protected]> wrote:
> Yeah I will Ack this.. but was waiting for your next version with fixed log. :)
Do you really want a new one. If so, please let me know how you would
like this to be re-worded. I have already used this for others and
thought it was clear enough :)
--
With warm regards,
Sachin
On 28 September 2013 17:38, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
> Cc: Viresh Kumar <[email protected]>
> ---
> drivers/pinctrl/spear/pinctrl-plgpio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
> index 0a7f0bdb..ff2940e 100644
> --- a/drivers/pinctrl/spear/pinctrl-plgpio.c
> +++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
> @@ -735,7 +735,7 @@ static struct platform_driver plgpio_driver = {
> .owner = THIS_MODULE,
> .name = "spear-plgpio",
> .pm = &plgpio_dev_pm_ops,
> - .of_match_table = of_match_ptr(plgpio_of_match),
> + .of_match_table = plgpio_of_match,
> },
> };
Acked-by: Viresh Kumar <[email protected]>
On 3 October 2013 12:40, Sachin Kamat <[email protected]> wrote:
> Do you really want a new one. If so, please let me know how you would
> like this to be re-worded. I have already used this for others and
> thought it was clear enough :)
Ahh.. misread your subject and then your reply :)..
On 3 October 2013 12:42, Viresh Kumar <[email protected]> wrote:
> On 3 October 2013 12:40, Sachin Kamat <[email protected]> wrote:
>> Do you really want a new one. If so, please let me know how you would
>> like this to be re-worded. I have already used this for others and
>> thought it was clear enough :)
>
> Ahh.. misread your subject and then your reply :)..
No problem. Thanks anyway.
--
With warm regards,
Sachin
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
> The data structure of_match_ptr() protects is always compiled in.
> Hence of_match_ptr() is not needed.
>
> Signed-off-by: Sachin Kamat <[email protected]>
> Cc: Viresh Kumar <[email protected]>
I have applied this as well, and after that I squashed all 11 commits into
one.
I think these 11 patches should have been one from the beginning
but it's quicker for me to simply squash them than to ask you to
resubmit a squashed patch.
Yours,
Linus Walleij
On 8 October 2013 13:36, Linus Walleij <[email protected]> wrote:
> On Sat, Sep 28, 2013 at 2:08 PM, Sachin Kamat <[email protected]> wrote:
>
>> The data structure of_match_ptr() protects is always compiled in.
>> Hence of_match_ptr() is not needed.
>>
>> Signed-off-by: Sachin Kamat <[email protected]>
>> Cc: Viresh Kumar <[email protected]>
>
> I have applied this as well, and after that I squashed all 11 commits into
> one.
>
> I think these 11 patches should have been one from the beginning
> but it's quicker for me to simply squash them than to ask you to
> resubmit a squashed patch.
Thanks for doing it. I wasn't sure if you preferred single one to
driver-wise split.
Squashing is simpler compared to splitting :)
--
With warm regards,
Sachin