Re: [PATCH v4 2/3] dt-bindings: net: bluetooth: Add NXP bluetooth support

From: Neeraj sanjay kale
Date: Thu Feb 23 2023 - 06:00:40 EST


Hi Krzysztof,

Thank you for reviewing this patch.

>
> I think list of compatibles changed... now they are nxp,88w8987-bt, so
> shouldn't the filename be "nxp,88w8987-bt.yaml"?
Updated file name.

> > +examples:
> > + - |
> > + uart2 {
>
> This is a friendly reminder during the review process.
>
> It seems my previous comments were not fully addressed. Maybe my
> feedback got lost between the quotes, maybe you just forgot to apply it.
> Please go back to the previous discussion and either implement all requested
> changes or keep discussing them.
>
Changed "uart2" to "serial" in v5 patch.

Thanks,
Neeraj