Re: [PATCH] staging: winbond: wb35reg.c Coding style fixes.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

> -void
> -Wb35Reg_Update(struct hw_data * pHwData,  u16 RegisterNo,  u32 RegisterValue)
> +void Wb35Reg_Update(struct hw_data *pHwData,  u16 RegisterNo,  u32 RegisterValue)
>  {
>  	struct wb35_reg *reg = &pHwData->reg;
>  	switch (RegisterNo) {
> -	case 0x3b0: reg->U1B0 = RegisterValue; break;
> -	case 0x3bc: reg->U1BC_LEDConfigure = RegisterValue; break;
> -	case 0x400: reg->D00_DmaControl = RegisterValue; break;
> -	case 0x800: reg->M00_MacControl = RegisterValue; break;
> -	case 0x804: reg->M04_MulticastAddress1 = RegisterValue; break;
> -	case 0x808: reg->M08_MulticastAddress2 = RegisterValue; break;
> -	case 0x824: reg->M24_MacControl = RegisterValue; break;
> -	case 0x828: reg->M28_MacControl = RegisterValue; break;
> -	case 0x82c: reg->M2C_MacControl = RegisterValue; break;
> -	case 0x838: reg->M38_MacControl = RegisterValue; break;
> -	case 0x840: reg->M40_MacControl = RegisterValue; break;
> -	case 0x844: reg->M44_MacControl = RegisterValue; break;
> -	case 0x848: reg->M48_MacControl = RegisterValue; break;
> -	case 0x84c: reg->M4C_MacStatus = RegisterValue; break;
> -	case 0x860: reg->M60_MacControl = RegisterValue; break;
> -	case 0x868: reg->M68_MacControl = RegisterValue; break;
> -	case 0x870: reg->M70_MacControl = RegisterValue; break;
> -	case 0x874: reg->M74_MacControl = RegisterValue; break;
> -	case 0x878: reg->M78_ERPInformation = RegisterValue; break;
> -	case 0x87C: reg->M7C_MacControl = RegisterValue; break;
> -	case 0x880: reg->M80_MacControl = RegisterValue; break;
> -	case 0x884: reg->M84_MacControl = RegisterValue; break;
> -	case 0x888: reg->M88_MacControl = RegisterValue; break;
> -	case 0x898: reg->M98_MacControl = RegisterValue; break;
> -	case 0x100c: reg->BB0C = RegisterValue; break;
> -	case 0x102c: reg->BB2C = RegisterValue; break;
> -	case 0x1030: reg->BB30 = RegisterValue; break;
> -	case 0x103c: reg->BB3C = RegisterValue; break;
> -	case 0x1048: reg->BB48 = RegisterValue; break;
> -	case 0x104c: reg->BB4C = RegisterValue; break;
> -	case 0x1050: reg->BB50 = RegisterValue; break;
> -	case 0x1054: reg->BB54 = RegisterValue; break;
> -	case 0x1058: reg->BB58 = RegisterValue; break;
> -	case 0x105c: reg->BB5C = RegisterValue; break;
> -	case 0x1060: reg->BB60 = RegisterValue; break;
> +	case 0x3b0:
> +		reg->U1B0 = RegisterValue;
> +		break;
> +	case 0x3bc:
> +		reg->U1BC_LEDConfigure = RegisterValue;
> +		break;
> +	case 0x400:
> +		reg->D00_DmaControl = RegisterValue;
> +		break;
> +	case 0x800:
> +		reg->M00_MacControl = RegisterValue;
> +		break;
> +	case 0x804:
> +		reg->M04_MulticastAddress1 = RegisterValue;
> +		break;
> +	case 0x808:
> +		reg->M08_MulticastAddress2 = RegisterValue;
> +		break;
> +	case 0x824:
> +		reg->M24_MacControl = RegisterValue;
> +		break;
> +	case 0x828:
> +		reg->M28_MacControl = RegisterValue;
> +		break;
> +	case 0x82c:
> +		reg->M2C_MacControl = RegisterValue;
> +		break;
> +	case 0x838:
> +		reg->M38_MacControl = RegisterValue;
> +		break;
> +	case 0x840:
> +		reg->M40_MacControl = RegisterValue;
> +		break;
> +	case 0x844:
> +		reg->M44_MacControl = RegisterValue;
> +		break;
> +	case 0x848:
> +		reg->M48_MacControl = RegisterValue;
> +		break;
> +	case 0x84c:
> +		reg->M4C_MacStatus = RegisterValue;
> +		break;
> +	case 0x860:
> +		reg->M60_MacControl = RegisterValue;
> +		break;
> +	case 0x868:
> +		reg->M68_MacControl = RegisterValue;
> +		break;
> +	case 0x870:
> +		reg->M70_MacControl = RegisterValue;
> +		break;
> +	case 0x874:
> +		reg->M74_MacControl = RegisterValue;
> +		break;
> +	case 0x878:
> +		reg->M78_ERPInformation = RegisterValue;
> +		break;
> +	case 0x87C:
> +		reg->M7C_MacControl = RegisterValue;
> +		break;
> +	case 0x880:
> +		reg->M80_MacControl = RegisterValue;
> +		break;
> +	case 0x884:
> +		reg->M84_MacControl = RegisterValue;
> +		break;
> +	case 0x888:
> +		reg->M88_MacControl = RegisterValue;
> +		break;
> +	case 0x898:
> +		reg->M98_MacControl = RegisterValue;
> +		break;
> +	case 0x100c:
> +		reg->BB0C = RegisterValue;
> +		break;
> +	case 0x102c:
> +		reg->BB2C = RegisterValue;
> +		break;
> +	case 0x1030:
> +		reg->BB30 = RegisterValue;
> +		break;
> +	case 0x103c:
> +		reg->BB3C = RegisterValue;
> +		break;
> +	case 0x1048:
> +		reg->BB48 = RegisterValue;
> +		break;
> +	case 0x104c:
> +		reg->BB4C = RegisterValue;
> +		break;
> +	case 0x1050:
> +		reg->BB50 = RegisterValue;
> +		break;
> +	case 0x1054:
> +		reg->BB54 = RegisterValue;
> +		break;
> +	case 0x1058:
> +		reg->BB58 = RegisterValue;
> +		break;
> +	case 0x105c:
> +		reg->BB5C = RegisterValue;
> +		break;
> +	case 0x1060:
> +		reg->BB60 = RegisterValue;
> +		break;
>  	}
>  }

I actually prefer the old version here. Yes, checkpatch will complain,
but so what -- it is better to have shorter function for something
this trivial.

									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux