Re: [PATCH] staging: fbtft: fbtft-bus: replace function-defining macro with concrete functions
From: Greg KH
Date: Fri Apr 24 2026 - 05:49:37 EST
On Fri, Apr 24, 2026 at 09:40:48AM -0000, Alexandru Hossu wrote:
> On Fri, Apr 24, 2026 at 12:38:45PM +0300, Andy Shevchenko wrote:
> > I'm not sure this patch improves the code. What I see it's harder to follow.
> > NAK.
>
> Fair point. Three near-identical functions hide the pattern the macro
> makes explicit. I'll drop this approach.
>
> If there's appetite for a minimal fix, I can send a v2 that moves
> EXPORT_SYMBOL() outside the macro body only, leaving the function
> definition untouched. Otherwise I'll leave it as-is and wait for
> Greg's take.
Please leave as-is, there's nothing wrong with the existing code here.
thanks,
greg k-h