Re: [PATCH] media: venus: use contig vb2 ops

2021-03-01 Thread Tomasz Figa
On Mon, Mar 1, 2021 at 7:22 PM Stanimir Varbanov wrote: > > > > On 3/1/21 11:23 AM, Tomasz Figa wrote: > > Hi Alex, Stanimir, > > > > On Wed, Dec 16, 2020 at 12:15 PM Tomasz Figa wrote: > >> > >> On Wed, Dec 16, 2020 at 4:21 AM Nicolas Dufresne > >> wrote: > >>> > >>> Le mardi 15 décembre 2020

Re: [PATCH] media: venus: use contig vb2 ops

2021-03-01 Thread Stanimir Varbanov
On 3/1/21 11:23 AM, Tomasz Figa wrote: > Hi Alex, Stanimir, > > On Wed, Dec 16, 2020 at 12:15 PM Tomasz Figa wrote: >> >> On Wed, Dec 16, 2020 at 4:21 AM Nicolas Dufresne >> wrote: >>> >>> Le mardi 15 décembre 2020 à 15:54 +0200, Stanimir Varbanov a écrit : Hi Tomasz, On 12/15

Re: [PATCH] media: venus: use contig vb2 ops

2021-03-01 Thread Tomasz Figa
Hi Alex, Stanimir, On Wed, Dec 16, 2020 at 12:15 PM Tomasz Figa wrote: > > On Wed, Dec 16, 2020 at 4:21 AM Nicolas Dufresne wrote: > > > > Le mardi 15 décembre 2020 à 15:54 +0200, Stanimir Varbanov a écrit : > > > Hi Tomasz, > > > > > > On 12/15/20 1:47 PM, Tomasz Figa wrote: > > > > On Tue, Dec

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Tomasz Figa
On Wed, Dec 16, 2020 at 4:21 AM Nicolas Dufresne wrote: > > Le mardi 15 décembre 2020 à 15:54 +0200, Stanimir Varbanov a écrit : > > Hi Tomasz, > > > > On 12/15/20 1:47 PM, Tomasz Figa wrote: > > > On Tue, Dec 15, 2020 at 8:16 PM Stanimir Varbanov > > > wrote: > > > > > > > > Hi, > > > > > > > >

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Nicolas Dufresne
Le mardi 15 décembre 2020 à 15:54 +0200, Stanimir Varbanov a écrit : > Hi Tomasz, > > On 12/15/20 1:47 PM, Tomasz Figa wrote: > > On Tue, Dec 15, 2020 at 8:16 PM Stanimir Varbanov > > wrote: > > > > > > Hi, > > > > > > Cc: Robin > > > > > > On 12/14/20 2:57 PM, Alexandre Courbot wrote: > > > >

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Stanimir Varbanov
Hi Tomasz, On 12/15/20 1:47 PM, Tomasz Figa wrote: > On Tue, Dec 15, 2020 at 8:16 PM Stanimir Varbanov > wrote: >> >> Hi, >> >> Cc: Robin >> >> On 12/14/20 2:57 PM, Alexandre Courbot wrote: >>> This driver uses the SG vb2 ops, but effectively only ever accesses the >>> first entry of the SG table

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Robin Murphy
On 2020-12-15 11:47, Tomasz Figa wrote: On Tue, Dec 15, 2020 at 8:16 PM Stanimir Varbanov wrote: Hi, Cc: Robin On 12/14/20 2:57 PM, Alexandre Courbot wrote: This driver uses the SG vb2 ops, but effectively only ever accesses the first entry of the SG table, indicating that it expects a flat

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Tomasz Figa
On Tue, Dec 15, 2020 at 8:16 PM Stanimir Varbanov wrote: > > Hi, > > Cc: Robin > > On 12/14/20 2:57 PM, Alexandre Courbot wrote: > > This driver uses the SG vb2 ops, but effectively only ever accesses the > > first entry of the SG table, indicating that it expects a flat layout. > > Switch it to u

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Robin Murphy
On 2020-12-15 11:16, Stanimir Varbanov wrote: Hi, Cc: Robin On 12/14/20 2:57 PM, Alexandre Courbot wrote: This driver uses the SG vb2 ops, but effectively only ever accesses the first entry of the SG table, indicating that it expects a flat layout. Switch it to use the contiguous ops to make s

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Stanimir Varbanov
Hi, Cc: Robin On 12/14/20 2:57 PM, Alexandre Courbot wrote: > This driver uses the SG vb2 ops, but effectively only ever accesses the > first entry of the SG table, indicating that it expects a flat layout. > Switch it to use the contiguous ops to make sure this expected invariant Under what cir

Re: [PATCH] media: venus: use contig vb2 ops

2020-12-15 Thread Tomasz Figa
On Mon, Dec 14, 2020 at 9:57 PM Alexandre Courbot wrote: > > This driver uses the SG vb2 ops, but effectively only ever accesses the > first entry of the SG table, indicating that it expects a flat layout. > Switch it to use the contiguous ops to make sure this expected invariant > is always enfor

[PATCH] media: venus: use contig vb2 ops

2020-12-14 Thread Alexandre Courbot
This driver uses the SG vb2 ops, but effectively only ever accesses the first entry of the SG table, indicating that it expects a flat layout. Switch it to use the contiguous ops to make sure this expected invariant is always enforced. Since the device is supposed to be behind an IOMMU this should