[PATCH 0/2] fbdev: make fb_ops const

STALE3309d

5 messages, 2 authors, 2017-08-21 · open the first message on its own page

[PATCH 0/2] fbdev: make fb_ops const

From: Bhumika Goyal <hidden>
Date: 2017-08-10 09:17:02

Make these const.

Bhumika Goyal (2):
  fbdev: matrox: make fb_ops const
  fbdev: i810: make fb_ops const

 drivers/video/fbdev/i810/i810_main.c       | 2 +-
 drivers/video/fbdev/matrox/matroxfb_base.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
1.9.1

[PATCH 1/2] fbdev: matrox: make fb_ops const

From: Bhumika Goyal <hidden>
Date: 2017-08-10 09:17:10

Make fb_ops const as it is only used during a copy operation.
Done using Coccinelle.

Signed-off-by: Bhumika Goyal <redacted>
---
 drivers/video/fbdev/matrox/matroxfb_base.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/matrox/matroxfb_base.c b/drivers/video/fbdev/matrox/matroxfb_base.c
index 00ea4e4..49ef48d 100644
--- a/drivers/video/fbdev/matrox/matroxfb_base.c
+++ b/drivers/video/fbdev/matrox/matroxfb_base.c
@@ -1198,7 +1198,7 @@ static int matroxfb_blank(int blank, struct fb_info *info)
 	return 0;
 }
 
-static struct fb_ops matroxfb_ops = {
+static const struct fb_ops matroxfb_ops = {
 	.owner =	THIS_MODULE,
 	.fb_open =	matroxfb_open,
 	.fb_release =	matroxfb_release,
-- 
1.9.1

[PATCH 2/2] fbdev: i810: make fb_ops const

From: Bhumika Goyal <hidden>
Date: 2017-08-10 09:17:16

Make the structure const as it is only used during a copy operation.
Done using Coccinelle.

Signed-off-by: Bhumika Goyal <redacted>
---
 drivers/video/fbdev/i810/i810_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/i810/i810_main.c b/drivers/video/fbdev/i810/i810_main.c
index f0a758a..d18f7b3 100644
--- a/drivers/video/fbdev/i810/i810_main.c
+++ b/drivers/video/fbdev/i810/i810_main.c
@@ -1542,7 +1542,7 @@ static int i810fb_cursor(struct fb_info *info, struct fb_cursor *cursor)
 	return 0;
 }
 
-static struct fb_ops i810fb_ops = {
+static const struct fb_ops i810fb_ops = {
 	.owner =             THIS_MODULE,
 	.fb_open =           i810fb_open,
 	.fb_release =        i810fb_release,
-- 
1.9.1

Re: [PATCH 1/2] fbdev: matrox: make fb_ops const

From: Bartlomiej Zolnierkiewicz <hidden>
Date: 2017-08-21 14:30:39

On Thursday, August 10, 2017 02:46:44 PM Bhumika Goyal wrote:
Make fb_ops const as it is only used during a copy operation.
Done using Coccinelle.

Signed-off-by: Bhumika Goyal <redacted>
Patch queued for 4.14, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

Re: [PATCH 2/2] fbdev: i810: make fb_ops const

From: Bartlomiej Zolnierkiewicz <hidden>
Date: 2017-08-21 14:32:29

On Thursday, August 10, 2017 02:46:45 PM Bhumika Goyal wrote:
Make the structure const as it is only used during a copy operation.
Done using Coccinelle.

Signed-off-by: Bhumika Goyal <redacted>
Patch queued for 4.14, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help