Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi Hans, On Wed, 2017-09-20 at 15:12 +0200, Hans Verkuil wrote: [...] > I don't like it :-) > > Currently g_mbus_config returns (and I quote from v4l2-mediabus.h): "How > many lanes the client can use". I.e. the capabilities of the HW. > > If we are going to use this to communicate how many

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi Hans, On Wed, Sep 20, 2017 at 03:12:03PM +0200, Hans Verkuil wrote: > On 09/20/17 14:50, Sakari Ailus wrote: > > Hi Hans and others, > > > > On Wed, Sep 20, 2017 at 01:24:02PM +0200, Hans Verkuil wrote: > >> On 09/20/17 13:00, Dave Stevenson wrote: > >>> On 20 September 2017 at 11:23, Philipp

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On 09/20/17 14:50, Sakari Ailus wrote: > Hi Hans and others, > > On Wed, Sep 20, 2017 at 01:24:02PM +0200, Hans Verkuil wrote: >> On 09/20/17 13:00, Dave Stevenson wrote: >>> On 20 September 2017 at 11:23, Philipp Zabel wrote: Hi, On Wed, 2017-09-20 at

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi Hans and others, On Wed, Sep 20, 2017 at 01:24:02PM +0200, Hans Verkuil wrote: > On 09/20/17 13:00, Dave Stevenson wrote: > > On 20 September 2017 at 11:23, Philipp Zabel wrote: > >> Hi, > >> > >> On Wed, 2017-09-20 at 10:14 +0100, Dave Stevenson wrote: > >>> Hi Mauro

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On 09/20/17 14:23, Dave Stevenson wrote: > On 20 September 2017 at 12:24, Hans Verkuil wrote: >> On 09/20/17 13:00, Dave Stevenson wrote: >>> On 20 September 2017 at 11:23, Philipp Zabel wrote: Hi, On Wed, 2017-09-20 at 10:14 +0100, Dave

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On Wed, 2017-09-20 at 13:24 +0200, Hans Verkuil wrote: > On 09/20/17 13:00, Dave Stevenson wrote: [...] > > It is communicated over the subdevice API - tc358743_g_mbus_config > > reports back the appropriate number of lanes to the receiver > > subdevice. > > A suitable

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On 20 September 2017 at 12:24, Hans Verkuil wrote: > On 09/20/17 13:00, Dave Stevenson wrote: >> On 20 September 2017 at 11:23, Philipp Zabel wrote: >>> Hi, >>> >>> On Wed, 2017-09-20 at 10:14 +0100, Dave Stevenson wrote: Hi Mauro & Philipp

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On 09/20/17 13:00, Dave Stevenson wrote: > On 20 September 2017 at 11:23, Philipp Zabel wrote: >> Hi, >> >> On Wed, 2017-09-20 at 10:14 +0100, Dave Stevenson wrote: >>> Hi Mauro & Philipp >>> >>> On 19 September 2017 at 17:49, Mauro Carvalho Chehab >>>

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

On 20 September 2017 at 11:23, Philipp Zabel wrote: > Hi, > > On Wed, 2017-09-20 at 10:14 +0100, Dave Stevenson wrote: >> Hi Mauro & Philipp >> >> On 19 September 2017 at 17:49, Mauro Carvalho Chehab >> wrote: >> > Em Tue, 19 Sep 2017 17:24:45

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi, On Wed, 2017-09-20 at 10:14 +0100, Dave Stevenson wrote: > Hi Mauro & Philipp > > On 19 September 2017 at 17:49, Mauro Carvalho Chehab > wrote: > > Em Tue, 19 Sep 2017 17:24:45 +0200 > > Philipp Zabel escreveu: > > > > > Hi Dave, > > > >

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi Mauro & Philipp On 19 September 2017 at 17:49, Mauro Carvalho Chehab wrote: > Em Tue, 19 Sep 2017 17:24:45 +0200 > Philipp Zabel escreveu: > >> Hi Dave, >> >> On Tue, 2017-09-19 at 14:08 +0100, Dave Stevenson wrote: >> > The existing fixed

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Em Tue, 19 Sep 2017 17:24:45 +0200 Philipp Zabel escreveu: > Hi Dave, > > On Tue, 2017-09-19 at 14:08 +0100, Dave Stevenson wrote: > > The existing fixed value of 16 worked for UYVY 720P60 over > > 2 lanes at 594MHz, or UYVY 1080P60 over 4 lanes. (RGB888 > > 1080P60

Re: [PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

Hi Dave, On Tue, 2017-09-19 at 14:08 +0100, Dave Stevenson wrote: > The existing fixed value of 16 worked for UYVY 720P60 over > 2 lanes at 594MHz, or UYVY 1080P60 over 4 lanes. (RGB888 > 1080P60 needs 6 lanes at 594MHz). > It doesn't allow for lower resolutions to work as the FIFO > underflows.

[PATCH 2/3] [media] tc358743: Increase FIFO level to 300.

The existing fixed value of 16 worked for UYVY 720P60 over 2 lanes at 594MHz, or UYVY 1080P60 over 4 lanes. (RGB888 1080P60 needs 6 lanes at 594MHz). It doesn't allow for lower resolutions to work as the FIFO underflows. Using a value of 300 works for all resolutions down to VGA60, and the