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
signature.asc
Description: PGP signature
