From: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>
Cc: ath12k@lists.infradead.org, linux-wireless@vger.kernel.org,
Kalle Valo <kvalo@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Jeff Johnson <jjohnson@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Saravanakumar Duraisamy <quic_saradura@quicinc.com>
Subject: Re: [RFC PATCH 3/5] wifi: ath12k: add ath12k_hw_regs for IPQ5424
Date: Thu, 30 Jan 2025 11:03:21 -0800 [thread overview]
Message-ID: <b70445c0-ec0d-4a7a-b23e-3939cae87bcf@oss.qualcomm.com> (raw)
In-Reply-To: <20250130-groovy-competent-mastiff-19cc9e@krzk-bin>
On 1/30/2025 12:38 AM, Krzysztof Kozlowski wrote:
> On Thu, Jan 30, 2025 at 10:48:36AM +0530, Raj Kumar Bhagat wrote:
>> From: Saravanakumar Duraisamy <quic_saradura@quicinc.com>
>>
>> Add register addresses (ath12k_hw_regs) for new ath12k AHB based
>> WiFi device IPQ5424.
>>
>> Tested-on: IPQ5424 hw1.0 AHB WLAN.WBE.1.5-01053-QCAHKSWPL_SILICONZ-1
>> Tested-on: IPQ5332 hw1.0 AHB WLAN.WBE.1.3.1-00130-QCAHKSWPL_SILICONZ-1
>>
>> Signed-off-by: Saravanakumar Duraisamy <quic_saradura@quicinc.com>
>> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>
>> ---
>> drivers/net/wireless/ath/ath12k/hal.h | 3 +
>> drivers/net/wireless/ath/ath12k/hw.c | 87 ++++++++++++++++++++++++++-
>> 2 files changed, 89 insertions(+), 1 deletion(-)
>>
>
> That's not a separate patch. Regs do not make any sense on their own,
> so this commit makes no sense on its own.
This approach was suggested during internal review so that each addition can
be reviewed without distraction from the other pieces. None of this is
actually enabled until the final patch, so "trickling in" the pieces seems
preferable to me.
/jeff
next prev parent reply other threads:[~2025-01-30 19:03 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-30 5:18 [RFC PATCH 0/5] wifi: ath12k: Enable IPQ5424 AHB WiFi device Raj Kumar Bhagat
2025-01-30 5:18 ` [RFC PATCH 1/5] dt-bindings: net: wireless: describe the ath12k wifi device IPQ5424 Raj Kumar Bhagat
2025-01-30 18:50 ` Conor Dooley
2025-02-25 7:14 ` Raj Kumar Bhagat
2025-01-30 5:18 ` [RFC PATCH 2/5] wifi: ath12k: Add ath12k_hw_params for IPQ5424 Raj Kumar Bhagat
2025-01-30 5:18 ` [RFC PATCH 3/5] wifi: ath12k: add ath12k_hw_regs " Raj Kumar Bhagat
2025-01-30 8:38 ` Krzysztof Kozlowski
2025-01-30 19:03 ` Jeff Johnson [this message]
2025-01-30 5:18 ` [RFC PATCH 4/5] wifi: ath12k: Add CE remap hardware parameters " Raj Kumar Bhagat
2025-01-30 5:18 ` [RFC PATCH 5/5] wifi: ath12k: Enable IPQ5424 WiFi device support Raj Kumar Bhagat
2025-01-30 8:39 ` Krzysztof Kozlowski
2025-02-26 17:23 ` Raj Kumar Bhagat
2025-01-30 7:37 ` [RFC PATCH 0/5] wifi: ath12k: Enable IPQ5424 AHB WiFi device Krzysztof Kozlowski
2025-02-26 18:10 ` Raj Kumar Bhagat
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b70445c0-ec0d-4a7a-b23e-3939cae87bcf@oss.qualcomm.com \
--to=jeff.johnson@oss.qualcomm.com \
--cc=ath12k@lists.infradead.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jjohnson@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=kvalo@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=quic_rajkbhag@quicinc.com \
--cc=quic_saradura@quicinc.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox