[RFC PATCH 0/6] media: davinci: VPIF: add DT support
hverkuil at xs4all.nl
Fri Nov 11 07:36:31 PST 2016
On 10/26/2016 01:55 AM, Kevin Hilman wrote:
> This series attempts to add DT support to the davinci VPIF capture
> I'm not sure I've completely grasped the proper use of the ports and
> endpoints stuff, so this RFC is primarily to get input on whether I'm
> on the right track.
> The last patch is the one where all my questions are, the rest are
> just prep work to ge there.
> Tested on da850-lcdk and was able to do basic frame capture from the
> composite input.
> Series applies on v4.9-rc1
> Kevin Hilman (6):
> [media] davinci: add support for DT init
> ARM: davinci: da8xx: VPIF: enable DT init
> ARM: dts: davinci: da850: add VPIF
> ARM: dts: davinci: da850-lcdk: enable VPIF capture
> [media] davinci: vpif_capture: don't lock over s_stream
> [media] davinci: vpif_capture: get subdevs from DT
Looks good, but wouldn't it be better to do the dts changes last when all the
supporting code is in?
> arch/arm/boot/dts/da850-lcdk.dts | 30 ++++++
> arch/arm/boot/dts/da850.dtsi | 28 +++++
> arch/arm/mach-davinci/da8xx-dt.c | 17 +++
> drivers/media/platform/davinci/vpif.c | 9 ++
> drivers/media/platform/davinci/vpif_capture.c | 150 +++++++++++++++++++++++++-
> include/media/davinci/vpif_types.h | 9 +-
> 6 files changed, 236 insertions(+), 7 deletions(-)
More information about the linux-arm-kernel