Re: [PATCH v2 2/4] phy: rockchip-samsung-dcphy: factor MIPI D-PHY power on/off into helpers

From
Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Date
2026-08-07 09:09:41
Also in
linux-devicetree, linux-phy, linux-rockchip, lkml
Reply to
Sebastian Reichel, 2026-08-06
On 06/08/2026 20:31, Sebastian Reichel wrote:
quoted
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
Thread (16 messages) 16 messages, 6 authors, last reply 2026-08-10
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help