Hi,
On Wed, May 15, 2019 at 11:20 AM Sean Paul <[email protected]> wrote:
> > + if (IS_ERR(hdmi->default_state) &&
> > + !IS_ERR(hdmi->unwedge_state)) {
> > + dev_warn(dev,
> > + "Unwedge requires default
> > pinctrl\n");
>
> Can you downgrade this message to info or dbg? Given how rare this issue is,
> we
> probably don't want to spam everyone who is happily using dw-hdmi.
I don't think it will spam anyone, will it? It will only spam if you
_do_ specify an unwedge state and you _don't_ specify a default state.
This seems like something you'd want a pretty serious warning about
because it meant that you wanted to use unwedge but you didn't specify
it properly.
> Reviewed-by: Sean Paul <[email protected]>
Thanks!
-Doug
