Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-24 Thread Hans Verkuil
On 7/24/19 7:09 PM, Joe Perches wrote: > On Tue, 2019-07-09 at 22:04 -0700, Joe Perches wrote: >> Arguments are supposed to be ordered high then low. >> >> Signed-off-by: Joe Perches >> --- >> drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-

Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-24 Thread Greg Kroah-Hartman
On Wed, Jul 24, 2019 at 11:39:03AM -0700, Joe Perches wrote: > On Wed, 2019-07-24 at 20:35 +0200, Greg Kroah-Hartman wrote: > > On Wed, Jul 24, 2019 at 10:09:44AM -0700, Joe Perches wrote: > > > On Tue, 2019-07-09 at 22:04 -0700, Joe Perches wrote: > > > > Arguments are supposed to be ordered high

Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-24 Thread Joe Perches
On Wed, 2019-07-24 at 20:35 +0200, Greg Kroah-Hartman wrote: > On Wed, Jul 24, 2019 at 10:09:44AM -0700, Joe Perches wrote: > > On Tue, 2019-07-09 at 22:04 -0700, Joe Perches wrote: > > > Arguments are supposed to be ordered high then low. > > > > > > Signed-off-by: Joe Perches > > > --- > > > d

Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-24 Thread Greg Kroah-Hartman
On Wed, Jul 24, 2019 at 10:09:44AM -0700, Joe Perches wrote: > On Tue, 2019-07-09 at 22:04 -0700, Joe Perches wrote: > > Arguments are supposed to be ordered high then low. > > > > Signed-off-by: Joe Perches > > --- > > drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 2 +- > > 1 file changed,

Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-24 Thread Joe Perches
On Tue, 2019-07-09 at 22:04 -0700, Joe Perches wrote: > Arguments are supposed to be ordered high then low. > > Signed-off-by: Joe Perches > --- > drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/media/sunx

Re: [PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-10 Thread Paul Kocialkowski
Hi, On Tue 09 Jul 19, 22:04, Joe Perches wrote: > Arguments are supposed to be ordered high then low. > > Signed-off-by: Joe Perches Good catch, thanks! Acked-by: Paul Kocialkowski Cheers, Paul > --- > drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 2 +- > 1 file changed, 1 insertion(

[PATCH 11/12] staging: media: cedrus: Fix misuse of GENMASK macro

2019-07-09 Thread Joe Perches
Arguments are supposed to be ordered high then low. Signed-off-by: Joe Perches --- drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_regs.h b/drivers/staging/media/sunxi/cedrus/cedrus_re