RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> -Original Message-
> From: Karicheri, Muralidharan
> Sent: Monday, December 21, 2009 7:00 PM
> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> Cc: hverk...@xs4all.nl
> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
>
> Vaibhav,
>
> Did you address my comments against this patch? What was
> the resolution. I haven't seen any response from you. Please
> forward me any response that you had sent.
>
[Hiremath, Vaibhav] Your comment was,
> >> For this and below, You are using every 4th location in the array
> >> for saving register values which is not necessary if you use
> >> something like.
> >> ccdc_ctx[CCDC_PCR >> 2] = regr(CCDC_PCR);
> >> ccdc_ctx[CCDC_SYN_MODE >> 2] = regr(CCDC_SYN_MODE);
> >> Any reason why you do this way?
Actually the main reason why I did not do this was "VP_OUT" register, the
VP_OUT offset is 0x94.
But I think I can do something like you suggested for others and for VP_OUT
will anyway will come to last.
I will incorporate the change in next post.
Thanks,
Vaibhav
> Murali Karicheri
> Software Design Engineer
> Texas Instruments Inc.
> Germantown, MD 20874
> phone: 301-407-9583
> email: m-kariche...@ti.com
> >-Original Message-
> >From: Hiremath, Vaibhav
> >Sent: Monday, December 21, 2009 1:29 AM
> >To: Karicheri, Muralidharan; linux-media@vger.kernel.org
> >Cc: hverk...@xs4all.nl
> >Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
> >
> >> -Original Message-
> >> From: Karicheri, Muralidharan
> >> Sent: Friday, November 20, 2009 3:31 AM
> >> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> >> Cc: hverk...@xs4all.nl
> >> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> >> Support
> >>
> >> Vaibhav,
> >>
> >> I have some comments. I have tested this patch for normal
> >> use case of tvp5146 capture on DM355. It looks ok. We
> >> don't have support for power management on DM355. So I couldn't
> >> test the suspend & resume operations.
> >>
> >[Hiremath, Vaibhav] Murali,
> >
> >If you don't any further comments/analysis, this patch should go
> in. I will
> >resubmit the patch against the tip.
> >
> >Thanks,
> >Vaibhav
> >
> >> >
> >> > struct ccdc_hw_device {
> >> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
> >> >b/drivers/media/video/davinci/dm644x_ccdc.c
> >> >index 5dff8d9..fdab823 100644
> >> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
> >> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
> >> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> >> > static int ccdc_addr_size;
> >> > static enum vpfe_hw_if_type ccdc_if_type;
> >> >
> >> >+#define CCDC_SZ_REGS SZ_1K
> >> >+
> >> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
> >>
> >> The last register is at 0x94 on DM6446. So do we need 256
> >> entries when we have only 37 registers?
> >>
> >> >+
> >> > /* register access routines */
> >> > static inline u32 regr(u32 offset)
> >> > {
> >> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
> >> >vpfe_hw_if_param *params)
> >> > return 0;
> >> > }
> >> >
> >> >+static void ccdc_save_context(void)
> >> >+{
> >> >+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
> >>
> >>
> >> For this and below, You are using every 4th location in the array
> >> for saving register values which is not necessary if you use
> >> something like.
> >> ccdc_ctx[CCDC_PCR >> 2] = regr(CCDC_PCR);
> >> ccdc_ctx[CCDC_SYN_MODE >> 2] = regr(CCDC_SYN_MODE);
> >> Any reason why you do this way?
> >>
> >> >+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
> >> >+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
> >> >+ ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
> >> >+ ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
> >> >+ ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
> >> >+ ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
> >> >+ ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
> >> >+ ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
> >> >+ ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
> >> >+ ccdc_ctx[CCDC_SDR
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
Vaibhav,
Did you address my comments against this patch? What was
the resolution. I haven't seen any response from you. Please
forward me any response that you had sent.
Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-kariche...@ti.com
>-Original Message-
>From: Hiremath, Vaibhav
>Sent: Monday, December 21, 2009 1:29 AM
>To: Karicheri, Muralidharan; linux-media@vger.kernel.org
>Cc: hverk...@xs4all.nl
>Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
>
>> -Original Message-
>> From: Karicheri, Muralidharan
>> Sent: Friday, November 20, 2009 3:31 AM
>> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
>> Cc: hverk...@xs4all.nl
>> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> Support
>>
>> Vaibhav,
>>
>> I have some comments. I have tested this patch for normal
>> use case of tvp5146 capture on DM355. It looks ok. We
>> don't have support for power management on DM355. So I couldn't
>> test the suspend & resume operations.
>>
>[Hiremath, Vaibhav] Murali,
>
>If you don't any further comments/analysis, this patch should go in. I will
>resubmit the patch against the tip.
>
>Thanks,
>Vaibhav
>
>> >
>> > struct ccdc_hw_device {
>> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
>> >b/drivers/media/video/davinci/dm644x_ccdc.c
>> >index 5dff8d9..fdab823 100644
>> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
>> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
>> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
>> > static int ccdc_addr_size;
>> > static enum vpfe_hw_if_type ccdc_if_type;
>> >
>> >+#define CCDC_SZ_REGS SZ_1K
>> >+
>> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
>>
>> The last register is at 0x94 on DM6446. So do we need 256
>> entries when we have only 37 registers?
>>
>> >+
>> > /* register access routines */
>> > static inline u32 regr(u32 offset)
>> > {
>> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
>> >vpfe_hw_if_param *params)
>> >return 0;
>> > }
>> >
>> >+static void ccdc_save_context(void)
>> >+{
>> >+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
>>
>>
>> For this and below, You are using every 4th location in the array
>> for saving register values which is not necessary if you use
>> something like.
>> ccdc_ctx[CCDC_PCR >> 2] = regr(CCDC_PCR);
>> ccdc_ctx[CCDC_SYN_MODE >> 2] = regr(CCDC_SYN_MODE);
>> Any reason why you do this way?
>>
>> >+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
>> >+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
>> >+ ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
>> >+ ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
>> >+ ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
>> >+ ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
>> >+ ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
>> >+ ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
>> >+ ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
>> >+ ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
>> >+ ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
>> >+ ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
>> >+ ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
>> >+ ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
>> >+ ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
>> >+ ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
>> >+ ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
>> >+ ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
>> >+ ccdc_ctx[CCDC_REC656IF] = regr(CCDC_REC656IF);
>> >+ ccdc_ctx[CCDC_CCDCFG] = regr(CCDC_CCDCFG);
>> >+ ccdc_ctx[CCDC_FMTCFG] = regr(CCDC_FMTCFG);
>> >+ ccdc_ctx[CCDC_FMT_HORZ] = regr(CCDC_FMT_HORZ);
>> >+ ccdc_ctx[CCDC_FMT_VERT] = regr(CCDC_FMT_VERT);
>> >+ ccdc_ctx[CCDC_FMT_ADDR0] = regr(CCDC_FMT_ADDR0);
>> >+ ccdc_ctx[CCDC_FMT_ADDR1] = regr(CCDC_FMT_ADDR1);
>> >+ ccdc_ctx[CCDC_FMT_ADDR2] = regr(CCDC_FMT_ADDR2);
>> >+ ccdc_ctx[CCDC_FMT_ADDR3] = regr(CCDC_FMT_ADDR3);
>> >+ ccdc_ctx[CCDC_FMT_ADDR4] = regr(CCDC_FMT_ADDR4);
>> >+ ccdc_ctx[CCDC_FMT_ADDR5] = regr(CCDC_FMT_ADDR5);
>> >+ ccdc_ctx[CCDC_FMT_ADDR6] = regr(CCDC_FMT_ADDR6);
>> >+ ccdc_ctx[CCDC_FMT_ADDR7] = regr(CCDC_FMT_ADDR7);
>> >+ ccdc_ctx[CCDC_PRGEVEN_0] = regr(CCDC_PRGEVEN_0);
>
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> -Original Message-
> From: Karicheri, Muralidharan
> Sent: Friday, November 20, 2009 3:31 AM
> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> Cc: hverk...@xs4all.nl
> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
>
> Vaibhav,
>
> I have some comments. I have tested this patch for normal
> use case of tvp5146 capture on DM355. It looks ok. We
> don't have support for power management on DM355. So I couldn't
> test the suspend & resume operations.
>
[Hiremath, Vaibhav] Murali,
If you don't any further comments/analysis, this patch should go in. I will
resubmit the patch against the tip.
Thanks,
Vaibhav
> >
> > struct ccdc_hw_device {
> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
> >b/drivers/media/video/davinci/dm644x_ccdc.c
> >index 5dff8d9..fdab823 100644
> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> > static int ccdc_addr_size;
> > static enum vpfe_hw_if_type ccdc_if_type;
> >
> >+#define CCDC_SZ_REGSSZ_1K
> >+
> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
>
> The last register is at 0x94 on DM6446. So do we need 256
> entries when we have only 37 registers?
>
> >+
> > /* register access routines */
> > static inline u32 regr(u32 offset)
> > {
> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
> >vpfe_hw_if_param *params)
> > return 0;
> > }
> >
> >+static void ccdc_save_context(void)
> >+{
> >+ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
>
>
> For this and below, You are using every 4th location in the array
> for saving register values which is not necessary if you use
> something like.
> ccdc_ctx[CCDC_PCR >> 2] = regr(CCDC_PCR);
> ccdc_ctx[CCDC_SYN_MODE >> 2] = regr(CCDC_SYN_MODE);
> Any reason why you do this way?
>
> >+ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
> >+ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
> >+ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
> >+ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
> >+ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
> >+ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
> >+ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
> >+ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
> >+ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
> >+ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
> >+ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
> >+ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
> >+ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
> >+ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
> >+ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
> >+ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
> >+ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
> >+ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
> >+ccdc_ctx[CCDC_REC656IF] = regr(CCDC_REC656IF);
> >+ccdc_ctx[CCDC_CCDCFG] = regr(CCDC_CCDCFG);
> >+ccdc_ctx[CCDC_FMTCFG] = regr(CCDC_FMTCFG);
> >+ccdc_ctx[CCDC_FMT_HORZ] = regr(CCDC_FMT_HORZ);
> >+ccdc_ctx[CCDC_FMT_VERT] = regr(CCDC_FMT_VERT);
> >+ccdc_ctx[CCDC_FMT_ADDR0] = regr(CCDC_FMT_ADDR0);
> >+ccdc_ctx[CCDC_FMT_ADDR1] = regr(CCDC_FMT_ADDR1);
> >+ccdc_ctx[CCDC_FMT_ADDR2] = regr(CCDC_FMT_ADDR2);
> >+ccdc_ctx[CCDC_FMT_ADDR3] = regr(CCDC_FMT_ADDR3);
> >+ccdc_ctx[CCDC_FMT_ADDR4] = regr(CCDC_FMT_ADDR4);
> >+ccdc_ctx[CCDC_FMT_ADDR5] = regr(CCDC_FMT_ADDR5);
> >+ccdc_ctx[CCDC_FMT_ADDR6] = regr(CCDC_FMT_ADDR6);
> >+ccdc_ctx[CCDC_FMT_ADDR7] = regr(CCDC_FMT_ADDR7);
> >+ccdc_ctx[CCDC_PRGEVEN_0] = regr(CCDC_PRGEVEN_0);
> >+ccdc_ctx[CCDC_PRGEVEN_1] = regr(CCDC_PRGEVEN_1);
> >+ccdc_ctx[CCDC_PRGODD_0] = regr(CCDC_PRGODD_0);
> >+ccdc_ctx[CCDC_PRGODD_1] = regr(CCDC_PRGODD_1);
> >+ccdc_ctx[CCDC_VP_OUT] = regr(CCDC_VP_OUT);
> >+}
> >+
> >+static void ccdc_restore_context(void)
> >+{
> >+regw(ccdc_ctx[CCDC_SYN_MODE], CCDC_SYN_MODE);
> >+regw(ccdc_ctx[CCDC_HD_VD_WID], CCDC_HD_VD_WID);
> >+regw(ccdc_ctx[CCDC_PIX_LINES], CCDC_PIX_LINES);
> >+regw(ccdc_ctx[CCDC_HORZ_INFO], CCDC_HORZ_INFO);
> >+regw(ccdc_ctx[CCDC_VERT_START], CCDC_VERT_START);
> >+regw(ccdc_ctx[CCDC_VERT_LINES], CCDC_VERT_LINES);
> >+regw(ccdc_ctx[CCDC_CULLING], CCDC_CULLING);
> >+regw(ccdc_ctx[CCDC_HSIZE_OFF], CCDC_HSIZE_OFF);
> >+regw(ccdc_ctx[CCDC_SDOFST], CCDC_SDOFST);
> >+regw(ccdc_ctx[CCDC_SDR_ADDR],
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
Vaibhav,
I have some comments. I have tested this patch for normal
use case of tvp5146 capture on DM355. It looks ok. We
don't have support for power management on DM355. So I couldn't
test the suspend & resume operations.
>
> struct ccdc_hw_device {
>diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
>b/drivers/media/video/davinci/dm644x_ccdc.c
>index 5dff8d9..fdab823 100644
>--- a/drivers/media/video/davinci/dm644x_ccdc.c
>+++ b/drivers/media/video/davinci/dm644x_ccdc.c
>@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> static int ccdc_addr_size;
> static enum vpfe_hw_if_type ccdc_if_type;
>
>+#define CCDC_SZ_REGS SZ_1K
>+
>+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
The last register is at 0x94 on DM6446. So do we need 256
entries when we have only 37 registers?
>+
> /* register access routines */
> static inline u32 regr(u32 offset)
> {
>@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
>vpfe_hw_if_param *params)
> return 0;
> }
>
>+static void ccdc_save_context(void)
>+{
>+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
For this and below, You are using every 4th location in the array
for saving register values which is not necessary if you use something like.
ccdc_ctx[CCDC_PCR >> 2] = regr(CCDC_PCR);
ccdc_ctx[CCDC_SYN_MODE >> 2] = regr(CCDC_SYN_MODE);
Any reason why you do this way?
>+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
>+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
>+ ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
>+ ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
>+ ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
>+ ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
>+ ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
>+ ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
>+ ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
>+ ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
>+ ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
>+ ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
>+ ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
>+ ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
>+ ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
>+ ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
>+ ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
>+ ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
>+ ccdc_ctx[CCDC_REC656IF] = regr(CCDC_REC656IF);
>+ ccdc_ctx[CCDC_CCDCFG] = regr(CCDC_CCDCFG);
>+ ccdc_ctx[CCDC_FMTCFG] = regr(CCDC_FMTCFG);
>+ ccdc_ctx[CCDC_FMT_HORZ] = regr(CCDC_FMT_HORZ);
>+ ccdc_ctx[CCDC_FMT_VERT] = regr(CCDC_FMT_VERT);
>+ ccdc_ctx[CCDC_FMT_ADDR0] = regr(CCDC_FMT_ADDR0);
>+ ccdc_ctx[CCDC_FMT_ADDR1] = regr(CCDC_FMT_ADDR1);
>+ ccdc_ctx[CCDC_FMT_ADDR2] = regr(CCDC_FMT_ADDR2);
>+ ccdc_ctx[CCDC_FMT_ADDR3] = regr(CCDC_FMT_ADDR3);
>+ ccdc_ctx[CCDC_FMT_ADDR4] = regr(CCDC_FMT_ADDR4);
>+ ccdc_ctx[CCDC_FMT_ADDR5] = regr(CCDC_FMT_ADDR5);
>+ ccdc_ctx[CCDC_FMT_ADDR6] = regr(CCDC_FMT_ADDR6);
>+ ccdc_ctx[CCDC_FMT_ADDR7] = regr(CCDC_FMT_ADDR7);
>+ ccdc_ctx[CCDC_PRGEVEN_0] = regr(CCDC_PRGEVEN_0);
>+ ccdc_ctx[CCDC_PRGEVEN_1] = regr(CCDC_PRGEVEN_1);
>+ ccdc_ctx[CCDC_PRGODD_0] = regr(CCDC_PRGODD_0);
>+ ccdc_ctx[CCDC_PRGODD_1] = regr(CCDC_PRGODD_1);
>+ ccdc_ctx[CCDC_VP_OUT] = regr(CCDC_VP_OUT);
>+}
>+
>+static void ccdc_restore_context(void)
>+{
>+ regw(ccdc_ctx[CCDC_SYN_MODE], CCDC_SYN_MODE);
>+ regw(ccdc_ctx[CCDC_HD_VD_WID], CCDC_HD_VD_WID);
>+ regw(ccdc_ctx[CCDC_PIX_LINES], CCDC_PIX_LINES);
>+ regw(ccdc_ctx[CCDC_HORZ_INFO], CCDC_HORZ_INFO);
>+ regw(ccdc_ctx[CCDC_VERT_START], CCDC_VERT_START);
>+ regw(ccdc_ctx[CCDC_VERT_LINES], CCDC_VERT_LINES);
>+ regw(ccdc_ctx[CCDC_CULLING], CCDC_CULLING);
>+ regw(ccdc_ctx[CCDC_HSIZE_OFF], CCDC_HSIZE_OFF);
>+ regw(ccdc_ctx[CCDC_SDOFST], CCDC_SDOFST);
>+ regw(ccdc_ctx[CCDC_SDR_ADDR], CCDC_SDR_ADDR);
>+ regw(ccdc_ctx[CCDC_CLAMP], CCDC_CLAMP);
>+ regw(ccdc_ctx[CCDC_DCSUB], CCDC_DCSUB);
>+ regw(ccdc_ctx[CCDC_COLPTN], CCDC_COLPTN);
>+ regw(ccdc_ctx[CCDC_BLKCMP], CCDC_BLKCMP);
>+ regw(ccdc_ctx[CCDC_FPC], CCDC_FPC);
>+ regw(ccdc_ctx[CCDC_FPC_ADDR], CCDC_FPC_ADDR);
>+ regw(ccdc_ctx[CCDC_VDINT], CCDC_VDINT);
>+ regw(ccdc_ctx[CCDC_ALAW], CCDC_ALAW);
>+ regw(ccdc_ctx[CCDC_REC656IF], CCDC_REC656IF);
>+ regw(ccdc_ctx[CCDC_CCDCFG], CCDC_CCDCFG);
>+ regw(ccdc_ctx[CCDC_FMTCFG], CCDC_FMTCFG);
>+ regw(ccdc_ctx[CCDC_FMT_HORZ], CCDC_FMT_HORZ);
>+ regw(ccdc_ctx[CCDC_FMT_VERT], CCDC_FMT_VERT);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR0], CCDC_FMT_ADDR0);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR1], CCDC_FMT_ADDR1);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR2], CCDC_FMT_ADDR2);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR3], CCDC_FMT_ADDR3);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR4], CCDC_FMT_ADDR4);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR5], CCDC_FMT_ADDR5);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR6], CCDC_FMT_ADDR6);
>+ regw(ccdc_ctx[CCDC_FMT_ADDR7]
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> -Original Message- > From: Karicheri, Muralidharan > Sent: Wednesday, November 18, 2009 10:36 PM > To: Hiremath, Vaibhav; linux-media@vger.kernel.org > Cc: hverk...@xs4all.nl > Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > Support > > > > Murali Karicheri > Software Design Engineer > Texas Instruments Inc. > Germantown, MD 20874 > phone: 301-407-9583 > email: m-kariche...@ti.com > > >-Original Message- > >From: Hiremath, Vaibhav > >Sent: Wednesday, November 18, 2009 11:52 AM > >To: Karicheri, Muralidharan; linux-media@vger.kernel.org > >Cc: hverk...@xs4all.nl > >Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > Support > > > > > >> -Original Message- > >> From: Karicheri, Muralidharan > >> Sent: Wednesday, November 18, 2009 9:55 PM > >> To: Hiremath, Vaibhav; linux-media@vger.kernel.org > >> Cc: hverk...@xs4all.nl > >> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > >> Support > >> > >> Vaibhav, > >> > >> Just wondering how to test this on DaVinci platforms. > >> Could you tell what you did to test this? > >> > >[Hiremath, Vaibhav] Enable CONFIG_PM in your defconfig and issue > following > >command - > > > ># echo mem > /sys/power/state > > > > What is the expected result? [Hiremath, Vaibhav] This puts all the modules into suspend mode and on next interrupt (normally in our case, it is serial key stroke), it will resume from suspend mode. Below is the sample output from AM3517 - r...@omap3517-evm:~# r...@omap3517-evm:~# echo mem > /sys/power/state PM: Syncing filesystems ... done. Freezing user space processes ... (elapsed 0.00 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. Suspending console(s) (use no_console_suspend to debug) vpfe-capture: vpfe capture clocks disabled Successfully put all powerdomains to target state vpfe-capture: vpss clock vpfe_ck enabled vpfe-capture: vpss clock vpfe_pck enabled eth0: attached PHY driver [Generic PHY] (mii_bus:phy_addr=1:00, id=7c0f1) Restarting tasks ... done. PHY: 1:00 - Link is UP r...@omap3517-evm:~# Thanks, Vaibhav > > Thanks > > Murali > >Thanks, > >Vaibhav > > > >> Murali Karicheri > >> Software Design Engineer > >> Texas Instruments Inc. > >> Germantown, MD 20874 > >> phone: 301-407-9583 > >> email: m-kariche...@ti.com > >> > >> >-Original Message- > >> >From: Hiremath, Vaibhav > >> >Sent: Wednesday, November 18, 2009 10:36 AM > >> >To: Karicheri, Muralidharan; linux-media@vger.kernel.org > >> >Cc: hverk...@xs4all.nl > >> >Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > >> Support > >> > > >> > > >> >> -Original Message- > >> >> From: Karicheri, Muralidharan > >> >> Sent: Wednesday, November 18, 2009 8:55 PM > >> >> To: Hiremath, Vaibhav; linux-media@vger.kernel.org > >> >> Cc: hverk...@xs4all.nl > >> >> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > >> >> Support > >> >> > >> >> Vaibhav, > >> >> > >> >> Did you validate suspend & resume operations on AM3517? > >> >> > >> >[Hiremath, Vaibhav] yes, I think I mentioned in my patch. Do you > >> see any > >> >issues? > >> > > >> >Thanks, > >> >Vaibhav > >> > > >> >> Murali Karicheri > >> >> Software Design Engineer > >> >> Texas Instruments Inc. > >> >> Germantown, MD 20874 > >> >> phone: 301-407-9583 > >> >> email: m-kariche...@ti.com > >> >> > >> >> >-Original Message- > >> >> >From: Hiremath, Vaibhav > >> >> >Sent: Wednesday, November 18, 2009 6:35 AM > >> >> >To: linux-media@vger.kernel.org > >> >> >Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath, > >> Vaibhav > >> >> >Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume > >> Support > >> >> > > >> >> >From: Vaibhav Hiremath > >> >> > > >> >> >Validated on AM3517 Platform. > >> >> > > >> >> >Signed-off-by: Vaibhav Hiremath > >> >> >--- > >> >> > drivers/media/video/davinci/ccdc_hw_de
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-kariche...@ti.com
>-Original Message-
>From: Hiremath, Vaibhav
>Sent: Wednesday, November 18, 2009 11:52 AM
>To: Karicheri, Muralidharan; linux-media@vger.kernel.org
>Cc: hverk...@xs4all.nl
>Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
>
>
>> -Original Message-
>> From: Karicheri, Muralidharan
>> Sent: Wednesday, November 18, 2009 9:55 PM
>> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
>> Cc: hverk...@xs4all.nl
>> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> Support
>>
>> Vaibhav,
>>
>> Just wondering how to test this on DaVinci platforms.
>> Could you tell what you did to test this?
>>
>[Hiremath, Vaibhav] Enable CONFIG_PM in your defconfig and issue following
>command -
>
># echo mem > /sys/power/state
>
What is the expected result?
Thanks
Murali
>Thanks,
>Vaibhav
>
>> Murali Karicheri
>> Software Design Engineer
>> Texas Instruments Inc.
>> Germantown, MD 20874
>> phone: 301-407-9583
>> email: m-kariche...@ti.com
>>
>> >-Original Message-----
>> >From: Hiremath, Vaibhav
>> >Sent: Wednesday, November 18, 2009 10:36 AM
>> >To: Karicheri, Muralidharan; linux-media@vger.kernel.org
>> >Cc: hverk...@xs4all.nl
>> >Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> Support
>> >
>> >
>> >> -Original Message-
>> >> From: Karicheri, Muralidharan
>> >> Sent: Wednesday, November 18, 2009 8:55 PM
>> >> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
>> >> Cc: hverk...@xs4all.nl
>> >> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> >> Support
>> >>
>> >> Vaibhav,
>> >>
>> >> Did you validate suspend & resume operations on AM3517?
>> >>
>> >[Hiremath, Vaibhav] yes, I think I mentioned in my patch. Do you
>> see any
>> >issues?
>> >
>> >Thanks,
>> >Vaibhav
>> >
>> >> Murali Karicheri
>> >> Software Design Engineer
>> >> Texas Instruments Inc.
>> >> Germantown, MD 20874
>> >> phone: 301-407-9583
>> >> email: m-kariche...@ti.com
>> >>
>> >> >-Original Message-
>> >> >From: Hiremath, Vaibhav
>> >> >Sent: Wednesday, November 18, 2009 6:35 AM
>> >> >To: linux-media@vger.kernel.org
>> >> >Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath,
>> Vaibhav
>> >> >Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> Support
>> >> >
>> >> >From: Vaibhav Hiremath
>> >> >
>> >> >Validated on AM3517 Platform.
>> >> >
>> >> >Signed-off-by: Vaibhav Hiremath
>> >> >---
>> >> > drivers/media/video/davinci/ccdc_hw_device.h |4 +
>> >> > drivers/media/video/davinci/dm644x_ccdc.c| 87
>> >> >++
>> >> > drivers/media/video/davinci/vpfe_capture.c | 29 ++---
>> >> > 3 files changed, 112 insertions(+), 8 deletions(-)
>> >> >
>> >> >diff --git a/drivers/media/video/davinci/ccdc_hw_device.h
>> >> >b/drivers/media/video/davinci/ccdc_hw_device.h
>> >> >index 86b9b35..2a1ead4 100644
>> >> >--- a/drivers/media/video/davinci/ccdc_hw_device.h
>> >> >+++ b/drivers/media/video/davinci/ccdc_hw_device.h
>> >> >@@ -91,6 +91,10 @@ struct ccdc_hw_ops {
>> >> > void (*setfbaddr) (unsigned long addr);
>> >> > /* Pointer to function to get field id */
>> >> > int (*getfid) (void);
>> >> >+
>> >> >+/* suspend/resume support */
>> >> >+void (*save_context)(void);
>> >> >+void (*restore_context)(void);
>> >> > };
>> >> >
>> >> > struct ccdc_hw_device {
>> >> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
>> >> >b/drivers/media/video/davinci/dm644x_ccdc.c
>> >> >index 5dff8d9..fdab823 100644
>> >> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
>> >> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
>> >> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> -Original Message-
> From: Karicheri, Muralidharan
> Sent: Wednesday, November 18, 2009 9:55 PM
> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> Cc: hverk...@xs4all.nl
> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
>
> Vaibhav,
>
> Just wondering how to test this on DaVinci platforms.
> Could you tell what you did to test this?
>
[Hiremath, Vaibhav] Enable CONFIG_PM in your defconfig and issue following
command -
# echo mem > /sys/power/state
Thanks,
Vaibhav
> Murali Karicheri
> Software Design Engineer
> Texas Instruments Inc.
> Germantown, MD 20874
> phone: 301-407-9583
> email: m-kariche...@ti.com
>
> >-Original Message-
> >From: Hiremath, Vaibhav
> >Sent: Wednesday, November 18, 2009 10:36 AM
> >To: Karicheri, Muralidharan; linux-media@vger.kernel.org
> >Cc: hverk...@xs4all.nl
> >Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
> >
> >
> >> -Original Message-
> >> From: Karicheri, Muralidharan
> >> Sent: Wednesday, November 18, 2009 8:55 PM
> >> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> >> Cc: hverk...@xs4all.nl
> >> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> >> Support
> >>
> >> Vaibhav,
> >>
> >> Did you validate suspend & resume operations on AM3517?
> >>
> >[Hiremath, Vaibhav] yes, I think I mentioned in my patch. Do you
> see any
> >issues?
> >
> >Thanks,
> >Vaibhav
> >
> >> Murali Karicheri
> >> Software Design Engineer
> >> Texas Instruments Inc.
> >> Germantown, MD 20874
> >> phone: 301-407-9583
> >> email: m-kariche...@ti.com
> >>
> >> >-Original Message-
> >> >From: Hiremath, Vaibhav
> >> >Sent: Wednesday, November 18, 2009 6:35 AM
> >> >To: linux-media@vger.kernel.org
> >> >Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath,
> Vaibhav
> >> >Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
> >> >
> >> >From: Vaibhav Hiremath
> >> >
> >> >Validated on AM3517 Platform.
> >> >
> >> >Signed-off-by: Vaibhav Hiremath
> >> >---
> >> > drivers/media/video/davinci/ccdc_hw_device.h |4 +
> >> > drivers/media/video/davinci/dm644x_ccdc.c| 87
> >> >++
> >> > drivers/media/video/davinci/vpfe_capture.c | 29 ++---
> >> > 3 files changed, 112 insertions(+), 8 deletions(-)
> >> >
> >> >diff --git a/drivers/media/video/davinci/ccdc_hw_device.h
> >> >b/drivers/media/video/davinci/ccdc_hw_device.h
> >> >index 86b9b35..2a1ead4 100644
> >> >--- a/drivers/media/video/davinci/ccdc_hw_device.h
> >> >+++ b/drivers/media/video/davinci/ccdc_hw_device.h
> >> >@@ -91,6 +91,10 @@ struct ccdc_hw_ops {
> >> > void (*setfbaddr) (unsigned long addr);
> >> > /* Pointer to function to get field id */
> >> > int (*getfid) (void);
> >> >+
> >> >+ /* suspend/resume support */
> >> >+ void (*save_context)(void);
> >> >+ void (*restore_context)(void);
> >> > };
> >> >
> >> > struct ccdc_hw_device {
> >> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
> >> >b/drivers/media/video/davinci/dm644x_ccdc.c
> >> >index 5dff8d9..fdab823 100644
> >> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
> >> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
> >> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> >> > static int ccdc_addr_size;
> >> > static enum vpfe_hw_if_type ccdc_if_type;
> >> >
> >> >+#define CCDC_SZ_REGS SZ_1K
> >> >+
> >> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
> >> >+
> >> > /* register access routines */
> >> > static inline u32 regr(u32 offset)
> >> > {
> >> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
> >> >vpfe_hw_if_param *params)
> >> > return 0;
> >> > }
> >> >
> >> >+static void ccdc_save_context(void)
> >> >+{
> >> >+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
> >> >+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
> >> >+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
> >> >+ ccdc_ctx[CCDC_PIX_LINE
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
Vaibhav,
Just wondering how to test this on DaVinci platforms.
Could you tell what you did to test this?
Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-kariche...@ti.com
>-Original Message-
>From: Hiremath, Vaibhav
>Sent: Wednesday, November 18, 2009 10:36 AM
>To: Karicheri, Muralidharan; linux-media@vger.kernel.org
>Cc: hverk...@xs4all.nl
>Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
>
>
>> -Original Message-
>> From: Karicheri, Muralidharan
>> Sent: Wednesday, November 18, 2009 8:55 PM
>> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
>> Cc: hverk...@xs4all.nl
>> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
>> Support
>>
>> Vaibhav,
>>
>> Did you validate suspend & resume operations on AM3517?
>>
>[Hiremath, Vaibhav] yes, I think I mentioned in my patch. Do you see any
>issues?
>
>Thanks,
>Vaibhav
>
>> Murali Karicheri
>> Software Design Engineer
>> Texas Instruments Inc.
>> Germantown, MD 20874
>> phone: 301-407-9583
>> email: m-kariche...@ti.com
>>
>> >-Original Message-
>> >From: Hiremath, Vaibhav
>> >Sent: Wednesday, November 18, 2009 6:35 AM
>> >To: linux-media@vger.kernel.org
>> >Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath, Vaibhav
>> >Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
>> >
>> >From: Vaibhav Hiremath
>> >
>> >Validated on AM3517 Platform.
>> >
>> >Signed-off-by: Vaibhav Hiremath
>> >---
>> > drivers/media/video/davinci/ccdc_hw_device.h |4 +
>> > drivers/media/video/davinci/dm644x_ccdc.c| 87
>> >++
>> > drivers/media/video/davinci/vpfe_capture.c | 29 ++---
>> > 3 files changed, 112 insertions(+), 8 deletions(-)
>> >
>> >diff --git a/drivers/media/video/davinci/ccdc_hw_device.h
>> >b/drivers/media/video/davinci/ccdc_hw_device.h
>> >index 86b9b35..2a1ead4 100644
>> >--- a/drivers/media/video/davinci/ccdc_hw_device.h
>> >+++ b/drivers/media/video/davinci/ccdc_hw_device.h
>> >@@ -91,6 +91,10 @@ struct ccdc_hw_ops {
>> >void (*setfbaddr) (unsigned long addr);
>> >/* Pointer to function to get field id */
>> >int (*getfid) (void);
>> >+
>> >+ /* suspend/resume support */
>> >+ void (*save_context)(void);
>> >+ void (*restore_context)(void);
>> > };
>> >
>> > struct ccdc_hw_device {
>> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
>> >b/drivers/media/video/davinci/dm644x_ccdc.c
>> >index 5dff8d9..fdab823 100644
>> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
>> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
>> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
>> > static int ccdc_addr_size;
>> > static enum vpfe_hw_if_type ccdc_if_type;
>> >
>> >+#define CCDC_SZ_REGS SZ_1K
>> >+
>> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
>> >+
>> > /* register access routines */
>> > static inline u32 regr(u32 offset)
>> > {
>> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
>> >vpfe_hw_if_param *params)
>> >return 0;
>> > }
>> >
>> >+static void ccdc_save_context(void)
>> >+{
>> >+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
>> >+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
>> >+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
>> >+ ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
>> >+ ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
>> >+ ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
>> >+ ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
>> >+ ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
>> >+ ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
>> >+ ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
>> >+ ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
>> >+ ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
>> >+ ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
>> >+ ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
>> >+ ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
>> >+ ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
>> >+ ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
>> >+ ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
>> >+ ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> -Original Message-
> From: Karicheri, Muralidharan
> Sent: Wednesday, November 18, 2009 8:55 PM
> To: Hiremath, Vaibhav; linux-media@vger.kernel.org
> Cc: hverk...@xs4all.nl
> Subject: RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume
> Support
>
> Vaibhav,
>
> Did you validate suspend & resume operations on AM3517?
>
[Hiremath, Vaibhav] yes, I think I mentioned in my patch. Do you see any issues?
Thanks,
Vaibhav
> Murali Karicheri
> Software Design Engineer
> Texas Instruments Inc.
> Germantown, MD 20874
> phone: 301-407-9583
> email: m-kariche...@ti.com
>
> >-Original Message-
> >From: Hiremath, Vaibhav
> >Sent: Wednesday, November 18, 2009 6:35 AM
> >To: linux-media@vger.kernel.org
> >Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath, Vaibhav
> >Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
> >
> >From: Vaibhav Hiremath
> >
> >Validated on AM3517 Platform.
> >
> >Signed-off-by: Vaibhav Hiremath
> >---
> > drivers/media/video/davinci/ccdc_hw_device.h |4 +
> > drivers/media/video/davinci/dm644x_ccdc.c| 87
> >++
> > drivers/media/video/davinci/vpfe_capture.c | 29 ++---
> > 3 files changed, 112 insertions(+), 8 deletions(-)
> >
> >diff --git a/drivers/media/video/davinci/ccdc_hw_device.h
> >b/drivers/media/video/davinci/ccdc_hw_device.h
> >index 86b9b35..2a1ead4 100644
> >--- a/drivers/media/video/davinci/ccdc_hw_device.h
> >+++ b/drivers/media/video/davinci/ccdc_hw_device.h
> >@@ -91,6 +91,10 @@ struct ccdc_hw_ops {
> > void (*setfbaddr) (unsigned long addr);
> > /* Pointer to function to get field id */
> > int (*getfid) (void);
> >+
> >+/* suspend/resume support */
> >+void (*save_context)(void);
> >+void (*restore_context)(void);
> > };
> >
> > struct ccdc_hw_device {
> >diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
> >b/drivers/media/video/davinci/dm644x_ccdc.c
> >index 5dff8d9..fdab823 100644
> >--- a/drivers/media/video/davinci/dm644x_ccdc.c
> >+++ b/drivers/media/video/davinci/dm644x_ccdc.c
> >@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> > static int ccdc_addr_size;
> > static enum vpfe_hw_if_type ccdc_if_type;
> >
> >+#define CCDC_SZ_REGSSZ_1K
> >+
> >+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
> >+
> > /* register access routines */
> > static inline u32 regr(u32 offset)
> > {
> >@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
> >vpfe_hw_if_param *params)
> > return 0;
> > }
> >
> >+static void ccdc_save_context(void)
> >+{
> >+ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
> >+ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
> >+ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
> >+ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
> >+ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
> >+ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
> >+ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
> >+ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
> >+ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
> >+ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
> >+ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
> >+ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
> >+ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
> >+ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
> >+ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
> >+ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
> >+ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
> >+ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
> >+ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
> >+ccdc_ctx[CCDC_REC656IF] = regr(CCDC_REC656IF);
> >+ccdc_ctx[CCDC_CCDCFG] = regr(CCDC_CCDCFG);
> >+ccdc_ctx[CCDC_FMTCFG] = regr(CCDC_FMTCFG);
> >+ccdc_ctx[CCDC_FMT_HORZ] = regr(CCDC_FMT_HORZ);
> >+ccdc_ctx[CCDC_FMT_VERT] = regr(CCDC_FMT_VERT);
> >+ccdc_ctx[CCDC_FMT_ADDR0] = regr(CCDC_FMT_ADDR0);
> >+ccdc_ctx[CCDC_FMT_ADDR1] = regr(CCDC_FMT_ADDR1);
> >+ccdc_ctx[CCDC_FMT_ADDR2] = regr(CCDC_FMT_ADDR2);
> >+ccdc_ctx[CCDC_FMT_ADDR3] = regr(CCDC_FMT_ADDR3);
> >+ccdc_ctx[CCDC_FMT_ADDR4] = regr(CCDC_FMT_ADDR4);
> >+ccdc_ctx[CCDC_FMT_ADDR5] = regr(CCDC_FMT_ADDR5);
> >+ccdc_ctx[CCDC_FMT_ADDR6] = regr(CCDC_FMT_ADDR6);
> >+ccdc_ctx[CCDC_FMT_ADDR7] = regr(CCDC_FMT_ADDR7);
> >+ccdc_ctx[CCDC_PRGEVEN_0] = regr(CCDC_PRGEVEN_0);
> >+ccdc_ctx[CCDC_PR
RE: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
Vaibhav,
Did you validate suspend & resume operations on AM3517?
Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
phone: 301-407-9583
email: m-kariche...@ti.com
>-Original Message-
>From: Hiremath, Vaibhav
>Sent: Wednesday, November 18, 2009 6:35 AM
>To: linux-media@vger.kernel.org
>Cc: hverk...@xs4all.nl; Karicheri, Muralidharan; Hiremath, Vaibhav
>Subject: [PATCH] Davinci VPFE Capture: Add Suspend/Resume Support
>
>From: Vaibhav Hiremath
>
>Validated on AM3517 Platform.
>
>Signed-off-by: Vaibhav Hiremath
>---
> drivers/media/video/davinci/ccdc_hw_device.h |4 +
> drivers/media/video/davinci/dm644x_ccdc.c| 87
>++
> drivers/media/video/davinci/vpfe_capture.c | 29 ++---
> 3 files changed, 112 insertions(+), 8 deletions(-)
>
>diff --git a/drivers/media/video/davinci/ccdc_hw_device.h
>b/drivers/media/video/davinci/ccdc_hw_device.h
>index 86b9b35..2a1ead4 100644
>--- a/drivers/media/video/davinci/ccdc_hw_device.h
>+++ b/drivers/media/video/davinci/ccdc_hw_device.h
>@@ -91,6 +91,10 @@ struct ccdc_hw_ops {
> void (*setfbaddr) (unsigned long addr);
> /* Pointer to function to get field id */
> int (*getfid) (void);
>+
>+ /* suspend/resume support */
>+ void (*save_context)(void);
>+ void (*restore_context)(void);
> };
>
> struct ccdc_hw_device {
>diff --git a/drivers/media/video/davinci/dm644x_ccdc.c
>b/drivers/media/video/davinci/dm644x_ccdc.c
>index 5dff8d9..fdab823 100644
>--- a/drivers/media/video/davinci/dm644x_ccdc.c
>+++ b/drivers/media/video/davinci/dm644x_ccdc.c
>@@ -88,6 +88,10 @@ static void *__iomem ccdc_base_addr;
> static int ccdc_addr_size;
> static enum vpfe_hw_if_type ccdc_if_type;
>
>+#define CCDC_SZ_REGS SZ_1K
>+
>+static u32 ccdc_ctx[CCDC_SZ_REGS / sizeof(u32)];
>+
> /* register access routines */
> static inline u32 regr(u32 offset)
> {
>@@ -834,6 +838,87 @@ static int ccdc_set_hw_if_params(struct
>vpfe_hw_if_param *params)
> return 0;
> }
>
>+static void ccdc_save_context(void)
>+{
>+ ccdc_ctx[CCDC_PCR] = regr(CCDC_PCR);
>+ ccdc_ctx[CCDC_SYN_MODE] = regr(CCDC_SYN_MODE);
>+ ccdc_ctx[CCDC_HD_VD_WID] = regr(CCDC_HD_VD_WID);
>+ ccdc_ctx[CCDC_PIX_LINES] = regr(CCDC_PIX_LINES);
>+ ccdc_ctx[CCDC_HORZ_INFO] = regr(CCDC_HORZ_INFO);
>+ ccdc_ctx[CCDC_VERT_START] = regr(CCDC_VERT_START);
>+ ccdc_ctx[CCDC_VERT_LINES] = regr(CCDC_VERT_LINES);
>+ ccdc_ctx[CCDC_CULLING] = regr(CCDC_CULLING);
>+ ccdc_ctx[CCDC_HSIZE_OFF] = regr(CCDC_HSIZE_OFF);
>+ ccdc_ctx[CCDC_SDOFST] = regr(CCDC_SDOFST);
>+ ccdc_ctx[CCDC_SDR_ADDR] = regr(CCDC_SDR_ADDR);
>+ ccdc_ctx[CCDC_CLAMP] = regr(CCDC_CLAMP);
>+ ccdc_ctx[CCDC_DCSUB] = regr(CCDC_DCSUB);
>+ ccdc_ctx[CCDC_COLPTN] = regr(CCDC_COLPTN);
>+ ccdc_ctx[CCDC_BLKCMP] = regr(CCDC_BLKCMP);
>+ ccdc_ctx[CCDC_FPC] = regr(CCDC_FPC);
>+ ccdc_ctx[CCDC_FPC_ADDR] = regr(CCDC_FPC_ADDR);
>+ ccdc_ctx[CCDC_VDINT] = regr(CCDC_VDINT);
>+ ccdc_ctx[CCDC_ALAW] = regr(CCDC_ALAW);
>+ ccdc_ctx[CCDC_REC656IF] = regr(CCDC_REC656IF);
>+ ccdc_ctx[CCDC_CCDCFG] = regr(CCDC_CCDCFG);
>+ ccdc_ctx[CCDC_FMTCFG] = regr(CCDC_FMTCFG);
>+ ccdc_ctx[CCDC_FMT_HORZ] = regr(CCDC_FMT_HORZ);
>+ ccdc_ctx[CCDC_FMT_VERT] = regr(CCDC_FMT_VERT);
>+ ccdc_ctx[CCDC_FMT_ADDR0] = regr(CCDC_FMT_ADDR0);
>+ ccdc_ctx[CCDC_FMT_ADDR1] = regr(CCDC_FMT_ADDR1);
>+ ccdc_ctx[CCDC_FMT_ADDR2] = regr(CCDC_FMT_ADDR2);
>+ ccdc_ctx[CCDC_FMT_ADDR3] = regr(CCDC_FMT_ADDR3);
>+ ccdc_ctx[CCDC_FMT_ADDR4] = regr(CCDC_FMT_ADDR4);
>+ ccdc_ctx[CCDC_FMT_ADDR5] = regr(CCDC_FMT_ADDR5);
>+ ccdc_ctx[CCDC_FMT_ADDR6] = regr(CCDC_FMT_ADDR6);
>+ ccdc_ctx[CCDC_FMT_ADDR7] = regr(CCDC_FMT_ADDR7);
>+ ccdc_ctx[CCDC_PRGEVEN_0] = regr(CCDC_PRGEVEN_0);
>+ ccdc_ctx[CCDC_PRGEVEN_1] = regr(CCDC_PRGEVEN_1);
>+ ccdc_ctx[CCDC_PRGODD_0] = regr(CCDC_PRGODD_0);
>+ ccdc_ctx[CCDC_PRGODD_1] = regr(CCDC_PRGODD_1);
>+ ccdc_ctx[CCDC_VP_OUT] = regr(CCDC_VP_OUT);
>+}
>+
>+static void ccdc_restore_context(void)
>+{
>+ regw(ccdc_ctx[CCDC_SYN_MODE], CCDC_SYN_MODE);
>+ regw(ccdc_ctx[CCDC_HD_VD_WID], CCDC_HD_VD_WID);
>+ regw(ccdc_ctx[CCDC_PIX_LINES], CCDC_PIX_LINES);
>+ regw(ccdc_ctx[CCDC_HORZ_INFO], CCDC_HORZ_INFO);
>+ regw(ccdc_ctx[CCDC_VERT_START], CCDC_VERT_START);
>+ regw(ccdc_ctx[CCDC_VERT_LINES], CCDC_VERT_LINES);
>+ regw(ccdc_ctx[CCDC_CULLING], CCDC_CULLING);
>+ regw(ccdc_ctx[CCDC_HSIZE_OFF], CCDC_HSIZE_OFF);
>+ regw(ccdc_ctx[CCDC_SDOFST], CCDC_SDOFST);
>+ regw(ccdc_ctx[CCDC_SDR_ADDR], CCDC_SDR_ADDR);
>+ regw(ccdc_ctx[CCDC_CLAMP], CCDC_CLAMP);
>+ regw(ccdc_ctx[CCDC_DCSUB], CCDC_DCSUB);
>+ regw(ccdc_ctx[CCDC_COLPTN], CCDC_COLPTN);
>+ regw(ccdc_ctx[CCDC_BLKCMP], CCDC_BLKCMP);
>+ regw(ccdc_ctx[CCDC_FPC], CCDC_FPC);
>+ regw(ccdc_ctx[CCDC_FPC_ADDR], CCDC_FPC_ADDR);
>+ regw(ccdc_ctx[CCDC_
