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 Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BB0FFECAAD3 for ; Wed, 31 Aug 2022 08:03:09 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 494FC84882; Wed, 31 Aug 2022 10:03:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=tinet.cat Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id 8435B849F1; Wed, 31 Aug 2022 10:03:04 +0200 (CEST) Received: from mx1.tinet.cat (mx1.tinet.cat [195.77.216.146]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id DE43B849CD for ; Wed, 31 Aug 2022 10:03:01 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=tinet.cat Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=xdrudis@tinet.cat X-ASG-Debug-ID: 1661932980-163e7b4765ad0520001-4l7tJC Received: from smtp01.tinet.cat (smtp.tinet.cat [195.77.216.131]) by mx1.tinet.cat with ESMTP id nspd4uKLwvlkCDBU; Wed, 31 Aug 2022 10:03:00 +0200 (CEST) X-Barracuda-Envelope-From: xdrudis@tinet.cat X-Barracuda-Effective-Source-IP: smtp.tinet.cat[195.77.216.131] X-Barracuda-Apparent-Source-IP: 195.77.216.131 Received: from begut (99.red-79-152-185.dynamicip.rima-tde.net [79.152.185.99]) by smtp01.tinet.cat (Postfix) with ESMTPSA id 4048E605E0C0; Wed, 31 Aug 2022 10:03:00 +0200 (CEST) Date: Wed, 31 Aug 2022 10:02:47 +0200 From: Xavier Drudis Ferran To: Kever Yang Cc: Xavier Drudis Ferran , u-boot@lists.denx.de, Simon Glass , Philipp Tomsich , Lukasz Majewski , Sean Anderson , Marek Vasut , =?utf-8?B?5ZC06Imv5bOw?= Subject: Re: [PATCH] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2. Message-ID: X-ASG-Orig-Subj: Re: [PATCH] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2. References: <20220701185959.GC1700@begut> <6b1207ad-51d0-f609-3fb6-7f12d70902d7@rock-chips.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <6b1207ad-51d0-f609-3fb6-7f12d70902d7@rock-chips.com> X-Barracuda-Connect: smtp.tinet.cat[195.77.216.131] X-Barracuda-Start-Time: 1661932980 X-Barracuda-URL: https://webmail.tinet.cat:443/cgi-mod/mark.cgi X-Barracuda-Scan-Msg-Size: 1182 X-Barracuda-BRTS-Status: 1 X-Barracuda-Bayes: INNOCENT GLOBAL 0.5138 1.0000 0.7500 X-Barracuda-Spam-Score: 0.75 X-Barracuda-Spam-Status: No, SCORE=0.75 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=6.0 KILL_LEVEL=8.0 tests= X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.100454 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean El Sat, Aug 27, 2022 at 11:20:17AM +0800, Kever Yang deia: > The idea for dtsi now is to sync the kernel and U-Boot dts, and Linux > distribution will > use the dts from U-Boot. If we change the dts property in -u-boot.dts, it > will also pass > to kernel, which make kernel function not available. So we would like to > sync the driver > for kernel and u-boot to use the same dts nodes. > That's news to me. I was thinking all the time that the reason for having some dtsi? files with -u-boot suffix was that these were exclusive for u-boot and the others were shared with linux. If now the content of .*-u-boot.dtsi? gets passed to linux too, I no longer know why we separate the files. Is this for Rockchip or ARM64 or a general rule for all of U-Boot? Should I (or someone) try to change documentation or is that already done? I thought I read something different few months ago ? Maybe I misunderstood... > So we can try option 1, output warning instead of error return for clock get > fail, since the clock > driver for kernel and u-boot are totally different. > Ok. I hope I find time to try that some day. Thanks for looking at the patch.