Home Explore Help
Sign In
GfA
/
linux_kernel
5
0
Fork 0
Files Issues 0 Pull Requests 0 Wiki
Browse Source

media: s5p-jpeg: fix number of components macro

The value to be processed must be first masked and then shifted,
not the other way round.

Fixes: 6c96dbbc2aa9f5b4a ("[media] s5p-jpeg: add support for 5433")

Signed-off-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Andrzej Pietrasiewicz 8 years ago
parent
787d862091
commit
6c83c1d8e7
1 changed files with 1 additions and 1 deletions
Unified View Show Diff Stats
  1. 1 1
      drivers/media/platform/s5p-jpeg/jpeg-regs.h

+ 1 - 1
drivers/media/platform/s5p-jpeg/jpeg-regs.h
View File 

@@ -371,7 +371,7 @@
 
 #define EXYNOS4_NF_SHIFT			16
 
 #define EXYNOS4_NF_SHIFT			16
 
 #define EXYNOS4_NF_MASK				0xff
 
 #define EXYNOS4_NF_MASK				0xff
 
 #define EXYNOS4_NF(x)				\
 
 #define EXYNOS4_NF(x)				\
 
-	(((x) << EXYNOS4_NF_SHIFT) & EXYNOS4_NF_MASK)
 
+	(((x) & EXYNOS4_NF_MASK) << EXYNOS4_NF_SHIFT)
 
 
 
 /* JPEG quantizer table register */
 
 /* JPEG quantizer table register */
 
 #define EXYNOS4_QTBL_CONTENT(n)	(0x100 + (n) * 0x40)
 
 #define EXYNOS4_QTBL_CONTENT(n)	(0x100 + (n) * 0x40)