From:   Neeraj sanjay kale <neeraj.sanjaykale@nxp.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com" <edumazet@google.com>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "marcel@holtmann.org" <marcel@holtmann.org>,
        "johan.hedberg@gmail.com" <johan.hedberg@gmail.com>,
        "luiz.dentz@gmail.com" <luiz.dentz@gmail.com>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "jirislaby@kernel.org" <jirislaby@kernel.org>,
        "alok.a.tiwari@oracle.com" <alok.a.tiwari@oracle.com>,
        "hdanton@sina.com" <hdanton@sina.com>,
        "ilpo.jarvinen@linux.intel.com" <ilpo.jarvinen@linux.intel.com>,
        "leon@kernel.org" <leon@kernel.org>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        "linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
        Amitkumar Karwar <amitkumar.karwar@nxp.com>,
        Rohit Fule <rohit.fule@nxp.com>,
        Sherry Sun <sherry.sun@nxp.com>
Subject: Re: [PATCH v4 2/3] dt-bindings: net: bluetooth: Add NXP bluetooth
 support
Thread-Topic: [PATCH v4 2/3] dt-bindings: net: bluetooth: Add NXP bluetooth
 support
Thread-Index: AQHZR3YLu/OL7KREHk+ciGqO0e7Ycw==
Date:   Thu, 23 Feb 2023 11:00:30 +0000
Message-ID: <AM9PR04MB86032C020C9C7DCB5D84B6E7E7AB9@AM9PR04MB8603.eurprd04.prod.outlook.com>
References: <20230221162541.3039992-1-neeraj.sanjaykale@nxp.com>
 <20230221162541.3039992-3-neeraj.sanjaykale@nxp.com>
 <ef019382-61a8-c663-773b-21791413889d@linaro.org>
In-Reply-To: <ef019382-61a8-c663-773b-21791413889d@linaro.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?3waX7u1Qbi+s40F8uy9w29qqhYkzTIuD2o4sIAHjwKBVLPCRKeg/V8biPYsQ?=
 =?us-ascii?Q?o+9F9iId3HDxbI2WZdGrmvaC8IQPLVoPXmKt7KnMSd3lLBhJBh094J9EE0Dv?=
 =?us-ascii?Q?LRM4MRN/yU+oTdfeZfmHI3kjAhafkDSjNeKGdxv6j0G9gz13tZVy3KzOfvJe?=
 =?us-ascii?Q?TcsI9ZV7NuAMIJJ/wgXF05fUK3TuSNhwvJBjWCBiaWkVoOeV6XRK/SzPAqv3?=
 =?us-ascii?Q?ubRFHNMj3alf8Pt+2M0tHZo14HMBm4HRPSvjWM9TwQoR22hEdW19JXS1n1k5?=
 =?us-ascii?Q?BfafuP2RREIZz4TWcWzsrv+P05h2ayOl88YpdMwvD6C11SlC1xJCiP1LCbNS?=
 =?us-ascii?Q?r47MSZjY7BGpFXCml4othXXfKTHB337cPwHuaLnkxxIocuPuVLH1y9F6yB7A?=
 =?us-ascii?Q?8XLhJ4SDm6iTuCn7BjD/G2nbnJC5e0hEaAydWklhgsx0hkp3EDGCSkZBNQli?=
 =?us-ascii?Q?+kjxF+bcmY84vSyrPz41NJm+LLmWMRCm4iX/GsK/ZQ0QmljR0D6qgDFEfqbe?=
 =?us-ascii?Q?alwkgcl5XG4li2z3FZFgWlPOrq9hr2BKOhbmR3pYZguva/ZXvx/hXemaCi84?=
 =?us-ascii?Q?NjJVDL6A4as+yGVcWVewM4OsdtO9RvhSM+MpvBszWUe2tPgAHQT8OYHMw8f0?=
 =?us-ascii?Q?tgt3o6DQzQ0ZDF4y70mYfbepXAKV9r+bLpnw6hxrYMBTPde13I5iQyyy90yA?=
 =?us-ascii?Q?aoiGQY9EvKQEHmF7QcEouq5Vzb+stoeHOjxDDJNYA1eOwRKLcB31P9Y6ZUW6?=
 =?us-ascii?Q?tIrT1rj+T79/37u4FhUxfFBV2YZbLIgetXG/RVHrBF+k+k3h1BTuDMmA/RWA?=
 =?us-ascii?Q?b46OtBTxUIiyPr5y9XuVdb5FnrLh1Ce9W7U15PDEZ5oz+B2arV4yvFeHBLo+?=
 =?us-ascii?Q?DFSoaqt2XkM+8/Di8Q7znyodMkjjc9orEYoW6GiMtRac9GQEolF9eNxl8iyC?=
 =?us-ascii?Q?F4HuNQm4UOxcTSnAbsl3p+Q0tWvEYrgsF/P/ihONYgLUyrylV5jVGNF0E9CW?=
 =?us-ascii?Q?8sH8GHnE9l6HZtGKtLpZauhlkDSV+XfDgpY9GJ8AQtzI6tCLbi+tTi5FxlIO?=
 =?us-ascii?Q?KJl11Bjo+LrUcIwOgp87iGB6IpVH6xkB/wWhR1h1YPOBK50lsyMBCqR2L9qQ?=
 =?us-ascii?Q?ow4vWVh8LjzdCq/usrnyFKirvBrIZFwJxHbXhIhDRV+ey1JOoe4neoJScwGl?=
 =?us-ascii?Q?LhkVUVm8tYuNF1rlhQ6xq54wgtsco/qyGXC4zCpA/L6lYyTNKWxQZMGjYqrd?=
 =?us-ascii?Q?VmaQaWvHnaWOFe9CWcTAkAWpoWYhfFBza0OQlFQVWlupb2rYy9F5RQmdzUwe?=
 =?us-ascii?Q?fPdc0ss4xVCd2TcA4RLFaH4+oMhA3eGubnLLU0XtUDrCOfjt7hMUBcaSx/qF?=
 =?us-ascii?Q?6Gsy1xZwXcpcB8phFXIRq/W3mMhE4SW9bqwWT3SuIKWNHZc6Aa8DnvdfN5b+?=
 =?us-ascii?Q?+pJfjs5UTOaEr9G6NtL/anUD1I0mE4WKcP0jte3u8tF+U8H193K2NfKhqsAq?=
 =?us-ascii?Q?hcyZDYKa2gnOSzI+RRAS23mi1C85jtpjD4qhvWXJidqoOCeejAHLYJprrrvR?=
 =?us-ascii?Q?fqdHvgD2mUH2HYzpIWYKZmTwUz2J2ILfYSEWRGY9U26azcSLR0IQYO7XYgNf?=
 =?us-ascii?Q?Sw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB8603.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d3350c04-6c68-4442-1e95-08db158d2d94
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Feb 2023 11:00:30.1686
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: z/MawPt1QrdSii77vyFIWvOjsmiMJjg4UJdrsZH/I42cD4F04AJ3Bk7AV8TCtOaU6iwNbEKZusKJH8NRHMHnCzZayHLXnci/sATqb6PwABU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8793
Precedence: bulk
List-ID: <linux-bluetooth.vger.kernel.org>
X-Mailing-List: linux-bluetooth@vger.kernel.org

Hi Krzysztof,

Thank you for reviewing this patch.

>=20
> 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 {
>=20
> This is a friendly reminder during the review process.
>=20
> 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 reques=
ted
> changes or keep discussing them.
>=20
Changed "uart2" to "serial" in v5 patch.

Thanks,
Neeraj