From patchwork Thu Nov 22 19:17:44 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Fabio Estevam X-Patchwork-Id: 53106 Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1gPuU7-0006zC-9h; Thu, 22 Nov 2018 19:17:55 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2392191AbeKWF6g (ORCPT + 1 other); Fri, 23 Nov 2018 00:58:36 -0500 Received: from mail-ot1-f66.google.com ([209.85.210.66]:34650 "EHLO mail-ot1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726009AbeKWF6g (ORCPT ); Fri, 23 Nov 2018 00:58:36 -0500 Received: by mail-ot1-f66.google.com with SMTP id t5so8893110otk.1 for ; Thu, 22 Nov 2018 11:17:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Rucf7INNyZlIh8QsXGSWmufoKDW6jWUSg+2WFyaqUM4=; b=gGGIIvXvQ0usZwkfx696lzzoaDh+ENzMMiisGlvMXYMK+37lkfzbJKhqSXLkYhV3H8 dWjVDnXvAVv6o4ttcUyN2uquylAfFO1ZLtwDWTYzuiUiUfKHqOyrQujRCgMDdPAeZ8HD eI0AJ9twUKCGs0t2WTa20An3wgW4fsicu77fA/Yy+5coES2PNzcMHbv0T9p9Ojp1cwGd ZSb60G5msTQNqnAuNADvksAtEKOxVKrlzxJL7p4ZYJmmlg8tuHG198ii10PZamDnk9wK LwVfMhv3SM6yFLTdNZKHUPAiOK/OvR5xTj0CDozWTYf+1zdjZzwdUhYEuAthTi4SEi7a CslA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Rucf7INNyZlIh8QsXGSWmufoKDW6jWUSg+2WFyaqUM4=; b=Ztp4X8ZsJ1nUl08yWef9x4elbB58DeFGN9VYwDPlr53RvYFNVxvgKhzkf2fJ5bsp4A 7RObaUx4eZmqkQP/eq0Ma9rWeJSd6q5YNOKF5ygi4q5r1PdtoBsIx7NTqPR3T5DReG+a Pz/a3A2JRg90Pxdzf/8g8u3ErZbQVukX3w/j0kV/fZvj54ysnJPSa49is6GaTI8M3ZqA hlNOg9jO9G1lDiHH+6F5HJGZjk6kMyc3MWX3vnPxTCW+EVvdPVhfauXtetUKKtaoHwal 5SzFZsZWWSonD8citGoLr1EAi4gCPvrloejHv9eEqhdghCxcMHRXivmtLU80Uz36SBQ1 0eEA== X-Gm-Message-State: AA+aEWYCIfk893IkRcqCvnrD6wRug1ugZSRie5MYMYEhUoW+YnLGIdSL nEfodwEMsg502j1zTQLDvpzllq9NL5J613ilqY4= X-Google-Smtp-Source: AFSGD/WF1+d3HbzkRai0ehz9yyqhhILkaymsbXyx6xEqVndEag69VB1ORyciF1FHW5Jju8LQDxouiCeQptqSyHI8KUQ= X-Received: by 2002:a9d:7e87:: with SMTP id m7mr7680339otp.225.1542914272213; Thu, 22 Nov 2018 11:17:52 -0800 (PST) MIME-Version: 1.0 References: <1542904065.16720.2.camel@pengutronix.de> In-Reply-To: <1542904065.16720.2.camel@pengutronix.de> From: Fabio Estevam Date: Thu, 22 Nov 2018 17:17:44 -0200 Message-ID: Subject: Re: 'bad remote port parent' warnings To: Philipp Zabel Cc: Sakari Ailus , Steve Longerbeam , Hans Verkuil , linux-media Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Philipp, On Thu, Nov 22, 2018 at 2:27 PM Philipp Zabel wrote: > There are empty endpoint nodes (without remote-endpoint property) > labeled ipu1_csi[01]_mux_from_parallel_sensor in the i.MX6 device trees > for board DT implementers' convenience. See commit 2539f517acbdc ("ARM: > dts: imx6qdl: Add video multiplexers, mipi_csi, and their connections"). > > We had a discussion about this issue in February when this caused a > probing error: https://patchwork.kernel.org/patch/10234469/ Thanks for the clarification. We could demote the warning to a debug message, make the wording a bit > less misleading (there is no bad remote port parent, there is just no > remote endpoint at all), or we could just accept the error message for Something like this? } And how should we treat these error probes? [ 3.449564] imx-ipuv3 2400000.ipu: driver could not parse port@1/endpoint@0 (-22) [ 3.457342] imx-ipuv3-csi: probe of imx-ipuv3-csi.1 failed with error -22 [ 3.464498] imx-ipuv3 2800000.ipu: driver could not parse port@0/endpoint@0 (-22) [ 3.472120] imx-ipuv3-csi: probe of imx-ipuv3-csi.4 failed with error -22 Thanks --- a/drivers/media/v4l2-core/v4l2-fwnode.c +++ b/drivers/media/v4l2-core/v4l2-fwnode.c @@ -613,7 +613,7 @@ v4l2_async_notifier_fwnode_parse_endpoint(struct device *dev, asd->match.fwnode = fwnode_graph_get_remote_port_parent(endpoint); if (!asd->match.fwnode) { - dev_warn(dev, "bad remote port parent\n"); + dev_dbg(dev, "no remote endpoint found\n"); ret = -ENOTCONN; goto out_err;