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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 22261C636CD for ; Tue, 31 Jan 2023 08:14:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230225AbjAaIO3 (ORCPT ); Tue, 31 Jan 2023 03:14:29 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35882 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229776AbjAaIO3 (ORCPT ); Tue, 31 Jan 2023 03:14:29 -0500 Received: from mail-qt1-f181.google.com (mail-qt1-f181.google.com [209.85.160.181]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 73E4A2F7B8; Tue, 31 Jan 2023 00:14:27 -0800 (PST) Received: by mail-qt1-f181.google.com with SMTP id cr22so454087qtb.10; Tue, 31 Jan 2023 00:14:27 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=w0ccHC8reNWYQgpAYbVr9s42CKygVFy7/fkfStrl4kw=; b=xVL0rmSqmFG8Fy/wiT3oZvwoIe1JjlG5hQIkwCmocnAhSNXKWtHa3N2wtLDwc/qMRR JpEOl5nzatGLY5cAnJs7NzmFXgNbvn/ZH7/TNg2nYZq98JiFZF9yqc5VfBEFzyQfKc1Z NUL/3j/Jgem7tIbgLm+35L2tLbb6tWFZfQEsZAmH7K1vUurfSk79rY8OaoJUtjSk6TLC FOlDkGZDTm4W2yIHy5/BxPny8vNkrYyaS0eqwWB+5rXYFtwPBFT6yCdrLVnf7uXPQv8R 8nwevTn/qW1IZH1Wkitv8haab7IQ2G+3r10Q855z9pYiN8xqnFb4ofrhbdBfKK+dAP7G hGOA== X-Gm-Message-State: AO0yUKX6VwooeJCof0ImhvQRmE5KWjXm7kNo4Zz39j0TighdQb8hn8N/ qpZUD1FAaAqR0n1KMhlJs3rnnSw3qUgCQA== X-Google-Smtp-Source: AK7set8cehzt6XtCtfsggb0kK7lrgGwoHmuqS4400CGzLNf17X1rKn1ag4jN14d+y444+9cn1Jtosw== X-Received: by 2002:ac8:5956:0:b0:3b8:4144:fe72 with SMTP id 22-20020ac85956000000b003b84144fe72mr18462380qtz.9.1675152866404; Tue, 31 Jan 2023 00:14:26 -0800 (PST) Received: from mail-yb1-f182.google.com (mail-yb1-f182.google.com. [209.85.219.182]) by smtp.gmail.com with ESMTPSA id f8-20020ac840c8000000b003b84b92052asm5396355qtm.57.2023.01.31.00.14.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 31 Jan 2023 00:14:26 -0800 (PST) Received: by mail-yb1-f182.google.com with SMTP id b1so17081658ybn.11; Tue, 31 Jan 2023 00:14:26 -0800 (PST) X-Received: by 2002:a25:9801:0:b0:7d5:b884:3617 with SMTP id a1-20020a259801000000b007d5b8843617mr4679605ybo.380.1675152855693; Tue, 31 Jan 2023 00:14:15 -0800 (PST) MIME-Version: 1.0 References: <20230127001141.407071-1-saravanak@google.com> <20230127001141.407071-4-saravanak@google.com> In-Reply-To: From: Geert Uytterhoeven Date: Tue, 31 Jan 2023 09:14:03 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 03/11] soc: renesas: Move away from using OF_POPULATED for fw_devlink To: Saravana Kannan Cc: Greg Kroah-Hartman , "Rafael J. Wysocki" , Sudeep Holla , Cristian Marussi , Linus Walleij , Bartosz Golaszewski , Thomas Gleixner , Marc Zyngier , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Rob Herring , Frank Rowand , Geert Uytterhoeven , Magnus Damm , Len Brown , Andy Shevchenko , Daniel Scally , Heikki Krogerus , Sakari Ailus , Tony Lindgren , Linux Kernel Functional Testing , Naresh Kamboju , Abel Vesa , Alexander Stein , John Stultz , Doug Anderson , Guenter Roeck , Dmitry Baryshkov , Maxim Kiselev , Maxim Kochetkov , Miquel Raynal , Luca Weiss , Colin Foster , Martin Kepplinger , Jean-Philippe Brucker , kernel-team@android.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-acpi@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org Hi Saravana, On Mon, Jan 30, 2023 at 9:00 PM Saravana Kannan wrote: > On Mon, Jan 30, 2023 at 12:43 AM Geert Uytterhoeven > wrote: > > On Sat, Jan 28, 2023 at 8:19 AM Saravana Kannan wrote: > > > On Fri, Jan 27, 2023 at 12:11 AM Geert Uytterhoeven > > > wrote: > > > > On Fri, Jan 27, 2023 at 1:11 AM Saravana Kannan wrote: > > > > > The OF_POPULATED flag was set to let fw_devlink know that the device > > > > > tree node will not have a struct device created for it. This information > > > > > is used by fw_devlink to avoid deferring the probe of consumers of this > > > > > device tree node. > > > > > > > > > > Let's use fwnode_dev_initialized() instead because it achieves the same > > > > > effect without using OF specific flags. This allows more generic code to > > > > > be written in driver core. > > > > > > > > > > Signed-off-by: Saravana Kannan > > > > > > > > Thanks for your patch! > > > > > > > > > --- a/drivers/soc/renesas/rcar-sysc.c > > > > > +++ b/drivers/soc/renesas/rcar-sysc.c > > > > > @@ -437,7 +437,7 @@ static int __init rcar_sysc_pd_init(void) > > > > > > > > > > error = of_genpd_add_provider_onecell(np, &domains->onecell_data); > > > > > if (!error) > > > > > - of_node_set_flag(np, OF_POPULATED); > > > > > + fwnode_dev_initialized(&np->fwnode, true); > > > > > > > > As drivers/soc/renesas/rmobile-sysc.c is already using this method, > > > > it should work fine. > > > > > > > > Reviewed-by: Geert Uytterhoeven > > > > i.e. will queue in renesas-devel for v6.4. > > I hope you meant queue it up for 6.3 and not 6.4? V6.4. The deadline for submitting pull requests for the soc tree is rc6. Sorry, your series was posted too late to make that. > > > Thanks! Does that mean I should drop this from this series? If two > > > maintainers pick the same patch up, will it cause problems? I'm > > > eventually expecting this series to be picked up by Greg into > > > driver-core-next. > > > > Indeed. Patches for drivers/soc/renesas/ are supposed to go upstream > > through the renesas-devel and soc trees. This patch has no dependencies > > on anything else in the series (or vice versa), so there is no reason > > to deviate from that, and possibly cause conflicts later. > > This series is supposed to fix a bunch of issues and I vaguely think > the series depends on this patch to work correctly on some Renesas > systems. You are my main renesas person, so it's probably some issue > you hit. Is you pick it up outside of this series I need to keep > asking folks to pick up two different patch threads. I don't have a > strong opinion, just a FYI. If you can take this patch soon, I don't > have any concerns. Oh right, you do remove OF_POPULATED handling in "[PATCH v2 09/11] of: property: Simplify of_link_to_phandle()". It might be wise to postpone that removal, as after your series, there are stillseveral users left, some of them might be impacted. I do plan to test your full series on all my boards, but probably that won't happen this week. > > BTW, I will convert to of_node_to_fwnode() while applying. > > Sounds good. If you still want this to land in v6,3 (with the of_node_to_fwnode() conversion): Acked-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds 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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 8E6C6C38142 for ; Tue, 31 Jan 2023 08:15:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2FxyxNNDQUWyAqqLcHYQDYqMbxOkJBrEC44o4Uh9E7s=; b=WUtlFf51z/rDRc q3eZjIkSNt4DVHezhQkRmbSIU9CIw11V8k0fv3eGaP79v6oiZVqnIi+MwxwiYN8cZuBC+ivD/HCib K4zrr+xLoZJTXgvjJ6wfHZLtPSudDxt3NnJEi14EivKaHHWV5tf+uQ9srBD/v1d+Q8cTbTkjLUCeO wFdWZybHaTebyNN4miXFhxBRe5mtet/u1bNcLvVLbdOdjo5s4DIQJnX6lJRBW1QUIS75fzBh6nG03 EAPnvRn3qeObaGKmZgfhlAj73C4w40F4ev+oB0BwmK7ZEEu0h2JDNzXgZ9FO81Iq2CZrK1s5McAIg QIuzE3pq5/tKpbqefXqw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pMlmv-006enc-4Z; Tue, 31 Jan 2023 08:14:45 +0000 Received: from mail-oa1-f51.google.com ([209.85.160.51]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pMlmj-006elm-6k for linux-arm-kernel@lists.infradead.org; Tue, 31 Jan 2023 08:14:34 +0000 Received: by mail-oa1-f51.google.com with SMTP id 586e51a60fabf-163adf0cfc4so7137890fac.7 for ; Tue, 31 Jan 2023 00:14:30 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=w0ccHC8reNWYQgpAYbVr9s42CKygVFy7/fkfStrl4kw=; b=0F7bPiAZWNcGWATP2zgK2518eTCzpaupPOv6vxdD4SBKOwwkrv6125J8Oe+x67DmvN zN+E1HsySil8TEZYCHIuLqwkmm5freJqd7DGp4wiBm8RbKzViZYulDir/6Li1Cno4PS7 E2VyiulRmt2p1Vii3pQAIkOxkLc7kcPORnFVbJCAF+BzgAJHOXoTm48PWVcdqTIzZMZQ ICjqVsEcnRbC23tuRnGAXsY0WU08zIgDtdb8vlH5F6pOCFNW4FO8csHq8IzHLU9i4W7R kTOBzJiCNVHLHUxteiR6BOtrfs4vhOaf9N5K8m2agDf8qn/0rNwOQlg3g6BN7xerXs9S QV2Q== X-Gm-Message-State: AO0yUKU0oNHPnhLWP8hK6IMSOqmPvQ7F8t0vbtGdzDmXvSH8szUdeeZh mF4oedjocJiVwt+mrZkCI+PjJwHIUK526g== X-Google-Smtp-Source: AK7set//khIjtmQgxdYfq/JQocJvhn1UZ/im0rM8+tRh3lo42k8Mdtd8bCu4/5j2moQuEvxn+x8FEA== X-Received: by 2002:a05:6870:c227:b0:163:d071:c477 with SMTP id z39-20020a056870c22700b00163d071c477mr2356717oae.43.1675152867059; Tue, 31 Jan 2023 00:14:27 -0800 (PST) Received: from mail-ot1-f47.google.com (mail-ot1-f47.google.com. [209.85.210.47]) by smtp.gmail.com with ESMTPSA id ld24-20020a0568702b1800b00163b3472300sm2373493oab.2.2023.01.31.00.14.26 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 31 Jan 2023 00:14:26 -0800 (PST) Received: by mail-ot1-f47.google.com with SMTP id 14-20020a9d010e000000b0068bdddfa263so640498otu.2 for ; Tue, 31 Jan 2023 00:14:26 -0800 (PST) X-Received: by 2002:a25:9801:0:b0:7d5:b884:3617 with SMTP id a1-20020a259801000000b007d5b8843617mr4679605ybo.380.1675152855693; Tue, 31 Jan 2023 00:14:15 -0800 (PST) MIME-Version: 1.0 References: <20230127001141.407071-1-saravanak@google.com> <20230127001141.407071-4-saravanak@google.com> In-Reply-To: From: Geert Uytterhoeven Date: Tue, 31 Jan 2023 09:14:03 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 03/11] soc: renesas: Move away from using OF_POPULATED for fw_devlink To: Saravana Kannan Cc: Greg Kroah-Hartman , "Rafael J. Wysocki" , Sudeep Holla , Cristian Marussi , Linus Walleij , Bartosz Golaszewski , Thomas Gleixner , Marc Zyngier , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Rob Herring , Frank Rowand , Geert Uytterhoeven , Magnus Damm , Len Brown , Andy Shevchenko , Daniel Scally , Heikki Krogerus , Sakari Ailus , Tony Lindgren , Linux Kernel Functional Testing , Naresh Kamboju , Abel Vesa , Alexander Stein , John Stultz , Doug Anderson , Guenter Roeck , Dmitry Baryshkov , Maxim Kiselev , Maxim Kochetkov , Miquel Raynal , Luca Weiss , Colin Foster , Martin Kepplinger , Jean-Philippe Brucker , kernel-team@android.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-acpi@vger.kernel.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230131_001433_266520_6F736E02 X-CRM114-Status: GOOD ( 50.80 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Saravana, On Mon, Jan 30, 2023 at 9:00 PM Saravana Kannan wrote: > On Mon, Jan 30, 2023 at 12:43 AM Geert Uytterhoeven > wrote: > > On Sat, Jan 28, 2023 at 8:19 AM Saravana Kannan wrote: > > > On Fri, Jan 27, 2023 at 12:11 AM Geert Uytterhoeven > > > wrote: > > > > On Fri, Jan 27, 2023 at 1:11 AM Saravana Kannan wrote: > > > > > The OF_POPULATED flag was set to let fw_devlink know that the device > > > > > tree node will not have a struct device created for it. This information > > > > > is used by fw_devlink to avoid deferring the probe of consumers of this > > > > > device tree node. > > > > > > > > > > Let's use fwnode_dev_initialized() instead because it achieves the same > > > > > effect without using OF specific flags. This allows more generic code to > > > > > be written in driver core. > > > > > > > > > > Signed-off-by: Saravana Kannan > > > > > > > > Thanks for your patch! > > > > > > > > > --- a/drivers/soc/renesas/rcar-sysc.c > > > > > +++ b/drivers/soc/renesas/rcar-sysc.c > > > > > @@ -437,7 +437,7 @@ static int __init rcar_sysc_pd_init(void) > > > > > > > > > > error = of_genpd_add_provider_onecell(np, &domains->onecell_data); > > > > > if (!error) > > > > > - of_node_set_flag(np, OF_POPULATED); > > > > > + fwnode_dev_initialized(&np->fwnode, true); > > > > > > > > As drivers/soc/renesas/rmobile-sysc.c is already using this method, > > > > it should work fine. > > > > > > > > Reviewed-by: Geert Uytterhoeven > > > > i.e. will queue in renesas-devel for v6.4. > > I hope you meant queue it up for 6.3 and not 6.4? V6.4. The deadline for submitting pull requests for the soc tree is rc6. Sorry, your series was posted too late to make that. > > > Thanks! Does that mean I should drop this from this series? If two > > > maintainers pick the same patch up, will it cause problems? I'm > > > eventually expecting this series to be picked up by Greg into > > > driver-core-next. > > > > Indeed. Patches for drivers/soc/renesas/ are supposed to go upstream > > through the renesas-devel and soc trees. This patch has no dependencies > > on anything else in the series (or vice versa), so there is no reason > > to deviate from that, and possibly cause conflicts later. > > This series is supposed to fix a bunch of issues and I vaguely think > the series depends on this patch to work correctly on some Renesas > systems. You are my main renesas person, so it's probably some issue > you hit. Is you pick it up outside of this series I need to keep > asking folks to pick up two different patch threads. I don't have a > strong opinion, just a FYI. If you can take this patch soon, I don't > have any concerns. Oh right, you do remove OF_POPULATED handling in "[PATCH v2 09/11] of: property: Simplify of_link_to_phandle()". It might be wise to postpone that removal, as after your series, there are stillseveral users left, some of them might be impacted. I do plan to test your full series on all my boards, but probably that won't happen this week. > > BTW, I will convert to of_node_to_fwnode() while applying. > > Sounds good. If you still want this to land in v6,3 (with the of_node_to_fwnode() conversion): Acked-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel