From owner-freebsd-arm@freebsd.org Sat Oct 21 23:09:50 2017 Return-Path: Delivered-To: freebsd-arm@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 6CC34E3C84C for ; Sat, 21 Oct 2017 23:09:50 +0000 (UTC) (envelope-from markmi@dsl-only.net) Received: from asp.reflexion.net (outbound-mail-211-161.reflexion.net [208.70.211.161]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 1A2ED6EC56 for ; Sat, 21 Oct 2017 23:09:49 +0000 (UTC) (envelope-from markmi@dsl-only.net) Received: (qmail 21013 invoked from network); 21 Oct 2017 23:09:47 -0000 Received: from unknown (HELO mail-cs-02.app.dca.reflexion.local) (10.81.19.2) by 0 (rfx-qmail) with SMTP; 21 Oct 2017 23:09:47 -0000 Received: by mail-cs-02.app.dca.reflexion.local (Reflexion email security v8.40.3) with SMTP; Sat, 21 Oct 2017 19:09:47 -0400 (EDT) Received: (qmail 28822 invoked from network); 21 Oct 2017 23:09:47 -0000 Received: from unknown (HELO iron2.pdx.net) (69.64.224.71) by 0 (rfx-qmail) with (AES256-SHA encrypted) SMTP; 21 Oct 2017 23:09:47 -0000 Received: from [192.168.1.25] (c-76-115-7-162.hsd1.or.comcast.net [76.115.7.162]) by iron2.pdx.net (Postfix) with ESMTPSA id E3BE5EC8F7A; Sat, 21 Oct 2017 16:09:46 -0700 (PDT) From: Mark Millard Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\)) Subject: Re: svn commit: r324822 - head/sys/modules/dtb/allwinner [removal of sinovoip-bpi-m3.dts from sys/modules/dtb/allwinner/Makefile DTS list] Date: Sat, 21 Oct 2017 16:09:46 -0700 References: <3AD6B1F8-512C-43BB-AC76-7721454AD02F@dsl-only.net> <20171021195812.5bdb902401b8e756b6abfe40@bidouilliste.com> <20171021204356.47e3cd6066144bcd07f46699@bidouilliste.com> <50728566-11C2-45EB-8367-00CAF38D4548@dsl-only.net> To: Emmanuel Vadot , freebsd-arm In-Reply-To: <50728566-11C2-45EB-8367-00CAF38D4548@dsl-only.net> Message-Id: X-Mailer: Apple Mail (2.3273) X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 21 Oct 2017 23:09:50 -0000 On 2017-Oct-21, at 2:06 PM, Mark Millard wrote: > On 2017-Oct-21, at 11:43 AM, Emmanuel Vadot = wrote: >=20 >> On Sat, 21 Oct 2017 11:26:57 -0700 >> Mark Millard wrote: >>=20 >>> On 2017-Oct-21, at 10:58 AM, Emmanuel Vadot wrote: >>>=20 >>>>>> . . . >>>=20 >>> If I understand the current status correctly, the >>> recent changes to use upstream DTS materials took >>> the BPI-M3 from having, e.g., USB working to USB >>> not working. -r323641 removed AWUSBPHY_TYPE_A83T >>> from enum awusbphy_type because of what was missing >>> from upstream DTS materials and such. >>=20 >> Uses of upstream DTS is active since ~2 years ago. >> I think I removed support for USB on A83T because I couldn't test, = but >> maybe the code just work and adding the compatible string again might >> do it. Feel free to send a patch on phabricator if it does. >=20 > . . . Note: My "normal" environment (such as /usr/src/ ) is currently at head -r324743 . That is what I'm comparing -r317015 to for now. I've started by seeing what if anything changed in .dt* files. Unfortunately I ran into changes in .dt* files used for bpi-m3 since -r317015 (indirect via #includes). At issue are at least: #include "sunxi-common-regulators.dtsi" and #include "sun8i-a83t.dtsi" (but the last one is complicated by multiple files with that name in different places, one having updates and the other not). It looks like I'll need a hint if any of the .dt* file that I've found so far that have differences since -r317015 have implications about the sysutils/u-boot-sinovoip-bpi-m3 being sufficient or not (for example). There might be implications for kernel versions as well. It might be that to have things work would require substituting the old content into the bpi-m3 specific files instead of including modern. I'm not so sure anyone that works on the code regularly would want such a thing: They might prefer that support be dropped for BPI-M3, including removal of: sysutils/u-boot-sinovoip-bpi-m3 More detail if needed. . . While: sys/boot/fdt/dts/arm/ does not have differences that I've found for bpi-m3 or a83t some gnu/dts/arm/ material that is included does: #include "sunxi-common-regulators.dtsi" The included file was updated in -r32130 , -r320834 , and -r324820 (today). The following may have differences in the included file: #include "sun8i-a83t.dtsi" Why "may"? # find /usr/src/sys/ -name sun8i-a83t.dtsi -print /usr/src/sys/boot/fdt/dts/arm/sun8i-a83t.dtsi /usr/src/sys/gnu/dts/arm/sun8i-a83t.dtsi There are two files with the matching name that might be found. I've not figured out which is used yet. The boot/fdt one is unchanged from -r299748 (2016-May-14 when it was created) but the gnu one has been updated in -r32130 , -r320834 , and -r324820 (today), much like for sunxi-common-regulators.dtsi . (I might not have traced every include that has potential updates but the above ones probably give a big hint at compatibility implications for someone that knows what they are looking at.) =3D=3D=3D Mark Millard markmi at dsl-only.net