From patchwork Mon Feb 20 03:50:50 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marek Vasut X-Patchwork-Id: 655205 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 2FF1CC64EC4 for ; Mon, 20 Feb 2023 03:51:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230081AbjBTDvP (ORCPT ); Sun, 19 Feb 2023 22:51:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44020 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230086AbjBTDvO (ORCPT ); Sun, 19 Feb 2023 22:51:14 -0500 Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 34EC1CA27 for ; Sun, 19 Feb 2023 19:51:10 -0800 (PST) Received: from tr.lan (ip-86-49-120-218.bb.vodafone.cz [86.49.120.218]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: marex@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id E72DA857DC; Mon, 20 Feb 2023 04:51:06 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1676865067; bh=BPUiVJ+7yinQifTeus3/lXFO6uiCyUkEUgB/v5vyxz8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=GS3FTU2yHEo+PpX6OUYgMtMHclPUWiC5zKwqbcGhtFn9jJfGKLUorX6zTTh7CfWSN mCblQl8BIoTuGpmAJMSprGVqa2nXF0VLXcXbofMLxMc1nwEXNhI+lHqkjaR3lL9MN/ h7H0QSpSCta2GvMQacD4/ET6Pu91pS+CWGPODejJGU3UCga/8NCza0yqJy8l2cGNSS 4rcn4ksY2Iq2yz/cVCnYZN+ySuAM1ut9phCK1zYz4NQH7CfGaN4RtsG/pjaVgZ9OU7 bNI8J/pdsmZ+GY/UFEPeC64QkkUNYQ6mKGC1+0LJWgKApPyok42la+e2LRz2TpJbNt yKl+GIp+QF4og== From: Marek Vasut To: linux-arm-kernel@lists.infradead.org Cc: Marek Vasut , Alexander Stein , Fabio Estevam , Krzysztof Kozlowski , Laurent Pinchart , Lucas Stach , NXP Linux Team , Paul Elder , Peng Fan , Pengutronix Kernel Team , Richard Cochran , Richard Zhu , Rob Herring , Sascha Hauer , Shawn Guo , devicetree@vger.kernel.org Subject: [PATCH v2 3/4] soc: imx: imx8m-blk-ctrl: Scan subnodes and bind drivers to them Date: Mon, 20 Feb 2023 04:50:50 +0100 Message-Id: <20230220035051.327847-3-marex@denx.de> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230220035051.327847-1-marex@denx.de> References: <20230220035051.327847-1-marex@denx.de> MIME-Version: 1.0 X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org This particular block can have DT subnodes describing the LVDS LDB bridge. Instead of misusing simple-bus to scan for those nodes, do the scan within the driver. Fixes: 94e6197dadc9 ("arm64: dts: imx8mp: Add LCDIF2 & LDB nodes") Signed-off-by: Marek Vasut Reviewed-by: Liu Ying --- Cc: Alexander Stein Cc: Fabio Estevam Cc: Krzysztof Kozlowski Cc: Laurent Pinchart Cc: Lucas Stach Cc: NXP Linux Team Cc: Paul Elder Cc: Peng Fan Cc: Pengutronix Kernel Team Cc: Richard Cochran Cc: Richard Zhu Cc: Rob Herring Cc: Sascha Hauer Cc: Shawn Guo Cc: devicetree@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org --- V2: - Turn this into 3/4 - Warn and continue in case of error --- drivers/soc/imx/imx8m-blk-ctrl.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/soc/imx/imx8m-blk-ctrl.c b/drivers/soc/imx/imx8m-blk-ctrl.c index 399cb85105a18..4f5736e612fb0 100644 --- a/drivers/soc/imx/imx8m-blk-ctrl.c +++ b/drivers/soc/imx/imx8m-blk-ctrl.c @@ -169,7 +169,9 @@ static int imx8m_blk_ctrl_probe(struct platform_device *pdev) { const struct imx8m_blk_ctrl_data *bc_data; struct device *dev = &pdev->dev; + struct platform_device *child; struct imx8m_blk_ctrl *bc; + struct device_node *np; void __iomem *base; int i, ret; @@ -310,6 +312,13 @@ static int imx8m_blk_ctrl_probe(struct platform_device *pdev) dev_set_drvdata(dev, bc); + for_each_child_of_node(dev->of_node, np) { + child = of_platform_device_create(np, NULL, dev); + if (child) + continue; + dev_warn(dev, "failed to create device for %pOF\n", np); + } + return 0; cleanup_provider: