From: Krzysztof Kozlowski <krzk@kernel.org>
To: Jeff Johnson <jeff.johnson@oss.qualcomm.com>,
Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>,
ath12k@lists.infradead.org
Cc: Johannes Berg <johannes@sipsolutions.net>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Jeff Johnson <jjohnson@kernel.org>,
linux-wireless@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Balamurugan S <quic_bselvara@quicinc.com>,
P Praneesh <quic_ppranees@quicinc.com>,
Vasanthakumar Thiagarajan
<vasanthakumar.thiagarajan@oss.qualcomm.com>
Subject: Re: [PATCH ath-next v11 08/13] wifi: ath12k: add AHB driver support for IPQ5332
Date: Tue, 18 Mar 2025 19:19:18 +0100 [thread overview]
Message-ID: <31b4ca9c-892a-4ba5-b006-95deed6deceb@kernel.org> (raw)
In-Reply-To: <332db13c-81db-42ad-afcf-3a4262344bf2@kernel.org>
On 18/03/2025 19:16, Krzysztof Kozlowski wrote:
> On 18/03/2025 18:55, Jeff Johnson wrote:
>> On 3/18/2025 8:50 AM, Krzysztof Kozlowski wrote:
>>> On 18/03/2025 16:44, Jeff Johnson wrote:
>>>> On 3/17/2025 1:46 PM, Raj Kumar Bhagat wrote:
>>>>> + hw_rev = (enum ath12k_hw_rev)of_device_get_match_data(&pdev->dev);
>>>>
>>>> kernel test robot warns:
>>>> cast to smaller integer type 'enum ath12k_hw_rev' from 'const void *'
>>>>
>>>> looks like others have fixed this by first casting to (uintptr_t)
>>>> a few examples:
>>>>
>>> Cast via (kernel_ulong_t)
>>>
>>> But another point is that this patch at stage v11 should not have
>>> compiler warnings and it's not our tools who should point it out. Except
>>> W=1, all standard static analyzers (sparse, smatch and coccinelle) are
>>> expected to be run.
>>
>> I ran what I thought was a reasonable cross-section of builds and did not see
>> this issue. Seems this issue is only flagged with config: um-allmodconfig ??
>>
>> Guess I need to add that configuration to my builds...
>
> This should be visible on every build on 32 bit archs.
Patchset does not apply neither on next nor on on mainline, so cannot
verify... Probably another problem to solve here - some sort of dependency?
Best regards,
Krzysztof
next prev parent reply other threads:[~2025-03-18 18:19 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-17 20:46 [PATCH ath-next v11 00/13] wifi: ath12k: add Ath12k AHB driver support for IPQ5332 Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 01/13] dt-bindings: net: wireless: describe the ath12k AHB module " Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 02/13] wifi: ath12k: fix incorrect CE addresses Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 03/13] wifi: ath12k: refactor ath12k_hw_regs structure Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 04/13] wifi: ath12k: add ath12k_hw_params for IPQ5332 Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 05/13] wifi: ath12k: avoid m3 firmware download in AHB device IPQ5332 Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 06/13] wifi: ath12k: Add hw_params to remap CE register space for IPQ5332 Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 07/13] wifi: ath12k: add support for fixed QMI firmware memory Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 08/13] wifi: ath12k: add AHB driver support for IPQ5332 Raj Kumar Bhagat
2025-03-18 15:44 ` Jeff Johnson
2025-03-18 15:50 ` Krzysztof Kozlowski
2025-03-18 17:55 ` Jeff Johnson
2025-03-18 18:16 ` Krzysztof Kozlowski
2025-03-18 18:19 ` Krzysztof Kozlowski [this message]
2025-03-18 18:53 ` Jeff Johnson
2025-03-18 19:06 ` Krzysztof Kozlowski
2025-03-18 20:03 ` Jeff Johnson
2025-03-21 10:05 ` Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 09/13] wifi: ath12k: Power up root PD Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 10/13] wifi: ath12k: Register various userPD interrupts and save SMEM entries Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 11/13] wifi: ath12k: Power up userPD Raj Kumar Bhagat
2025-03-18 4:01 ` Vasanthakumar Thiagarajan
2025-03-17 20:46 ` [PATCH ath-next v11 12/13] wifi: ath12k: Power down userPD Raj Kumar Bhagat
2025-03-17 20:46 ` [PATCH ath-next v11 13/13] wifi: ath12k: enable ath12k AHB support Raj Kumar Bhagat
2025-03-18 4:02 ` Vasanthakumar Thiagarajan
2025-03-18 8:00 ` [PATCH ath-next v11 00/13] wifi: ath12k: add Ath12k AHB driver support for IPQ5332 Krzysztof Kozlowski
2025-03-21 10:07 ` 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=31b4ca9c-892a-4ba5-b006-95deed6deceb@kernel.org \
--to=krzk@kernel.org \
--cc=ath12k@lists.infradead.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jeff.johnson@oss.qualcomm.com \
--cc=jjohnson@kernel.org \
--cc=johannes@sipsolutions.net \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=quic_bselvara@quicinc.com \
--cc=quic_ppranees@quicinc.com \
--cc=quic_rajkbhag@quicinc.com \
--cc=robh@kernel.org \
--cc=vasanthakumar.thiagarajan@oss.qualcomm.com \
/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