: Lucas Stach
---
v2: Add comments why it is safe to skip the CPU sync.
---
.../common/videobuf2/videobuf2-dma-contig.c | 23 +--
1 file changed, 16 insertions(+), 7 deletions(-)
diff --git a/drivers/media/common/videobuf2/videobuf2-dma-contig.c
b/drivers/media/common/videobuf2
Hi Hans,
Am Dienstag, den 12.03.2019, 11:03 +0100 schrieb Lucas Stach:
> Hi Hans,
>
> Am Dienstag, den 12.03.2019, 08:57 +0100 schrieb Hans Verkuil:
> > Pawel and/or Marek, can you take a look at this?
Seems nobody else is interested in this patch. Would you be willing to
Hi Hans,
Am Dienstag, den 12.03.2019, 08:57 +0100 schrieb Hans Verkuil:
> Pawel and/or Marek, can you take a look at this?
>
> It looks sane to me, but I'd like to have a second opinion as well before
> merging this.
>
> On 2/28/19 8:19 AM, Lucas Stach wrote:
> >
.
Signed-off-by: Lucas Stach
---
drivers/media/common/videobuf2/videobuf2-dma-contig.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/media/common/videobuf2/videobuf2-dma-contig.c
b/drivers/media/common/videobuf2/videobuf2-dma-contig.c
index aff0ab7bf83d
Am Freitag, den 19.10.2018, 12:54 +0200 schrieb Michael Grzeschik:
> > From: Steven Toth
>
> This patch is based on the work of Steven Toth. He reverse engineered
> the driver by tracing the windows driver.
>
> https://github.com/stoth68000/hdcapm/
>
> > Signed-off-by: Steven Toth
> > Signed-o
: Lucas Stach
---
drivers/media/platform/coda/coda-common.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/media/platform/coda/coda-common.c
b/drivers/media/platform/coda/coda-common.c
index c7631e117dd3..1ae15d4ec5ed 100644
--- a/drivers/media/platform/coda/coda
Am Mittwoch, den 16.05.2018, 11:42 +0200 schrieb Daniel Vetter:
> On Tue, May 15, 2018 at 01:16:30PM +0100, Chris Wilson wrote:
> > Quoting Ezequiel Garcia (2018-05-14 22:28:31)
> > > On Mon, 2018-05-14 at 18:48 +0200, Daniel Vetter wrote:
> > > > On Fri, May 11, 2018 at 08:27:41AM +0100, Chris Wil
Am Mittwoch, den 25.04.2018, 13:44 -0400 schrieb Alex Deucher:
> On Wed, Apr 25, 2018 at 2:41 AM, Christoph Hellwig > wrote:
> > On Wed, Apr 25, 2018 at 02:24:36AM -0400, Alex Deucher wrote:
> > > > It has a non-coherent transaction mode (which the chipset can opt to
> > > > not implement and stil
v2 with that fixed.
Regards,
Lucas
> Regards,
> Christian.
>
> Am 11.01.2018 um 11:43 schrieb Lucas Stach:
> > Did this fall through the cracks over the holidays? It really has made
> > my work much easier while reworking some of the reservation object
> > handlin
Did this fall through the cracks over the holidays? It really has made
my work much easier while reworking some of the reservation object
handling in etnaviv and I think it might benefit others.
Regards,
Lucas
Am Freitag, den 01.12.2017, 12:12 +0100 schrieb Lucas Stach:
> This adds lock
This adds lockdep asserts to the reservation functions which state in their
documentation that obj->lock must be held. Allows builds with PROVE_LOCKING
enabled to check that the locking requirements are met.
Signed-off-by: Lucas Stach
---
drivers/dma-buf/reservation.c | 8
incl
Am Donnerstag, den 20.07.2017, 14:51 +0200 schrieb Daniel Vetter:
> It makes debugging a massive pain.
It is also considered very bad style to BUG the kernel on anything other
than filesystem eating catastrophic failures.
Reviewed-by: Lucas Stach
> Signed-off-by: Daniel Vetter
>
Am Donnerstag, den 20.04.2017, 11:12 -0300 schrieb Ezequiel Garcia:
> On 20 April 2017 at 07:10, Anuradha Ranasinghe
> wrote:
> > Dear All,
> >
> > This issue is associated to the Linux Mainline Kernel 4.1.15.2 (branch2)
> > tw686x upstream driver and IMX6Q platform.
> >
> > We have an analog cam
Am Donnerstag, den 06.04.2017, 16:03 +0200 schrieb Philipp Zabel:
> TRY_FMT already disables the YUYV format if the VDOA is not available.
> ENUM_FMT must do the same.
>
> Fixes: d40e98c13b3e ("[media] coda: support YUYV output if VDOA is used")
> Signed-off-by: Philipp Z
The h.264 standard allows up to 16 reference frame for the high profile
and we need one additional internal framebuffer when the VDOA is in use.
Lift the current maximum of 8 internal framebuffers to allow playback
of those video streams.
Signed-off-by: Lucas Stach
---
drivers/media/platform
and
before destroying the currently active context.
Signed-off-by: Lucas Stach
---
drivers/media/platform/coda/imx-vdoa.c | 49 +++---
1 file changed, 33 insertions(+), 16 deletions(-)
diff --git a/drivers/media/platform/coda/imx-vdoa.c
b/drivers/media/platform/coda
The mvcol buffer needs to be placed behind the chroma plane(s), so
use the real offset including any required rounding.
Signed-off-by: Lucas Stach
---
drivers/media/platform/coda/coda-bit.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/media/platform/coda
decode run we would hold the context and wait for userspace to queue more
buffers.
Signed-off-by: Lucas Stach
---
drivers/media/platform/coda/coda-bit.c| 20
drivers/media/platform/coda/coda-common.c | 3 +++
drivers/media/platform/coda/coda.h| 1 +
3 files changed
Am Mittwoch, den 05.04.2017, 13:18 +0200 schrieb Pavel Machek:
> Hi!
>
> > + * video stream multiplexer controlled via gpio or syscon
> > + *
> > + * Copyright (C) 2013 Pengutronix, Sascha Hauer
> > + * Copyright (C) 2016 Pengutronix, Philipp Zabel
>
> This is actually quite interesting. Same e
Am Montag, den 14.03.2016, 15:19 -0300 schrieb Javier Martinez Canillas:
> Hello Lucas,
>
> On Mon, Mar 14, 2016 at 12:23 PM, Lucas Stach wrote:
> > From: Philipp Zabel
> >
> > By looking at the endpoint flags, it can be determined whether the link
> > s
From: Philipp Zabel
Make sure to not overwrite decoder->norm when setting the standard
in hardware, but only when instructed by V4L2 API calls.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 56 +
1 f
.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 34 --
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c
index 67312c9d83c1..f6720d1d09ea 100644
--- a
From: Philipp Zabel
Regmap provides built in debugging, caching and provides dedicated accessors
for bit manipulations in registers, which make the following changes a lot
simpler.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 194
From: Philipp Zabel
This patch adds userspace V4L2 subdevice API support.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 282 +++-
1 file changed, 223 insertions(+), 59 deletions(-)
diff --git a/drivers/media
From: Philipp Zabel
This patch adds an optional interrupt handler to handle the sync
lock interrupt and sync lock status.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 103 ++--
drivers/media/i2c
From: Philipp Zabel
To avoid short frames on stream start, keep output pins at high impedance
while we are not properly locked onto the input signal.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 12 +---
1 file changed, 9 insertions(+), 3
From: Philipp Zabel
To trigger standard autodetection only the reset part of the routine
is necessary. Split this out to make it callable on its own.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 17 +
1 file changed, 13 insertions
From: Philipp Zabel
If cropping isn't set explicitly by userspace, reset it to the maximum
possible rectangle in subdevice open if a standard change is detected.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 18 +-
1 file ch
From: Philipp Zabel
To allow optional interrupt support, we want to configure the pin settings
dynamically. Move those register accesses out of the static initialization
tables.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 19
From: Philipp Zabel
This patch adds an optional interrupt handler to handle the sync
lock interrupt and sync lock status.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 103 ++--
drivers/media/i2c
From: Philipp Zabel
Regmap provides built in debugging, caching and provides dedicated accessors
for bit manipulations in registers, which make the following changes a lot
simpler.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 194 ++--
From: Philipp Zabel
If cropping isn't set explicitly by userspace, reset it to the maximum
possible rectangle in subdevice open if a standard change is detected.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 18 +-
1 file changed, 13 insertions(+), 5 deletions(
From: Philipp Zabel
To allow optional interrupt support, we want to configure the pin settings
dynamically. Move those register accesses out of the static initialization
tables.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 19 +++
drivers/media/i2c/tvp5150
From: Philipp Zabel
To avoid short frames on stream start, keep output pins at high impedance
while we are not properly locked onto the input signal.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/dr
From: Philipp Zabel
To trigger standard autodetection only the reset part of the routine
is necessary. Split this out to make it callable on its own.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --gi
From: Philipp Zabel
By looking at the endpoint flags, it can be determined whether the link
should be of V4L2_MBUS_PARALLEL or V4L2_MBUS_BT656 type. Disable the
dedicated HSYNC/VSYNC outputs in BT.656 mode.
For devices that are not instantiated through DT the current behavior
is preserved.
Sign
From: Philipp Zabel
This patch adds userspace V4L2 subdevice API support.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
v2: Allow the driver to be built without MEDIA_CONTROLLER and
VIDEO_V4L2_SUBDEV_API, to keep it working for devices that
don't want or nee
From: Philipp Zabel
Make sure to not overwrite decoder->norm when setting the standard
in hardware, but only when instructed by V4L2 API calls.
Signed-off-by: Philipp Zabel
---
drivers/media/i2c/tvp5150.c | 56 +
1 file changed, 31 insertions(+), 25
Am Donnerstag, den 19.11.2015, 01:06 +0800 schrieb kbuild test robot:
> Hi Philipp,
>
> [auto build test ERROR on: v4.4-rc1]
> [also build test ERROR on: next-20151118]
> [cannot apply to: linuxtv-media/master]
>
> url:
> https://github.com/0day-ci/linux/commits/Luc
From: Philipp Zabel
This patch adds an optional interrupt handler to handle the sync
lock interrupt and sync lock status.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 103 ++--
drivers/media/i2c
From: Philipp Zabel
Regmap provides built in debugging, caching and provides dedicated accessors
for bit manipulations in registers, which make the following changes a lot
simpler.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 194
From: Philipp Zabel
To avoid short frames on stream start, keep output pins at high impedance
while we are not properly locked onto the input signal.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 12 +---
1 file changed, 9 insertions(+), 3
From: Philipp Zabel
To allow optional interrupt support, we want to configure the pin settings
dynamically. Move those register accesses out of the static initialization
tables.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 19
From: Philipp Zabel
This patch adds userspace V4L2 subdevice API support.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 259 ++--
1 file changed, 200 insertions(+), 59 deletions(-)
diff --git a/drivers/media
From: Philipp Zabel
Make sure to not overwrite decoder->norm when setting the standard
in hardware, but only when instructed by V4L2 API calls.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 56 +
1 f
From: Philipp Zabel
To trigger standard autodetection only the reset part of the routine
is necessary. Split this out to make it callable on its own.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 17 +
1 file changed, 13 insertions
.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 34 --
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c
index 8670b478dcd6..21cde350e385 100644
--- a
From: Philipp Zabel
If cropping isn't set explicitly by userspace, reset it to the maximum
possible rectangle in subdevice open if a standard change is detected.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
drivers/media/i2c/tvp5150.c | 18 +-
1 file ch
valid(struct drm_connector
> *connector,
> if (ret)
> return MODE_BAD;
>
> + if (mode->clock * 1000 > hdata->max_pixel_clock)
> + return MODE_BAD;
> +
This should be MODE_CLOCK_HIGH
Regards,
Lucas
--
Pengutronix e.K.
art, which is used for example for programming the DMA
channels is using V4L2 pixel formats as a common base. We have patches
to split this out and make this fact more visible. (The IPU core will be
placed aside the Tegra host1x driver)
Regards,
Lucas
--
Pengutronix e.K.
Am Dienstag, den 06.08.2013, 10:14 -0400 schrieb Rob Clark:
> On Tue, Aug 6, 2013 at 8:18 AM, Lucas Stach wrote:
> > Am Dienstag, den 06.08.2013, 12:31 +0100 schrieb Tom Cooksey:
> >> Hi Rob,
> >>
> >> +lkml
> >>
> >> > >>
at/etc.. that is really userspace level knowledge.
> > There are a few exceptions when the kernel needs to know how to setup
> > GTT/etc for tiled buffers, but normally this sort of information is up
> > at the next level up (userspace, and drm_framebuffer in case of
> >
> dma-buf-mgr
> 2. kick off GPU
> processing
> 3. qbuf buf1 into V4L2
> 3. reference buf1
> for reading
>
through dma-buf-mgr
2. kick off GPU processing
3. qbuf buf1 into V4L2
3. reference buf1 for reading
3.1 wait for fence from GPU to
ace for
every syncpoint.
>
> With dmabuf-sync is:
>
> Task A
>
> 1. dma_buf_sync_lock <- synpoint (call by user side)
> 2. CPU writes something to buf1
> 3. dma_buf_sync_unlock <- syncpoint (call by user side)
> 4. copy buf1 to buf2
Am Donnerstag, den 20.06.2013, 09:17 +0100 schrieb Russell King - ARM
Linux:
> On Thu, Jun 20, 2013 at 09:47:07AM +0200, Lucas Stach wrote:
> > Am Donnerstag, den 20.06.2013, 15:43 +0900 schrieb Inki Dae:
> > >
> > > > -Original Message-
> > > >
qbuf (dragging buffer into
device domain, flush caches,
reserve buffer etc.)
|
wait for device oper
Am Mittwoch, den 19.06.2013, 19:44 +0900 schrieb Inki Dae:
>
> > -Original Message-
> > From: Lucas Stach [mailto:l.st...@pengutronix.de]
> > Sent: Wednesday, June 19, 2013 7:22 PM
> > To: Inki Dae
> > Cc: 'Russell King - ARM Linux'; 'linu
Am Mittwoch, den 19.06.2013, 14:45 +0900 schrieb Inki Dae:
>
> > -Original Message-
> > From: Lucas Stach [mailto:l.st...@pengutronix.de]
> > Sent: Tuesday, June 18, 2013 6:47 PM
> > To: Inki Dae
> > Cc: 'Russell King - ARM Linux'; 'linu
cross-device sync.
Regards,
Lucas
--
Pengutronix e.K. | Lucas Stach |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-5076 |
Amtsgericht Hildesheim, HRA 2686
, that's imo a bug with the other driver. If it doesn't export
> something really simple (e.g. contiguous memory which doesn't require any
> mmio resources at all) it should have a cache of exported dma_buf fds so
> that it hands out the same dma_buf every t
61 matches
Mail list logo