2015-07-07 13:01:31

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id

The platform_device_id is not modified by the driver and core uses it as
const.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/video/fbdev/s3c-fb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/s3c-fb.c b/drivers/video/fbdev/s3c-fb.c
index 7e3a05fc47aa..f72dd12456f9 100644
--- a/drivers/video/fbdev/s3c-fb.c
+++ b/drivers/video/fbdev/s3c-fb.c
@@ -1938,7 +1938,7 @@ static struct s3c_fb_driverdata s3c_fb_data_s3c2443 = {
},
};

-static struct platform_device_id s3c_fb_driver_ids[] = {
+static const struct platform_device_id s3c_fb_driver_ids[] = {
{
.name = "s3c-fb",
.driver_data = (unsigned long)&s3c_fb_data_64xx,
--
2.1.4


2015-07-08 04:34:48

by Jingoo Han

[permalink] [raw]
Subject: Re: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id


On 2015. 7. 7., at PM 10:00, Krzysztof Kozlowski <[email protected]> wrote:
>
> The platform_device_id is not modified by the driver and core uses it as
> const.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Acked-by: Jingoo Han <[email protected]>

Best regards,
Jingoo Han

> ---
> drivers/video/fbdev/s3c-fb.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/video/fbdev/s3c-fb.c b/drivers/video/fbdev/s3c-fb.c
> index 7e3a05fc47aa..f72dd12456f9 100644
> --- a/drivers/video/fbdev/s3c-fb.c
> +++ b/drivers/video/fbdev/s3c-fb.c
> @@ -1938,7 +1938,7 @@ static struct s3c_fb_driverdata s3c_fb_data_s3c2443 = {
> },
> };
>
> -static struct platform_device_id s3c_fb_driver_ids[] = {
> +static const struct platform_device_id s3c_fb_driver_ids[] = {
> {
> .name = "s3c-fb",
> .driver_data = (unsigned long)&s3c_fb_data_64xx,
> --
> 2.1.4
>

2015-07-24 06:12:26

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id

2015-07-08 13:34 GMT+09:00 Jingoo Han <[email protected]>:
>
> On 2015. 7. 7., at PM 10:00, Krzysztof Kozlowski <[email protected]> wrote:
>>
>> The platform_device_id is not modified by the driver and core uses it as
>> const.
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> Acked-by: Jingoo Han <[email protected]>
>
> Best regards,
> Jingoo Han

Seems nobody is picking it up, so I merge it through Samsung-SoC tree.
If there are any objections please let me know.

Best regards,
Krzysztof

>
>> ---
>> drivers/video/fbdev/s3c-fb.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/video/fbdev/s3c-fb.c b/drivers/video/fbdev/s3c-fb.c
>> index 7e3a05fc47aa..f72dd12456f9 100644
>> --- a/drivers/video/fbdev/s3c-fb.c
>> +++ b/drivers/video/fbdev/s3c-fb.c
>> @@ -1938,7 +1938,7 @@ static struct s3c_fb_driverdata s3c_fb_data_s3c2443 = {
>> },
>> };
>>
>> -static struct platform_device_id s3c_fb_driver_ids[] = {
>> +static const struct platform_device_id s3c_fb_driver_ids[] = {
>> {
>> .name = "s3c-fb",
>> .driver_data = (unsigned long)&s3c_fb_data_64xx,
>> --
>> 2.1.4
>>

2015-08-20 09:16:28

by Tomi Valkeinen

[permalink] [raw]
Subject: Re: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id



On 24/07/15 09:12, Krzysztof Kozlowski wrote:
> 2015-07-08 13:34 GMT+09:00 Jingoo Han <[email protected]>:
>>
>> On 2015. 7. 7., at PM 10:00, Krzysztof Kozlowski <[email protected]> wrote:
>>>
>>> The platform_device_id is not modified by the driver and core uses it as
>>> const.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>
>> Acked-by: Jingoo Han <[email protected]>
>>
>> Best regards,
>> Jingoo Han
>
> Seems nobody is picking it up, so I merge it through Samsung-SoC tree.
> If there are any objections please let me know.

I can pick this up if you didn't already do that.

Tomi


Attachments:
signature.asc (819.00 B)
OpenPGP digital signature

2015-08-21 01:46:31

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id

On 20.08.2015 18:16, Tomi Valkeinen wrote:
>
>
> On 24/07/15 09:12, Krzysztof Kozlowski wrote:
>> 2015-07-08 13:34 GMT+09:00 Jingoo Han <[email protected]>:
>>>
>>> On 2015. 7. 7., at PM 10:00, Krzysztof Kozlowski <[email protected]> wrote:
>>>>
>>>> The platform_device_id is not modified by the driver and core uses it as
>>>> const.
>>>>
>>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>>
>>> Acked-by: Jingoo Han <[email protected]>
>>>
>>> Best regards,
>>> Jingoo Han
>>
>> Seems nobody is picking it up, so I merge it through Samsung-SoC tree.
>> If there are any objections please let me know.
>
> I can pick this up if you didn't already do that.

Could you? It would be great.
Kukjin did not pick it up so it is still waiting for some attention. :)

Best regards,
Krzysztof

2015-08-21 05:57:20

by Tomi Valkeinen

[permalink] [raw]
Subject: Re: [PATCH RESEND] video: fbdev: s3c-fb: Constify platform_device_id



On 21/08/15 04:46, Krzysztof Kozlowski wrote:
> On 20.08.2015 18:16, Tomi Valkeinen wrote:
>>
>>
>> On 24/07/15 09:12, Krzysztof Kozlowski wrote:
>>> 2015-07-08 13:34 GMT+09:00 Jingoo Han <[email protected]>:
>>>>
>>>> On 2015. 7. 7., at PM 10:00, Krzysztof Kozlowski <[email protected]> wrote:
>>>>>
>>>>> The platform_device_id is not modified by the driver and core uses it as
>>>>> const.
>>>>>
>>>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>>>
>>>> Acked-by: Jingoo Han <[email protected]>
>>>>
>>>> Best regards,
>>>> Jingoo Han
>>>
>>> Seems nobody is picking it up, so I merge it through Samsung-SoC tree.
>>> If there are any objections please let me know.
>>
>> I can pick this up if you didn't already do that.
>
> Could you? It would be great.
> Kukjin did not pick it up so it is still waiting for some attention. :)

Ok. I've queued this for 4.3.

Tomi


Attachments:
signature.asc (819.00 B)
OpenPGP digital signature