[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <31b4ca9c-892a-4ba5-b006-95deed6deceb@kernel.org>
Date: Tue, 18 Mar 2025 19:19:18 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Jeff Johnson <jeff.johnson@....qualcomm.com>,
Raj Kumar Bhagat <quic_rajkbhag@...cinc.com>, ath12k@...ts.infradead.org
Cc: Johannes Berg <johannes@...solutions.net>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Jeff Johnson <jjohnson@...nel.org>,
linux-wireless@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, Balamurugan S <quic_bselvara@...cinc.com>,
P Praneesh <quic_ppranees@...cinc.com>,
Vasanthakumar Thiagarajan <vasanthakumar.thiagarajan@....qualcomm.com>
Subject: Re: [PATCH ath-next v11 08/13] wifi: ath12k: add AHB driver support
for IPQ5332
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
Powered by blists - more mailing lists