On 03/04/13 15:50, Stephen Boyd wrote: > >> + >> + evt->name = "dummy timer"; >> + evt->features = CLOCK_EVT_FEAT_PERIODIC | >> + CLOCK_EVT_FEAT_ONESHOT | >> + CLOCK_EVT_FEAT_DUMMY; >> + evt->rating = 100; >> + evt->set_mode = dummy_set_mode; >> + evt->cpumask = cpumask_of(cpu); >> + >> + clockevents_register_device(evt); >> +} >> + >> +static int __cpuinit dummy_cpu_notify(struct notifier_block *self, >> + unsigned long action, void *hcpu) >> +{ >> + if ((action & ~CPU_TASKS_FROZEN) == CPU_STARTING) >> + dummy_setup(); > There are already two dummy_setup() functions. Perhaps we can > s/dummy/dummy_broadcast/ throughout this file? > Sorry, meant to say dummy_timer not dummy_broadcast. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html