[PATCH 10/11] Staging: bcm: Remove typedef for _LEDColors and call directly.

Kevin McKinney klmckinney1 at gmail.com
Sun Dec 16 22:08:02 UTC 2012


On Sun, Dec 16, 2012 at 09:48:44PM +0300, Dan Carpenter wrote:
> On Fri, Dec 14, 2012 at 07:26:53PM -0500, Kevin McKinney wrote:
> > This patch removes typedef for _LEDColors, and changes
> > the name of the enum to bcm_led_colors. In addition,
> > any calls to LEDColors are changed to call directly.
> > 
> 
> There aren't any calls changed...  You could probably just delete
> this enum.  :P
>

Yeah, I noticed this but when I removed this enum, I was unable to compile because
of these lines of code in led_controlc:
		switch (ucGPIOInfo[ucIndex]) {
		case RED_LED:
			GPIO_Array[RED_LED] = ucIndex;
			Adapter->gpioBitMap |= (1 << ucIndex);
			break;
		case BLUE_LED:
			GPIO_Array[BLUE_LED] = ucIndex;
			Adapter->gpioBitMap |= (1 << ucIndex);
			break;
		case YELLOW_LED:
			GPIO_Array[YELLOW_LED] = ucIndex;
			Adapter->gpioBitMap |= (1 << ucIndex);
			break;
		case GREEN_LED:
			GPIO_Array[GREEN_LED] = ucIndex;
			Adapter->gpioBitMap |= (1 << ucIndex);
			break;
		default:
			break;
		}

It appears this enum is being referenced here.

Thanks,
Kevin



More information about the devel mailing list