Received: by 2002:a05:7412:40d:b0:e2:908c:2ebd with SMTP id 13csp525999rdf; Tue, 21 Nov 2023 08:54:16 -0800 (PST) X-Google-Smtp-Source: AGHT+IH0MJdHVhU/Xe4k//nqkrBOSl8e4ut6vSOZokzmaKgCjikjbvuiyu2IkXgk7FA+4Tf/4bRt X-Received: by 2002:a05:6a20:9385:b0:18b:2032:e7a8 with SMTP id x5-20020a056a20938500b0018b2032e7a8mr676320pzh.24.1700585656297; Tue, 21 Nov 2023 08:54:16 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700585656; cv=none; d=google.com; s=arc-20160816; b=sh9Yw1xiBDWs5UxpoX6WCQZ+dc+tuwIddwj1hRwsOFKFH9R5zYxxJyk3hf/MOikDHK NehH6OHOHsR03onnoHwr/0pQ7wIB2JxxdkfbJKR4we7aEmg4Wycp6iKzU147z484VmnY nQMIEyxnt2/U8Yiu1vKe9YLFD4O87iHePX/9Fk6WJeLttN/BM3rZIkEpPB9eKwEQOasH LkSFcQJgQJygQDNq/81XIfG58EPNMK1lKl9pfUznrvZhiKqy1Hu1atWVhRzovdaz8dSM Dqw5Iea2ZdivO1KFWGzRLXpcZ/BW9fXdRLtg8m69Q10Pz1v/H5Y7FW74wR3oGZp0Bspo TtFQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:references:from:to:cc:subject :message-id:date:content-transfer-encoding:mime-version :dkim-signature; bh=n02adsIvyblEozYaL9kieTrZl/WHapBpg8ySkC5Rsmo=; fh=VTRRORkzhxBzu5FjfE131rkCuL9CG7/FAPBO5q+LQQI=; b=aakNcH1OXmZH+dSGiK0fGZel2XBgerB0al2zXVk5G5Gj1FCl8dz5cGZEM0DuFKV0TM YAPItgaQELgZxhQ0BDR3U4tBqSDtb5nhbMp/BaUknxW+CWh32Sv2PIesg59dumDvmgsh Kva/ae9J7PFD+kH/BhZixoJUaYAADzemPP+37dn9UU7Bjgvs1Ok7qylPYHSfpN+IWEnk VNnlrgdsfAaznuREuBvVdQGO1aDnMz13ogPBX8RinnnenZFkg3hugm3BQNdRtuOq4uUW bALhSK/0jJahrGx6tYxVtgU/XU8kuESyYLIcg9ZSlqOAfFQdQvNzjbVysEqM/XHf1Y+0 JqDA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=iqdm9xd8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [2620:137:e000::3:3]) by mx.google.com with ESMTPS id bq20-20020a056a02045400b005774978dd75si12392787pgb.175.2023.11.21.08.54.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Nov 2023 08:54:16 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3; Authentication-Results: mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=iqdm9xd8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id D4DCF80CC105; Tue, 21 Nov 2023 08:54:12 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231250AbjKUQxl (ORCPT + 99 others); Tue, 21 Nov 2023 11:53:41 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46864 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230399AbjKUQxk (ORCPT ); Tue, 21 Nov 2023 11:53:40 -0500 Received: from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net [217.70.183.195]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DFE39113; Tue, 21 Nov 2023 08:53:35 -0800 (PST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 4CE3C6000B; Tue, 21 Nov 2023 16:53:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1700585614; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=n02adsIvyblEozYaL9kieTrZl/WHapBpg8ySkC5Rsmo=; b=iqdm9xd88QUoCRBrFpluE/q44QmcnEPu0yhe/WVswF6dVuLGela12+s0emF5H4mYpLynIy s0yZfQL3IKnUZh+hBRCCF9elHteEPEKsDgrGH7PvItYqYQzh5lb67lIiKJhhCyztZA64MW 15Mt9HDl1umMGmiDi/Ad9jdiiGgnt9aq1Au4vgb/Pgb+aARJPTIWoLVlNRCWcj0+lgMdDg y0iuv9chCqQN7Z0sIo/++nymEIxNqLAlbDIG8oT1Rqf5XY6zTMFlEPQfbsRNWf+Xh7Eiaq 7UK0500YQMzyNMiHQRDsEshFGKh7YF5nDTtU+6h5xalvcxCbBBKfKiqyEjnFcw== Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 21 Nov 2023 17:53:32 +0100 Message-Id: Subject: Re: [PATCH v2 1/7] dt-bindings: usb: ti,j721e-usb: add ti,j7200-usb compatible Cc: , , , , =?utf-8?q?Gr=C3=A9gory_Clement?= , "Conor Dooley" To: "Krzysztof Kozlowski" , "Greg Kroah-Hartman" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Roger Quadros" , "Peter Chen" , "Pawel Laszczak" , "Nishanth Menon" , "Vignesh Raghavendra" , "Tero Kristo" From: =?utf-8?q?Th=C3=A9o_Lebrun?= X-Mailer: aerc 0.15.2 References: <20231120-j7200-usb-suspend-v2-0-038c7e4a3df4@bootlin.com> <20231120-j7200-usb-suspend-v2-1-038c7e4a3df4@bootlin.com> <6f0da181-717c-4b14-ba3f-d287efe4105b@linaro.org> In-Reply-To: <6f0da181-717c-4b14-ba3f-d287efe4105b@linaro.org> X-GND-Sasl: theo.lebrun@bootlin.com X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Tue, 21 Nov 2023 08:54:13 -0800 (PST) Hello, On Mon Nov 20, 2023 at 6:32 PM CET, Krzysztof Kozlowski wrote: > On 20/11/2023 18:06, Th=C3=A9o Lebrun wrote: > > On this platform, the controller & its wrapper are reset on resume. Thi= s > > makes it have a different behavior from other platforms. > >=20 > > We allow using the new compatible with a fallback onto the original > > ti,j721e-usb compatible. We therefore allow using an older kernel with > > Where is fallback ti,j721e-usb used? Please point me to the code. No fallback is implemented in code. Using a kernel that doesn't have this patch series but a more recent devicetree: DT has both devicetrees & the kernel will know which driver to use. That is opposed to having only compatible =3D "ti,j7200-usb". If using an old kernel, it would not know what driver to match it to. [...] > > --- a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml > > +++ b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml > > @@ -12,11 +12,15 @@ maintainers: > > properties: > > compatible: > > oneOf: > > + - const: ti,j7200-usb > > - const: ti,j721e-usb > > - const: ti,am64-usb > > - items: > > - const: ti,j721e-usb > > - const: ti,am64-usb > > + - items: > > + - const: ti,j721e-usb > > This makes little sense. It's already on the list. Twice! Don't add it > third time. > > I am sorry, but this binding makes no sense. I mean, existing binding > makes no sense, but your change is not making it anyhow better. The goal of the DT schema pre-patch was to allow all three: compatible =3D "ti,j721e-usb"; compatible =3D "ti,am64-usb"; compatible =3D "ti,j721e-usb", "ti,am64-usb"; I've followed the same scheme & added both of those: compatible =3D "ti,j7200-usb"; compatible =3D "ti,j7200-usb", "ti,j721e-usb"; I messed up the ordering in the added 'items' options, but the logic seems right to me. And dtbs_check agrees. Am I missing something? Thanks, -- Th=C3=A9o Lebrun, Bootlin Embedded Linux and Kernel engineering https://bootlin.com