From owner-svn-src-head@freebsd.org Sun Jan 28 21:11:07 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id B0325EDF551 for ; Sun, 28 Jan 2018 21:11:07 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-it0-x236.google.com (mail-it0-x236.google.com [IPv6:2607:f8b0:4001:c0b::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 352A56EAD9 for ; Sun, 28 Jan 2018 21:11:07 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-it0-x236.google.com with SMTP id w14so7077363itc.3 for ; Sun, 28 Jan 2018 13:11:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=DPKQFeLTZdABNXoANKpzyE89Q6jdLJxeZXN0xKUprQQ=; b=Sw56yLw5wHKGwT8XacW+v4HSBiOKuTWiDNhctrdaSsv36tlGtrn7whwlPEhqtUQsbb Zw9Ybc8hWUhSh236mUnBvPiKy2+zHeXYFQuWacwBQYUx4tc+L1Yf2L4ou8R2I8EF0RAd Kl8Y4se5iY4Xe1+mTenI7sCXVNPk21IJtXgbQg5R3uQnUKBPbNDly9dppy5oX5WfBE/4 6kdIYEVGRco4TqmfIxN101+z//IXXubmrHQ2EzPiMzS45QSIfgLtQ0xU2A2ao9fe9tVY Ow0YW1HHharp5lPHZ/I/BMsu6nAY6RM3aHticfW5BxgHEgX/0DwNbrdFpZH+EUQOziGR uevg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=DPKQFeLTZdABNXoANKpzyE89Q6jdLJxeZXN0xKUprQQ=; b=S6WtK6bZTfLJIgu3xY3iOhwtoVA30ert1lyw34EtNnUjsper34iS4R1u0gHlmphQ3n Sfnm+Xagjm0e9wibV63vhsidrA53uY7sEzvpuXqHd2vAuiRf+lx8lAhHEE6ROf66eqwJ l03atZ0/Wvw0ovpyLgI9QAaDqQBpQGXTi8gkg+gop6HMMVBX2bn70Gs9LfGI864fL9b8 BKmYepfnF+9e7tRj8Zyk/PzcC4xLt5vRzaNDTNQA+sGpk10/vGRT93Nsw9Fl0Tv4Pwip VMuKatMVfXQ7mjC4Bt0FJVa7g/++8W9oJ0U2EAuZEUMu78NfME08k5LRqtgc2pt/A7Vg fUkQ== X-Gm-Message-State: AKwxyteFJKOsgMurZMAVf7V3UDnDzES98p29KBGWnNR7y5szVn0Sw7Aq CGtGkta/vc0srWK8G5hKr7JyT/J3ZOqKzbnlYOJ6Jg== X-Google-Smtp-Source: AH8x227VBaG07xEAs43lhZPR+MBh7XTtvxxo1trGoavAjKfXlLEqEFWgwkwtXAtMNY0uUdDNIIhOekeAY4JPuyeMcuE= X-Received: by 10.36.74.200 with SMTP id k191mr26233743itb.69.1517173866360; Sun, 28 Jan 2018 13:11:06 -0800 (PST) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 10.79.201.67 with HTTP; Sun, 28 Jan 2018 13:11:05 -0800 (PST) X-Originating-IP: [50.253.99.174] In-Reply-To: References: <201801220710.w0M7AUm9091853@repo.freebsd.org> <90451.1516663240@critter.freebsd.dk> <2987003.eeGRFBb6N8@ralph.baldwin.cx> <93949.1516733748@critter.freebsd.dk> <20180127210801.37b8001125dd0a2c92372f98@bidouilliste.com> <72042.1517094867@critter.freebsd.dk> <8d8ae9d10058fd72ce3ec467181c9f22@megadrive.org> From: Warner Losh Date: Sun, 28 Jan 2018 14:11:05 -0700 X-Google-Sender-Auth: XWJbiozyd66xGjXFdx-85gmoyLs Message-ID: Subject: Re: svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules To: Adrian Chadd Cc: Emmanuel Vadot , Poul-Henning Kamp , John Baldwin , Ravi Pokala , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org, owner-src-committers@freebsd.org Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Jan 2018 21:11:08 -0000 On Sun, Jan 28, 2018 at 2:01 PM, Adrian Chadd wrote: > [snip] > > And this is the root bit that's missing - dynamic pinmux/pinctrl stuff > at runtime. > Well, you gotta walk before you can run. We don't even have static pinmx on rpi. Though someone else is working on that. > Warner's already said he's WIP'ing it and phk seems like a good test > case for kicking those tyres, so it sounds like those tyres are about > to be kicked. > > In the meantime, hacks allow people to make some progress, and as long > as they're not on by default, it's okay. The challenge is finding the > middle ground between "right" and "working". Some people are happy to > do the legwork to do things right first; others are happy to do the > end bits and then backfill the supporting infrastructure. > No. Such hacks are actively getting in the way. My WIP already breaks PHK's hacks (unintentionally, but discovered in hindsight) because they are outside the mainstream. I have 0 interest in preserving short-term hacks through the longer-term fixes, and while I'd rather not step on toes, I don't really feel bad in cases like this... However, my WiP is more about fixing some issues in NEWBUS / devctl and trying to have some order in the "Let's change 'disabled' it at runtime" with enough hooks so that we do more than just turn on the device driver for the device, but also integrate the new state into the dependencies, like pinmux/pinctl. We'll need that regardless of whether we need a quick hack to turn them on/off or we allow loading DTB overlays at runtime. Unlike x86 where you might have not attached fxp0 and just need to call fxp_attach() (basically) to make it work, in the embedded space, you'll have lots of unhappy campers if you don't also deal with the dependent power, clock and pin resources. > Fun times, fun times! I'm just happy to see more RPI support. That > platform still isn't dying. > Yea, otherwise we could kill armv6 completely. Warner