Re: [PATCH net-next v5 00/22] ethtool netlink interface, part 1

Jiri Pirko
2019-03-27T15:19:41Z

Wed, Mar 27, 2019 at 03:28:06PM CET, mkubecek@suse.cz wrote:
>On Wed, Mar 27, 2019 at 02:09:28PM +0100, Jiri Pirko wrote:
>> Could you please perhaps split the patchset so it contains max ~15 patches?
>
>Would 1-17 still be acceptable? The thing is that patch 16 is the first
>where some "normal" notification (i.e. not EVENT one) is sent and
>patch 17 is the first implementation of a request modifying data.
>I believe having at least one example of both querying and setting data
>would help the review discussion.
>
>Or perhaps I could leave GET_INFO related patches (at least second and
>third) for later.

I think 17 is okay. Please split if possible.

>
>Michal