[PATCH RFC] drm/vc4: hdmi: Avoid ASoC error messages on startup

[Mark Brown]

On Fri, Jan 15, 2021 at 07:14:37PM +0100, Maxime Ripard wrote:
> On Wed, Jan 13, 2021 at 11:42:23AM +0000, Mark Brown wrote:
> > On Wed, Jan 13, 2021 at 10:19:57AM +0100, Maxime Ripard wrote:

> > > I'd like to get Mark's opinion before merging though

> > I'm not sure what the question is here?  I get CCed on a bunch of not
> > obviously relevant DRM patches so there's a good chance I've just
> > deleted some relevnat discussion.

> The patch is question is here:
> https://lore.kernel.org/dri-devel/1609256210-19863-1-git-send-email-stefan.wahren@i2se.com/

> In particular, I'm not so sure whether it's fine to return -EPROBE_DEFER
> in the startup hook.

I wouldn't expect that to do anything useful and IIRC that error code
will end up in userspace which isn't good.  If the driver wants to defer
probe it should defer it from the probe() routine, after the driver has
been instantiated I'm not sure what the expectation would be.  In
general a driver should acquire all resources it needs when probing.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20210115/fd1c1450/attachment.sig>