Re: [Outreachy kernel] [PATCH v2 2/2] staging: media: zoran: remove and add comments; align code
> Mam, my new patch is ready but I am not sure how to send, the v1/2 is > not on top, I have not made changes in it because it was not required. Now, > how to > send these two mails as a patchset, since in between these two git > commits, I have another commits too. Suppose you have relevant patch1 patch on some other file relevant patch2 Then you can use git rebase -i HEAD~3 to reorder them, ie to put relevant patch2 before patch on some other file. When you do the git rebase, just move the lines for the different patches around and save the changes. julia > > > julia > > > > > > > +#define ZR36057_VFESPFR_RGB565 (2 << 3) > > > > > +#define ZR36057_VFESPFR_RGB555 (3 << 3) > > > > > +#define ZR36057_VFESPFR_ERR_DIF BIT(2) > > > > > +#define ZR36057_VFESPFR_PACK24 BIT(1) > > > > > +#define ZR36057_VFESPFR_LITTLE_ENDIANBIT(0) > > > > > + > > > > > +/* Video Display "Top" Register */ > > > > > +#define ZR36057_VDTR 0x00c > > > > > + > > > > > +/* Video Display "Bottom" Register */ > > > > > +#define ZR36057_VDBR 0x010 > > > > > + > > > > > +/* Video Stride, Status, and Frame Grab Register */ > > > > > +#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_GRABBIT(0) > > > > > + > > > > > +/* Video Display Configuration Register */ > > > > > +#define ZR36057_VDCR 0x018 > > > > > +#define ZR36057_VDCR_VID_EN BIT(31) > > > > > +#define ZR36057_VDCR_MIN_PIX 24 > > > > > +#define ZR36057_VDCR_TRITON BIT(24) > > > > > +#define ZR36057_VDCR_VID_WIN_HT 12 > > > > > > > > These don't look well aligned either. > > > > > > > > Please check on the rest. > > > > > > > Yes Ma'am, I am rechecking this. > > > > > > > julia > > > > > > > > > > > > > +#define ZR36057_VDCR_VID_WIN_WID 0 > > > > > + > > > > > +/* Masking Map "Top" Register */ > > > > > +#define ZR36057_MMTR 0x01c > > > > > + > > > > > +/* Masking Map "Bottom" Register */ > > > > > +#define ZR36057_MMBR 0x020 > > > > > + > > > > > +/* Overlay Control Register */ > > > > > +#define ZR36057_OCR 0x024 > > > > > +#define ZR36057_OCR_OVL_ENABLE BIT(15) > > > > > +#define ZR36057_OCR_MASK_STRIDE 0 > > > > > + > > > > > +/* System, PCI, and General Purpose Pins Control Register */ > > > > > +#define ZR36057_SPGPPCR 0x028 > > > > > +#define ZR36057_SPGPPCR_SOFT_RESET BIT(24) > > > > > + > > > > > +/* General Purpose Pins and GuestBus Control Register (1) */ > > > > > +#define ZR36057_GPPGCR1 0x02c > > > > > + > > > > > +/* MPEG Code Source Address Register */ > > > > > +#define ZR36057_MCSAR0x030 > > > > > + > > > > > +/* MPEG Code Transfer Control Register */ > > > > > +#define ZR36057_MCTCR0x034 > > > > > +#define ZR36057_MCTCR_COD_TIME BIT(30) > > > > > +#define ZR36057_MCTCR_C_EMPTYBIT(29) > > > > > +#define ZR36057_MCTCR_C_FLUSHBIT(28) > > > > > #define ZR36057_MCTCR_COD_GUEST_ID 20 > > > > > #define ZR36057_MCTCR_COD_GUEST_REG 16 > > > > > > > > > > -#define ZR36057_MCMPR 0x038/* MPEG Code Memory > > > > > Pointer Register */ > > > > > - > > > > > -#define ZR36057_ISR 0x03c/* Interrupt Status > > > > > Register */ > > > > > -#define ZR36057_ISR_GIRQ1BIT(30) > > > > > -#define ZR36057_ISR_GIRQ0BIT(29) > > > > > -#define ZR36057_ISR_COD_REP_IRQBIT(28) > > > > > -#define ZR36057_ISR_JPEG_REP_IRQ BIT(27) > > > > > - > > > > > -#define ZR36057_ICR 0x040/* Interrupt Control > > > > > Register */ > > > > > -#define ZR36057_ICR_GIRQ1BIT(30) > > > > > -#define ZR36057_ICR_GIRQ0BIT(29) > > > > > -#define ZR36057_ICR_COD_REP_IRQBIT(28) > > > > > -#define ZR36057_ICR_JPEG_REP_IRQ BIT(27) > > > > > -#define ZR36057_ICR_INT_PIN_EN BIT(24) > > > > > - > > > > > -#define ZR36057_I2CBR 0x044/* I2C Bus Register */ > > > > > -#define ZR36057_I2CBR_SDA BIT(1) > > > > > -#define ZR36057_I2CBR_SCL BIT(0) > > > > > - > > > > > -#define ZR36057_JMC 0x100/* JPEG Mode and > > > > > Control */ > > > > > -#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) > > > > >
Re: [Outreachy kernel] [PATCH v2 2/2] staging: media: zoran: remove and add comments; align code
On Sat, Apr 10, 2021 at 01:56:24PM +0200, Julia Lawall wrote: > > > On Sat, 10 Apr 2021, Mitali Borkar wrote: > > > On Fri, Apr 09, 2021 at 10:12:12PM +0200, Julia Lawall wrote: > > > > > > > > > On Sat, 10 Apr 2021, Mitali Borkar wrote: > > > > > > > Removed comments from the same line and added them to new line above the > > > > blocks, aligned everything properly by using tabs to make code neater > > > > and improve readability. > > > > > > > > Signed-off-by: Mitali Borkar > > > > --- > > > > drivers/staging/media/zoran/zr36057.h | 293 ++ > > > > 1 file changed, 162 insertions(+), 131 deletions(-) > > > > > > > > diff --git a/drivers/staging/media/zoran/zr36057.h > > > > b/drivers/staging/media/zoran/zr36057.h > > > > index 93075459f910..198d344a8879 100644 > > > > --- a/drivers/staging/media/zoran/zr36057.h > > > > +++ b/drivers/staging/media/zoran/zr36057.h > > > > @@ -12,145 +12,176 @@ > > > > > > > > /* Zoran ZR36057 registers */ > > > > > > > > -#define ZR36057_VFEHCR 0x000 /* Video Front End, Horizontal > > > > Configuration Register */ > > > > -#define ZR36057_VFEHCR_HS_POL BIT(30) > > > > -#define ZR36057_VFEHCR_H_START 10 > > > > +/* Video Front End, Horizontal Configuration Register */ > > > > +#define ZR36057_VFEHCR 0x000 > > > > +#define ZR36057_VFEHCR_HS_POL BIT(30) > > > > > > It looks like the alignment didn't work out here? Check that the use of > > > tabs is the same as on the nearby lines. > > > > > Do I need to align BIT(30), 10, 0x000 and rest in same column or should I > > align them separately? > > If there are a bunch of #defines with no blank line between them and > similar names (these all start with ZR36057_VFEHCR), then it can be nice > to line them all up. > > > > > > > +#define ZR36057_VFEHCR_H_START 10 > > > > #define ZR36057_VFEHCR_H_END 0 > > > > #define ZR36057_VFEHCR_HMASK 0x3ff > > > > > > > > -#define ZR36057_VFEVCR 0x004 /* Video Front End, Vertical > > > > Configuration Register */ > > > > -#define ZR36057_VFEVCR_VS_POL BIT(30) > > > > -#define ZR36057_VFEVCR_V_START 10 > > > > +/* Video Front End, Vertical Configuration Register */ > > > > +#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_VFESPFR 0x008 /* Video Front End, Scaler and > > > > Pixel Format Register */ > > > > -#define ZR36057_VFESPFR_EXT_FLBIT(26) > > > > -#define ZR36057_VFESPFR_TOP_FIELD BIT(25) > > > > -#define ZR36057_VFESPFR_VCLK_POL BIT(24) > > > > -#define ZR36057_VFESPFR_H_FILTER 21 > > > > -#define ZR36057_VFESPFR_HOR_DCM 14 > > > > -#define ZR36057_VFESPFR_VER_DCM 8 > > > > -#define ZR36057_VFESPFR_DISP_MODE6 > > > > -#define ZR36057_VFESPFR_YUV422 (0 << 3) > > > > -#define ZR36057_VFESPFR_RGB888 BIT(3) > > > > -#define ZR36057_VFESPFR_RGB565 (2 << 3) > > > > -#define ZR36057_VFESPFR_RGB555 (3 << 3) > > > > -#define ZR36057_VFESPFR_ERR_DIF BIT(2) > > > > -#define ZR36057_VFESPFR_PACK24 BIT(1) > > > > -#define ZR36057_VFESPFR_LITTLE_ENDIANBIT(0) > > > > - > > > > -#define ZR36057_VDTR0x00c /* Video Display "Top" Register > > > > */ > > > > - > > > > -#define ZR36057_VDBR0x010 /* Video Display "Bottom" > > > > Register */ > > > > - > > > > -#define ZR36057_VSSFGR 0x014 /* Video Stride, Status, and > > > > Frame Grab Register */ > > > > -#define ZR36057_VSSFGR_DISP_STRIDE 16 > > > > -#define ZR36057_VSSFGR_VID_OVFBIT(8) > > > > -#define ZR36057_VSSFGR_SNAP_SHOT BIT(1) > > > > -#define ZR36057_VSSFGR_FRAME_GRAB BIT(0) > > > > - > > > > -#define ZR36057_VDCR0x018 /* Video Display Configuration > > > > Register */ > > > > -#define ZR36057_VDCR_VID_EN BIT(31) > > > > -#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_MMTR0x01c /* Masking Map "Top" Register */ > > > > - > > > > -#define ZR36057_MMBR0x020 /* Masking Map "Bottom" > > > > Register */ > > > > - > > > > -#define ZR36057_OCR 0x024 /* Overlay Control Register */ > > > > -#define ZR36057_OCR_OVL_ENABLEBIT(15) > > > > -#define ZR36057_OCR_MASK_STRIDE 0 > > > > - > > > > -#define ZR36057_SPGPPCR 0x028 /* System, PCI, and General > > > > Purpose Pins Control Register */ > > > > -#define ZR36057_SPGPPCR_SOFT_RESET BIT(24) > > > > - > > > >
Re: [Outreachy kernel] [PATCH v2 2/2] staging: media: zoran: remove and add comments; align code
On Sat, 10 Apr 2021, Mitali Borkar wrote: > On Fri, Apr 09, 2021 at 10:12:12PM +0200, Julia Lawall wrote: > > > > > > On Sat, 10 Apr 2021, Mitali Borkar wrote: > > > > > Removed comments from the same line and added them to new line above the > > > blocks, aligned everything properly by using tabs to make code neater > > > and improve readability. > > > > > > Signed-off-by: Mitali Borkar > > > --- > > > drivers/staging/media/zoran/zr36057.h | 293 ++ > > > 1 file changed, 162 insertions(+), 131 deletions(-) > > > > > > diff --git a/drivers/staging/media/zoran/zr36057.h > > > b/drivers/staging/media/zoran/zr36057.h > > > index 93075459f910..198d344a8879 100644 > > > --- a/drivers/staging/media/zoran/zr36057.h > > > +++ b/drivers/staging/media/zoran/zr36057.h > > > @@ -12,145 +12,176 @@ > > > > > > /* Zoran ZR36057 registers */ > > > > > > -#define ZR36057_VFEHCR 0x000/* Video Front End, Horizontal > > > Configuration Register */ > > > -#define ZR36057_VFEHCR_HS_POL BIT(30) > > > -#define ZR36057_VFEHCR_H_START 10 > > > +/* Video Front End, Horizontal Configuration Register */ > > > +#define ZR36057_VFEHCR 0x000 > > > +#define ZR36057_VFEHCR_HS_POLBIT(30) > > > > It looks like the alignment didn't work out here? Check that the use of > > tabs is the same as on the nearby lines. > > > Do I need to align BIT(30), 10, 0x000 and rest in same column or should I > align them separately? If there are a bunch of #defines with no blank line between them and similar names (these all start with ZR36057_VFEHCR), then it can be nice to line them all up. > > > > +#define ZR36057_VFEHCR_H_START 10 > > > #define ZR36057_VFEHCR_H_END 0 > > > #define ZR36057_VFEHCR_HMASK 0x3ff > > > > > > -#define ZR36057_VFEVCR 0x004/* Video Front End, Vertical > > > Configuration Register */ > > > -#define ZR36057_VFEVCR_VS_POL BIT(30) > > > -#define ZR36057_VFEVCR_V_START 10 > > > +/* Video Front End, Vertical Configuration Register */ > > > +#define ZR36057_VFEVCR 0x004 > > > +#define ZR36057_VFEVCR_VS_POLBIT(30) > > > +#define ZR36057_VFEVCR_V_START 10 > > > #define ZR36057_VFEVCR_V_END 0 > > > #define ZR36057_VFEVCR_VMASK 0x3ff > > > > > > -#define ZR36057_VFESPFR 0x008/* Video Front End, Scaler and > > > Pixel Format Register */ > > > -#define ZR36057_VFESPFR_EXT_FLBIT(26) > > > -#define ZR36057_VFESPFR_TOP_FIELD BIT(25) > > > -#define ZR36057_VFESPFR_VCLK_POL BIT(24) > > > -#define ZR36057_VFESPFR_H_FILTER 21 > > > -#define ZR36057_VFESPFR_HOR_DCM 14 > > > -#define ZR36057_VFESPFR_VER_DCM 8 > > > -#define ZR36057_VFESPFR_DISP_MODE6 > > > -#define ZR36057_VFESPFR_YUV422 (0 << 3) > > > -#define ZR36057_VFESPFR_RGB888 BIT(3) > > > -#define ZR36057_VFESPFR_RGB565 (2 << 3) > > > -#define ZR36057_VFESPFR_RGB555 (3 << 3) > > > -#define ZR36057_VFESPFR_ERR_DIF BIT(2) > > > -#define ZR36057_VFESPFR_PACK24 BIT(1) > > > -#define ZR36057_VFESPFR_LITTLE_ENDIANBIT(0) > > > - > > > -#define ZR36057_VDTR0x00c/* Video Display "Top" Register > > > */ > > > - > > > -#define ZR36057_VDBR0x010/* Video Display "Bottom" > > > Register */ > > > - > > > -#define ZR36057_VSSFGR 0x014/* Video Stride, Status, and > > > Frame Grab Register */ > > > -#define ZR36057_VSSFGR_DISP_STRIDE 16 > > > -#define ZR36057_VSSFGR_VID_OVFBIT(8) > > > -#define ZR36057_VSSFGR_SNAP_SHOT BIT(1) > > > -#define ZR36057_VSSFGR_FRAME_GRAB BIT(0) > > > - > > > -#define ZR36057_VDCR0x018/* Video Display Configuration > > > Register */ > > > -#define ZR36057_VDCR_VID_EN BIT(31) > > > -#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_MMTR0x01c/* Masking Map "Top" Register */ > > > - > > > -#define ZR36057_MMBR0x020/* Masking Map "Bottom" > > > Register */ > > > - > > > -#define ZR36057_OCR 0x024/* Overlay Control Register */ > > > -#define ZR36057_OCR_OVL_ENABLEBIT(15) > > > -#define ZR36057_OCR_MASK_STRIDE 0 > > > - > > > -#define ZR36057_SPGPPCR 0x028/* System, PCI, and General > > > Purpose Pins Control Register */ > > > -#define ZR36057_SPGPPCR_SOFT_RESETBIT(24) > > > - > > > -#define ZR36057_GPPGCR1 0x02c/* General Purpose Pins and > > > GuestBus Control Register (1) */ > > > - > > > -#define ZR36057_MCSAR 0x030/* MPEG Code Source Address > > > Register */ > > > - > > > -#define ZR36057_MCTCR
Re: [Outreachy kernel] [PATCH v2 2/2] staging: media: zoran: remove and add comments; align code
On Fri, Apr 09, 2021 at 10:12:12PM +0200, Julia Lawall wrote: > > > On Sat, 10 Apr 2021, Mitali Borkar wrote: > > > Removed comments from the same line and added them to new line above the > > blocks, aligned everything properly by using tabs to make code neater > > and improve readability. > > > > Signed-off-by: Mitali Borkar > > --- > > drivers/staging/media/zoran/zr36057.h | 293 ++ > > 1 file changed, 162 insertions(+), 131 deletions(-) > > > > diff --git a/drivers/staging/media/zoran/zr36057.h > > b/drivers/staging/media/zoran/zr36057.h > > index 93075459f910..198d344a8879 100644 > > --- a/drivers/staging/media/zoran/zr36057.h > > +++ b/drivers/staging/media/zoran/zr36057.h > > @@ -12,145 +12,176 @@ > > > > /* Zoran ZR36057 registers */ > > > > -#define ZR36057_VFEHCR 0x000 /* Video Front End, Horizontal > > Configuration Register */ > > -#define ZR36057_VFEHCR_HS_POL BIT(30) > > -#define ZR36057_VFEHCR_H_START 10 > > +/* Video Front End, Horizontal Configuration Register */ > > +#define ZR36057_VFEHCR 0x000 > > +#define ZR36057_VFEHCR_HS_POL BIT(30) > > It looks like the alignment didn't work out here? Check that the use of > tabs is the same as on the nearby lines. > Do I need to align BIT(30), 10, 0x000 and rest in same column or should I align them separately? > > +#define ZR36057_VFEHCR_H_START 10 > > #define ZR36057_VFEHCR_H_END 0 > > #define ZR36057_VFEHCR_HMASK 0x3ff > > > > -#define ZR36057_VFEVCR 0x004 /* Video Front End, Vertical > > Configuration Register */ > > -#define ZR36057_VFEVCR_VS_POL BIT(30) > > -#define ZR36057_VFEVCR_V_START 10 > > +/* Video Front End, Vertical Configuration Register */ > > +#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_VFESPFR 0x008 /* Video Front End, Scaler and > > Pixel Format Register */ > > -#define ZR36057_VFESPFR_EXT_FLBIT(26) > > -#define ZR36057_VFESPFR_TOP_FIELD BIT(25) > > -#define ZR36057_VFESPFR_VCLK_POL BIT(24) > > -#define ZR36057_VFESPFR_H_FILTER 21 > > -#define ZR36057_VFESPFR_HOR_DCM 14 > > -#define ZR36057_VFESPFR_VER_DCM 8 > > -#define ZR36057_VFESPFR_DISP_MODE6 > > -#define ZR36057_VFESPFR_YUV422 (0 << 3) > > -#define ZR36057_VFESPFR_RGB888 BIT(3) > > -#define ZR36057_VFESPFR_RGB565 (2 << 3) > > -#define ZR36057_VFESPFR_RGB555 (3 << 3) > > -#define ZR36057_VFESPFR_ERR_DIF BIT(2) > > -#define ZR36057_VFESPFR_PACK24 BIT(1) > > -#define ZR36057_VFESPFR_LITTLE_ENDIANBIT(0) > > - > > -#define ZR36057_VDTR0x00c /* Video Display "Top" Register > > */ > > - > > -#define ZR36057_VDBR0x010 /* Video Display "Bottom" > > Register */ > > - > > -#define ZR36057_VSSFGR 0x014 /* Video Stride, Status, and > > Frame Grab Register */ > > -#define ZR36057_VSSFGR_DISP_STRIDE 16 > > -#define ZR36057_VSSFGR_VID_OVFBIT(8) > > -#define ZR36057_VSSFGR_SNAP_SHOT BIT(1) > > -#define ZR36057_VSSFGR_FRAME_GRAB BIT(0) > > - > > -#define ZR36057_VDCR0x018 /* Video Display Configuration > > Register */ > > -#define ZR36057_VDCR_VID_EN BIT(31) > > -#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_MMTR0x01c /* Masking Map "Top" Register */ > > - > > -#define ZR36057_MMBR0x020 /* Masking Map "Bottom" > > Register */ > > - > > -#define ZR36057_OCR 0x024 /* Overlay Control Register */ > > -#define ZR36057_OCR_OVL_ENABLEBIT(15) > > -#define ZR36057_OCR_MASK_STRIDE 0 > > - > > -#define ZR36057_SPGPPCR 0x028 /* System, PCI, and General > > Purpose Pins Control Register */ > > -#define ZR36057_SPGPPCR_SOFT_RESET BIT(24) > > - > > -#define ZR36057_GPPGCR1 0x02c /* General Purpose Pins and > > GuestBus Control Register (1) */ > > - > > -#define ZR36057_MCSAR 0x030 /* MPEG Code Source Address > > Register */ > > - > > -#define ZR36057_MCTCR 0x034 /* MPEG Code Transfer Control > > Register */ > > -#define ZR36057_MCTCR_COD_TIMEBIT(30) > > -#define ZR36057_MCTCR_C_EMPTY BIT(29) > > -#define ZR36057_MCTCR_C_FLUSH BIT(28) > > +/* Video Front End, Scaler and Pixel Format Register */ > > +#define ZR36057_VFESPFR0x008 > > +#define ZR36057_VFESPFR_EXT_FL
Re: [Outreachy kernel] [PATCH v2 2/2] staging: media: zoran: remove and add comments; align code
On Sat, 10 Apr 2021, Mitali Borkar wrote: > Removed comments from the same line and added them to new line above the > blocks, aligned everything properly by using tabs to make code neater > and improve readability. > > Signed-off-by: Mitali Borkar > --- > drivers/staging/media/zoran/zr36057.h | 293 ++ > 1 file changed, 162 insertions(+), 131 deletions(-) > > diff --git a/drivers/staging/media/zoran/zr36057.h > b/drivers/staging/media/zoran/zr36057.h > index 93075459f910..198d344a8879 100644 > --- a/drivers/staging/media/zoran/zr36057.h > +++ b/drivers/staging/media/zoran/zr36057.h > @@ -12,145 +12,176 @@ > > /* Zoran ZR36057 registers */ > > -#define ZR36057_VFEHCR 0x000/* Video Front End, Horizontal > Configuration Register */ > -#define ZR36057_VFEHCR_HS_POL BIT(30) > -#define ZR36057_VFEHCR_H_START 10 > +/* Video Front End, Horizontal Configuration Register */ > +#define ZR36057_VFEHCR 0x000 > +#define ZR36057_VFEHCR_HS_POLBIT(30) It looks like the alignment didn't work out here? Check that the use of tabs is the same as on the nearby lines. > +#define ZR36057_VFEHCR_H_START 10 > #define ZR36057_VFEHCR_H_END 0 > #define ZR36057_VFEHCR_HMASK 0x3ff > > -#define ZR36057_VFEVCR 0x004/* Video Front End, Vertical > Configuration Register */ > -#define ZR36057_VFEVCR_VS_POL BIT(30) > -#define ZR36057_VFEVCR_V_START 10 > +/* Video Front End, Vertical Configuration Register */ > +#define ZR36057_VFEVCR 0x004 > +#define ZR36057_VFEVCR_VS_POLBIT(30) > +#define ZR36057_VFEVCR_V_START 10 > #define ZR36057_VFEVCR_V_END 0 > #define ZR36057_VFEVCR_VMASK 0x3ff > > -#define ZR36057_VFESPFR 0x008/* Video Front End, Scaler and > Pixel Format Register */ > -#define ZR36057_VFESPFR_EXT_FLBIT(26) > -#define ZR36057_VFESPFR_TOP_FIELD BIT(25) > -#define ZR36057_VFESPFR_VCLK_POL BIT(24) > -#define ZR36057_VFESPFR_H_FILTER 21 > -#define ZR36057_VFESPFR_HOR_DCM 14 > -#define ZR36057_VFESPFR_VER_DCM 8 > -#define ZR36057_VFESPFR_DISP_MODE6 > -#define ZR36057_VFESPFR_YUV422 (0 << 3) > -#define ZR36057_VFESPFR_RGB888 BIT(3) > -#define ZR36057_VFESPFR_RGB565 (2 << 3) > -#define ZR36057_VFESPFR_RGB555 (3 << 3) > -#define ZR36057_VFESPFR_ERR_DIF BIT(2) > -#define ZR36057_VFESPFR_PACK24 BIT(1) > -#define ZR36057_VFESPFR_LITTLE_ENDIANBIT(0) > - > -#define ZR36057_VDTR0x00c/* Video Display "Top" Register > */ > - > -#define ZR36057_VDBR0x010/* Video Display "Bottom" > Register */ > - > -#define ZR36057_VSSFGR 0x014/* Video Stride, Status, and > Frame Grab Register */ > -#define ZR36057_VSSFGR_DISP_STRIDE 16 > -#define ZR36057_VSSFGR_VID_OVFBIT(8) > -#define ZR36057_VSSFGR_SNAP_SHOT BIT(1) > -#define ZR36057_VSSFGR_FRAME_GRAB BIT(0) > - > -#define ZR36057_VDCR0x018/* Video Display Configuration > Register */ > -#define ZR36057_VDCR_VID_EN BIT(31) > -#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_MMTR0x01c/* Masking Map "Top" Register */ > - > -#define ZR36057_MMBR0x020/* Masking Map "Bottom" > Register */ > - > -#define ZR36057_OCR 0x024/* Overlay Control Register */ > -#define ZR36057_OCR_OVL_ENABLEBIT(15) > -#define ZR36057_OCR_MASK_STRIDE 0 > - > -#define ZR36057_SPGPPCR 0x028/* System, PCI, and General > Purpose Pins Control Register */ > -#define ZR36057_SPGPPCR_SOFT_RESETBIT(24) > - > -#define ZR36057_GPPGCR1 0x02c/* General Purpose Pins and > GuestBus Control Register (1) */ > - > -#define ZR36057_MCSAR 0x030/* MPEG Code Source Address > Register */ > - > -#define ZR36057_MCTCR 0x034/* MPEG Code Transfer Control > Register */ > -#define ZR36057_MCTCR_COD_TIMEBIT(30) > -#define ZR36057_MCTCR_C_EMPTY BIT(29) > -#define ZR36057_MCTCR_C_FLUSH BIT(28) > +/* Video Front End, Scaler and Pixel Format Register */ > +#define ZR36057_VFESPFR 0x008 > +#define ZR36057_VFESPFR_EXT_FL BIT(26) > +#define ZR36057_VFESPFR_TOP_FIELDBIT(25) > +#define ZR36057_VFESPFR_VCLK_POL BIT(24) > +#define ZR36057_VFESPFR_H_FILTER 21 > +#define ZR36057_VFESPFR_HOR_DCM 14 > +#define ZR36057_VFESPFR_VER_DCM 8 > +#define ZR36057_VFESPFR_DISP_MODE6 The above four lines also look odd.