From: Greg KH > Sent: 17 October 2022 15:10 > > On Mon, Oct 17, 2022 at 06:52:50PM +0530, Deepak R Varma wrote: > > Reformat long running computation instructions to improve code readability. > > Address following checkpatch script complaints: > > CHECK: line length of 171 exceeds 100 columns > > CHECK: line length of 113 exceeds 100 columns > > > > Signed-off-by: Deepak R Varma <drv@xxxxxxxxx> > > --- > > drivers/staging/r8188eu/core/rtw_br_ext.c | 20 +++++++++++++------- > > 1 file changed, 13 insertions(+), 7 deletions(-) > > > > diff --git a/drivers/staging/r8188eu/core/rtw_br_ext.c b/drivers/staging/r8188eu/core/rtw_br_ext.c > > index 79daf8f269d6..427da7e8ba4c 100644 > > --- a/drivers/staging/r8188eu/core/rtw_br_ext.c > > +++ b/drivers/staging/r8188eu/core/rtw_br_ext.c > > @@ -211,8 +211,10 @@ static int __nat25_network_hash(unsigned char *network_addr) > > } else if (network_addr[0] == NAT25_IPX) { > > unsigned long x; > > > > - x = network_addr[1] ^ network_addr[2] ^ network_addr[3] ^ network_addr[4] ^ > network_addr[5] ^ > > - network_addr[6] ^ network_addr[7] ^ network_addr[8] ^ network_addr[9] ^ > network_addr[10]; > > + x = network_addr[1] ^ network_addr[2] ^ network_addr[3] ^ > > Why not go out to [4] here and then you are one line shorter? and/or use a shorter variable name.... David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)