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: [PATCH net-next v6 0/9] [RFT] net: dpaa: Convert to phylink
To:     Jakub Kicinski <kuba@kernel.org>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        Camelia Alexandra Groza <camelia.groza@nxp.com>,
        netdev@vger.kernel.org, Russell King <linux@armlinux.org.uk>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        "linuxppc-dev @ lists . ozlabs . org" <linuxppc-dev@lists.ozlabs.org>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Ioana Ciornei <ioana.ciornei@nxp.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Li Yang <leoyang.li@nxp.com>,
        Michael Ellerman <mpe@ellerman.id.au>,
        Paul Mackerras <paulus@samba.org>,
        Rob Herring <robh+dt@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>, devicetree@vger.kernel.org
References: <20220930200933.4111249-1-sean.anderson@seco.com>
 <0b47fd86-f775-e6ad-4e5f-e40479f2d301@seco.com>
 <20221004095245.1e9918bf@kernel.org>
From:   Sean Anderson <sean.anderson@seco.com>
Message-ID: <bf6b58ed-a42f-9848-993f-e074779e8264@seco.com>
Date:   Tue, 4 Oct 2022 13:26:35 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <20221004095245.1e9918bf@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?SlN3N2tkQVk2bkowVHFhRnQxblZLTzVaYTlCbGxRMFozWkkwNmo5OUtHekFl?=
 =?utf-8?B?TDQyR1NPYWdaZjdRWkFTWk1rNEpSTktOTkRvdU9ncHNNdHNZbjZkczhqSGs3?=
 =?utf-8?B?NWM0WDBLUTB0QmJLS0ZUYTlyVlhVb0wrOXhhZWZwZE5NQThvOHA0RTZsMG05?=
 =?utf-8?B?SXJaUjlWRlF3TUdydFJvZzFBZmQyZEZISEVtem53R0xDWUNRZWFFa2JYWVAx?=
 =?utf-8?B?TGprQy9CeXQ2bit4NWRQMHBCdmZIa2szdGZyWWc2Tm5ucUROVXRhRDNvSEhB?=
 =?utf-8?B?Q0NIV2R4T0RrRWdleU52bmpYZjF0aXM4YWJiT0h1bUdQaFZBdWo4dmlZWlZv?=
 =?utf-8?B?QlR1OUQxUjFqNVdnLzJSZVRCa3B2NllhS010NzcrME5CTFA4NnJzaHZKUWp2?=
 =?utf-8?B?QnBKZnVSMTU3TWI4dXhqa2hNUWx5dkVndkE3aDUzNTA2NFI0bldudHZRNGlm?=
 =?utf-8?B?YzlSbjhFdkFoR00yVVJ5eDEyamd5bzluSlRoSkZ3b3RHcXdUL29FM0pyMjZH?=
 =?utf-8?B?S2Nzdm1sb2pGYjRtTXFtYnJLdDd0U1Flc0ZWYmwvR1l1K0Q4Zi9Yd1doa1lh?=
 =?utf-8?B?dWpWNXRkQWpxR2RLeHpNQ3pITlVHQ0RpUlZmam82YlZSUTNCc0UwclRUdzRt?=
 =?utf-8?B?MnlIcUgySytuVTZXdnN2L2ozcytPSER6V1pYOVpTV28wQ3NqUFlLM2tya3dD?=
 =?utf-8?B?dGhYZktIbThYSGNtSG95dlExZ1dGZHhQNEUzRkthaTgwQnR2bzRFTndJSHJE?=
 =?utf-8?B?UlJTN3dqQmRrZ1JJK0hJdG9ZSkFsdVhDWGNEKzlLWE80YS9sd3ZqdG9lc2Yz?=
 =?utf-8?B?bHdVejJ6a3gvVWVDTmIvZE03OGVGaGc1SzhkSXpSSVFQbHNPMTZPQ0J4bWdw?=
 =?utf-8?B?MytxY3dKdWZtMjVqSlFQb3QxTllvOGVGUWxadlI1c0Q4Y2tLa3RVYm1JRGFR?=
 =?utf-8?B?T2hUNEd4Zk9sTDRsek9FZlJ0b01YMndXbzZQU3ZVQTVXQ2lucTFSb3BOSlR5?=
 =?utf-8?B?dHRXVHJTSDVZVE0reFFKYTFoeEJpUWp2WmRsRmFkN0NhcjJUa01GbHQ0endS?=
 =?utf-8?B?MVFlckFDbCtrUXZKL3VFREpYS2hqVzhKV1NDTlhzV3ZJdTg2RGFGWnd2dStC?=
 =?utf-8?B?NmNRT2NoRmRTZXk2a1VZazlXbFBYS1lCTVl5ZG5ZbERvclpmcEhEcGh1Z0xw?=
 =?utf-8?B?ekk4RTJULytLVVBmNzZSV0U1M1dibHh5SlIwM0RhYzE0NnRaaDhnbHVoaXB0?=
 =?utf-8?B?SFdmRjVZd2U0dk9oaThoMituYWZ0RW16clRMSTZ0SHpOOEsxaFhNNVk0VVdL?=
 =?utf-8?B?a05KNFZ6QVJyNkh6bCticW94OWl3am9NM3RFY1lONEZ3Y0lOQTlBdjRHTDZV?=
 =?utf-8?B?RS9vcm9VUWZFeWl4MzN1Z1Jud0p2TW9HbXV0UlZ6QkxGY3E5SU1oZHkxcndD?=
 =?utf-8?B?dlc3NkVaQUZKR3F2YkIzODg4b0p3eHZYRHhlSU1RS2xONnRIZmlnUUJxK0xx?=
 =?utf-8?B?bDBHRTk4dWhNdVBRcnl0aEZ5VDl6OENZTWsrek1iQWlMajhzL0NpdkkxQ1Nz?=
 =?utf-8?B?MHlFcTdLeXJXSjFwdm1iUnY5elRacUswdHVTcWJpckZkcFBRdWlPOFl2bnlK?=
 =?utf-8?B?b0lzMy94N3NkeDdrajVPVHlrNWtIb2hDZ2xvblI0cDNKT2NSQ2tKbFo1RXFj?=
 =?utf-8?B?STNDM3RrYi9pMzlqK0xSRE1TRHZhNm5WZHdsK1ovNW92MjBFeWJKTE1kNzV0?=
 =?utf-8?B?ZklRVVpVemtSdERJYXdPcmJMcVVBNUJudG9BR0VXTG9pUGRZQ2JVSmx0RjM5?=
 =?utf-8?B?ejBIWVNTKzZVckVudE84Sjh3SG5SVWxnZ0FPMDBZenFORWFXelVHekg1eHJS?=
 =?utf-8?B?d3ZzWUk4WjlOUjIvemZHVExUTXBURXdaSENSNWpUb3VJL3FDeFNQUTUxSWxj?=
 =?utf-8?B?Vm5WT0xoZ255aUJHcXdXbGdabmRnaXE4bC9vMmc1RHFiTDA0bGZ5VzZLRmdO?=
 =?utf-8?B?TUVic1RwZ1R4Wkl3SVdpczBqM25kL0JGUGdwZ3RrWVJoZXhZWG5QZEJia3FV?=
 =?utf-8?B?bkUwUkp6a2hVbDBWNVp2QnM5VkVnTnJaNWhMMHdhbzdnbzFqcVZUT1hFbG96?=
 =?utf-8?B?R3JERjM2SGJCVEJBak1XRmt0SEFCajJhbm5Ib05wR0w4MWVqQmRTTXZQNUpp?=
 =?utf-8?B?S3c9PQ==?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3104820d-3653-41e3-abd5-08daa62d9b43
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Oct 2022 17:26:43.6707
 (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: rAe1rSMw2LZqdq5D+/3jOLlKawv49VQCWQDQTsLQ2aocye4TXPvbRcS5MXLRNV2/5CE9WHNd0pWTc6mW1vbnww==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR03MB8273
X-Spam-Status: No, score=-4.6 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 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


On 10/4/22 12:52 PM, Jakub Kicinski wrote:
> On Tue, 4 Oct 2022 11:28:19 -0400 Sean Anderson wrote:
>> I noticed that this series was marked "RFC" in patchwork.
> 
> Because the cover letter has RTF in the subject, presumably.
> 
>> I consider this series ready to apply. I am requesting *testing*, in
>> particular on 10gec/dtsec boards (P-series). Since no one seems to
>> have tried that over the past 4 months that I've been working on this
>> series, perhaps the best way for it to get tested is to apply it...
> 
> You know the situation the best as the author, you should make 
> a clear call on the nature of the posting. It's either RFC/RFT 
> or a ready-to-go-in posting.

Well, I consider the memac stuff to be well tested, but I don't
have 10gec/dtsec hardware. I was hoping that someone with the hardware
might look at this series if I stuck RFT in the subject. I suspect
there are still some bugs in those drivers.

> Maybe in smaller subsystems you can post an RFC/RTF and then it 
> gets applied after some time without a repost but we don't do that.
> The normal processing time for a patch is 1-3 days while we like
> to give people a week to test. So the patches would have to rot in 
> the review queue for extra half a week. At our patch rate this is
> unsustainable.
> 

Well, I have gotten reviews for the device tree stuff, but the core
changes (what I consider to be the actual content of the series) is
missing Reviewed-bys. I don't anticipate making any major changes to
the series unless I get some feedback one way or another. If having
RFT in the subject is preventing that review, I will remove it.

--Sean