RFC media: davinci: VPIF: add DT support
Get series with b4
b4 am [email protected]Versions newest first
- v1 RFC Latest0 of 6 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 6 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [RFC PATCH 0/6] media: davinci: VPIF: add DT support
Quoted 26 lines · Hans Verkuil
Hi Kevin,
Quoted 15 lines · Kevin Hilman
This series attempts to add DT support to the davinci VPIF capture driver.
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
Quoted 7 lines
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 overs_stream[media] davinci:vpif_capture: get subdevs from DT
Quoted 2 lines
Looks good, but wouldn't it be better to do the dts changes last when all the supporting code is in?
I guess it doesn't really matter in this case, because the DT nodes will be nops until the driver changes are in.
Either way, next week I'll repost a non-RFC version and separate out the arch and DT patches, since those will go through Sekhar's davinci tree, and then via arm-soc.
Thanks for the review,
Kevin