On 07/07/14 18:32, Jacek Anaszewski wrote: > +void exynos3250_jpeg_dec_scaling_ratio(void __iomem *regs, > + unsigned int sratio) > +{ > + switch (sratio) { > + case 1: > + sratio = EXYNOS3250_DEC_SCALE_FACTOR_8_8; > + break; > + case 2: > + sratio = EXYNOS3250_DEC_SCALE_FACTOR_4_8; > + break; > + case 4: > + sratio = EXYNOS3250_DEC_SCALE_FACTOR_2_8; > + break; > + case 8: > + sratio = EXYNOS3250_DEC_SCALE_FACTOR_1_8; > + break; > + } Missing the 'default' case ? > + writel(sratio & EXYNOS3250_DEC_SCALE_FACTOR_MASK, > + regs + EXYNOS3250_DEC_SCALING_RATIO); > +} -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html