Messages in this thread | | | From | Andy Shevchenko <> | Date | Mon, 24 Sep 2018 23:47:22 +0300 | Subject | Re: [PATCH v2 2/2] Bluetooth: btqcomsmd: Get the BD address with device_get_bd_address() |
| |
On Mon, Sep 24, 2018 at 10:37 PM Matthias Kaehlcke <mka@chromium.org> wrote: > > Use the new API to get the BD address instead of reading it directly > from the device tree. > > Also remove an unncessary pair of braces in the same area of code. >
This looks nice!
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Signed-off-by: Matthias Kaehlcke <mka@chromium.org> > --- > Changes in v2: > - pass bdaddr_t instead of byte pointer + len > --- > drivers/bluetooth/btqcomsmd.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/bluetooth/btqcomsmd.c b/drivers/bluetooth/btqcomsmd.c > index 7df3eed1ef5e..ff74d2c46991 100644 > --- a/drivers/bluetooth/btqcomsmd.c > +++ b/drivers/bluetooth/btqcomsmd.c > @@ -172,11 +172,9 @@ static int btqcomsmd_probe(struct platform_device *pdev) > /* The local-bd-address property is usually injected by the > * bootloader which has access to the allocated BD address. > */ > - if (!of_property_read_u8_array(pdev->dev.of_node, "local-bd-address", > - (u8 *)&btq->bdaddr, sizeof(bdaddr_t))) { > + if (!device_get_bd_address(&pdev->dev, &btq->bdaddr)) > dev_info(&pdev->dev, "BD address %pMR retrieved from device-tree", > &btq->bdaddr); > - } > > hdev = hci_alloc_dev(); > if (!hdev) > -- > 2.19.0.444.g18242da7ef-goog >
-- With Best Regards, Andy Shevchenko
| |