On Tue, 10 Sep 2024, Doug Anderson <[email protected]> wrote: > Hi, > > On Tue, Sep 10, 2024 at 3:04 AM Jani Nikula <[email protected]> wrote: >> >> Building with clang, W=1, CONFIG_PM=n and CONFIG_OF_GPIO=n leads to >> warning about unused ti_sn_pwm_pin_request() and >> ti_sn_pwm_pin_release(). Fix by annotating them with __maybe_unused. >> >> See also commit 6863f5643dd7 ("kbuild: allow Clang to find unused static >> inline functions for W=1 build"). >> >> Signed-off-by: Jani Nikula <[email protected]> >> >> --- >> >> Cc: Douglas Anderson <[email protected]> >> Cc: Andrzej Hajda <[email protected]> >> Cc: Neil Armstrong <[email protected]> >> Cc: Robert Foss <[email protected]> >> Cc: Laurent Pinchart <[email protected]> >> Cc: Jonas Karlman <[email protected]> >> Cc: Jernej Skrabec <[email protected]> >> Cc: Nathan Chancellor <[email protected]> >> --- >> drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > Reviewed-by: Douglas Anderson <[email protected]> > > I'm happy to land this in drm-misc-next unless there are any extra > dependencies. Let me know. In some sense I guess this could even be > considered a "Fix", though I guess given the history of the compiler > options that might be a bit of a stretch.
drm-misc-next is fine. Agree on not considering this a fix. BR, Jani. -- Jani Nikula, Intel
