Re: [PATCH 00/10] ARM: davinci: add vpif display support

[Hans Verkuil]

On 09/03/17 12:00, Bartosz Golaszewski wrote:
> 2017-03-09 11:53 GMT+01:00 Hans Verkuil :
>> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>>> The following series adds support for v4l2 display on da850-evm with
>>> a UI board in device tree boot mode.
>>
>> As far as I could tell from the comments this patch series will see a
>> second version, so I am marking it as 'Changes Requested' in patchwork.
>>
>> If I'm wrong, then please let me know.
>>
>> Regards,
>>
>> Hans
>>
> 
> Hi Hans,
> 
> this series has since been split into several smaller ones and most of
> the code is already merged into Sekhar Nori's DaVinci tree.
> 
> There are two series that still need merging with the media tree[1][2].

Merged, thanks!

Regards,

Hans

> 
> Thanks,
> Bartosz Golaszewski
> 
> [1] https://lkml.org/lkml/2017/2/28/426
> [2] https://patchwork.kernel.org/patch/9577965/
> 

Re: [PATCH 00/10] ARM: davinci: add vpif display support

[Hans Verkuil]

On 07/02/17 17:41, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.

As far as I could tell from the comments this patch series will see a
second version, so I am marking it as 'Changes Requested' in patchwork.

If I'm wrong, then please let me know.

Regards,

Hans

> 
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
> 
> Patch 6/10 enables the relevant modules in the defconfig file.
> 
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
> 
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
> 
> The last patch adds the pdata quirks necessary to enable v4l2 display.
> 
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
> 
> [1] https://github.com/brgl/yavta davinci/vpif-display
> 
> Bartosz Golaszewski (10):
>   media: dt-bindings: vpif: fix whitespace errors
>   ARM: dts: da850-evm: fix whitespace errors
>   media: dt-bindings: vpif: extend the example with an output port
>   ARM: dts: da850-evm: add the output port to the vpif node
>   ARM: dts: da850: add vpif video display pins
>   ARM: davinci_all_defconfig: enable VPIF display modules
>   ARM: davinci: fix a whitespace error
>   ARM: davinci: fix the DT boot on da850-evm
>   media: vpif: use a configurable i2c_adapter_id for vpif display
>   ARM: davinci: add pdata-quirks for da850-evm vpif display
> 
>  .../devicetree/bindings/media/ti,da850-vpif.txt| 45 ---
>  arch/arm/boot/dts/da850-evm.dts| 26 +++---
>  arch/arm/boot/dts/da850.dtsi   | 25 +-
>  arch/arm/configs/davinci_all_defconfig |  2 +
>  arch/arm/mach-davinci/board-da850-evm.c|  1 +
>  arch/arm/mach-davinci/pdata-quirks.c   | 92 
> --
>  drivers/media/platform/davinci/vpif_display.c  |  2 +-
>  include/media/davinci/vpif_types.h |  1 +
>  8 files changed, 164 insertions(+), 30 deletions(-)
> 

Re: [PATCH 00/10] ARM: davinci: add vpif display support

[Bartosz Golaszewski]

2017-03-09 11:53 GMT+01:00 Hans Verkuil :
> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>
> As far as I could tell from the comments this patch series will see a
> second version, so I am marking it as 'Changes Requested' in patchwork.
>
> If I'm wrong, then please let me know.
>
> Regards,
>
> Hans
>

Hi Hans,

this series has since been split into several smaller ones and most of
the code is already merged into Sekhar Nori's DaVinci tree.

There are two series that still need merging with the media tree[1][2].

Thanks,
Bartosz Golaszewski

[1] https://lkml.org/lkml/2017/2/28/426
[2] https://patchwork.kernel.org/patch/9577965/

Re: [PATCH 00/10] ARM: davinci: add vpif display support

[Bartosz Golaszewski]

2017-02-13 10:22 GMT+01:00 Sekhar Nori :
> Hi Bartosz,
>
> On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>>
>> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
>> errors in dts files and bindings, extend the example and the dts for
>> da850-evm with the output port and address the pinmuxing.
>>
>> Patch 6/10 enables the relevant modules in the defconfig file.
>>
>> Patches 7/10 and 8/10 fix two already existing bugs encountered
>> during development.
>>
>> Patch 9/10 make it possible to use a different i2c adapter in the
>> vpif display driver.
>>
>> The last patch adds the pdata quirks necessary to enable v4l2 display.
>>
>> Tested with a modified version of yavta[1] as gstreamer support for
>> v4l2 seems to be broken and results in picture artifacts.
>>
>> [1] https://github.com/brgl/yavta davinci/vpif-display
>
> Can you also share the command line you used ?
>
> Thanks,
> Sekhar

Will do. I'll also send separate sets of patches for your different
branches as advised by Kevin.

Thanks,
Bartosz

Re: [PATCH 00/10] ARM: davinci: add vpif display support

[Sekhar Nori]

Hi Bartosz,

On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
> 
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
> 
> Patch 6/10 enables the relevant modules in the defconfig file.
> 
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
> 
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
> 
> The last patch adds the pdata quirks necessary to enable v4l2 display.
> 
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
> 
> [1] https://github.com/brgl/yavta davinci/vpif-display

Can you also share the command line you used ?

Thanks,
Sekhar

[PATCH 00/10] ARM: davinci: add vpif display support

[Bartosz Golaszewski]

The following series adds support for v4l2 display on da850-evm with
a UI board in device tree boot mode.

Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
errors in dts files and bindings, extend the example and the dts for
da850-evm with the output port and address the pinmuxing.

Patch 6/10 enables the relevant modules in the defconfig file.

Patches 7/10 and 8/10 fix two already existing bugs encountered
during development.

Patch 9/10 make it possible to use a different i2c adapter in the
vpif display driver.

The last patch adds the pdata quirks necessary to enable v4l2 display.

Tested with a modified version of yavta[1] as gstreamer support for
v4l2 seems to be broken and results in picture artifacts.

[1] https://github.com/brgl/yavta davinci/vpif-display

Bartosz Golaszewski (10):
  media: dt-bindings: vpif: fix whitespace errors
  ARM: dts: da850-evm: fix whitespace errors
  media: dt-bindings: vpif: extend the example with an output port
  ARM: dts: da850-evm: add the output port to the vpif node
  ARM: dts: da850: add vpif video display pins
  ARM: davinci_all_defconfig: enable VPIF display modules
  ARM: davinci: fix a whitespace error
  ARM: davinci: fix the DT boot on da850-evm
  media: vpif: use a configurable i2c_adapter_id for vpif display
  ARM: davinci: add pdata-quirks for da850-evm vpif display

 .../devicetree/bindings/media/ti,da850-vpif.txt| 45 ---
 arch/arm/boot/dts/da850-evm.dts| 26 +++---
 arch/arm/boot/dts/da850.dtsi   | 25 +-
 arch/arm/configs/davinci_all_defconfig |  2 +
 arch/arm/mach-davinci/board-da850-evm.c|  1 +
 arch/arm/mach-davinci/pdata-quirks.c   | 92 --
 drivers/media/platform/davinci/vpif_display.c  |  2 +-
 include/media/davinci/vpif_types.h |  1 +
 8 files changed, 164 insertions(+), 30 deletions(-)

-- 
2.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html