Re: [RESEND PATCH] drm/i915: do AUD_FREQ_CNTRL state save on all gen9+ platforms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 09, 2020 at 05:14:01PM +0300, Kai Vehmanen wrote:
> Hey,
> 
> On Mon, 30 Mar 2020, Kai Vehmanen wrote:
> 
> > Replace the TGL/ICL specific platform checks with a more generic check
> > using INTEL_GEN(). Fixes bug with broken audio after S3 resume on JSL
> > platforms.
> 
> I would be (gently) beaten with a stick on alsa-devel for sending this
> type of content free ping, but I still dare to seek your input on what is 
> the proper way to get attention to a patch that are seemingly forever 
> stuck on the review sideline.

And what is this?
https://patchwork.freedesktop.org/patch/347148/?series=71527&rev=1

> 
> I've sent this on 13.3., resend on 30.3.. Should I just keep on sending 
> resends and let the system work (this is the alsa-devel practise), or 
> should I start to contact potential reviewers with more direct asks?

Just ping on original patch or ping someone on irc. Resending
the same patch over and over does no good. At least my brain just
ignores anything that looks like it's just a resend w/o any clear
justification.

> 
> Tests seem to all pass and this is pretty important for anyone using JSL 
> platforms (you lose HDMI/DP audio after first S3 suspend otherwise):
> https://patchwork.freedesktop.org/series/74664/
> 
> Br, Kai

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux