Messages in this thread | | | Date | Tue, 12 Mar 2019 17:33:00 +0200 | From | Andy Shevchenko <> | Subject | Re: [PATCH v2 2/5] auxdisplay: charlcd: Move to_priv() to charlcd namespace |
| |
On Tue, Mar 12, 2019 at 04:07:37PM +0100, Geert Uytterhoeven wrote: > Hi Andy, > > On Tue, Mar 12, 2019 at 3:44 PM Andy Shevchenko > <andriy.shevchenko@linux.intel.com> wrote: > > In order to be more particular in names, rename to_priv() macro > > to charlcd_to_priv(). > > As this is a macro, not a function, the name doesn't end up as a symbol in > the binary anyway, and it's for internal use by the driver only.
I understand this, and I did in order to avoid potential collisions in the future.
> > > No functional change intended. > > > > Cc: Geert Uytterhoeven <geert@linux-m68k.org> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Regardless: > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks!
> > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds
-- With Best Regards, Andy Shevchenko
| |