2017-03-07 17:14:04

by Lad, Prabhakar

Subject: Re: [PATCH] media: vpif: use a configurable i2c_adapter_id for vpif display

Hi Bartosz,

Thanks for the patch.

On Thu, Feb 16, 2017 at 6:08 PM, Bartosz Golaszewski
<[email protected]> wrote:
>
> The vpif display driver uses a static i2c adapter ID of 1 but on the
> da850-evm board in DT boot mode the i2c adapter ID is actually 0.
>
> Make the adapter ID configurable like it already is for vpif capture.
>
> Signed-off-by: Bartosz Golaszewski <[email protected]>
> Acked-by: Kevin Hilman <[email protected]>
> ---
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>

Acked-by: Lad, Prabhakar <[email protected]>

Cheers,
--Prabhakar Lad

2017-03-08 12:48:23

by Bartosz Golaszewski

Subject: Re: [PATCH] media: vpif: use a configurable i2c_adapter_id for vpif display

2017-03-07 18:12 GMT+01:00 Lad, Prabhakar <[email protected]>:
> Hi Bartosz,
>
> Thanks for the patch.
>
> On Thu, Feb 16, 2017 at 6:08 PM, Bartosz Golaszewski
> <[email protected]> wrote:
>>
>> The vpif display driver uses a static i2c adapter ID of 1 but on the
>> da850-evm board in DT boot mode the i2c adapter ID is actually 0.
>>
>> Make the adapter ID configurable like it already is for vpif capture.
>>
>> Signed-off-by: Bartosz Golaszewski <[email protected]>
>> Acked-by: Kevin Hilman <[email protected]>
>> ---
>> arch/arm/mach-davinci/board-da850-evm.c | 1 +
>> drivers/media/platform/davinci/vpif_display.c | 2 +-
>> include/media/davinci/vpif_types.h | 1 +
>> 3 files changed, 3 insertions(+), 1 deletion(-)
>>
>
> Acked-by: Lad, Prabhakar <[email protected]>
>

Thanks!

Sekhar let me know, that this should be picked up by media. Also:
there are still two DT bindings patches for vpif that can be picked up
- Rob Herring acked them.

Thanks,
Bartosz