lkml.org 
[lkml]   [2018]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v5 2/3] dt-bindings: net: bluetooth: Add qualcomm-bluetooth
    On Tue 20 Mar 23:58 HKT 2018, Marcel Holtmann wrote:
    > > Signed-off-by: Thierry Escande <thierry.escande@linaro.org>
    [..]
    > > + - clocks: clock phandle for SUSCLK_32KHZ
    >
    > if I compare this with broadcom-bluetooth.txt or ti-bluetooth.txt then
    > besides compatible, everything else is optional. The
    > nokia-bluetooth.txt has everything required, but that is also a really
    > specific platform.
    >
    > Can we be less restrictive for a QCA general purpose chip?
    >

    The way we deal with this in other bindings is that we tie such
    requirements to the compatible; i.e. we would say that qcom,qca6174-bt
    requires a clock and we would have something like qcom,qca-bt that makes
    it optional.

    The beauty of this is that the driver will tell you if you forgot to
    specify the clock when it actually is required, which saves you
    considerable amount of debugging time.


    NB. The way the bcm driver handles this is insufficient, as it treats
    any error from clk_get as "there's no clock specified". The driver
    should accept a clock not being specified, but should fail properly when
    a clock is specified but can't be acquired (e.g. due to clk_get()
    returning EPROBE_DEFER).

    > > +
    > > +Example:
    > > +
    > > +serial@7570000 {
    > > + pinctrl-names = "default", "sleep";
    > > + pinctrl-0 = <&blsp1_uart1_default>;
    > > + pinctrl-1 = <&blsp1_uart1_sleep>;
    > > +
    > > + bluetooth {
    > > + compatible = "qcom,qca6174-bt";
    > > +
    > > + enable-gpios = <&pm8994_gpios 19 GPIO_ACTIVE_HIGH>;
    > > +
    > > + pinctrl-names = "default";
    > > + pinctrl-0 = <&bt_en_pin_a>;
    >
    > This one I do not understand and you might want to shed some light
    > into why this is done that way.
    >

    This is completely generic and only relates to getting the electrical
    properties of the gpio pin set up correctly. So I would recommend that
    we omit this from the binding and example (including the pinctrl
    properties for the serial above).

    Regards,
    Bjorn

    \
     
     \ /
      Last update: 2018-03-27 00:51    [W:3.073 / U:1.116 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site