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.

Do you have an int method() {} with a final path that returns no value ?

Tooling/compilers will definitely and rightly complain about that.
int function() {
...

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