Re: [PATCH] drm/i915/sdvo: Fix harmless build warning

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

 



Dongdong Deng wrote:
> Removing the following harmless build warning to let compiler happy.

> @@ -1170,7 +1170,8 @@ static void intel_sdvo_mode_set(struct drm_encoder *encoder,
>  	switch (sdvo_pixel_multiply) {
>  	case 1: rate = SDVO_CLOCK_RATE_MULT_1X; break;
>  	case 2: rate = SDVO_CLOCK_RATE_MULT_2X; break;
> -	case 4: rate = SDVO_CLOCK_RATE_MULT_4X; break;
> +	case 4:
> +	default: rate = SDVO_CLOCK_RATE_MULT_4X;
>  	}
>  	if (!intel_sdvo_set_clock_rate_mult(intel_sdvo, rate))
>  		return;

Even if the default case can not trigger, is 4x a good default multiplier?

Also, shouldn't there be a break; also in the default: case?


//Peter
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux