From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 385B2C10F14 for ; Tue, 8 Oct 2019 17:03:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 18A55206C2 for ; Tue, 8 Oct 2019 17:03:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728465AbfJHRDE (ORCPT ); Tue, 8 Oct 2019 13:03:04 -0400 Received: from muru.com ([72.249.23.125]:36050 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725900AbfJHRDE (ORCPT ); Tue, 8 Oct 2019 13:03:04 -0400 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 50D6E8081; Tue, 8 Oct 2019 17:03:36 +0000 (UTC) Date: Tue, 8 Oct 2019 10:02:59 -0700 From: Tony Lindgren To: Jeroen Hofstee Cc: Grygorii Strashko , "linux-omap@vger.kernel.org" , Koen Kooi , =?utf-8?Q?Beno=C3=AEt?= Cousson , open list Subject: Re: [PATCH] ARM: dts: am335x-sancloud-bbe: Fix PHY mode for ethernet Message-ID: <20191008170259.GI5610@atomide.com> References: <20191002095416.19603-1-jhofstee@victronenergy.com> <436f1712-7dec-db40-d08f-1a3032af3596@victronenergy.com> <20191008142349.GE5610@atomide.com> <1f779973-1702-47eb-158f-7000770c435a@victronenergy.com> <20191008161416.GG5610@atomide.com> <4fa5393d-0b2c-4b8c-128a-81ab05a14a82@victronenergy.com> <20191008165154.GH5610@atomide.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.1 (2019-06-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Jeroen Hofstee [191008 17:00]: > Hi, > > On 10/8/19 6:51 PM, Tony Lindgren wrote: > > * Jeroen Hofstee [191008 16:43]: > >> Hello Tony, > >> > >> On 10/8/19 6:14 PM, Tony Lindgren wrote: > >>> * Jeroen Hofstee [191008 16:03]: > >>>> Hello Tony, > >>>> > >>>> On 10/8/19 4:23 PM, Tony Lindgren wrote: > >>>>> * Grygorii Strashko [191003 02:32]: > >>>>>> On 03/10/2019 11:16, Jeroen Hofstee wrote: > >>>>>>> Furthermore 4.19 is fine, so there is no need to include it in stable > >>>>>>> and have a note to make sure also other patches are required etc. > >>>>>> Hence all above patches went in 5.1 it would be correct to mention only > >>>>>> 6d4cd041f0af net: phy: at803x: disable delay only for RGMII mode > >>>>> Jeroen, can you please send an updated patch with the fixes > >>>>> tag changed? > >>>>> > >>>> For completeness, there is no "Fixes tag" as you mentioned. > >>>> The commit only refers to another commit which introduces > >>>> a problem. > >>> Well please add the fixes tag, that way this will get > >>> properly applied to earlier stable kernels too :) > >> But 4.19 is fine, this is an issue in 5.1 as in EOL... > >> I really don't understand why I should waste time > >> to figure out what happened exactly during the 5.1 > >> release cycle... > > Hmm so what's the issue with just adding the fixes tag Grygorii > > suggested: > > > > 6d4cd041f0af ("net: phy: at803x: disable delay only for RGMII mode") > > > > No need to dig further? > > Grygorii doesn't suggest to add a fixes tag, just to change the referenced > commit to another. Obviously I would like to understand why another commit > should be referenced. And then I should read and parse the response, so there > is no special reason, just time... OK sure. Well once you guys have the commit figured out, let me know what to apply. And we know Grygorii is mostly right based on his history of comments so best to not ignore that :) Cheers, Tony