On Mon, Aug 31, 2026 at 05:34:23PM +0800, Andy Chang wrote: > On Fri, Aug 28, 2026 at 12:38 AM Conor Dooley wrote: > > > > On Thu, Aug 27, 2026 at 07:02:41PM +0800, Andy Chang wrote: > > > Add the brcm,bcm4384-bt compatible string to support the > > > Synaptics BCM4384 Bluetooth controller. > > > > Please explain why a fallback compatible is not suitable here, or > > alternatively if it is, use one. > > > > pw-bot: changes-requested > > > > Thanks, > > Conor. > Hi Conor, > A fallback compatible is not suitable here because the BCM4384 > controller requires a dedicated firmware patch file (.hcd) matching > its specific hardware ID and revision. > Falling back to an existing compatible would cause the driver to > request the wrong firmware binary and fail the controller > initialization. > Therefore, it requires a distinct compatible string. Cool, please put this info in the commit message. Thanks, Conor.