On 12/6/2024 6:25 PM, Krzysztof Kozlowski wrote: > On 06/12/2024 12:07, Raj Kumar Bhagat wrote: >> On 10/16/2024 12:27 PM, Krzysztof Kozlowski wrote: >>> On Tue, Oct 15, 2024 at 11:56:15PM +0530, Raj Kumar Bhagat wrote: >>>> Currently, Ath12k driver only supports WiFi devices that are based on >>>> PCI bus. New Ath12k device IPQ5332 is based on AHB bus. Hence, add >>>> Ath12k AHB support for IPQ5332. >>>> >>>> IPQ5332 is IEEE802.11be 2 GHz 2x2 Wifi device. To bring-up IPQ5332 >>>> device: >>>> - Add hardware parameters for IPQ5332. >>>> - CE and CMEM register address space in IPQ5332 is separate from WCSS >>>> register space. Hence, add logic to remap CE and CMEM register >>>> address. >>>> - Add support for fixed QMI firmware memory for IPQ5332. >>>> - Support userPD handling for WCSS secure PIL driver to enable ath12k >>>> AHB support. >>>> >>>> Depends-On: [PATCH V7 0/5] remove unnecessary q6 clocks >>>> Depends-On: [PATCH V2 0/4] Add new driver for WCSS secure PIL loading >>>> Link: https://lore.kernel.org/all/20240820055618.267554-1-quic_gokulsri@xxxxxxxxxxx/ >>>> Link: https://lore.kernel.org/all/20240829134021.1452711-1-quic_gokulsri@xxxxxxxxxxx/ >>> >>> These are series targetting other subsystems. I do not understand why >>> you created such dependency. It does not look needed and for sure is not >>> good: nothing here can be tested, nothing can be applied. >> >> To validate this series, the dependencies mentioned above were necessary, which >> is why they were included. > > What does it mean "validate"? You are supposed to describe how upstream > can consume this. > "validate" here means building an image, bring-up DUT in AP, STA or Mesh mode, associate Station and run bi-directional data traffic.
