[PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

On 06/09/2017 04:23 PM, Pavel Machek wrote:

Hi!

quoted

quoted

quoted

quoted

Steve,

You need to remove the fim node now that you've moved this to V4L2 controls.

Yep, I caught this just after sending the v8 patchset. I'll send
a v9 of this patch.

This needs ack from devicetree people, then it can be merged. Can you
be a bit more forceful getting the ack?

OK, I need an Ack please, he said, in a forceful way. :)

I'd tune the force up a tiny bit more. This is not FreeBSD ;-). You
can read some emails from Linus for inspiration. Or drink few beers
and look at Al Viro's emails.

quoted

In fact Ack's are needed for all the changes to dts sources,
patches 4-14.

Actually, are they? Those should not need acks from device tree
people, just acks from ARM people, and those are easier to get... in
fact they should not need any acks, you should just send them to arm
people and get them merged.

Hans said he prefers to have the dts patches as part of the whole set
rather than submitted separately. But I did add --to's for the ARM
people for 4-14.

1-4 is just a documentation, and you have acks there. (Except 2?)
That's ready to be merged, probably via the media tree? Just make it
clear you want it merged.

Yes, 1-3 now have Acks (binding docs).

15,16 should be ready to. Media tree, too, I guess.

Yes, those have Acks (video-mux entity functions and subdev driver).

drivers/staging is greg. Advantage is these do _not_ need to go after
the dts changes. It is a driver. Actually I'd normally add dts support
after the driver. So you can push it now.

Right, Hans agrees, except that for staging drivers we don't really need
an Ack from Greg.

quoted

quoted

I don't think it makes sense to resubmit v9 before that. This is not a
rocket science.

I guess that makes sense, I'll wait for Ack's from all these patches
before submitting the entire patchset as v9.

You may want to split the series, according to mainainters, or just
ask the maintainers to merge the relevant parts. I believe most of it
can be pushed now.

Good luck,

Thanks!
Steve