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 <kbal...@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
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to