Re: [PATCH v2 2/4] phy: rockchip-samsung-dcphy: factor MIPI D-PHY power on/off into helpers
From: Bryan O'Donoghue
Date: Fri Aug 07 2026 - 05:09:39 EST
On 06/08/2026 20:31, Sebastian Reichel wrote:
This looks a bit funny.int function() {
Do you have an int method() {} with a final path that returns no value ?
Tooling/compilers will definitely and rightly complain about that.
...
switch(foo) {
case 42:
return 1;
default:
return 0;
}
/* cannot be reached */
}
Greetings,
-- Sebastian
Hmm.
The LLM tells me there is no compiler version that will generate a warning for this.
Entertaining overdone too
"Across compilers that will ever see this code: yes, and I can now show it rather than assert it — I installed the spread. gcc 12, 13, 14 and clang 18, from -Wall -Wextra -Wpedantic upward: clean. Then the decisive one: clang -Weverything — every diagnostic clang possesses, a strict superset of any CI configuration that can exist — on Sebastian's exact function: zero output."
Meh
---
bod