* [PATCH][RESEND v2] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF
@ 2016-05-25 1:15 Kuninori Morimoto
2016-06-03 3:09 ` Kuninori Morimoto
0 siblings, 1 reply; 3+ messages in thread
From: Kuninori Morimoto @ 2016-05-25 1:15 UTC (permalink / raw)
To: Rob Herring, Frank Rowand, Grant Likely, devicetree,
linux-kernel, Guennadi Liakhovetski
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
commit 93c667ca2598bd84f1bd3f2fa176af69707699fe
("of: *node argument to of_parse_phandle_with_args should be const")
changed to const for struct device node *np,
but it cares CONFIG_OF case only, !CONFIG_OF case need it too.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
include/linux/of.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/of.h b/include/linux/of.h
index 7fcb681..4f2ce1b 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -585,7 +585,7 @@ static inline struct device_node *of_parse_phandle(const struct device_node *np,
return NULL;
}
-static inline int of_parse_phandle_with_args(struct device_node *np,
+static inline int of_parse_phandle_with_args(const struct device_node *np,
const char *list_name,
const char *cells_name,
int index,
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND v2] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF
2016-05-25 1:15 [PATCH][RESEND v2] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF Kuninori Morimoto
@ 2016-06-03 3:09 ` Kuninori Morimoto
2016-06-08 13:42 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Kuninori Morimoto @ 2016-06-03 3:09 UTC (permalink / raw)
To: Rob Herring
Cc: Frank Rowand, Grant Likely, devicetree, linux-kernel,
Guennadi Liakhovetski
Hi Rob
Should I re-send this patch ?
It still no response...
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> commit 93c667ca2598bd84f1bd3f2fa176af69707699fe
> ("of: *node argument to of_parse_phandle_with_args should be const")
> changed to const for struct device node *np,
> but it cares CONFIG_OF case only, !CONFIG_OF case need it too.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> include/linux/of.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/of.h b/include/linux/of.h
> index 7fcb681..4f2ce1b 100644
> --- a/include/linux/of.h
> +++ b/include/linux/of.h
> @@ -585,7 +585,7 @@ static inline struct device_node *of_parse_phandle(const struct device_node *np,
> return NULL;
> }
>
> -static inline int of_parse_phandle_with_args(struct device_node *np,
> +static inline int of_parse_phandle_with_args(const struct device_node *np,
> const char *list_name,
> const char *cells_name,
> int index,
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND v2] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF
2016-06-03 3:09 ` Kuninori Morimoto
@ 2016-06-08 13:42 ` Rob Herring
0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring @ 2016-06-08 13:42 UTC (permalink / raw)
To: Kuninori Morimoto
Cc: Frank Rowand, Grant Likely, devicetree, linux-kernel,
Guennadi Liakhovetski
On Thu, Jun 2, 2016 at 10:09 PM, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>
> Hi Rob
>
> Should I re-send this patch ?
> It still no response...
Sorry, for the delay. Now applied.
Rob
>
>> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>>
>> commit 93c667ca2598bd84f1bd3f2fa176af69707699fe
>> ("of: *node argument to of_parse_phandle_with_args should be const")
>> changed to const for struct device node *np,
>> but it cares CONFIG_OF case only, !CONFIG_OF case need it too.
>>
>> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>> ---
>> include/linux/of.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/linux/of.h b/include/linux/of.h
>> index 7fcb681..4f2ce1b 100644
>> --- a/include/linux/of.h
>> +++ b/include/linux/of.h
>> @@ -585,7 +585,7 @@ static inline struct device_node *of_parse_phandle(const struct device_node *np,
>> return NULL;
>> }
>>
>> -static inline int of_parse_phandle_with_args(struct device_node *np,
>> +static inline int of_parse_phandle_with_args(const struct device_node *np,
>> const char *list_name,
>> const char *cells_name,
>> int index,
>> --
>> 1.9.1
>>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-06-08 13:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-25 1:15 [PATCH][RESEND v2] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF Kuninori Morimoto
2016-06-03 3:09 ` Kuninori Morimoto
2016-06-08 13:42 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).