Re: [PATCH v6 00/13] TVP5150 new features

[Mauro Carvalho Chehab <mchehab+samsung@kernel.org>]

Em Tue, 14 May 2019 22:58:24 +0200
Marco Felsch <m.felsch@pengutronix.de> escreveu:

> Hi Mauro,
> 
> On 19-05-14 17:20, Mauro Carvalho Chehab wrote:
> > Em Tue, 14 May 2019 14:18:24 -0300
> > Mauro Carvalho Chehab <mchehab@kernel.org> escreveu:
> >   
> > > Em Mon, 6 May 2019 07:47:13 +0200
> > > Marco Felsch <m.felsch@pengutronix.de> escreveu:
> > >   
> > > > Hi Mauro,
> > > > 
> > > > I know you are busy but can you have a look on it?    
> > > 
> > > You should really trust on the sub-maintainers for such kind of
> > > reviews :-)  
> 
> I trust all of them and many thanks to Hans and Jacopo for the reviews
> :) I will integrate them this week. The point is that you are the
> maintainer and in that case the one who picks the patches.

Actually, Hans is the one that usually picks V4L2 patches after reviewing.
I pull from his tree and from other media maintainers.

> 
> > > 
> > > I'll take a look today.  
> > 
> > Done. Please notice that I didn't run any test here.  
> 
> Thanks for that will integrate it too and prepare a v7.

Thanks!
Mauro
> 
> Regards,
>   Marco
> 
> >   
> > >   
> > > > 
> > > > Regards,
> > > >   Marco
> > > > 
> > > > On 19-04-15 14:44, Marco Felsch wrote:    
> > > > > Hi,
> > > > > 
> > > > > many thanks to Hans and Jacopo for the feedack :) this v6 address the
> > > > > comments both made on my v5 [1].
> > > > > 
> > > > > In short this is round fixes just some minor issues rather than major
> > > > > ones so the diff to the v5 is really small. The changed patches contain
> > > > > the changelog so I omit it here.
> > > > > 
> > > > > I've tested it on a custom hardware but I can't test the em28xx usb
> > > > > use-case since I haven't such a device. So other testers are welcome :)
> > > > > 
> > > > > Looking forward for your feedack,
> > > > > 
> > > > > 	Marco
> > > > > 
> > > > > [1] https://patchwork.kernel.org/cover/10886903/
> > > > > 
> > > > > Javier Martinez Canillas (1):
> > > > >   partial revert of "[media] tvp5150: add HW input connectors support"
> > > > > 
> > > > > Marco Felsch (11):
> > > > >   dt-bindings: connector: analog: add tv norms property
> > > > >   media: v4l2-fwnode: add v4l2_fwnode_connector
> > > > >   media: v4l2-fwnode: add initial connector parsing support
> > > > >   media: tvp5150: add input source selection of_graph support
> > > > >   media: dt-bindings: tvp5150: Add input port connectors DT bindings
> > > > >   media: tvp5150: add FORMAT_TRY support for get/set selection handlers
> > > > >   media: tvp5150: add s_power callback
> > > > >   media: dt-bindings: tvp5150: cleanup bindings stlye
> > > > >   media: dt-bindings: tvp5150: add optional tvnorms documentation
> > > > >   media: tvp5150: add support to limit tv norms on connector
> > > > >   media: tvp5150: make debug output more readable
> > > > > 
> > > > > Michael Tretter (1):
> > > > >   media: tvp5150: initialize subdev before parsing device tree
> > > > > 
> > > > >  .../display/connector/analog-tv-connector.txt |   4 +
> > > > >  .../devicetree/bindings/media/i2c/tvp5150.txt | 125 +++-
> > > > >  drivers/media/i2c/tvp5150.c                   | 672 +++++++++++++-----
> > > > >  drivers/media/v4l2-core/v4l2-fwnode.c         | 111 +++
> > > > >  include/dt-bindings/media/tvnorms.h           |  56 ++
> > > > >  include/dt-bindings/media/tvp5150.h           |   2 -
> > > > >  include/media/v4l2-connector.h                |  30 +
> > > > >  include/media/v4l2-fwnode.h                   |  49 ++
> > > > >  8 files changed, 859 insertions(+), 190 deletions(-)
> > > > >  create mode 100644 include/dt-bindings/media/tvnorms.h
> > > > >  create mode 100644 include/media/v4l2-connector.h
> > > > > 
> > > > > -- 
> > > > > 2.20.1
> > > > > 
> > > > > 
> > > > >       
> > > >     
> > > 
> > > 
> > > 
> > > Thanks,
> > > Mauro  
> > 
> > 
> > 
> > Thanks,
> > Mauro
> >   
> 



Thanks,
Mauro