> -----Original Message-----
> From: Wu, Fengguang
> Sent: Thursday, September 13, 2012 11:14 AM
> To: Wang, Xingchao
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; Zhao, Yakui; Fu, Michael
> Subject: Re: [PATCH] drm/i915: HDMI - Clear Audio Enable bit for Hot Plug
> unconditionally
> 
> >     u32 enable_bits = SDVO_ENABLE;
> >
> > -   if (intel_hdmi->has_audio)
> > -           enable_bits |= SDVO_AUDIO_ENABLE;
> > +   enable_bits |= SDVO_AUDIO_ENABLE;
> 
> The two lines can be combined:
> 
>       u32 enable_bits = SDVO_ENABLE | SDVO_AUDIO_ENABLE;

Looks better, I send out V2 patch for Daniel.

-- Xingchao
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to