Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [RESEND PATCH net-next v4 00/25] net: dpaa: Cleanups in
 preparation for phylink conversion
To:     Jakub Kicinski <kuba@kernel.org>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>, netdev@vger.kernel.org,
        Camelia Groza <camelia.groza@nxp.com>,
        open list <linux-kernel@vger.kernel.org>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        linuxppc-dev@lists.ozlabs.org,
        linux-arm-kernel@lists.infradead.org,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Li Yang <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>,
        devicetree@vger.kernel.org
References: <20220818161649.2058728-1-sean.anderson@seco.com>
 <20220818112054.29cd77fb@kernel.org>
From:   Sean Anderson <sean.anderson@seco.com>
Message-ID: <f085609c-24c9-a9fb-e714-18ba7f3ef48a@seco.com>
Date:   Thu, 18 Aug 2022 14:37:23 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <20220818112054.29cd77fb@kernel.org>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S0NzNks4SEhibHZDaUZPZzZ1QXczU04zNmFJMzFDakk3akkzQU56cG5oYXpq?=
 =?utf-8?B?V2RXL2NGT3huZ0IxUDU5MmJic2tDQ1hvMmo5UFh6NGQvRHZDWXNpelVhS09p?=
 =?utf-8?B?YkxONjJsSDE0WXVLZmh2QVAwSy9IRmkycms2ZGF5NlVQQTdDTFZuTzFqT2xE?=
 =?utf-8?B?V3M4d1lybHVwNGxYaTcxbVpmNWl3MU0wTmEzbXhUZjZ2Z3pOd1k5MVBRRGZH?=
 =?utf-8?B?SWtoVHJTQ0N2UVJjWEdlNDByQ1BWblBJK0lkNFovQ1VLa2dHSGg3QkJwc3JR?=
 =?utf-8?B?NkdsdDluU1dzWUU1TllFVGk5enpUWTljMXBOOVZFalNUbTQzOEJaZEtXdElD?=
 =?utf-8?B?Q2ZSVzRzV1JtRDdDVWZodmc3MmZhVE5GVmFXRTRPK3EwemlkWmd1WlhLc2ow?=
 =?utf-8?B?VStoWHYrL0RSWCtTSG1yNCtHWllsbzV3aHJlOWRtanhVeTRpeTE0NVZvdVNY?=
 =?utf-8?B?VktZcWhHRHZocFlUd08vRzhKSTdkb2FJWk0rOGlxdWtqd2tvNWt1UjNPbjRI?=
 =?utf-8?B?RnlyYWhueGtEMzFCTi9lM292dHVBSGFyZEp6M3U2aTlubVp0NWU0MWVYVkRt?=
 =?utf-8?B?QlUzRnZCMEpyMTV0em9oc0d3YUtuZ2F1MTU1V21TYnV3b3hmZEtDZlVtWlFa?=
 =?utf-8?B?YjNTYUN0bzhFUndKNHVkcnhqY2lGd2J2NHN1UWYxTzhsazg4RzJYNEcwMnRr?=
 =?utf-8?B?dnJ5U2pEdWtYZGhaTHpGb1FrWVpIWGVXK3pkQXZRTE9oem5vOFhJY2ltRG5a?=
 =?utf-8?B?N3BtUnNaaS8wY0hZNFh0TDA4NnJTZm16d3pOMWExL3ozeUxjcE8rY2NtVHRI?=
 =?utf-8?B?cGpQNzZXeWlyakk4RjdTL1Q4QjZiK09ubHEyU0cyWENtU1hTTkdlNnEzYytR?=
 =?utf-8?B?UmIyU2VXNGVnSDl2VlExTHJLZUE3T2gyRExDdlk3VnZWMEZBTzY4K0RqQ0N5?=
 =?utf-8?B?d01jazA3NXNFUzJ4bWQ1ZHZseEtYSUtpMXZabEY3anBkUmQrVU5Mek9CcVZw?=
 =?utf-8?B?MVpOcnFjbS9NKzBoYmVUeXFHeDVPN2plT1g1WHQ4bzR4U0V3QVVYL0ZKeTRp?=
 =?utf-8?B?YVdsZFhMcytaZkR4cGxzeFBaRnR1NExWQlNZUEQrUStTTUhmNk1uWnVCL3hW?=
 =?utf-8?B?T3ZEVDJMbFgyelYrcFhOcmtXV0lqdENkVERERUowOFhvSGdydDZ5L3BackQv?=
 =?utf-8?B?QzFMZ2dNdnNpelZKUXZOSFEwVWliV0ZBL2s3ckZHMzhnUEFSNEFtVjRKSmho?=
 =?utf-8?B?Qm0zQ0hlY1VSSUxEbDBmWmx3eW94Wnd2YlRVWnBTKzI3MEJvUFNLdnRSK2pD?=
 =?utf-8?B?ZmQzTnZZM29zbG9uQS8xOC9oQlVrSHMrSXB1WUlFaFVzdHRpenowVDZKMk5i?=
 =?utf-8?B?LzdQTXJ3N1VtZ0oyYzcvNENQR1Z3UWEyakZFTm16NExUcmFwM0dSQ3NySHJl?=
 =?utf-8?B?SmJFYkgzU0lkWTJTZUJUaU9HYUdTNGUxZDZJU1ltaXVLWkh3WlhwTjRXaEhz?=
 =?utf-8?B?SjZQU2R5RWVHdGpPYTZJSHVkQytHSHFlUjdSb1dlR2doVTdoQUV1VzJ3VTRS?=
 =?utf-8?B?R1ozcGtsbEdJZXNDKzhuTDJyN2lsQjdlQTMvU2hBRDJjMlQvZStVRW9KM0hn?=
 =?utf-8?B?RnBBQ1JETHBDTitJeGZ4TDNMbWx5TWRvQzNadzdVaWh6em9BRlh3MjBkRlFM?=
 =?utf-8?B?bkM3d2JuSTZBQVFISGl4MjFIMkNucmRMUUtoZTJiT3BodTJodTRJZC9MZmR3?=
 =?utf-8?B?UjRPa051eWEzNnFEZWlTZVFwUjJJUkRXZlJuZFZiVVN5OXk1K2lIOVU4SmxQ?=
 =?utf-8?B?c2hiUTR5dVo0MGtYUTUrWCtuYURLTXd1bzBBRDRDbmpmWEZFelBFZGNDT3Y0?=
 =?utf-8?B?SnBnU3NiTEp2NElZSUY0cy9vUEVOcEN2UEhkaEc2YTJCSHkzMXQwNHdoVWFH?=
 =?utf-8?B?ajI3cndCd056RkpORWlJNS9hUDYxSEo0WFRWSDd3czF5SVdMdUxSMUZ5cUxW?=
 =?utf-8?B?WXZSbWVVUG4zTWlGNnB3Y3ZDUndFZUNBam1qKzVVeWJNL2RHNFkxR2E4ZGJD?=
 =?utf-8?B?SkJGUXRJLzlVMDVmM3JjSGZHUHl2STZWdTNtNHJtREVzNjEwVkV3UTRvbC9s?=
 =?utf-8?B?RTh2M0lvdC9adFNlRjRlZG9GSGhpK2JmVWhmK0MxRzRwMG9ZNFp1OTNPM0Zz?=
 =?utf-8?B?Y1E9PQ==?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 32d9379b-8058-4dd3-dba1-08da8148b2d6
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2022 18:37:26.5494
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Kduk6WBz3Ssn6k22LgmsiUZpEOU2xZApFgwexHerdHu/Nz/EifhJhZHzwvig58mewbMpgK2PIrUOkKi55mIs2w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR03MB3600
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Jakub,

On 8/18/22 2:20 PM, Jakub Kicinski wrote:
> On Thu, 18 Aug 2022 12:16:24 -0400 Sean Anderson wrote:
>> This series contains several cleanup patches for dpaa/fman. While they
>> are intended to prepare for a phylink conversion, they stand on their
>> own. This series was originally submitted as part of [1].
> 
> Still over the limit of patches in a patch series, and looks pretty
> easy to chunk up. We review and apply patches in netdev in 1-3 days,
> it really is more efficient to post smaller series. 

Last time I offered to arbitrarily chunk things [1], but I did not receive
a response for 3 weeks.

> And with the other series you sent to the list we have nearly 50
> patches from you queued for review. I don't think this is reasonable,
> people reviewing this code are all volunteers trying to get their
> work done as well :(

These patches have been sent to the list in one form or another since
I first sent out an RFC for DPAA conversion back in June [2]. I have not
substantially modified any of them (although I did add a few more
since v2). It's not like I came up with these just now; I have been
seeking feedback on these series for 2-3 months so far. The only
reviews I have gotten were from Camelia Groza, who has provided some
helpful feedback.

--Sean

[1] https://lore.kernel.org/netdev/51c26e19-e4ea-7596-1147-7d95eec9e6a9@seco.com/
[2] https://lore.kernel.org/netdev/20220617203312.3799646-1-sean.anderson@seco.com/