Re: [PATCH] drm/etnaviv: fix some off by one bugs

2019-01-07 Thread Daniel Vetter
On Mon, Jan 07, 2019 at 09:42:00AM +0100, Lucas Stach wrote: > Hi Daniel, > > Am Montag, den 24.12.2018, 10:32 +0100 schrieb Daniel Vetter: > > On Fri, Dec 21, 2018 at 9:24 PM Dan Carpenter > om> wrote: > > > > > > I don't think anyone responded to this one? > > > > Maybe time to move etnaviv i

Re: [PATCH] drm/etnaviv: fix some off by one bugs

2019-01-07 Thread Lucas Stach
Hi Daniel, Am Montag, den 24.12.2018, 10:32 +0100 schrieb Daniel Vetter: > On Fri, Dec 21, 2018 at 9:24 PM Dan Carpenter om> wrote: > > > > I don't think anyone responded to this one? > > Maybe time to move etnaviv into drm-misc so that there's a notch more > redundancy in maintainers? Lucas, C

Re: [PATCH] drm/etnaviv: fix some off by one bugs

2018-12-28 Thread Christian Gmeiner
Am Fr., 13. Juli 2018 um 17:00 Uhr schrieb Dan Carpenter : > > The ->nr_signal is the supposed to be the number of elements in the > ->signal array. There was one place where it was 5 but it was supposed > to be 4. That looks like a copy and paste bug. There were also two > checks that were off

Re: [PATCH] drm/etnaviv: fix some off by one bugs

2018-12-24 Thread Daniel Vetter
On Fri, Dec 21, 2018 at 9:24 PM Dan Carpenter wrote: > > I don't think anyone responded to this one? Maybe time to move etnaviv into drm-misc so that there's a notch more redundancy in maintainers? Lucas, Christian, others? -Daniel > > regards, > dan carpenter > > On Fri, Jul 13, 2018 at 06:00:1

Re: [PATCH] drm/etnaviv: fix some off by one bugs

2018-12-21 Thread Dan Carpenter
I don't think anyone responded to this one? regards, dan carpenter On Fri, Jul 13, 2018 at 06:00:18PM +0300, Dan Carpenter wrote: > The ->nr_signal is the supposed to be the number of elements in the > ->signal array. There was one place where it was 5 but it was supposed > to be 4. That looks

[PATCH] drm/etnaviv: fix some off by one bugs

2018-07-13 Thread Dan Carpenter
The ->nr_signal is the supposed to be the number of elements in the ->signal array. There was one place where it was 5 but it was supposed to be 4. That looks like a copy and paste bug. There were also two checks that were off by one. Fixes: 9e2c2e273012 ("drm/etnaviv: add infrastructure to que