On Thu, Mar 16, 2023 at 08:55:13AM +0100, Krzysztof Kozlowski wrote: > On 15/03/2023 18:36, Chris Morgan wrote: > > From: Chris Morgan <macromorgan@xxxxxxxxxxx> > > > > Add compatible string for RTL8821CS for existing Realtek Bluetooth > > driver. > > > > Signed-off-by: Chris Morgan <macromorgan@xxxxxxxxxxx> > > --- > > .../bindings/net/realtek-bluetooth.yaml | 24 ++++++++++++------- > > 1 file changed, 15 insertions(+), 9 deletions(-) > > > > This is a friendly reminder during the review process. > > It looks like you received a tag and forgot to add it. I removed it on purpose and noted it in the cover letter because I changed the bindings. I apologize if that's not correct. I changed the fallback from "realtek,rtl8822cs-bt" to "realtek,rtl8723bs-bt". From the driver perspective that causes it to disable runtime power management, but otherwise it's identical to V4. Thank you. > > If you do not know the process, here is a short explanation: > Please add Acked-by/Reviewed-by/Tested-by tags when posting new > versions. However, there's no need to repost patches *only* to add the > tags. The upstream maintainer will do that for acks received on the > version they apply. > > https://elixir.bootlin.com/linux/v5.17/source/Documentation/process/submitting-patches.rst#L540 > > If a tag was not added on purpose, please state why and what changed. > > > Best regards, > Krzysztof >