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 AB21CC61DA7 for ; Fri, 27 Jan 2023 08:12:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232126AbjA0IMD (ORCPT ); Fri, 27 Jan 2023 03:12:03 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58644 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232109AbjA0IMC (ORCPT ); Fri, 27 Jan 2023 03:12:02 -0500 Received: from mail-qv1-f42.google.com (mail-qv1-f42.google.com [209.85.219.42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5D186761FB; Fri, 27 Jan 2023 00:11:58 -0800 (PST) Received: by mail-qv1-f42.google.com with SMTP id ll10so2245757qvb.6; Fri, 27 Jan 2023 00:11:58 -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=dBAPOHYTkWiPLnj81NzCswXH0Y6077YYaFE4ZOx/zRY=; b=cSJYl8O2nDpMg/CA3iceMsy1oxQllQ9Gjg9Nmq6jJ+kPrr71GxU5PxeL5DfGt3zDfa gPWuMZ3zzRic0MJOJj0flpXENXbJ/OobiqOD0JvpCHn8ift85z/WN5i/QgzRiPA5hLPM T2HEq8Kwkk2eZ3iP3pFF8GllTOLzR0s0cvAPUBAsVGuEdCWMWMeIkplPIRK8pHdQYe6l PiKsuN/lDJ203Wb1mNcQ4y/1RubjH2mQ8pAf0YL2H7BfEHz8otvTonhIBEEVjQBLIQlQ hb/D8yWbwg5rx0mxa1jhaJaCT7iugdUs2vsOlbMTNeFjQkNFXE82Ncf7I2C8yF8XxQIh bveA== X-Gm-Message-State: AO0yUKVFCiFEgINkl1Y5vLuqV6A25a2jfGBmvxfvgrjaROJJ7JKy1bO3 PKze7f9gQjFmi6bkda7L0W1+DoAsUoqj6AZW X-Google-Smtp-Source: AK7set9Bz62Q67ABrWdM7KPzU8QFgHRgxZTAr85sdaX8Gw1YpZ3MjQxtUT3lWXRtAgamQxPpSzFeGA== X-Received: by 2002:ad4:48d1:0:b0:537:727f:ac2b with SMTP id v17-20020ad448d1000000b00537727fac2bmr13020372qvx.41.1674807117361; Fri, 27 Jan 2023 00:11:57 -0800 (PST) Received: from mail-yw1-f175.google.com (mail-yw1-f175.google.com. [209.85.128.175]) by smtp.gmail.com with ESMTPSA id p25-20020a05620a22b900b00706c1f7a608sm190589qkh.89.2023.01.27.00.11.57 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 27 Jan 2023 00:11:57 -0800 (PST) Received: by mail-yw1-f175.google.com with SMTP id 00721157ae682-4fda31c3351so56901947b3.11; Fri, 27 Jan 2023 00:11:57 -0800 (PST) X-Received: by 2002:a25:37d4:0:b0:80b:8602:f3fe with SMTP id e203-20020a2537d4000000b0080b8602f3femr1568499yba.36.1674807106088; Fri, 27 Jan 2023 00:11:46 -0800 (PST) MIME-Version: 1.0 References: <20230127001141.407071-1-saravanak@google.com> <20230127001141.407071-4-saravanak@google.com> In-Reply-To: <20230127001141.407071-4-saravanak@google.com> From: Geert Uytterhoeven Date: Fri, 27 Jan 2023 09:11:34 +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 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. > > out_put: > of_node_put(np); 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 8079FC54EAA for ; Fri, 27 Jan 2023 08:13:16 +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=RSejYdbd018CQWRzf6WIc9Vabp1z0enV9hAIujPw9cc=; b=HPj+Myu774qqTA erphYiP0KJn9JeJ2JkZByZpJCW48w9wr45BmSvqF00PGL7E1AP1Zzl9TT40ylwuND+J8EUYbdsEZH CdkXJQ/f7bl9lCYY9+N9Wu03fbZ+AMoGb1AMHLRFeWgD/vZRk8DmdM69vbXJUeENa3cFbXfTD77dd Rv5Zf1r0+rabrIJDZp/vRhdhMTDaCBmlRQB+UgSko0qzDexzvLUR3IQfOl3ggksNXIx6Puc5ZOqVv 5KLrcQYSHtOnhjuYu9HsLm4sIQCPRN1xNBZ6N45gLejhiLh9AXT23K458hHndp2DB5xTLxWGmEUXk 0mNaJAl6ikvnPRcwUViQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pLJqF-00DUjm-8T; Fri, 27 Jan 2023 08:12:11 +0000 Received: from mail-qt1-f177.google.com ([209.85.160.177]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pLJqA-00DUip-Pq for linux-arm-kernel@lists.infradead.org; Fri, 27 Jan 2023 08:12:10 +0000 Received: by mail-qt1-f177.google.com with SMTP id jr19so3422186qtb.7 for ; Fri, 27 Jan 2023 00:11:58 -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=dBAPOHYTkWiPLnj81NzCswXH0Y6077YYaFE4ZOx/zRY=; b=R70UK0uSczUW2UqQLFOwT0OQIf6kr3TTD/1DoM5eXjER/LtUUkqlpSOXDo3Uvkste3 JK6Ez2dygLxak6oY/Gu9hwrkvVsKj03UTuiwoRhMjCp0r1nCmzA8LDDC5d9ByH/eIZ3s zSTRqiugTC8T0Yb6uVGZAutpj4aCyR05hfJIngBnHy3r7U27fZxnKDGs7obbvfvFvlaL s6XKuW0wQbXsWqbIJql2Y1NK67foaLnGioK9zQTzYpK1dY2hcB4FDxUFC4pc32rsPlll 8mruTIlWXhzskIcq9inpPPoKa6uarzElGVCLXIqQh3aeK2VyNWbBhNNNIDO7lAmj8lpI UBcQ== X-Gm-Message-State: AFqh2krzrtmJF35eGu/Nxnq1AgsXWnmZ82u93YsahLGrhetGDNSiivzX ZpB2B8WL7SlYMsA6cXlGZQHWn4siySMri/3g X-Google-Smtp-Source: AMrXdXsiaABHPDC4m1BJ3MfOfraXCC56yzy1X/NTln6RNH+7tajOk+UtbW0jxDoNT/sAa0cy5cx5wg== X-Received: by 2002:ac8:6b8a:0:b0:3a9:756b:3db9 with SMTP id z10-20020ac86b8a000000b003a9756b3db9mr54399781qts.45.1674807117173; Fri, 27 Jan 2023 00:11:57 -0800 (PST) Received: from mail-yb1-f179.google.com (mail-yb1-f179.google.com. [209.85.219.179]) by smtp.gmail.com with ESMTPSA id m20-20020aed27d4000000b003b6382f66b1sm2268076qtg.29.2023.01.27.00.11.57 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 27 Jan 2023 00:11:57 -0800 (PST) Received: by mail-yb1-f179.google.com with SMTP id d8so3108232ybe.9 for ; Fri, 27 Jan 2023 00:11:57 -0800 (PST) X-Received: by 2002:a25:37d4:0:b0:80b:8602:f3fe with SMTP id e203-20020a2537d4000000b0080b8602f3femr1568499yba.36.1674807106088; Fri, 27 Jan 2023 00:11:46 -0800 (PST) MIME-Version: 1.0 References: <20230127001141.407071-1-saravanak@google.com> <20230127001141.407071-4-saravanak@google.com> In-Reply-To: <20230127001141.407071-4-saravanak@google.com> From: Geert Uytterhoeven Date: Fri, 27 Jan 2023 09:11:34 +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-20230127_001206_862092_E64E9FBC X-CRM114-Status: GOOD ( 28.51 ) 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 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. > > out_put: > of_node_put(np); 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