Re: [PATCH v4 01/10] Battery: Add Battery Service Gatt Client
From: Johan Hedberg <hidden>
Date: 2012-08-16 11:39:41
Hi Chen, On Thu, Aug 16, 2012, chen.ganir@ti.com wrote:
quoted hunk ↗ jump to hunk
--- a/doc/device-api.txt +++ b/doc/device-api.txt@@ -179,3 +179,8 @@ Properties string Address [readonly] Note that this property can exhibit false-positives in the case of Bluetooth 2.1 (or newer) devices that have disabled Extended Inquiry Response support. + + array{string} Batteries [readonly] + + List of device battery object paths that represents the available + batteries on the remote device.
I don't think it's ok to pollute the Device interface or src/device.c
with profile-specific details. That should happen in profile-specific
plugins and interfaces. Since we're switching over to object manager
maybe an interface/property like this isn't needed at all? (even if it
would be needed the type should be array{object} and not array{string}
Johan