[PATCH 1/5] introduce-charging-led-behaviour.patch

[Holger Freyther]

nice.

> +		case PMU_EVT_CHARGER_IDLE:
> +			/* printk(KERN_ERR"PMU_EVT_CHARGER_IDLE\n"); */
> +			neo1973_gpb_setpin(GTA02_GPIO_AUX_LED, 0);
> +			break;
> +		case PMU_EVT_CHARGER_ACTIVE:
> +			/* printk(KERN_ERR"PMU_EVT_CHARGER_ACTIVE\n"); */
> +			neo1973_gpb_setpin(GTA02_GPIO_AUX_LED, 1);
> +			break;

commented out code :)




> +			mutex_lock(&pcf->working_lock_nobat);
> +			if (!pcf->working_nobat) {
> +				pcf->usb_removal_count_nobat =
> +							pcf->usb_removal_count;
> +
> +				if (!schedule_work(&pcf->work_nobat))
> +					DEBUGPC("failed to schedule nobat\n");

okay, when will schedule_work actually call our work function and from which 
context? Is it guranteed that we have unlocked the mutex or can unlock the 
mutex?