Quoting Shi, Yang A (2018-04-19 02:22:33) > >-----Original Message----- > >From: Jani Nikula [mailto:jani.nik...@linux.intel.com] > >Sent: Wednesday, April 18, 2018 8:44 PM > >To: Chris Wilson <ch...@chris-wilson.co.uk>; Shi, Yang A > ><yang.a....@intel.com>; > >intel-gfx@lists.freedesktop.org > >Cc: Shi, Yang A <yang.a....@intel.com> > >Subject: Re: [Intel-gfx] [PATCH] drm/i915: Refine the has_audio assignment > > > >On Fri, 13 Apr 2018, Chris Wilson <ch...@chris-wilson.co.uk> wrote: > >> Quoting Yang (2018-04-13 05:06:45) > >>> From: Yang Shi <yang.a....@intel.com> > >>> > >>> Refine the has_audio assignment for dp and hdmi. > >> > >> s/Refine/Ignore the user override for/ > >> > >> Why? > > > >Yang, we'll need the explanation before we continue with review or even > >consider > >merging. We routinely ignore patches where questions go unanswered. > > > >BR, > >Jani. > > This code change is just to make driver code more readable. > If(condition) value = a else value = condition looks redundancy.
Read the patch again, AUTO != ON. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
