Re: [PATCH] staging: media: atomisp: pci: Replace bytes macros with functions
From: Andy Shevchenko
Date: Tue Jan 17 2023 - 10:36:14 EST
On Tue, Jan 17, 2023 at 05:34:12PM +0200, Andy Shevchenko wrote:
> On Tue, Jan 17, 2023 at 10:08:41AM -0500, Brent Pappas wrote:
> > Replace the function-like macros FPNTBL_BYTES, SCTBL_BYTES, and
> > MORPH_PLANE_BYTES with static inline functions to comply with Linux coding
> > style standards.
>
> Thank you!
>
> But I think what you need, besides dropping unneeded parentheses, is to use some
> macros from overflow.h.
And drop "staging:" prefix from the patch since media maintainer uses (dumb)
script that adds "media:" if it doesn't lead the Subject.
--
With Best Regards,
Andy Shevchenko