Hi,
On Wed, Sep 11, 2024 at 1:02 AM Jani Nikula <[email protected]> wrote:
>
> 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.
It's only been on the list a day but it's simple so I just landed it
to drm-misc-next:
[4/8] drm/bridge: ti-sn65dsi86: annotate ti_sn_pwm_pin_{request,
release} with __maybe_unused
commit: 868cd000c19f77e4c25ce87c47b6f951facf4394
