Re: [PATCH v2 2/3] gpu: nova-core: fb: fix clippy::precedence warning in read_sysmem_flush_page_ga100()

[Alexandre Courbot]

On Sun Apr 5, 2026 at 9:32 PM JST, Danilo Krummrich wrote:
> On Sun Apr 5, 2026 at 1:42 PM CEST, Alexandre Courbot wrote:
>> On Sun Apr 5, 2026 at 6:35 AM JST, John Hubbard wrote:
>>> On 4/4/26 2:28 PM, John Hubbard wrote:
>>>> Clippy warns when shifts and bitwise-OR are mixed without parentheses,
>>>> because the relative precedence of << and | is easy to misread.
>>>> 
>>>> Add explicit parentheses to silence the warning.
>>>
>>> Please disregard (drop) this patch 2/3, because Miguel is removing
>>> the associated lint, so we won't need it. 
>>
>> I'm tempted to consider it anyway, because it is a readability
>> improvement. Any objection?
>
> I'd rather not. If we have the lint disabled - and I think having it enabled is
> a bit too much - then I don't want to set a precedence for unnecessary churn
> like adding or removing unnecessary braces.

Alright - dropping it for now then, thank you (and Miguel) for the input.