[patch v4 2/3] staging: xgifb: vb_setmode: fix long line warnings
Greg KH
gregkh at linuxfoundation.org
Mon Jan 15 14:57:26 UTC 2018
On Fri, Jan 12, 2018 at 03:34:44PM -0700, Kenny Ballou wrote:
> Fix a number of checkpatch warnings in xgfib/vb_setmode.c about line
> length being over 80 characters.
>
> Signed-off-by: Kenny Ballou <kballou at devnulllabs.io>
> ---
> drivers/staging/xgifb/vb_setmode.c | 64 +++++++++++++++++++++++++-------------
> 1 file changed, 42 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/staging/xgifb/vb_setmode.c b/drivers/staging/xgifb/vb_setmode.c
> index e9d930f150cb..db5d721cc17c 100644
> --- a/drivers/staging/xgifb/vb_setmode.c
> +++ b/drivers/staging/xgifb/vb_setmode.c
> @@ -74,7 +74,8 @@ static void XGI_SetSeqRegs(struct vb_device_info *pVBInfo)
> /* Get SR1,2,3,4 from file */
> /* SR1 is with screen off 0x20 */
> SRdata = XGI330_StandTable.SR[i];
> - xgifb_reg_set(pVBInfo->P3c4, i + 1, SRdata); /* Set SR 1 2 3 4 */
> + /* Set SR 1 2 3 4 */
> + xgifb_reg_set(pVBInfo->P3c4, i + 1, SRdata);
> }
> }
>
> @@ -1225,9 +1226,10 @@ static void const *XGI_GetLcdPtr(struct XGI330_LCDDataTablStruct const *table,
> return table[i].DATAPTR;
> }
>
> -static struct SiS_TVData const *XGI_GetTVPtr(unsigned short ModeIdIndex,
> - unsigned short RefreshRateTableIndex,
> - struct vb_device_info *pVBInfo)
> +static struct SiS_TVData const *XGI_GetTVPtr(
> + unsigned short ModeIdIndex,
> + unsigned short RefreshRateTableIndex,
> + struct vb_device_info *pVBInfo)
> {
> unsigned short i, tempdx, tempal, modeflag;
>
> @@ -1480,14 +1482,19 @@ static void XGI_SetLVDSRegs(unsigned short ModeIdIndex,
> if (tempcx >= tempax)
> tempcx -= tempax;
>
> - xgifb_reg_set(pVBInfo->Part1Port, 0x1b, (unsigned short)(tempbx & 0xff));
> - xgifb_reg_set(pVBInfo->Part1Port, 0x1c, (unsigned short)(tempcx & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port,
> + 0x1b,
> + (unsigned short)(tempbx & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port,
> + 0x1c,
> + (unsigned short)(tempcx & 0xff));
>
> tempbx = (tempbx >> 8) & 0x07;
> tempcx = (tempcx >> 8) & 0x07;
>
> - xgifb_reg_set(pVBInfo->Part1Port, 0x1d, (unsigned short)((tempcx << 3) |
> - tempbx));
> + xgifb_reg_set(pVBInfo->Part1Port,
> + 0x1d,
> + (unsigned short)((tempcx << 3) | tempbx));
>
> tempax = pVBInfo->VT;
> tempbx = LCDPtr1->LCDVRS;
> @@ -1501,8 +1508,10 @@ static void XGI_SetLVDSRegs(unsigned short ModeIdIndex,
> if (tempcx >= tempax)
> tempcx -= tempax;
>
> - xgifb_reg_set(pVBInfo->Part1Port, 0x18, (unsigned short)(tempbx & 0xff));
> - xgifb_reg_and_or(pVBInfo->Part1Port, 0x19, ~0x0f, (unsigned short)(tempcx & 0x0f));
> + xgifb_reg_set(pVBInfo->Part1Port, 0x18,
> + (unsigned short)(tempbx & 0xff));
> + xgifb_reg_and_or(pVBInfo->Part1Port, 0x19, ~0x0f,
> + (unsigned short)(tempcx & 0x0f));
>
> tempax = ((tempbx >> 8) & 0x07) << 3;
>
> @@ -1592,16 +1601,20 @@ static void XGI_SetLVDSRegs(unsigned short ModeIdIndex,
>
> tempax = ((tempbx >> 8) & 0xff) << 3;
> tempax |= (unsigned short)((temp3 >> 8) & 0x07);
> - xgifb_reg_set(pVBInfo->Part1Port, 0x20, (unsigned short)(tempax & 0xff));
> - xgifb_reg_set(pVBInfo->Part1Port, 0x21, (unsigned short)(tempbx & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port, 0x20,
> + (unsigned short)(tempax & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port, 0x21,
> + (unsigned short)(tempbx & 0xff));
>
> temp3 >>= 16;
>
> if (modeflag & HalfDCLK)
> temp3 >>= 1;
>
> - xgifb_reg_set(pVBInfo->Part1Port, 0x22, (unsigned short)((temp3 >> 8) & 0xff));
> - xgifb_reg_set(pVBInfo->Part1Port, 0x23, (unsigned short)(temp3 & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port, 0x22,
> + (unsigned short)((temp3 >> 8) & 0xff));
> + xgifb_reg_set(pVBInfo->Part1Port, 0x23,
> + (unsigned short)(temp3 & 0xff));
> }
>
> /*
> @@ -1856,7 +1869,8 @@ void XGI_GetVBType(struct vb_device_info *pVBInfo)
> pVBInfo->VBType = tempbx;
> }
>
> -static void XGI_GetVBInfo(unsigned short ModeIdIndex, struct vb_device_info *pVBInfo)
> +static void XGI_GetVBInfo(unsigned short ModeIdIndex,
> + struct vb_device_info *pVBInfo)
> {
> unsigned short tempax, push, tempbx, temp, modeflag;
>
> @@ -1981,7 +1995,8 @@ static void XGI_GetVBInfo(unsigned short ModeIdIndex, struct vb_device_info *pVB
> pVBInfo->VBInfo = tempbx;
> }
>
> -static void XGI_GetTVInfo(unsigned short ModeIdIndex, struct vb_device_info *pVBInfo)
> +static void XGI_GetTVInfo(unsigned short ModeIdIndex,
> + struct vb_device_info *pVBInfo)
> {
> unsigned short tempbx = 0, resinfo = 0, modeflag, index1;
>
> @@ -2688,7 +2703,7 @@ static unsigned short XGI_GetColorDepth(unsigned short ModeIdIndex)
>
> static unsigned short XGI_GetOffset(unsigned short ModeNo,
> unsigned short ModeIdIndex,
> - unsigned short RefreshRateTableIndex)
> + unsigned short RefreshRateTableIndex)
> {
> unsigned short temp, colordepth, modeinfo, index, infoflag,
> ColorDepth[] = { 0x01, 0x02, 0x04 };
> @@ -3633,7 +3648,8 @@ static void XGI_SetGroup2(unsigned short ModeNo, unsigned short ModeIdIndex,
> }
> }
>
> -static void XGI_SetLCDRegs(unsigned short ModeIdIndex, struct vb_device_info *pVBInfo)
> +static void XGI_SetLCDRegs(unsigned short ModeIdIndex,
> + struct vb_device_info *pVBInfo)
> {
> unsigned short pushbx, tempax, tempbx, tempcx, temp, tempah,
> tempbh, tempch;
> @@ -4527,8 +4543,10 @@ static unsigned short XGI_GetTVPtrIndex(struct vb_device_info *pVBInfo)
> * 1 : 301B/302B/301LV/302LV
> * Description :
> */
> -static void XGI_GetTVPtrIndex2(unsigned short *tempbx, unsigned char *tempcl,
> - unsigned char *tempch, struct vb_device_info *pVBInfo)
> +static void XGI_GetTVPtrIndex2(unsigned short *tempbx,
> + unsigned char *tempcl,
> + unsigned char *tempch,
> + struct vb_device_info *pVBInfo)
> {
> *tempbx = 0;
> *tempcl = 0;
> @@ -4637,7 +4655,8 @@ static void XGI_SetLCDCap_B(unsigned short tempcx,
> (unsigned short)(((tempcx & 0x00ff) >> 6) | 0x0c));
> else
> xgifb_reg_and_or(pVBInfo->Part2Port, 0x1A, 0xE0,
> - (unsigned short)(((tempcx & 0x00ff) >> 6) | 0x18));
> + (unsigned short)(((tempcx & 0x00ff) >> 6) |
> + 0x18));
Why did you only change this line, and not the one above it?
In reality, both should be fine, please don't make things harder to
read.
thanks,
greg k-h
More information about the devel
mailing list