* [PATCH 0/2] fbdev: make fb_ops const
@ 2017-08-10 9:16 Bhumika Goyal
2017-08-10 9:16 ` [PATCH 1/2] fbdev: matrox: " Bhumika Goyal
2017-08-10 9:16 ` [PATCH 2/2] fbdev: i810: " Bhumika Goyal
0 siblings, 2 replies; 5+ messages in thread
From: Bhumika Goyal @ 2017-08-10 9:16 UTC (permalink / raw)
To: julia.lawall, adaplas, b.zolnierkie, linux-fbdev, linux-kernel
Cc: Bhumika Goyal
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] fbdev: matrox: make fb_ops const
2017-08-10 9:16 [PATCH 0/2] fbdev: make fb_ops const Bhumika Goyal
@ 2017-08-10 9:16 ` Bhumika Goyal
2017-08-21 14:30 ` Bartlomiej Zolnierkiewicz
2017-08-10 9:16 ` [PATCH 2/2] fbdev: i810: " Bhumika Goyal
1 sibling, 1 reply; 5+ messages in thread
From: Bhumika Goyal @ 2017-08-10 9:16 UTC (permalink / raw)
To: julia.lawall, adaplas, b.zolnierkie, linux-fbdev, linux-kernel
Cc: Bhumika Goyal
Make fb_ops const as it is only used during a copy operation.
Done using Coccinelle.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] fbdev: i810: make fb_ops const
2017-08-10 9:16 [PATCH 0/2] fbdev: make fb_ops const Bhumika Goyal
2017-08-10 9:16 ` [PATCH 1/2] fbdev: matrox: " Bhumika Goyal
@ 2017-08-10 9:16 ` Bhumika Goyal
2017-08-21 14:32 ` Bartlomiej Zolnierkiewicz
1 sibling, 1 reply; 5+ messages in thread
From: Bhumika Goyal @ 2017-08-10 9:16 UTC (permalink / raw)
To: julia.lawall, adaplas, b.zolnierkie, linux-fbdev, linux-kernel
Cc: Bhumika Goyal
Make the structure const as it is only used during a copy operation.
Done using Coccinelle.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] fbdev: matrox: make fb_ops const
2017-08-10 9:16 ` [PATCH 1/2] fbdev: matrox: " Bhumika Goyal
@ 2017-08-21 14:30 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 5+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2017-08-21 14:30 UTC (permalink / raw)
To: Bhumika Goyal; +Cc: julia.lawall, adaplas, linux-fbdev, linux-kernel
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 <bhumirks@gmail.com>
Patch queued for 4.14, thanks.
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] fbdev: i810: make fb_ops const
2017-08-10 9:16 ` [PATCH 2/2] fbdev: i810: " Bhumika Goyal
@ 2017-08-21 14:32 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 5+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2017-08-21 14:32 UTC (permalink / raw)
To: Bhumika Goyal; +Cc: julia.lawall, adaplas, linux-fbdev, linux-kernel
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 <bhumirks@gmail.com>
Patch queued for 4.14, thanks.
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-08-21 14:32 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-10 9:16 [PATCH 0/2] fbdev: make fb_ops const Bhumika Goyal
2017-08-10 9:16 ` [PATCH 1/2] fbdev: matrox: " Bhumika Goyal
2017-08-21 14:30 ` Bartlomiej Zolnierkiewicz
2017-08-10 9:16 ` [PATCH 2/2] fbdev: i810: " Bhumika Goyal
2017-08-21 14:32 ` Bartlomiej Zolnierkiewicz
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).