diff options
| author | Nick Kralevich <nnk@google.com> | 2015-06-02 16:56:06 -0700 |
|---|---|---|
| committer | Nick Kralevich <nnk@google.com> | 2015-06-02 17:04:47 -0700 |
| commit | 2f7fa3ed77373fd49117b53fff402de0fe535385 (patch) | |
| tree | 7ec87e5165360f98be4c153283eff90a64082267 /native | |
| parent | 233241f13d5d7f841d3db80408429d4d0a528c78 (diff) | |
| download | frameworks_base-2f7fa3ed77373fd49117b53fff402de0fe535385.zip frameworks_base-2f7fa3ed77373fd49117b53fff402de0fe535385.tar.gz frameworks_base-2f7fa3ed77373fd49117b53fff402de0fe535385.tar.bz2 | |
UsbDeviceManager: Don't unnecessarily touch properties
Before setting the properties persist.sys.usb.config or sys.usb.config,
check the existing values. If the values are the same as what we'd set it
to, don't perform the set.
Any USB property set, even if setting the property to the value it already
has, triggers one of the "on property" triggers in system/core/rootdir/init.usb.rc .
The script then reconfigures the driver unnecessarily, which causes instability
and dropped connections when trying to use ADB.
Avoid this instability by not performing property sets which would have
no effect.
Bug: 21404762
Bug: 18905620
Change-Id: Id3c2543308df994a0114a0661e20ca799c2dc0e8
Diffstat (limited to 'native')
0 files changed, 0 insertions, 0 deletions