linux-kernel - Re: [RFC 7/9] dt-bindings: wireless: ath11k: describe QCA6390

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87o7duzx65.fsf@kernel.org>
Date: Tue, 09 Jan 2024 11:13:54 +0200
From: Kalle Valo <kvalo@...nel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc: Bartosz Golaszewski <brgl@...ev.pl>,  "David S . Miller"
 <davem@...emloft.net>,  Eric Dumazet <edumazet@...gle.com>,  Jakub
 Kicinski <kuba@...nel.org>,  Paolo Abeni <pabeni@...hat.com>,  Rob Herring
 <robh+dt@...nel.org>,  Krzysztof Kozlowski
 <krzysztof.kozlowski+dt@...aro.org>,  Conor Dooley <conor+dt@...nel.org>,
  Bjorn Andersson <andersson@...nel.org>,  Konrad Dybcio
 <konrad.dybcio@...aro.org>,  Catalin Marinas <catalin.marinas@....com>,
  Will Deacon <will@...nel.org>,  Bjorn Helgaas <bhelgaas@...gle.com>,
  Heiko Stuebner <heiko@...ech.de>,  Jernej Skrabec
 <jernej.skrabec@...il.com>,  Chris Morgan <macromorgan@...mail.com>,
  Linus Walleij <linus.walleij@...aro.org>,  Geert Uytterhoeven
 <geert+renesas@...der.be>,  Arnd Bergmann <arnd@...db.de>,  Neil Armstrong
 <neil.armstrong@...aro.org>,  Nícolas F . R . A . Prado
 <nfraprado@...labora.com>,  Marek Szyprowski <m.szyprowski@...sung.com>,
  Peng Fan <peng.fan@....com>,  Robert Richter <rrichter@....com>,  Dan
 Williams <dan.j.williams@...el.com>,  Jonathan Cameron
 <Jonathan.Cameron@...wei.com>,  Terry Bowman <terry.bowman@....com>,
  Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@...ux.intel.com>,
  Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,  Huacai
 Chen
 <chenhuacai@...nel.org>,  Alex Elder <elder@...aro.org>,  Srini Kandagatla
 <srinivas.kandagatla@...aro.org>,  Greg Kroah-Hartman
 <gregkh@...uxfoundation.org>,  linux-wireless@...r.kernel.org,
  netdev@...r.kernel.org,  devicetree@...r.kernel.org,
  linux-kernel@...r.kernel.org,  linux-arm-msm@...r.kernel.org,
  linux-arm-kernel@...ts.infradead.org,  linux-pci@...r.kernel.org,
  Bartosz Golaszewski <bartosz.golaszewski@...aro.org>
Subject: Re: [RFC 7/9] dt-bindings: wireless: ath11k: describe QCA6390

Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> writes:

> On 04/01/2024 14:01, Bartosz Golaszewski wrote:
>
>> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k-pci.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k-pci.yaml
>> index 817f02a8b481..f584c25f4276 100644
>> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k-pci.yaml
>> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k-pci.yaml
>> @@ -16,6 +16,7 @@ description: |
>>  properties:
>>    compatible:
>>      enum:
>> +      - pci17cb,1101  # QCA6390
>>        - pci17cb,1103  # WCN6855
>>  
>>    reg:
>> @@ -27,6 +28,19 @@ properties:
>>        string to uniquely identify variant of the calibration data for designs
>>        with colliding bus and device ids
>>  
>> +  enable-gpios:
>> +    description: GPIO line enabling the ATH11K module when asserted.
>> +    maxItems: 1
>> +
>> +  vddpmu-supply:
>> +    description: VDD_PMU supply regulator handle
>> +
>> +  vddpcie1-supply:
>> +    description: VDD_PCIE1 supply regulator handle
>> +
>> +  vddpcie2-supply:
>> +    description: VDD_PCIE2 supply regulator handle
>
> Looks like these are valid only for specific variant, so you should have
> allOf:if:then disallowing other properties when not applicable. The same
> applies for existing properties which might not exist on qca6390.

And it would be good to provide more information in the commit log about
the platforms where this is used, otherwise the context might not be
clear. Or maybe document that in the bindings file, not sure.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches