On Wed, Apr 13, 2022 at 11:06:53PM -0700, Aliya wrote: > On Thu, Apr 14, 2022 at 6:40 AM Alison Schofield > <alison.schofield@xxxxxxxxx> wrote: > > > > On Thu, Apr 14, 2022 at 02:09:15AM +0530, Aliya Rahmani wrote: > > > > Hi Aliya, > > > > Wondering why the recipient list different than this: > > > > $ perl scripts/get_maintainer.pl --nogit-fallback --no-rolestats -f drivers/staging/sm750fb/sm750.h > > Sudip Mukherjee <sudipm.mukherjee@xxxxxxxxx> > > Teddy Wang <teddy.wang@xxxxxxxxxxxxxxxxx> > > Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > linux-fbdev@xxxxxxxxxxxxxxx > > linux-staging@xxxxxxxxxxxxxxx > > linux-kernel@xxxxxxxxxxxxxxx > > > Do I need to include all the maintainers and mailing lists? Usually - yes. It's possible that a maintainer has instructed otherwise, and in that case the maintainers file can be updated. That's why I posed it as a 'wondering why' question :) Instructions for using get_maintainers.pl are here: https://kernelnewbies.org/Outreachyfirstpatch And, here's a prior round email discussion of same: https://lore.kernel.org/outreachy/20211015171331.GA431883@alison-desk/ Alison > > > > > Remove volatile to fix checkpatch.pl warning. > > > > > > Signed-off-by: Aliya Rahmani <aliyarahmani786@xxxxxxxxx> > > > --- > > > drivers/staging/sm750fb/sm750.h | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm750.h > > > index aff69661c8e6..7a11f8593596 100644 > > > --- a/drivers/staging/sm750fb/sm750.h > > > +++ b/drivers/staging/sm750fb/sm750.h > > > @@ -52,7 +52,7 @@ struct lynx_accel { > > > /* base virtual address of DPR registers */ > > > volatile unsigned char __iomem *dprBase; > > > /* base virtual address of de data port */ > > > - volatile unsigned char __iomem *dpPortBase; > > > + unsigned char __iomem *dpPortBase; > > > > > > /* function pointers */ > > > void (*de_init)(struct lynx_accel *accel); > > > -- > > > 2.25.1 > > > > > > > > Thanks, > Aliya
