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=-9.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 97E80C07E95 for ; Wed, 14 Jul 2021 02:34:34 +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 5DA0B61154 for ; Wed, 14 Jul 2021 02:34:34 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5DA0B61154 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org 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=b/QxT7nswPDz2k9edR0E4zjtOGxGE0uo4+id8YuvYDo=; b=MeZI6Wn5HGBqkx 7CQrELRQLFdRZb3H3TdyhKeIP3Fa1zKFeNyMmNr8d8SP7DRagyBVcbJf3p4kHOrszwMTGjwSLkoit yXZJ2+nbY0/MG6QSbINPTmIWU748h1UcvJVJs/CnCcHmccrP0uE6kS5pzrsuKPJy2sEYuP9KSd8I0 SlzFihV1RI5GZqVPFXFxYG/4CNN62yJhx/VeuEKKsEaLe0wxjb/EOFtjMZP88/Cqb/wPHXSwfBynV APm040YSMwn3hIlr1TV/T0KscSEkNStOXDlSeVr7FG4YLW4QE5FlcRuLGiAE6UI2yCYFDmDMY5vFG Zc0Ou3WAgXv10+BQDTTg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m3Ugl-00C1Y4-Tj; Wed, 14 Jul 2021 02:31:57 +0000 Received: from mail-oi1-x22f.google.com ([2607:f8b0:4864:20::22f]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m3Uci-00Bzem-Rp for linux-arm-kernel@lists.infradead.org; Wed, 14 Jul 2021 02:27:46 +0000 Received: by mail-oi1-x22f.google.com with SMTP id t25so570287oiw.13 for ; Tue, 13 Jul 2021 19:27:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=FKr65vfR2TdVe8GBpKpEMCo3zswr0/dtqzTR9ciGq6A=; b=uoBMWDRKB2h5zkZgtNGjB5PUPYoGSOLtb3zF4sUvR2vepVPa1PFSMQpuOZMtsNSMF5 Hzt5PAC4Ae4rtH3smb6oPZAjFy7mh1HuAQ8xAB79NBCW54xVS0DQMIV0mPtbvwlqJXGj ztHfmUKpK4Pr+PryEJH3QJWnCmFdsZUoM3TbuD+XrmZmzNG7Ryz6nkKixeQYSallRYsn BmV7MG1ULrw8rNkFguVubx17Sf4FNw0XqVwmqVFmClOc6/Lm2Js6fgMMXmLT9GXkXKHl 3cbmOrvpLvXMDQVEDlISNIlIaLHB5Fj4qBjWSOKwBJKvXjLuE8R3MsUzp5hS3MQNzsVF R1bw== 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=FKr65vfR2TdVe8GBpKpEMCo3zswr0/dtqzTR9ciGq6A=; b=DvsZIhNr8Vy6lvBg+Ctmve+gxgl7Vk4DU/gRHDw6cxFPnU+tME/OmC5eI880m/S5tG NkoDjdzE+FUodSbnhdySvx4TPyj6IMrqMVYSgQjas9HUvocD47CICaX8xoyFlY9i2Aq/ KhAqJGNmstboGHgR7OAPsfmo8gEqo0alEF/HVuCHJWtHs6E9Cq+lvgo95BYO/n9aNkEl 7o7ZtA8zfTiucNKFWdyiIZGkhsFA9Tzi2/fLdd2F2MFA75zftMHn52oIMVibnuxodM06 PHjlA28c4RTdTJ12YGGQqHgkKn0PYxuA5OJ9N1JU2Yr4uS796r00taEejptwhmREov09 rziQ== X-Gm-Message-State: AOAM533f3KBttQ7EJlR4kHliIkKURI+VlEHd7mfWDnPRWscfTs72b0Or 9pnhCg1oczxDeFtrPGOhEFy7/A== X-Google-Smtp-Source: ABdhPJz/ytXCZ8BRBiVtna/vi/6ubcxZ+G6RnZaRINMM7ogFKSWmRMxswDIcrHAlc372iWrarNnkiQ== X-Received: by 2002:aca:3d8a:: with SMTP id k132mr1058697oia.120.1626229663063; Tue, 13 Jul 2021 19:27:43 -0700 (PDT) Received: from yoga (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id v7sm172544ooj.46.2021.07.13.19.27.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Jul 2021 19:27:42 -0700 (PDT) Date: Tue, 13 Jul 2021 21:27:39 -0500 From: Bjorn Andersson To: John Stultz Cc: Serge Semin , Felipe Balbi , Krzysztof Kozlowski , Florian Fainelli , Rob Herring , Greg Kroah-Hartman , Andy Gross , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-arm-msm , Linux USB List , lkml , Serge Semin , linux-arm-kernel , Amit Pundir Subject: Re: [PATCH 29/29] arm64: dts: qcom: Harmonize DWC USB3 DT nodes name Message-ID: References: <20201020115959.2658-1-Sergey.Semin@baikalelectronics.ru> <20201020115959.2658-30-Sergey.Semin@baikalelectronics.ru> 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-20210713_192744_971146_E5B96BFF X-CRM114-Status: GOOD ( 26.97 ) 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 On Tue 13 Jul 19:07 CDT 2021, 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, I totally forgot that the name of that node is part of the USB gadget configfs interface. > In the exact same way an earlier patch broke HiKey960: > https://lore.kernel.org/lkml/CALAqxLWGujgR7p8Vb5S_RimRVYxwm5XF-c4NkKgMH-43wEBaWg@mail.gmail.com/ > > (which I still have to carry a revert for). > > I get that this change is useful so more dynamic userland can find > devices using consistent naming with future kernels (but doesn't the > dynamic userland have to handle the case for older kernels as well?) > But for userland that uses static configs, its painful as updating > userland to use the new node ids then causes older kernels to fail. > It won't help you, but having a mechanism to provide user friendly names would certainly be welcome. I always struggle with the question of what "6a00000.dwc3" (now .usb) actually is... > I'm looking into how we might be able to probe and set the property > dynamically, but AOSP's init system is far more aligned to static > configs. > > This will probably be ignored again, but it would be nice if we could > have a release where DTS changes don't break userland for one of my > boards. As it feels like its been awhile. > I don't have any preference to this being names "dwc3" or "usb" and we could back out the change in time for v5.14. But you will still have the problem for Hikey iiuc and the dts would then violate the binding - so we need to fix that, and all the other Qualcomm boards defined by the same binding. Regards, Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel