Re: [PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Tomi Valkeinen
On 19/03/14 09:47, Bo Shen wrote:
> Correct passing parameter sequence, which will avoid the id of
> fix screen info is overwritten.
> 
> Signed-off-by: Bo Shen 
> ---
>  drivers/video/atmel_lcdfb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
> index cd96162..695f28a 100644
> --- a/drivers/video/atmel_lcdfb.c
> +++ b/drivers/video/atmel_lcdfb.c
> @@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
> platform_device *pdev)
>   if (!sinfo->config)
>   goto free_info;
>  
> - strcpy(info->fix.id, sinfo->pdev->name);
>   info->flags = ATMEL_LCDFB_FBINFO_DEFAULT;
>   info->pseudo_palette = sinfo->pseudo_palette;
>   info->fbops = _lcdfb_ops;
>  
>   info->fix = atmel_lcdfb_fix;
> + strcpy(info->fix.id, sinfo->pdev->name);
>  
>   /* Enable LCDC Clocks */
>   sinfo->bus_clk = clk_get(dev, "hclk");
> 

Thanks, queued for 3.15.

 Tomi




signature.asc
Description: OpenPGP digital signature


Re: [PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Nicolas Ferre
On 19/03/2014 08:47, Bo Shen :
> Correct passing parameter sequence, which will avoid the id of
> fix screen info is overwritten.
> 
> Signed-off-by: Bo Shen 

Acked-by: Nicolas Ferre 

Thanks.

> ---
>  drivers/video/atmel_lcdfb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
> index cd96162..695f28a 100644
> --- a/drivers/video/atmel_lcdfb.c
> +++ b/drivers/video/atmel_lcdfb.c
> @@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
> platform_device *pdev)
>   if (!sinfo->config)
>   goto free_info;
>  
> - strcpy(info->fix.id, sinfo->pdev->name);
>   info->flags = ATMEL_LCDFB_FBINFO_DEFAULT;
>   info->pseudo_palette = sinfo->pseudo_palette;
>   info->fbops = _lcdfb_ops;
>  
>   info->fix = atmel_lcdfb_fix;
> + strcpy(info->fix.id, sinfo->pdev->name);
>  
>   /* Enable LCDC Clocks */
>   sinfo->bus_clk = clk_get(dev, "hclk");
> 


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Bo Shen
Correct passing parameter sequence, which will avoid the id of
fix screen info is overwritten.

Signed-off-by: Bo Shen 
---
 drivers/video/atmel_lcdfb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
index cd96162..695f28a 100644
--- a/drivers/video/atmel_lcdfb.c
+++ b/drivers/video/atmel_lcdfb.c
@@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
platform_device *pdev)
if (!sinfo->config)
goto free_info;
 
-   strcpy(info->fix.id, sinfo->pdev->name);
info->flags = ATMEL_LCDFB_FBINFO_DEFAULT;
info->pseudo_palette = sinfo->pseudo_palette;
info->fbops = _lcdfb_ops;
 
info->fix = atmel_lcdfb_fix;
+   strcpy(info->fix.id, sinfo->pdev->name);
 
/* Enable LCDC Clocks */
sinfo->bus_clk = clk_get(dev, "hclk");
-- 
1.8.5.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Bo Shen
Correct passing parameter sequence, which will avoid the id of
fix screen info is overwritten.

Signed-off-by: Bo Shen voice.s...@atmel.com
---
 drivers/video/atmel_lcdfb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
index cd96162..695f28a 100644
--- a/drivers/video/atmel_lcdfb.c
+++ b/drivers/video/atmel_lcdfb.c
@@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
platform_device *pdev)
if (!sinfo-config)
goto free_info;
 
-   strcpy(info-fix.id, sinfo-pdev-name);
info-flags = ATMEL_LCDFB_FBINFO_DEFAULT;
info-pseudo_palette = sinfo-pseudo_palette;
info-fbops = atmel_lcdfb_ops;
 
info-fix = atmel_lcdfb_fix;
+   strcpy(info-fix.id, sinfo-pdev-name);
 
/* Enable LCDC Clocks */
sinfo-bus_clk = clk_get(dev, hclk);
-- 
1.8.5.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Nicolas Ferre
On 19/03/2014 08:47, Bo Shen :
 Correct passing parameter sequence, which will avoid the id of
 fix screen info is overwritten.
 
 Signed-off-by: Bo Shen voice.s...@atmel.com

Acked-by: Nicolas Ferre nicolas.fe...@atmel.com

Thanks.

 ---
  drivers/video/atmel_lcdfb.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
 index cd96162..695f28a 100644
 --- a/drivers/video/atmel_lcdfb.c
 +++ b/drivers/video/atmel_lcdfb.c
 @@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
 platform_device *pdev)
   if (!sinfo-config)
   goto free_info;
  
 - strcpy(info-fix.id, sinfo-pdev-name);
   info-flags = ATMEL_LCDFB_FBINFO_DEFAULT;
   info-pseudo_palette = sinfo-pseudo_palette;
   info-fbops = atmel_lcdfb_ops;
  
   info-fix = atmel_lcdfb_fix;
 + strcpy(info-fix.id, sinfo-pdev-name);
  
   /* Enable LCDC Clocks */
   sinfo-bus_clk = clk_get(dev, hclk);
 


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] Video: atmel: avoid the id of fix screen info is overwritten

2014-03-19 Thread Tomi Valkeinen
On 19/03/14 09:47, Bo Shen wrote:
 Correct passing parameter sequence, which will avoid the id of
 fix screen info is overwritten.
 
 Signed-off-by: Bo Shen voice.s...@atmel.com
 ---
  drivers/video/atmel_lcdfb.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
 index cd96162..695f28a 100644
 --- a/drivers/video/atmel_lcdfb.c
 +++ b/drivers/video/atmel_lcdfb.c
 @@ -1190,12 +1190,12 @@ static int __init atmel_lcdfb_probe(struct 
 platform_device *pdev)
   if (!sinfo-config)
   goto free_info;
  
 - strcpy(info-fix.id, sinfo-pdev-name);
   info-flags = ATMEL_LCDFB_FBINFO_DEFAULT;
   info-pseudo_palette = sinfo-pseudo_palette;
   info-fbops = atmel_lcdfb_ops;
  
   info-fix = atmel_lcdfb_fix;
 + strcpy(info-fix.id, sinfo-pdev-name);
  
   /* Enable LCDC Clocks */
   sinfo-bus_clk = clk_get(dev, hclk);
 

Thanks, queued for 3.15.

 Tomi




signature.asc
Description: OpenPGP digital signature