* [PATCH] Staging: xgifb: XGI_main_26.c: Fixed over 80 column characters
@ 2018-03-18 11:06 Pratik Jain
2018-03-18 15:51 ` Joe Perches
0 siblings, 1 reply; 3+ messages in thread
From: Pratik Jain @ 2018-03-18 11:06 UTC (permalink / raw)
To: arnaud.patard; +Cc: trivial, linux-kernel, Pratik Jain
Fixed coding style issues detected by checkpatch.pl
Signed-off-by: Pratik Jain <pratik.jain0509@gmail.com>
---
drivers/staging/xgifb/XGI_main_26.c | 23 +++++++++++++++++------
1 file changed, 17 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c
index 10107de0119a..d4d289f6fe87 100644
--- a/drivers/staging/xgifb/XGI_main_26.c
+++ b/drivers/staging/xgifb/XGI_main_26.c
@@ -770,7 +770,9 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info)
xgifb_reg_and(vb->Part2Port, 0x3a, 0x1f);
if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
- xgifb_reg_and(vb->Part2Port, 0x30, 0xdf);
+ xgifb_reg_and(vb->Part2Port,
+ 0x30,
+ 0xdf);
} else if (xgifb_info->TV_plug
== TVPLUG_COMPOSITE) {
@@ -826,11 +828,15 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info)
xgifb_reg_and(vb->Part2Port, 0x3A, 0x1F);
if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
- xgifb_reg_and(vb->Part2Port, 0x30, 0xDF);
+ xgifb_reg_and(vb->Part2Port,
+ 0x30,
+ 0xDF);
} else if (xgifb_info->TV_plug
== TVPLUG_COMPOSITE) {
- xgifb_reg_or(vb->Part2Port, 0x30, 0x20);
+ xgifb_reg_or(vb->Part2Port,
+ 0x30,
+ 0x20);
switch (xgifb_info->video_width) {
case 640:
@@ -1024,7 +1030,9 @@ static int XGIfb_do_set_var(struct fb_var_screeninfo *var, int isactive,
xgifb_info->XGI310_AccelDepth = 0x00010000;
#if defined(__BIG_ENDIAN)
cr_data = xgifb_reg_get(vb->P3d4, 0x4D);
- xgifb_reg_set(vb->P3d4, 0x4D, ((cr_data & 0xE0) | 0x0B));
+ xgifb_reg_set(vb->P3d4,
+ 0x4D,
+ ((cr_data & 0xE0) | 0x0B));
#endif
xgifb_info->video_cmap_len = 16;
break;
@@ -1034,7 +1042,9 @@ static int XGIfb_do_set_var(struct fb_var_screeninfo *var, int isactive,
xgifb_info->video_cmap_len = 16;
#if defined(__BIG_ENDIAN)
cr_data = xgifb_reg_get(vb->P3d4, 0x4D);
- xgifb_reg_set(vb->P3d4, 0x4D, ((cr_data & 0xE0) | 0x15));
+ xgifb_reg_set(vb->P3d4,
+ 0x4D,
+ ((cr_data & 0xE0) | 0x15));
#endif
break;
default:
@@ -1894,7 +1904,8 @@ static int xgifb_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
xgifb_info->refresh_rate = refresh_rate;
if (xgifb_info->refresh_rate == 0)
xgifb_info->refresh_rate = 60;
- if (XGIfb_search_refresh_rate(xgifb_info, xgifb_info->refresh_rate) == 0) {
+ if (XGIfb_search_refresh_rate(xgifb_info, xgifb_info->refresh_rate)
+ == 0) {
xgifb_info->rate_idx = 1;
xgifb_info->refresh_rate = 60;
}
--
2.16.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging: xgifb: XGI_main_26.c: Fixed over 80 column characters
2018-03-18 11:06 [PATCH] Staging: xgifb: XGI_main_26.c: Fixed over 80 column characters Pratik Jain
@ 2018-03-18 15:51 ` Joe Perches
2018-03-18 16:02 ` Pratik Jain
0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2018-03-18 15:51 UTC (permalink / raw)
To: Pratik Jain, arnaud.patard; +Cc: trivial, linux-kernel
On Sun, 2018-03-18 at 16:36 +0530, Pratik Jain wrote:
> Fixed coding style issues detected by checkpatch.pl
Hi. Rather than just following what checkpatch says,
try to look for opportunities to improve the code a bit.
Instead of wrapping, try unwrapping by unindenting blocks.
> diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c
> []
> @@ -770,7 +770,9 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info)
> xgifb_reg_and(vb->Part2Port, 0x3a, 0x1f);
>
> if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
> - xgifb_reg_and(vb->Part2Port, 0x30, 0xdf);
> + xgifb_reg_and(vb->Part2Port,
> + 0x30,
> + 0xdf);
>
> } else if (xgifb_info->TV_plug
> == TVPLUG_COMPOSITE) {
For instance, these tests could return early and
so unindent the blocks and could be:
---
drivers/staging/xgifb/XGI_main_26.c | 225 ++++++++++++++----------------------
1 file changed, 86 insertions(+), 139 deletions(-)
diff --git a/drivers/staging/xgifb/XGI_main_26.c b/drivers/staging/xgifb/XGI_main_26.c
index 10107de0119a..73e389b8156f 100644
--- a/drivers/staging/xgifb/XGI_main_26.c
+++ b/drivers/staging/xgifb/XGI_main_26.c
@@ -697,6 +697,7 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info)
struct vb_device_info *vb = &xgifb_info->dev_info;
u8 reg;
unsigned char doit = 1;
+ int filter_tb;
if (xgifb_info->video_bpp == 8) {
/*
@@ -735,162 +736,108 @@ static void XGIfb_post_setmode(struct xgifb_video_info *xgifb_info)
xgifb_reg_and(vb->P3c4, IND_SIS_RAMDAC_CONTROL, ~0x04);
- if (xgifb_info->display2 == XGIFB_DISP_TV &&
- xgifb_info->hasVB == HASVB_301) {
- reg = xgifb_reg_get(vb->Part4Port, 0x01);
+ if (xgifb_info->display2 != XGIFB_DISP_TV ||
+ xgifb_info->hasVB != HASVB_301)
+ return;
- if (reg < 0xB0) { /* Set filter for XGI301 */
- int filter_tb;
+ reg = xgifb_reg_get(vb->Part4Port, 0x01);
+ if (reg >= 0xB0)
+ return;
+
+ /* Set filter for XGI301 */
+
+ switch (xgifb_info->video_width) {
+ case 320:
+ filter_tb = (xgifb_info->TV_type == TVMODE_NTSC) ? 4 : 12;
+ break;
+ case 640:
+ filter_tb = (xgifb_info->TV_type == TVMODE_NTSC) ? 5 : 13;
+ break;
+ case 720:
+ filter_tb = (xgifb_info->TV_type == TVMODE_NTSC) ? 6 : 14;
+ break;
+ case 800:
+ filter_tb = (xgifb_info->TV_type == TVMODE_NTSC) ? 7 : 15;
+ break;
+ default:
+ filter_tb = 0;
+ filter = -1;
+ break;
+ }
+ xgifb_reg_or(vb->Part1Port, SIS_CRT2_WENABLE_315, 0x01);
+
+ if (xgifb_info->TV_type == TVMODE_NTSC) {
+ xgifb_reg_and(vb->Part2Port, 0x3a, 0x1f);
+
+ if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
+ xgifb_reg_and(vb->Part2Port, 0x30, 0xdf);
+
+ } else if (xgifb_info->TV_plug == TVPLUG_COMPOSITE) {
+ xgifb_reg_or(vb->Part2Port, 0x30, 0x20);
switch (xgifb_info->video_width) {
- case 320:
- filter_tb = (xgifb_info->TV_type ==
- TVMODE_NTSC) ? 4 : 12;
- break;
case 640:
- filter_tb = (xgifb_info->TV_type ==
- TVMODE_NTSC) ? 5 : 13;
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xEB);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0x04);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x25);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0x18);
break;
case 720:
- filter_tb = (xgifb_info->TV_type ==
- TVMODE_NTSC) ? 6 : 14;
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xEE);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0x0C);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x22);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0x08);
break;
case 800:
- filter_tb = (xgifb_info->TV_type ==
- TVMODE_NTSC) ? 7 : 15;
- break;
- default:
- filter_tb = 0;
- filter = -1;
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xEB);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0x15);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x25);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0xF6);
break;
}
- xgifb_reg_or(vb->Part1Port, SIS_CRT2_WENABLE_315, 0x01);
+ }
- if (xgifb_info->TV_type == TVMODE_NTSC) {
- xgifb_reg_and(vb->Part2Port, 0x3a, 0x1f);
-
- if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
- xgifb_reg_and(vb->Part2Port, 0x30, 0xdf);
-
- } else if (xgifb_info->TV_plug
- == TVPLUG_COMPOSITE) {
- xgifb_reg_or(vb->Part2Port, 0x30, 0x20);
-
- switch (xgifb_info->video_width) {
- case 640:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xEB);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0x04);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x25);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0x18);
- break;
- case 720:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xEE);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0x0C);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x22);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0x08);
- break;
- case 800:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xEB);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0x15);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x25);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0xF6);
- break;
- }
- }
+ } else if (xgifb_info->TV_type == TVMODE_PAL) {
+ xgifb_reg_and(vb->Part2Port, 0x3A, 0x1F);
- } else if (xgifb_info->TV_type == TVMODE_PAL) {
- xgifb_reg_and(vb->Part2Port, 0x3A, 0x1F);
-
- if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
- xgifb_reg_and(vb->Part2Port, 0x30, 0xDF);
-
- } else if (xgifb_info->TV_plug
- == TVPLUG_COMPOSITE) {
- xgifb_reg_or(vb->Part2Port, 0x30, 0x20);
-
- switch (xgifb_info->video_width) {
- case 640:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xF1);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0xF7);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x1F);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0x32);
- break;
- case 720:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xF3);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0x00);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x1D);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0x20);
- break;
- case 800:
- xgifb_reg_set(vb->Part2Port,
- 0x35,
- 0xFC);
- xgifb_reg_set(vb->Part2Port,
- 0x36,
- 0xFB);
- xgifb_reg_set(vb->Part2Port,
- 0x37,
- 0x14);
- xgifb_reg_set(vb->Part2Port,
- 0x38,
- 0x2A);
- break;
- }
- }
- }
+ if (xgifb_info->TV_plug == TVPLUG_SVIDEO) {
+ xgifb_reg_and(vb->Part2Port, 0x30, 0xDF);
- if ((filter >= 0) && (filter <= 7)) {
- const u8 *f = XGI_TV_filter[filter_tb].filter[filter];
+ } else if (xgifb_info->TV_plug == TVPLUG_COMPOSITE) {
+ xgifb_reg_or(vb->Part2Port, 0x30, 0x20);
- pr_debug("FilterTable[%d]-%d: %*ph\n",
- filter_tb, filter, 4, f);
- xgifb_reg_set(vb->Part2Port, 0x35, f[0]);
- xgifb_reg_set(vb->Part2Port, 0x36, f[1]);
- xgifb_reg_set(vb->Part2Port, 0x37, f[2]);
- xgifb_reg_set(vb->Part2Port, 0x38, f[3]);
+ switch (xgifb_info->video_width) {
+ case 640:
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xF1);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0xF7);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x1F);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0x32);
+ break;
+ case 720:
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xF3);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0x00);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x1D);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0x20);
+ break;
+ case 800:
+ xgifb_reg_set(vb->Part2Port, 0x35, 0xFC);
+ xgifb_reg_set(vb->Part2Port, 0x36, 0xFB);
+ xgifb_reg_set(vb->Part2Port, 0x37, 0x14);
+ xgifb_reg_set(vb->Part2Port, 0x38, 0x2A);
+ break;
}
}
}
+
+ if ((filter >= 0) && (filter <= 7)) {
+ const u8 *f = XGI_TV_filter[filter_tb].filter[filter];
+
+ pr_debug("FilterTable[%d]-%d: %*ph\n", filter_tb, filter, 4, f);
+ xgifb_reg_set(vb->Part2Port, 0x35, f[0]);
+ xgifb_reg_set(vb->Part2Port, 0x36, f[1]);
+ xgifb_reg_set(vb->Part2Port, 0x37, f[2]);
+ xgifb_reg_set(vb->Part2Port, 0x38, f[3]);
+ }
}
static int XGIfb_do_set_var(struct fb_var_screeninfo *var, int isactive,
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging: xgifb: XGI_main_26.c: Fixed over 80 column characters
2018-03-18 15:51 ` Joe Perches
@ 2018-03-18 16:02 ` Pratik Jain
0 siblings, 0 replies; 3+ messages in thread
From: Pratik Jain @ 2018-03-18 16:02 UTC (permalink / raw)
To: Joe Perches; +Cc: arnaud.patard, trivial, linux-kernel
Thanks for the advice. I will work on it.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-03-18 16:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-03-18 11:06 [PATCH] Staging: xgifb: XGI_main_26.c: Fixed over 80 column characters Pratik Jain
2018-03-18 15:51 ` Joe Perches
2018-03-18 16:02 ` Pratik Jain
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox