[PATCH BlueZ 0/3] Prerequisites for playstation-peripheral _external_ plugin

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I proposed these changes back in September[1] but they didn't receive
a lot of comments, I know they are not very interesting to most of you,
but let me repeat the rationale behind them.

  1. It was decided to have the playstation-peripheral as an external
     plugin because it is going to be used by few people and it is not
     worth to keep it in the bluetoothd binary. Distributors could
     choose to put external plugins in their own packages so to provide
     a install-time choice instead of a compile-time choice to use the
     plugin services.

  2. external plugins can only access symbols exported as global in
     src/bluetooth.ver, like btd_* and very few others.

  3. It was suggested to add some helper functions to the bluez code
     instead of adding new global symbols to bluetooth.ver

Some notes about the implementation: I choose to pass only strings in
function signatures so that external plugins doesn't have to care about
bluez data structures too much and we can avoid exporting str2ba() and
other similar functions as global symbols in bluetooth.ver.

I hope you could comment on the patches and give me directions of how
you want me to do them if they are not OK as they are.

Once these —or equivalent— are merged then getting in the
playstation-peripheral plugin will take a very short time.

Thanks a lot,
   Antonio

[1] http://article.gmane.org/gmane.linux.bluez.kernel/16540

Changes since the RFC:
  - fixed a memory leak in btd_manager_get_default_adapter_str(), thanks to
    Daninele Forsi for looking at the code.
  - Removed unnecessary steps in btd_device_set_trusted()
  - Allow passing vendor_id_source in btd_device_set_trusted()

Antonio Ospite (3):
  manager: add a btd_manager_get_default_adapter_str() call
  storage: add a btd_store_record_string() call
  device: add a btd_device_set_trusted() call

 src/device.c  |   62 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 src/device.h  |    8 +++++++
 src/manager.c |   21 +++++++++++++++++++
 src/manager.h |    1 +
 src/storage.c |   17 +++++++++++++++
 src/storage.h |    1 +
 6 files changed, 110 insertions(+), 0 deletions(-)

-- 
Antonio Ospite
http://ao2.it

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?
--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux