On Wed, 2019-05-08 at 16:22 +0300, Alexandru Ardelean wrote:
On Wed, 2019-05-08 at 15:18 +0200, Greg KH wrote:
On Wed, May 08, 2019 at 04:11:28PM +0300, Andy Shevchenko wrote:
On Wed, May 08, 2019 at 02:28:29PM +0300, Alexandru Ardelean wrote:
This change re-introduces `match_string()` as a macro that uses ARRAY_SIZE() to compute the size of the array. The macro is added in all the places that do `match_string(_a, ARRAY_SIZE(_a), s)`, since the change is pretty straightforward.
Can you split include/linux/ change from the rest?
That would break the build, why do you want it split out? This makes sense all as a single patch to me.
Not really. It would be just be the new match_string() helper/macro in a new commit. And the conversions of the simple users of match_string() (the ones using ARRAY_SIZE()) in another commit.
I should have asked in my previous reply. Leave this as-is or re-formulate in 2 patches ?
No strong preference from my side.
Thanks Alex
Thanks Alex
thanks,
greg k-h