On 2/14/2018 2:17 PM, Chris Wilson wrote:
Quoting Kumar, Abhijeet (2018-02-14 04:53:57)

On 2/14/2018 9:36 AM, abhijeet.ku...@intel.com wrote:

     From: Abhijeet Kumar <abhijeet.ku...@intel.com>

      sound/pci/hda/hda_codec.c | 2 +-
      1 file changed, 1 insertion(+), 1 deletion(-)

     diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
     index 8c1b07e300a8..377d5719b4cd 100644
     --- a/sound/pci/hda/hda_codec.c
     +++ b/sound/pci/hda/hda_codec.c
     @@ -2714,7 +2714,7 @@ static unsigned int hda_sync_power_state(struct 
hda_codec *codec,
             int count;

             for (count = 0;count < 500; count++) {
     -               state = snd_hda_codec_read(codec, fg, 0,
     +               state = snd_hdac_codec_read(&codec->core, fg, 0,
                                                AC_VERB_GET_POWER_STATE, 0);
                     if (state & AC_PWRST_ERROR){

Both tests are passing on hsw and bdw devices.I can conclude that none of my
Where did you run this against CI? (Due to the nature of patchwork it
will not have picked this up as a new revision.)

You can find it here https://patchwork.freedesktop.org/series/38212/.
I've reverted my patch and made my changes in hda_codec inorder to demonstrate my changes is not
breaking it.

in "ALSA: hda: Make use of core codec functions to sync power state" is "
directly" causing the regression.
As this patch series changes the previously defined sync function similar to
the latest one (the one defined
in the defaulter patch).
If you have no answer, we will apply the revert to our CI so that we do
not lose coverage.

I guess, I don't have any issue by reverting this single patch alone as i already said this patch had no functional change! It just had few optimization which  i believe we can skip for now.  :)

+Takashi to comment.


Intel-gfx mailing list

Reply via email to