* [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used
@ 2014-07-05 12:28 Rickard Strandqvist
2014-07-05 12:29 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Rickard Strandqvist @ 2014-07-05 12:28 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
Cc: Rickard Strandqvist, Rickard Strandqvist, Jingoo Han,
Laurent Pinchart, Rob Clark, Daniel Vetter, linux-fbdev,
linux-kernel
From: Rickard Strandqvist <rickard.strandqvist@sonymobile.com>
Variable ar assigned a value that is never used.
I have also removed all the code that thereby serves no purpose.
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
---
drivers/video/fbdev/mb862xx/mb862xxfb_accel.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
index fe92eed..aa778ba 100644
--- a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
+++ b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
@@ -28,7 +28,6 @@
static void mb862xxfb_write_fifo(u32 count, u32 *data, struct fb_info *info)
{
- struct mb862xxfb_par *par = info->par;
static u32 free;
u32 total = 0;
@@ -188,7 +187,6 @@ static void mb86290fb_imageblit16(u32 *cmd, u16 step, u16 dx, u16 dy,
static void mb86290fb_imageblit(struct fb_info *info,
const struct fb_image *image)
{
- int mdr;
u32 *cmd = NULL;
void (*cmdfn) (u32 *, u16, u16, u16, u16, u16, u32, u32,
const struct fb_image *, struct fb_info *) = NULL;
@@ -200,7 +198,6 @@ static void mb86290fb_imageblit(struct fb_info *info,
u16 dx = image->dx, dy = image->dy;
int x2, y2, vxres, vyres;
- mdr = (GDC_ROP_COPY << 9);
x2 = image->dx + image->width;
y2 = image->dy + image->height;
vxres = info->var.xres_virtual;
@@ -309,8 +306,6 @@ static void mb86290fb_fillrect(struct fb_info *info,
void mb862xxfb_init_accel(struct fb_info *info, int xres)
{
- struct mb862xxfb_par *par = info->par;
-
if (info->var.bits_per_pixel = 32) {
info->fbops->fb_fillrect = cfb_fillrect;
info->fbops->fb_copyarea = cfb_copyarea;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: Cleaning up variable that is never used
2014-07-05 12:28 [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used Rickard Strandqvist
@ 2014-07-05 12:29 ` Rickard Strandqvist
2014-07-06 18:39 ` [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: " Rickard Strandqvist
2014-07-06 18:40 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
2 siblings, 0 replies; 4+ messages in thread
From: Rickard Strandqvist @ 2014-07-05 12:29 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
Cc: Rickard Strandqvist, Rickard Strandqvist, Grant Likely,
Rob Herring, Jingoo Han, Daniel Vetter, Anatolij Gustschin,
Laurent Pinchart, Sachin Kamat, linux-fbdev, linux-kernel,
devicetree
From: Rickard Strandqvist <rickard.strandqvist@sonymobile.com>
Variable ar assigned a value that is never used.
I have also removed all the code that thereby serves no purpose.
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
---
drivers/video/fbdev/mb862xx/mb862xxfbdrv.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
index 0cd4c33..24be945 100644
--- a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
+++ b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
@@ -273,7 +273,6 @@ static int mb862xxfb_set_par(struct fb_info *fbi)
static int mb862xxfb_pan(struct fb_var_screeninfo *var,
struct fb_info *info)
{
- struct mb862xxfb_par *par = info->par;
unsigned long reg;
reg = pack(var->yoffset, var->xoffset);
@@ -286,7 +285,6 @@ static int mb862xxfb_pan(struct fb_var_screeninfo *var,
static int mb862xxfb_blank(int mode, struct fb_info *fbi)
{
- struct mb862xxfb_par *par = fbi->par;
unsigned long reg;
dev_dbg(fbi->dev, "blank mode=%d\n", mode);
@@ -548,7 +546,6 @@ static ssize_t mb862xxfb_show_dispregs(struct device *dev,
struct device_attribute *attr, char *buf)
{
struct fb_info *fbi = dev_get_drvdata(dev);
- struct mb862xxfb_par *par = fbi->par;
char *ptr = buf;
unsigned int reg;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used
2014-07-05 12:28 [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used Rickard Strandqvist
2014-07-05 12:29 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
@ 2014-07-06 18:39 ` Rickard Strandqvist
2014-07-06 18:40 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
2 siblings, 0 replies; 4+ messages in thread
From: Rickard Strandqvist @ 2014-07-06 18:39 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
Cc: Rickard Strandqvist, Jingoo Han, Laurent Pinchart, Rob Clark,
linux-fbdev, linux-kernel
Variable ar assigned a value that is never used.
I have also removed all the code that thereby serves no purpose.
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
---
drivers/video/fbdev/mb862xx/mb862xxfb_accel.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
index fe92eed..aa778ba 100644
--- a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
+++ b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c
@@ -28,7 +28,6 @@
static void mb862xxfb_write_fifo(u32 count, u32 *data, struct fb_info *info)
{
- struct mb862xxfb_par *par = info->par;
static u32 free;
u32 total = 0;
@@ -188,7 +187,6 @@ static void mb86290fb_imageblit16(u32 *cmd, u16 step, u16 dx, u16 dy,
static void mb86290fb_imageblit(struct fb_info *info,
const struct fb_image *image)
{
- int mdr;
u32 *cmd = NULL;
void (*cmdfn) (u32 *, u16, u16, u16, u16, u16, u32, u32,
const struct fb_image *, struct fb_info *) = NULL;
@@ -200,7 +198,6 @@ static void mb86290fb_imageblit(struct fb_info *info,
u16 dx = image->dx, dy = image->dy;
int x2, y2, vxres, vyres;
- mdr = (GDC_ROP_COPY << 9);
x2 = image->dx + image->width;
y2 = image->dy + image->height;
vxres = info->var.xres_virtual;
@@ -309,8 +306,6 @@ static void mb86290fb_fillrect(struct fb_info *info,
void mb862xxfb_init_accel(struct fb_info *info, int xres)
{
- struct mb862xxfb_par *par = info->par;
-
if (info->var.bits_per_pixel = 32) {
info->fbops->fb_fillrect = cfb_fillrect;
info->fbops->fb_copyarea = cfb_copyarea;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: Cleaning up variable that is never used
2014-07-05 12:28 [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used Rickard Strandqvist
2014-07-05 12:29 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
2014-07-06 18:39 ` [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: " Rickard Strandqvist
@ 2014-07-06 18:40 ` Rickard Strandqvist
2 siblings, 0 replies; 4+ messages in thread
From: Rickard Strandqvist @ 2014-07-06 18:40 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard, Tomi Valkeinen
Cc: Rickard Strandqvist, Grant Likely, Rob Herring, Jingoo Han,
Daniel Vetter, Anatolij Gustschin, Sachin Kamat, linux-fbdev,
linux-kernel, devicetree
Variable ar assigned a value that is never used.
I have also removed all the code that thereby serves no purpose.
This was found using a static code analysis program called cppcheck
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
---
drivers/video/fbdev/mb862xx/mb862xxfbdrv.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
index 0cd4c33..24be945 100644
--- a/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
+++ b/drivers/video/fbdev/mb862xx/mb862xxfbdrv.c
@@ -273,7 +273,6 @@ static int mb862xxfb_set_par(struct fb_info *fbi)
static int mb862xxfb_pan(struct fb_var_screeninfo *var,
struct fb_info *info)
{
- struct mb862xxfb_par *par = info->par;
unsigned long reg;
reg = pack(var->yoffset, var->xoffset);
@@ -286,7 +285,6 @@ static int mb862xxfb_pan(struct fb_var_screeninfo *var,
static int mb862xxfb_blank(int mode, struct fb_info *fbi)
{
- struct mb862xxfb_par *par = fbi->par;
unsigned long reg;
dev_dbg(fbi->dev, "blank mode=%d\n", mode);
@@ -548,7 +546,6 @@ static ssize_t mb862xxfb_show_dispregs(struct device *dev,
struct device_attribute *attr, char *buf)
{
struct fb_info *fbi = dev_get_drvdata(dev);
- struct mb862xxfb_par *par = fbi->par;
char *ptr = buf;
unsigned int reg;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-07-06 18:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-05 12:28 [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: Cleaning up variable that is never used Rickard Strandqvist
2014-07-05 12:29 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
2014-07-06 18:39 ` [PATCH] video: fbdev: mb862xx: mb862xxfb_accel.c: " Rickard Strandqvist
2014-07-06 18:40 ` [PATCH] video: fbdev: mb862xx: mb862xxfbdrv.c: " Rickard Strandqvist
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox