On 2014년 11월 08일 19:13, Tc, Jenny wrote: >> +ATOMIC_NOTIFIER_HEAD(psy_battery_info_notifier); > > Isn't it good to reuse the existing power_supply_notifier for this? > >> +enum battery_info_notifier_events { >> + PSY_BATT_INFO_REGISTERED, >> + PSY_BATT_INFO_UNREGISTERED, >> +}; > > If we use the power_supply_notifier, then this can be moved to > enum power_supply_notifier_events >
It doesn't use power_supply_notifier, rather than it uses newly introduced notifier for battery information. Intention of making of new notifier block here is to extinguish event from power_supply_changed which might be noisy for battery information consumer. However, If it looks wasteful code, it's not a big deal to use existed power_supply_notifier. Thanks, Jonghwa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/