Peter Chen writes: > On Fri, Apr 01, 2016 at 03:21:49PM +0800, Baolin Wang wrote: > + >> +static struct attribute *usb_charger_attrs[] = { >> + &dev_attr_sdp_current.attr, >> + &dev_attr_dcp_current.attr, >> + &dev_attr_cdp_current.attr, >> + &dev_attr_aca_current.attr, >> + &dev_attr_charger_type.attr, >> + &dev_attr_charger_state.attr, >> + NULL >> +}; > > The user may only care about current limit, type and state, why they > need to care what type's current limit, it is the usb charger > framework handles, the framework judge the current according to > charger type and USB state (connect/configured/suspended). it might be useful if we want to know that $this charger doesn't really give us as much current as it advertises. -- balbi