On 25/02/2023 00:23, 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 | 23 +++++++++++-------- > 1 file changed, 14 insertions(+), 9 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/realtek-bluetooth.yaml b/Documentation/devicetree/bindings/net/realtek-bluetooth.yaml > index 143b5667abad..f91d06d629b9 100644 > --- a/Documentation/devicetree/bindings/net/realtek-bluetooth.yaml > +++ b/Documentation/devicetree/bindings/net/realtek-bluetooth.yaml > @@ -4,24 +4,29 @@ > $id: http://devicetree.org/schemas/net/realtek-bluetooth.yaml# > $schema: http://devicetree.org/meta-schemas/core.yaml# > > -title: RTL8723BS/RTL8723CS/RTL8822CS Bluetooth > +title: RTL8723BS/RTL8723CS/RTL8821CS/RTL8822CS Bluetooth > > maintainers: > - Vasily Khoruzhick <anarsoul@xxxxxxxxx> > - Alistair Francis <alistair@xxxxxxxxxxxxx> > > description: > - RTL8723CS/RTL8723CS/RTL8822CS is WiFi + BT chip. WiFi part is connected over > - SDIO, while BT is connected over serial. It speaks H5 protocol with few > - extra commands to upload firmware and change module speed. > + RTL8723CS/RTL8723CS/RTL8821CS/RTL8822CS is a WiFi + BT chip. WiFi part > + is connected over SDIO, while BT is connected over serial. It speaks > + H5 protocol with few extra commands to upload firmware and change > + module speed. > > properties: > compatible: > - enum: > - - realtek,rtl8723bs-bt > - - realtek,rtl8723cs-bt > - - realtek,rtl8723ds-bt > - - realtek,rtl8822cs-bt > + oneOf: > + - const: realtek,rtl8723bs-bt > + - const: realtek,rtl8723cs-bt > + - const: realtek,rtl8723ds-bt > + - const: realtek,rtl8822cs-bt These still should be an enum. Why changing them? > + - items: > + - enum: > + - realtek,rtl8821cs-bt > + - const: realtek,rtl8822cs-bt Best regards, Krzysztof