On Thu, Jun 17, 2021 at 07:41:09PM +0200, Stefan Wahren wrote: > Hi Maxime, > > Am 17.06.21 um 17:25 schrieb Maxime Ripard: > > Hi Stefan, > > > > On Sat, Jun 12, 2021 at 12:04:08PM +0200, Stefan Wahren wrote: > >> Hi Maxime, > >> > >> Am 04.06.21 um 11:02 schrieb Maxime Ripard: > >>> Hi Stefan, > >>> > >>> I would assume it's due to this: > >>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/gpu/drm/vc4/vc4_hdmi.c#n1083 > >>> > >>> It pre-dates my time working on the vc4 driver so I'm not really sure > >>> what this is supposed to prevent, but my guess is that it's there to > >>> avoid someone using the audio card before we have a display detected and > >>> connected, and its capabilities known (the first and more obvious one > >>> being does it support audio in the first place). > >>> > >>> It's nothing new though, maybe it's the error printing itself that is? > >> i'm sorry, i forgot about this discussion here: > >> > >> https://lists.freedesktop.org/archives/dri-devel/2020-December/292701.html > > It looks like there's no discussion on that link, is it the link you wanted to paste? > > it was the right patch, but the discussion get lost because of turn of > the year. Next try: > > https://www.spinics.net/lists/dri-devel/msg284535.html It looks like Mark wasn't particularly on-board with that change though :/ I'm not really sure what to do with it, really. The only issue I'm seeing is that there's some logging that doesn't look too nice. Is there anything else? Maxime
Attachment:
signature.asc
Description: PGP signature
