2012-10-22 12:13:44

by Lad, Prabhakar

[permalink] [raw]
Subject: [PATCH RESEND] ARM: dm365: replace V4L2_OUT_CAP_CUSTOM_TIMINGS with V4L2_OUT_CAP_DV_TIMINGS

From: Lad, Prabhakar <[email protected]>

This patch replaces V4L2_OUT_CAP_CUSTOM_TIMINGS macro with
V4L2_OUT_CAP_DV_TIMINGS. As V4L2_OUT_CAP_CUSTOM_TIMINGS is being phased
out.

Signed-off-by: Lad, Prabhakar <[email protected]>
Signed-off-by: Manjunath Hadli <[email protected]>
Cc: Sekhar Nori <[email protected]>
---
Resending the patch since, it didn't reach the DLOS mailing list.

This patch is based on the following patch series,
ARM: davinci: dm365 EVM: add support for VPBE display
(https://patchwork.kernel.org/patch/1295071/)

arch/arm/mach-davinci/board-dm365-evm.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c
index 2924d61..771abb5 100644
--- a/arch/arm/mach-davinci/board-dm365-evm.c
+++ b/arch/arm/mach-davinci/board-dm365-evm.c
@@ -514,7 +514,7 @@ static struct vpbe_output dm365evm_vpbe_outputs[] = {
.index = 1,
.name = "Component",
.type = V4L2_OUTPUT_TYPE_ANALOG,
- .capabilities = V4L2_OUT_CAP_CUSTOM_TIMINGS,
+ .capabilities = V4L2_OUT_CAP_DV_TIMINGS,
},
.subdev_name = VPBE_VENC_SUBDEV_NAME,
.default_mode = "480p59_94",
--
1.7.4.1


2012-10-23 09:49:33

by Sergei Shtylyov

[permalink] [raw]
Subject: Re: [PATCH RESEND] ARM: dm365: replace V4L2_OUT_CAP_CUSTOM_TIMINGS with V4L2_OUT_CAP_DV_TIMINGS

Hello.

On 22-10-2012 16:12, Prabhakar Lad wrote:

> From: Lad, Prabhakar <[email protected]>

> This patch replaces V4L2_OUT_CAP_CUSTOM_TIMINGS macro with
> V4L2_OUT_CAP_DV_TIMINGS. As V4L2_OUT_CAP_CUSTOM_TIMINGS is being phased
> out.

> Signed-off-by: Lad, Prabhakar <[email protected]>
> Signed-off-by: Manjunath Hadli <[email protected]>
> Cc: Sekhar Nori <[email protected]>
> ---
> Resending the patch since, it didn't reach the DLOS mailing list.

> This patch is based on the following patch series,
> ARM: davinci: dm365 EVM: add support for VPBE display
> (https://patchwork.kernel.org/patch/1295071/)

> arch/arm/mach-davinci/board-dm365-evm.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c
> index 2924d61..771abb5 100644
> --- a/arch/arm/mach-davinci/board-dm365-evm.c
> +++ b/arch/arm/mach-davinci/board-dm365-evm.c
> @@ -514,7 +514,7 @@ static struct vpbe_output dm365evm_vpbe_outputs[] = {
> .index = 1,
> .name = "Component",
> .type = V4L2_OUTPUT_TYPE_ANALOG,
> - .capabilities = V4L2_OUT_CAP_CUSTOM_TIMINGS,
> + .capabilities = V4L2_OUT_CAP_DV_TIMINGS,

Why this extra space after '='?

WBR, Sergei

2012-10-23 09:57:08

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH RESEND] ARM: dm365: replace V4L2_OUT_CAP_CUSTOM_TIMINGS with V4L2_OUT_CAP_DV_TIMINGS

Hi Sergei,

On Tue, Oct 23, 2012 at 3:18 PM, Sergei Shtylyov <[email protected]> wrote:
> Hello.
>
>
> On 22-10-2012 16:12, Prabhakar Lad wrote:
>
>> From: Lad, Prabhakar <[email protected]>
>
>
>> This patch replaces V4L2_OUT_CAP_CUSTOM_TIMINGS macro with
>> V4L2_OUT_CAP_DV_TIMINGS. As V4L2_OUT_CAP_CUSTOM_TIMINGS is being phased
>> out.
>
>
>> Signed-off-by: Lad, Prabhakar <[email protected]>
>> Signed-off-by: Manjunath Hadli <[email protected]>
>> Cc: Sekhar Nori <[email protected]>
>> ---
>> Resending the patch since, it didn't reach the DLOS mailing list.
>
>
>> This patch is based on the following patch series,
>> ARM: davinci: dm365 EVM: add support for VPBE display
>> (https://patchwork.kernel.org/patch/1295071/)
>
>
>> arch/arm/mach-davinci/board-dm365-evm.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/board-dm365-evm.c
>> b/arch/arm/mach-davinci/board-dm365-evm.c
>> index 2924d61..771abb5 100644
>> --- a/arch/arm/mach-davinci/board-dm365-evm.c
>> +++ b/arch/arm/mach-davinci/board-dm365-evm.c
>> @@ -514,7 +514,7 @@ static struct vpbe_output dm365evm_vpbe_outputs[] = {
>> .index = 1,
>> .name = "Component",
>> .type = V4L2_OUTPUT_TYPE_ANALOG,
>> - .capabilities = V4L2_OUT_CAP_CUSTOM_TIMINGS,
>> + .capabilities = V4L2_OUT_CAP_DV_TIMINGS,
>
>
> Why this extra space after '='?
>
My Bad, I'll post a v2 fixing it.

Regards,
--Prabhakar

> WBR, Sergei
>