* [PATCH v3 0/2] staging: media: zoran: aligned the code.
@ 2021-04-10 15:33 Mitali Borkar
2021-04-10 15:33 ` [PATCH v3 1/2] media: zoran: add spaces around '<<' Mitali Borkar
2021-04-10 15:34 ` [PATCH v3 2/2] staging: media: zoran: align code appropriately Mitali Borkar
0 siblings, 2 replies; 5+ messages in thread
From: Mitali Borkar @ 2021-04-10 15:33 UTC (permalink / raw)
To: clabbe, mchehab, gregkh
Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel, mitali_s
This patch fix the alignment of code in proper manner
Changes from v2:- Aligned the code using tabs.
Changes from v1:- Aligned the code using tabs and readjusted the
comments line.
Mitali Borkar (2):
media: zoran: add spaces around '<<'
staging: media: zoran: align code appropriately
drivers/staging/media/zoran/zr36057.h | 108 +++++++++++++-------------
1 file changed, 54 insertions(+), 54 deletions(-)
--
2.30.2
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 1/2] media: zoran: add spaces around '<<'
2021-04-10 15:33 [PATCH v3 0/2] staging: media: zoran: aligned the code Mitali Borkar
@ 2021-04-10 15:33 ` Mitali Borkar
2021-04-13 15:02 ` Hans Verkuil
2021-04-10 15:34 ` [PATCH v3 2/2] staging: media: zoran: align code appropriately Mitali Borkar
1 sibling, 1 reply; 5+ messages in thread
From: Mitali Borkar @ 2021-04-10 15:33 UTC (permalink / raw)
To: clabbe, mchehab, gregkh
Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel, mitali_s
Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
---
Changes from v2:- No changes.
Changes from v1:- NO changes.
drivers/staging/media/zoran/zr36057.h | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/media/zoran/zr36057.h b/drivers/staging/media/zoran/zr36057.h
index 71b651add35a..a2a75fd9f535 100644
--- a/drivers/staging/media/zoran/zr36057.h
+++ b/drivers/staging/media/zoran/zr36057.h
@@ -30,13 +30,13 @@
#define ZR36057_VFESPFR_HOR_DCM 14
#define ZR36057_VFESPFR_VER_DCM 8
#define ZR36057_VFESPFR_DISP_MODE 6
-#define ZR36057_VFESPFR_YUV422 (0<<3)
-#define ZR36057_VFESPFR_RGB888 (1<<3)
-#define ZR36057_VFESPFR_RGB565 (2<<3)
-#define ZR36057_VFESPFR_RGB555 (3<<3)
-#define ZR36057_VFESPFR_ERR_DIF (1<<2)
-#define ZR36057_VFESPFR_PACK24 (1<<1)
-#define ZR36057_VFESPFR_LITTLE_ENDIAN (1<<0)
+#define ZR36057_VFESPFR_YUV422 (0 << 3)
+#define ZR36057_VFESPFR_RGB888 (1 << 3)
+#define ZR36057_VFESPFR_RGB565 (2 << 3)
+#define ZR36057_VFESPFR_RGB555 (3 << 3)
+#define ZR36057_VFESPFR_ERR_DIF (1 << 2)
+#define ZR36057_VFESPFR_PACK24 (1 << 1)
+#define ZR36057_VFESPFR_LITTLE_ENDIAN (1 << 0)
#define ZR36057_VDTR 0x00c /* Video Display "Top" Register */
--
2.30.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v3 2/2] staging: media: zoran: align code appropriately
2021-04-10 15:33 [PATCH v3 0/2] staging: media: zoran: aligned the code Mitali Borkar
2021-04-10 15:33 ` [PATCH v3 1/2] media: zoran: add spaces around '<<' Mitali Borkar
@ 2021-04-10 15:34 ` Mitali Borkar
2021-04-13 14:47 ` Hans Verkuil
1 sibling, 1 reply; 5+ messages in thread
From: Mitali Borkar @ 2021-04-10 15:34 UTC (permalink / raw)
To: clabbe, mchehab, gregkh
Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel, mitali_s
Aligned the code properly by using tabs to make code neater and improve
readability.
Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
---
drivers/staging/media/zoran/zr36057.h | 108 +++++++++++++-------------
1 file changed, 54 insertions(+), 54 deletions(-)
diff --git a/drivers/staging/media/zoran/zr36057.h b/drivers/staging/media/zoran/zr36057.h
index 198d344a8879..db42f445248f 100644
--- a/drivers/staging/media/zoran/zr36057.h
+++ b/drivers/staging/media/zoran/zr36057.h
@@ -13,28 +13,28 @@
/* Zoran ZR36057 registers */
/* Video Front End, Horizontal Configuration Register */
-#define ZR36057_VFEHCR 0x000
+#define ZR36057_VFEHCR 0x000
#define ZR36057_VFEHCR_HS_POL BIT(30)
-#define ZR36057_VFEHCR_H_START 10
-#define ZR36057_VFEHCR_H_END 0
-#define ZR36057_VFEHCR_HMASK 0x3ff
+#define ZR36057_VFEHCR_H_START 10
+#define ZR36057_VFEHCR_H_END 0
+#define ZR36057_VFEHCR_HMASK 0x3ff
/* Video Front End, Vertical Configuration Register */
-#define ZR36057_VFEVCR 0x004
+#define ZR36057_VFEVCR 0x004
#define ZR36057_VFEVCR_VS_POL BIT(30)
-#define ZR36057_VFEVCR_V_START 10
-#define ZR36057_VFEVCR_V_END 0
-#define ZR36057_VFEVCR_VMASK 0x3ff
+#define ZR36057_VFEVCR_V_START 10
+#define ZR36057_VFEVCR_V_END 0
+#define ZR36057_VFEVCR_VMASK 0x3ff
/* Video Front End, Scaler and Pixel Format Register */
#define ZR36057_VFESPFR 0x008
#define ZR36057_VFESPFR_EXT_FL BIT(26)
#define ZR36057_VFESPFR_TOP_FIELD BIT(25)
#define ZR36057_VFESPFR_VCLK_POL BIT(24)
-#define ZR36057_VFESPFR_H_FILTER 21
+#define ZR36057_VFESPFR_H_FILTER 21
#define ZR36057_VFESPFR_HOR_DCM 14
#define ZR36057_VFESPFR_VER_DCM 8
-#define ZR36057_VFESPFR_DISP_MODE 6
+#define ZR36057_VFESPFR_DISP_MODE 6
#define ZR36057_VFESPFR_YUV422 (0 << 3)
#define ZR36057_VFESPFR_RGB888 BIT(3)
#define ZR36057_VFESPFR_RGB565 (2 << 3)
@@ -44,34 +44,34 @@
#define ZR36057_VFESPFR_LITTLE_ENDIAN BIT(0)
/* Video Display "Top" Register */
-#define ZR36057_VDTR 0x00c
+#define ZR36057_VDTR 0x00c
/* Video Display "Bottom" Register */
-#define ZR36057_VDBR 0x010
+#define ZR36057_VDBR 0x010
/* Video Stride, Status, and Frame Grab Register */
-#define ZR36057_VSSFGR 0x014
-#define ZR36057_VSSFGR_DISP_STRIDE 16
+#define ZR36057_VSSFGR 0x014
+#define ZR36057_VSSFGR_DISP_STRIDE 16
#define ZR36057_VSSFGR_VID_OVF BIT(8)
#define ZR36057_VSSFGR_SNAP_SHOT BIT(1)
#define ZR36057_VSSFGR_FRAME_GRAB BIT(0)
/* Video Display Configuration Register */
-#define ZR36057_VDCR 0x018
+#define ZR36057_VDCR 0x018
#define ZR36057_VDCR_VID_EN BIT(31)
-#define ZR36057_VDCR_MIN_PIX 24
+#define ZR36057_VDCR_MIN_PIX 24
#define ZR36057_VDCR_TRITON BIT(24)
#define ZR36057_VDCR_VID_WIN_HT 12
-#define ZR36057_VDCR_VID_WIN_WID 0
+#define ZR36057_VDCR_VID_WIN_WID 0
/* Masking Map "Top" Register */
-#define ZR36057_MMTR 0x01c
+#define ZR36057_MMTR 0x01c
/* Masking Map "Bottom" Register */
-#define ZR36057_MMBR 0x020
+#define ZR36057_MMBR 0x020
/* Overlay Control Register */
-#define ZR36057_OCR 0x024
+#define ZR36057_OCR 0x024
#define ZR36057_OCR_OVL_ENABLE BIT(15)
#define ZR36057_OCR_MASK_STRIDE 0
@@ -83,42 +83,42 @@
#define ZR36057_GPPGCR1 0x02c
/* MPEG Code Source Address Register */
-#define ZR36057_MCSAR 0x030
+#define ZR36057_MCSAR 0x030
/* MPEG Code Transfer Control Register */
-#define ZR36057_MCTCR 0x034
+#define ZR36057_MCTCR 0x034
#define ZR36057_MCTCR_COD_TIME BIT(30)
#define ZR36057_MCTCR_C_EMPTY BIT(29)
#define ZR36057_MCTCR_C_FLUSH BIT(28)
-#define ZR36057_MCTCR_COD_GUEST_ID 20
-#define ZR36057_MCTCR_COD_GUEST_REG 16
+#define ZR36057_MCTCR_COD_GUEST_ID 20
+#define ZR36057_MCTCR_COD_GUEST_REG 16
/* MPEG Code Memory Pointer Register */
-#define ZR36057_MCMPR 0x038
+#define ZR36057_MCMPR 0x038
/* Interrupt Status Register */
-#define ZR36057_ISR 0x03c
+#define ZR36057_ISR 0x03c
#define ZR36057_ISR_GIRQ1 BIT(30)
#define ZR36057_ISR_GIRQ0 BIT(29)
-#define ZR36057_ISR_COD_REP_IRQ BIT(28)
+#define ZR36057_ISR_COD_REP_IRQ BIT(28)
#define ZR36057_ISR_JPEG_REP_IRQ BIT(27)
/* Interrupt Control Register */
-#define ZR36057_ICR 0x040
+#define ZR36057_ICR 0x040
#define ZR36057_ICR_GIRQ1 BIT(30)
#define ZR36057_ICR_GIRQ0 BIT(29)
-#define ZR36057_ICR_COD_REP_IRQ BIT(28)
+#define ZR36057_ICR_COD_REP_IRQ BIT(28)
#define ZR36057_ICR_JPEG_REP_IRQ BIT(27)
#define ZR36057_ICR_INT_PIN_EN BIT(24)
/* I2C Bus Register */
-#define ZR36057_I2CBR 0x044
+#define ZR36057_I2CBR 0x044
#define ZR36057_I2CBR_SDA BIT(1)
#define ZR36057_I2CBR_SCL BIT(0)
/* JPEG Mode and Control */
-#define ZR36057_JMC 0x100
-#define ZR36057_JMC_JPG BIT(31)
+#define ZR36057_JMC 0x100
+#define ZR36057_JMC_JPG BIT(31)
#define ZR36057_JMC_JPG_EXP_MODE (0 << 29)
#define ZR36057_JMC_JPG_CMP_MODE BIT(29)
#define ZR36057_JMC_MJPG_EXP_MODE (2 << 29)
@@ -132,56 +132,56 @@
#define ZR36057_JMC_STLL_LIT_ENDIAN BIT(0)
/* JPEG Process Control */
-#define ZR36057_JPC 0x104
+#define ZR36057_JPC 0x104
#define ZR36057_JPC_P_RESET BIT(7)
-#define ZR36057_JPC_COD_TRNS_EN BIT(5)
+#define ZR36057_JPC_COD_TRNS_EN BIT(5)
#define ZR36057_JPC_ACTIVE BIT(0)
/* Vertical Sync Parameters */
-#define ZR36057_VSP 0x108
-#define ZR36057_VSP_VSYNC_SIZE 16
-#define ZR36057_VSP_FRM_TOT 0
+#define ZR36057_VSP 0x108
+#define ZR36057_VSP_VSYNC_SIZE 16
+#define ZR36057_VSP_FRM_TOT 0
/* Horizontal Sync Parameters */
-#define ZR36057_HSP 0x10c
+#define ZR36057_HSP 0x10c
#define ZR36057_HSP_HSYNC_START 16
-#define ZR36057_HSP_LINE_TOT 0
+#define ZR36057_HSP_LINE_TOT 0
/* Field Horizontal Active Portion */
-#define ZR36057_FHAP 0x110
-#define ZR36057_FHAP_NAX 16
-#define ZR36057_FHAP_PAX 0
+#define ZR36057_FHAP 0x110
+#define ZR36057_FHAP_NAX 16
+#define ZR36057_FHAP_PAX 0
/* Field Vertical Active Portion */
-#define ZR36057_FVAP 0x114
-#define ZR36057_FVAP_NAY 16
-#define ZR36057_FVAP_PAY 0
+#define ZR36057_FVAP 0x114
+#define ZR36057_FVAP_NAY 16
+#define ZR36057_FVAP_PAY 0
/* Field Process Parameters */
-#define ZR36057_FPP 0x118
+#define ZR36057_FPP 0x118
#define ZR36057_FPP_ODD_EVEN BIT(0)
/* JPEG Code Base Address */
-#define ZR36057_JCBA 0x11c
+#define ZR36057_JCBA 0x11c
/* JPEG Code FIFO Threshold */
-#define ZR36057_JCFT 0x120
+#define ZR36057_JCFT 0x120
/* JPEG Codec Guest ID */
-#define ZR36057_JCGI 0x124
-#define ZR36057_JCGI_JPE_GUEST_ID 4
-#define ZR36057_JCGI_JPE_GUEST_REG 0
+#define ZR36057_JCGI 0x124
+#define ZR36057_JCGI_JPE_GUEST_ID 4
+#define ZR36057_JCGI_JPE_GUEST_REG 0
/* GuestBus Control Register (2) */
-#define ZR36057_GCR2 0x12c
+#define ZR36057_GCR2 0x12c
/* Post Office Register */
-#define ZR36057_POR 0x200
+#define ZR36057_POR 0x200
#define ZR36057_POR_PO_PEN BIT(25)
#define ZR36057_POR_PO_TIME BIT(24)
#define ZR36057_POR_PO_DIR BIT(23)
/* "Still" Transfer Register */
-#define ZR36057_STR 0x300
+#define ZR36057_STR 0x300
#endif
--
2.30.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 2/2] staging: media: zoran: align code appropriately
2021-04-10 15:34 ` [PATCH v3 2/2] staging: media: zoran: align code appropriately Mitali Borkar
@ 2021-04-13 14:47 ` Hans Verkuil
0 siblings, 0 replies; 5+ messages in thread
From: Hans Verkuil @ 2021-04-13 14:47 UTC (permalink / raw)
To: Mitali Borkar, clabbe, mchehab, gregkh
Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel, mitali_s
On 10/04/2021 17:34, Mitali Borkar wrote:
> Aligned the code properly by using tabs to make code neater and improve
> readability.
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> ---
> drivers/staging/media/zoran/zr36057.h | 108 +++++++++++++-------------
> 1 file changed, 54 insertions(+), 54 deletions(-)
>
> diff --git a/drivers/staging/media/zoran/zr36057.h b/drivers/staging/media/zoran/zr36057.h
> index 198d344a8879..db42f445248f 100644
> --- a/drivers/staging/media/zoran/zr36057.h
> +++ b/drivers/staging/media/zoran/zr36057.h
> @@ -13,28 +13,28 @@
> /* Zoran ZR36057 registers */
>
> /* Video Front End, Horizontal Configuration Register */
> -#define ZR36057_VFEHCR 0x000
> +#define ZR36057_VFEHCR 0x000
> #define ZR36057_VFEHCR_HS_POL BIT(30)
Same issue as the other patches: this patch depends on other local changes
so won't apply to mainline code...
Regards,
Hans
> -#define ZR36057_VFEHCR_H_START 10
> -#define ZR36057_VFEHCR_H_END 0
> -#define ZR36057_VFEHCR_HMASK 0x3ff
> +#define ZR36057_VFEHCR_H_START 10
> +#define ZR36057_VFEHCR_H_END 0
> +#define ZR36057_VFEHCR_HMASK 0x3ff
>
> /* Video Front End, Vertical Configuration Register */
> -#define ZR36057_VFEVCR 0x004
> +#define ZR36057_VFEVCR 0x004
> #define ZR36057_VFEVCR_VS_POL BIT(30)
> -#define ZR36057_VFEVCR_V_START 10
> -#define ZR36057_VFEVCR_V_END 0
> -#define ZR36057_VFEVCR_VMASK 0x3ff
> +#define ZR36057_VFEVCR_V_START 10
> +#define ZR36057_VFEVCR_V_END 0
> +#define ZR36057_VFEVCR_VMASK 0x3ff
>
> /* Video Front End, Scaler and Pixel Format Register */
> #define ZR36057_VFESPFR 0x008
> #define ZR36057_VFESPFR_EXT_FL BIT(26)
> #define ZR36057_VFESPFR_TOP_FIELD BIT(25)
> #define ZR36057_VFESPFR_VCLK_POL BIT(24)
> -#define ZR36057_VFESPFR_H_FILTER 21
> +#define ZR36057_VFESPFR_H_FILTER 21
> #define ZR36057_VFESPFR_HOR_DCM 14
> #define ZR36057_VFESPFR_VER_DCM 8
> -#define ZR36057_VFESPFR_DISP_MODE 6
> +#define ZR36057_VFESPFR_DISP_MODE 6
> #define ZR36057_VFESPFR_YUV422 (0 << 3)
> #define ZR36057_VFESPFR_RGB888 BIT(3)
> #define ZR36057_VFESPFR_RGB565 (2 << 3)
> @@ -44,34 +44,34 @@
> #define ZR36057_VFESPFR_LITTLE_ENDIAN BIT(0)
>
> /* Video Display "Top" Register */
> -#define ZR36057_VDTR 0x00c
> +#define ZR36057_VDTR 0x00c
>
> /* Video Display "Bottom" Register */
> -#define ZR36057_VDBR 0x010
> +#define ZR36057_VDBR 0x010
>
> /* Video Stride, Status, and Frame Grab Register */
> -#define ZR36057_VSSFGR 0x014
> -#define ZR36057_VSSFGR_DISP_STRIDE 16
> +#define ZR36057_VSSFGR 0x014
> +#define ZR36057_VSSFGR_DISP_STRIDE 16
> #define ZR36057_VSSFGR_VID_OVF BIT(8)
> #define ZR36057_VSSFGR_SNAP_SHOT BIT(1)
> #define ZR36057_VSSFGR_FRAME_GRAB BIT(0)
>
> /* Video Display Configuration Register */
> -#define ZR36057_VDCR 0x018
> +#define ZR36057_VDCR 0x018
> #define ZR36057_VDCR_VID_EN BIT(31)
> -#define ZR36057_VDCR_MIN_PIX 24
> +#define ZR36057_VDCR_MIN_PIX 24
> #define ZR36057_VDCR_TRITON BIT(24)
> #define ZR36057_VDCR_VID_WIN_HT 12
> -#define ZR36057_VDCR_VID_WIN_WID 0
> +#define ZR36057_VDCR_VID_WIN_WID 0
>
> /* Masking Map "Top" Register */
> -#define ZR36057_MMTR 0x01c
> +#define ZR36057_MMTR 0x01c
>
> /* Masking Map "Bottom" Register */
> -#define ZR36057_MMBR 0x020
> +#define ZR36057_MMBR 0x020
>
> /* Overlay Control Register */
> -#define ZR36057_OCR 0x024
> +#define ZR36057_OCR 0x024
> #define ZR36057_OCR_OVL_ENABLE BIT(15)
> #define ZR36057_OCR_MASK_STRIDE 0
>
> @@ -83,42 +83,42 @@
> #define ZR36057_GPPGCR1 0x02c
>
> /* MPEG Code Source Address Register */
> -#define ZR36057_MCSAR 0x030
> +#define ZR36057_MCSAR 0x030
>
> /* MPEG Code Transfer Control Register */
> -#define ZR36057_MCTCR 0x034
> +#define ZR36057_MCTCR 0x034
> #define ZR36057_MCTCR_COD_TIME BIT(30)
> #define ZR36057_MCTCR_C_EMPTY BIT(29)
> #define ZR36057_MCTCR_C_FLUSH BIT(28)
> -#define ZR36057_MCTCR_COD_GUEST_ID 20
> -#define ZR36057_MCTCR_COD_GUEST_REG 16
> +#define ZR36057_MCTCR_COD_GUEST_ID 20
> +#define ZR36057_MCTCR_COD_GUEST_REG 16
>
> /* MPEG Code Memory Pointer Register */
> -#define ZR36057_MCMPR 0x038
> +#define ZR36057_MCMPR 0x038
>
> /* Interrupt Status Register */
> -#define ZR36057_ISR 0x03c
> +#define ZR36057_ISR 0x03c
> #define ZR36057_ISR_GIRQ1 BIT(30)
> #define ZR36057_ISR_GIRQ0 BIT(29)
> -#define ZR36057_ISR_COD_REP_IRQ BIT(28)
> +#define ZR36057_ISR_COD_REP_IRQ BIT(28)
> #define ZR36057_ISR_JPEG_REP_IRQ BIT(27)
>
> /* Interrupt Control Register */
> -#define ZR36057_ICR 0x040
> +#define ZR36057_ICR 0x040
> #define ZR36057_ICR_GIRQ1 BIT(30)
> #define ZR36057_ICR_GIRQ0 BIT(29)
> -#define ZR36057_ICR_COD_REP_IRQ BIT(28)
> +#define ZR36057_ICR_COD_REP_IRQ BIT(28)
> #define ZR36057_ICR_JPEG_REP_IRQ BIT(27)
> #define ZR36057_ICR_INT_PIN_EN BIT(24)
>
> /* I2C Bus Register */
> -#define ZR36057_I2CBR 0x044
> +#define ZR36057_I2CBR 0x044
> #define ZR36057_I2CBR_SDA BIT(1)
> #define ZR36057_I2CBR_SCL BIT(0)
>
> /* JPEG Mode and Control */
> -#define ZR36057_JMC 0x100
> -#define ZR36057_JMC_JPG BIT(31)
> +#define ZR36057_JMC 0x100
> +#define ZR36057_JMC_JPG BIT(31)
> #define ZR36057_JMC_JPG_EXP_MODE (0 << 29)
> #define ZR36057_JMC_JPG_CMP_MODE BIT(29)
> #define ZR36057_JMC_MJPG_EXP_MODE (2 << 29)
> @@ -132,56 +132,56 @@
> #define ZR36057_JMC_STLL_LIT_ENDIAN BIT(0)
>
> /* JPEG Process Control */
> -#define ZR36057_JPC 0x104
> +#define ZR36057_JPC 0x104
> #define ZR36057_JPC_P_RESET BIT(7)
> -#define ZR36057_JPC_COD_TRNS_EN BIT(5)
> +#define ZR36057_JPC_COD_TRNS_EN BIT(5)
> #define ZR36057_JPC_ACTIVE BIT(0)
>
> /* Vertical Sync Parameters */
> -#define ZR36057_VSP 0x108
> -#define ZR36057_VSP_VSYNC_SIZE 16
> -#define ZR36057_VSP_FRM_TOT 0
> +#define ZR36057_VSP 0x108
> +#define ZR36057_VSP_VSYNC_SIZE 16
> +#define ZR36057_VSP_FRM_TOT 0
>
> /* Horizontal Sync Parameters */
> -#define ZR36057_HSP 0x10c
> +#define ZR36057_HSP 0x10c
> #define ZR36057_HSP_HSYNC_START 16
> -#define ZR36057_HSP_LINE_TOT 0
> +#define ZR36057_HSP_LINE_TOT 0
>
> /* Field Horizontal Active Portion */
> -#define ZR36057_FHAP 0x110
> -#define ZR36057_FHAP_NAX 16
> -#define ZR36057_FHAP_PAX 0
> +#define ZR36057_FHAP 0x110
> +#define ZR36057_FHAP_NAX 16
> +#define ZR36057_FHAP_PAX 0
>
> /* Field Vertical Active Portion */
> -#define ZR36057_FVAP 0x114
> -#define ZR36057_FVAP_NAY 16
> -#define ZR36057_FVAP_PAY 0
> +#define ZR36057_FVAP 0x114
> +#define ZR36057_FVAP_NAY 16
> +#define ZR36057_FVAP_PAY 0
>
> /* Field Process Parameters */
> -#define ZR36057_FPP 0x118
> +#define ZR36057_FPP 0x118
> #define ZR36057_FPP_ODD_EVEN BIT(0)
>
> /* JPEG Code Base Address */
> -#define ZR36057_JCBA 0x11c
> +#define ZR36057_JCBA 0x11c
>
> /* JPEG Code FIFO Threshold */
> -#define ZR36057_JCFT 0x120
> +#define ZR36057_JCFT 0x120
>
> /* JPEG Codec Guest ID */
> -#define ZR36057_JCGI 0x124
> -#define ZR36057_JCGI_JPE_GUEST_ID 4
> -#define ZR36057_JCGI_JPE_GUEST_REG 0
> +#define ZR36057_JCGI 0x124
> +#define ZR36057_JCGI_JPE_GUEST_ID 4
> +#define ZR36057_JCGI_JPE_GUEST_REG 0
>
> /* GuestBus Control Register (2) */
> -#define ZR36057_GCR2 0x12c
> +#define ZR36057_GCR2 0x12c
>
> /* Post Office Register */
> -#define ZR36057_POR 0x200
> +#define ZR36057_POR 0x200
> #define ZR36057_POR_PO_PEN BIT(25)
> #define ZR36057_POR_PO_TIME BIT(24)
> #define ZR36057_POR_PO_DIR BIT(23)
>
> /* "Still" Transfer Register */
> -#define ZR36057_STR 0x300
> +#define ZR36057_STR 0x300
>
> #endif
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3 1/2] media: zoran: add spaces around '<<'
2021-04-10 15:33 ` [PATCH v3 1/2] media: zoran: add spaces around '<<' Mitali Borkar
@ 2021-04-13 15:02 ` Hans Verkuil
0 siblings, 0 replies; 5+ messages in thread
From: Hans Verkuil @ 2021-04-13 15:02 UTC (permalink / raw)
To: Mitali Borkar, clabbe, mchehab, gregkh
Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel, mitali_s
Missing commit message.
You used to have one in v1 of this patch!
Regards,
Hans
On 10/04/2021 17:33, Mitali Borkar wrote:
> Signed-off-by: Mitali Borkar <mitaliborkar810@gmail.com>
> ---
> Changes from v2:- No changes.
> Changes from v1:- NO changes.
>
> drivers/staging/media/zoran/zr36057.h | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/staging/media/zoran/zr36057.h b/drivers/staging/media/zoran/zr36057.h
> index 71b651add35a..a2a75fd9f535 100644
> --- a/drivers/staging/media/zoran/zr36057.h
> +++ b/drivers/staging/media/zoran/zr36057.h
> @@ -30,13 +30,13 @@
> #define ZR36057_VFESPFR_HOR_DCM 14
> #define ZR36057_VFESPFR_VER_DCM 8
> #define ZR36057_VFESPFR_DISP_MODE 6
> -#define ZR36057_VFESPFR_YUV422 (0<<3)
> -#define ZR36057_VFESPFR_RGB888 (1<<3)
> -#define ZR36057_VFESPFR_RGB565 (2<<3)
> -#define ZR36057_VFESPFR_RGB555 (3<<3)
> -#define ZR36057_VFESPFR_ERR_DIF (1<<2)
> -#define ZR36057_VFESPFR_PACK24 (1<<1)
> -#define ZR36057_VFESPFR_LITTLE_ENDIAN (1<<0)
> +#define ZR36057_VFESPFR_YUV422 (0 << 3)
> +#define ZR36057_VFESPFR_RGB888 (1 << 3)
> +#define ZR36057_VFESPFR_RGB565 (2 << 3)
> +#define ZR36057_VFESPFR_RGB555 (3 << 3)
> +#define ZR36057_VFESPFR_ERR_DIF (1 << 2)
> +#define ZR36057_VFESPFR_PACK24 (1 << 1)
> +#define ZR36057_VFESPFR_LITTLE_ENDIAN (1 << 0)
>
> #define ZR36057_VDTR 0x00c /* Video Display "Top" Register */
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-04-13 15:02 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-10 15:33 [PATCH v3 0/2] staging: media: zoran: aligned the code Mitali Borkar
2021-04-10 15:33 ` [PATCH v3 1/2] media: zoran: add spaces around '<<' Mitali Borkar
2021-04-13 15:02 ` Hans Verkuil
2021-04-10 15:34 ` [PATCH v3 2/2] staging: media: zoran: align code appropriately Mitali Borkar
2021-04-13 14:47 ` Hans Verkuil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).