On 08/04/2024 15:22, Neeraj Sanjay Kale wrote: > This adds a new optional device tree property called firware-name. > > Signed-off-by: Neeraj Sanjay Kale <neeraj.sanjaykale@xxxxxxx> BTW, there is no such device as btnxpuart. Bindings are for hardware. With corrected subject: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > --- > v2: Add maxItems, simplify description, remove "nxp/". (Krzysztof) > --- > .../devicetree/bindings/net/bluetooth/nxp,88w8987-bt.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/bluetooth/nxp,88w8987-bt.yaml b/Documentation/devicetree/bindings/net/bluetooth/nxp,88w8987-bt.yaml > index f01a3988538c..6774cc4d6a9e 100644 > --- a/Documentation/devicetree/bindings/net/bluetooth/nxp,88w8987-bt.yaml > +++ b/Documentation/devicetree/bindings/net/bluetooth/nxp,88w8987-bt.yaml > @@ -31,6 +31,11 @@ properties: > This property depends on the module vendor's > configuration. > > + firmware-name: > + maxItems: 1 > + description: > + Specify firmware file name. Drop description, redundant. You did not say anything different than property is saying already. Best regards, Krzysztof
