On Tue 13 Mar 09:44 PDT 2018, Thierry Escande wrote: > Add binding document for serial bluetooth chips using Qualcomm protocol. > > Signed-off-by: Thierry Escande <thierry.escande@xxxxxxxxxx> Looks good, just two minor things. [..] > +Required properties: > + > + - compatible: should contain one of the following: > + * "qcom,qca6174-bt" > + You're missing the "susclk" clock. > +Optional properties: > + > + - bt-disable-n-gpios: GPIO specifier, used to enable chip during probe Is this really optional? Regards, Bjorn -- 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