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=-10.8 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 6D9A7C07E9B for ; Wed, 21 Jul 2021 10:04:01 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 3ACC961019 for ; Wed, 21 Jul 2021 10:04:01 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3ACC961019 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2WOqWfxjw60TSigRwOW1B7w2Wp2pXNM9KRnoW9XDciU=; b=HYyn9084I1PdzW QMakCOl0g8QeMgjoEs0J8K5ChE9aUjRfS0WY1e8Ga69xeaVpJIMP9dPEjTjCErLNMUCjZCrtoxuEC v7j5TIYOE/X3JoPKH5z9nRyA1XehkINCKOizdeGh8PXDA3UHkKDhjwDdFIWzil23SylUWyg8pXZuw weOQPO8hHiAMSJiUsyZb9mxm0zuaAVhV/0Y+NZm22ipuiRQqW6wcbKW+PHYDOrlMVK2iLU+jMG2rS 21ILEyXXWN4Ik+bGpuCVmNhkO/i6r7Jps2DJjH3w1R1HkQojBCkxjF6fVt6muiicDKwYlGd2Au7Bg AdAvxbC3SW48RVP8V+xQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m693f-00FADO-PD; Wed, 21 Jul 2021 10:02:31 +0000 Received: from mail-lf1-x12a.google.com ([2a00:1450:4864:20::12a]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m693b-00FACG-O8 for linux-arm-kernel@lists.infradead.org; Wed, 21 Jul 2021 10:02:29 +0000 Received: by mail-lf1-x12a.google.com with SMTP id 8so2294765lfp.9 for ; Wed, 21 Jul 2021 03:02:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=uq/BDgHzeQirTDEogKe70scOce4C0cOi5x0MYXrCpbY=; b=Ke7nwN3tAtft8VpPMijva70KCg4QgprDjHpK3KURV7Tsib3GmYBjUI6FQAjUWgrhMn C6XaCl0VVvViBXLVZWPfh/7TkdpPei4bF8Z59SpvnyKrTHfXqYVpllMPX8Dz+5v6heAD 4oZPzUCmj3xOtoqpCXCgQ0i+H3/HXujIfqghC+aFA3LmMJ/a7rrDfgtR3hCmQKY/LZMD K6Xz1WXQBmNs56GlU+capX+PxgH4zuOFDgJe2Q7MmqrXVDWLOpFELkwbYKQox1fwvZge 7YZYtsE/6PX/r9CajyrWlSWs7Ts1nlgIBX5zwPKOsnf5lVUwEIPkAfE+VFKGj1FVKMpQ GRJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=uq/BDgHzeQirTDEogKe70scOce4C0cOi5x0MYXrCpbY=; b=IYHrufCpeZq/y3i4bbeC5kLPZxspuXzRukBUzxVAc4/1LCgCtS1CeO4txAyDSwxEj9 dRhxf4wrGIkOonp8TZk+QucYVwh34v6xXt9GSctYxAilKse9Zn9E5bYNMdNieO3x+d/1 JEk8wj1fHPF8KvCfD5NCiGWsbjIML5pNsXb4rsxzsDq/N2g8uFjdGljoeiKzbDyUzlSv PqiAKsVLzqhsEOPAz1rtEj9gSuG1ifz1mNfl042MxSR5UW+4JhcqhvGlvpmid3PUk+Wi mg/inOtcxMxQh0ULh1H2fSlpmIRmHdc4d9VsUCC+7jQzBdFEoRTx0607j/B1/IxcZu/L CVug== X-Gm-Message-State: AOAM530gPLh7JSPtgREjQjjHytjiGSrUO6O0AEI7g3p5KS9bjeoG0eC7 IljznrFc//1oazzf+6XJXJI= X-Google-Smtp-Source: ABdhPJwJ9oOZ8/cOl8mTGgS93guEhW0BZrD42ndG6/7urMIsSHF10NN5cZsYqdECl0IP7vfCOlqiDw== X-Received: by 2002:ac2:4107:: with SMTP id b7mr24842540lfi.609.1626861744103; Wed, 21 Jul 2021 03:02:24 -0700 (PDT) Received: from mobilestation ([95.79.127.110]) by smtp.gmail.com with ESMTPSA id t7sm2138866ljc.81.2021.07.21.03.02.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 21 Jul 2021 03:02:22 -0700 (PDT) Date: Wed, 21 Jul 2021 13:02:20 +0300 From: Serge Semin To: Greg Kroah-Hartman , Krzysztof Kozlowski , Rob Herring Cc: Serge Semin , John Stultz , Felipe Balbi , Florian Fainelli , Andy Gross , Bjorn Andersson , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-arm-msm , Linux USB List , lkml , linux-arm-kernel , Amit Pundir Subject: Re: [PATCH 29/29] arm64: dts: qcom: Harmonize DWC USB3 DT nodes name Message-ID: <20210721100220.ddfxwugivsndsedv@mobilestation> References: <20201020115959.2658-1-Sergey.Semin@baikalelectronics.ru> <20201020115959.2658-30-Sergey.Semin@baikalelectronics.ru> <20210714124807.o22mottsrg3tv6nt@mobilestation> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210721_030227_856990_15E5D718 X-CRM114-Status: GOOD ( 46.77 ) 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 Greg, @Krzysztof, @Rob, please join the discussion so to finally get done with the concerned issue. On Wed, Jul 21, 2021 at 09:38:54AM +0200, Greg Kroah-Hartman wrote: > On Wed, Jul 14, 2021 at 03:48:07PM +0300, Serge Semin wrote: > > Hello John, > > > > On Tue, Jul 13, 2021 at 05:07:00PM -0700, John Stultz wrote: > > > On Tue, Oct 20, 2020 at 5:10 AM Serge Semin > > > wrote: > > > > > > > > In accordance with the DWC USB3 bindings the corresponding node > > > > name is suppose to comply with the Generic USB HCD DT schema, which > > > > requires the USB nodes to have the name acceptable by the regexp: > > > > "^usb(@.*)?" . Make sure the "snps,dwc3"-compatible nodes are correctly > > > > named. > > > > > > > > Signed-off-by: Serge Semin > > > > > > > > I know folks like to ignore this, but this patch breaks AOSP on db845c. :( > > > > Sorry to hear that. Alas there is no much can be done about it. > > Yes there is, we can revert the change. We do not break existing > configurations, sorry. By reverting this patch we'll get back to the broken dt-bindings since it won't comply to the current USB DT-nodes requirements which at this state well describe the latest DT spec: https://github.com/devicetree-org/devicetree-specification/releases/tag/v0.3 Thus the dtbs_check will fail for these nodes. Originally this whole patchset was connected with finally getting the DT-node names in order to comply with the standard requirement and it was successful mostly except a few patches which still haven't been merged in. Anyway @Krzysztof has already responded to the complain regarding this issue here: https://lore.kernel.org/lkml/20201221210423.GA2504@kozik-lap/ but noone cared to respond on his reasonable questions in order to get to a suitable solution for everyone. Instead we are getting another email with the same request to revert the changes. Here is the quote from the Krzysztof email so we could continue the discussion: On Mon, 21 Dec 2020 13:04:27 -0800 (PST), Krzysztof Kozlowski wrote: > On Mon, Dec 21, 2020 at 12:24:11PM -0800, John Stultz wrote: > > On Sat, Dec 19, 2020 at 3:06 AM Krzysztof Kozlowski wrote: > > > ... > > > > > > The node names are not part of an ABI, are they? I expect only > > > compatibles and properties to be stable. If user-space looks for > > > something by name, it's a user-space's mistake. Not mentioning that you > > > also look for specific address... Imagine remapping of addresses with > > > ranges (for whatever reason) - AOSP also would be broken? Addresses are > > > definitely not an ABI. > > > > Though that is how it's exported through sysfs. > > The ABI is the format of sysfs file for example in /sys/devices. However > the ABI is not the exact address or node name of each device. > > > In AOSP it is then used to setup the configfs gadget by writing that > > value into /config/usb_gadget/g1/UDC. > > > > Given there may be multiple controllers on a device, or even if its > > just one and the dummy hcd driver is enabled, I'm not sure how folks > > reference the "right" one without the node name? > > I think it is the same type of problem as for all other subsystems, e.g. > mmc, hwmon/iio. They usually solve it either with aliases or with > special property with the name/label. > > > I understand the fuzziness with sysfs ABI, and I get that having > > consistent naming is important, but like the eth0 -> enp3s0 changes, > > it seems like this is going to break things. > > One could argue whether interface name is or is not ABI. But please tell > me how the address of a device in one's representation (for example DT) > is a part of a stable interface? > > > Greg? Is there some better way AOSP should be doing this? > > If you need to find specific device, maybe go through the given bus and > check compatibles? > > Best regards, > Krzysztof So the main question is how is the DT-node really connected with ABI and is supposed to be stable in that concern? As I see it even if it affects the configfs node name, then we may either need to break that connection and somehow deliver DT-node-name independent interface to the user-space or we have no choice but to export the node with an updated name and ask of user-space to deal with it. In both suggested cases the DT-node name will still conform to the USB-node name DT spec. Currently we are at the second one. Regards, -Sergey > > thanks, > > greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel