Hi Hemant/Johan: On Thu, Apr 26, 2012 at 6:32 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Hemant, > > On Wed, Apr 25, 2012, Hemant Gupta wrote: >> This patch stores the address type while storing the primary services >> of LE Device which is later used for re-creating device from primary >> service database. >> --- >> src/adapter.c | 10 ++++++++-- >> src/device.c | 9 +++++++-- >> 2 files changed, 15 insertions(+), 4 deletions(-) > > Would it make more sense (and I'm not asking just you but also the > INdTers) to have the address type as part of the key instead of the > value. After all it's basically an extension to the address and that way > tightly bound to it. Maybe something like the following: > > 11:22:33:44:55:66|1 <value1> > aa:bb:cc:dd:ee:ff|0 <value2> > > The same should then also be considered for the longtermkeys file. > > Johan I agree if we append the address type for all LE/GATT storage files(not only for primary and longtermkeys). Probably we will have minor problems between BlueZ releases, but IMO it is acceptable since LE it is not officially supported. btw: the separator commonly used in the storage files is "#" instead of "|". BR, Claudio -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
