From owner-svn-src-head@freebsd.org Mon Jan 29 06:54:08 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 86C0CED97C1 for ; Mon, 29 Jan 2018 06:54:08 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-it0-x22e.google.com (mail-it0-x22e.google.com [IPv6:2607:f8b0:4001:c0b::22e]) (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 07F7F8229D for ; Mon, 29 Jan 2018 06:54:08 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-it0-x22e.google.com with SMTP id 68so8093053ite.4 for ; Sun, 28 Jan 2018 22:54:08 -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=lErqnTVGEGpBbHloF8EvX7o/3cLwmVnVdQmRzJMQSIM=; b=BxmIEWKqThPuVKOHG4KjHZYHlZr8dx4RRSZQdelNj/zOa8Fl1f6dcZaCkE1Sgz2lfg 0eRrSmlMJbmhpb2qtpe72bNeK+q2Erqfb0Fe7aBxK0h0uhK9xwcOgOsgWZWzm6z1F2us Dfk5bY/BHKX7zrQQaIR+0HtWPQKXzRyBd44auHOcSKS4zd6y4r9w4+uoQbneBD/OwpDM 1kbMmvR/awFEBLgnv9Yct6acYMTIYFct+A/TRk16+xGRbTebAhqulyZyv/ezvdpY7Ufy 41LHnFYl9kDJu82LcZhZKbqjp76wECEFkJhe3Z1RHJ05Sbw+HNl2HgKxRng2NPur7ee0 EvNA== 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=lErqnTVGEGpBbHloF8EvX7o/3cLwmVnVdQmRzJMQSIM=; b=i5U4bX3NszeZ+4IwGOPPlw4/pAvWkfzZGdszln+SntMrIKj7+MulXPHDZ7iGdOXKNt UQUbeTCqSn+drwlPJjy6Cq7DASK5pCHiHU5OwLcKt4mT7e7qBKmm9SfznUKzJ2P0EvIm burwwUUS6i112R8WTrG0zCmtOmhsI0N8BfFk4Ok4hyr9mydNgqFQBn2ZKhBNu9Hq7fJq /1qgQLMAJUkbrRI4ivTaj9EWXVRKC1MeO5oaRZvbkNrI3j61BVxUBY/ws9zAbS0bzAg9 s/wwmWx8UmvlxbdfUX5k5TauXhqZH8gvH2LreaC607+TIp2R8+VN9kUzhwVmYIw4KLx8 72Wg== X-Gm-Message-State: AKwxytdlwNJKLLz943Gw51OX8GO6t+PYljKgCNvzWhZ9sUKrr3ihXKrV hcNyuGJWh7kFnQXexpjyaFI/TyZ1xdEpIMsxYQ2aZg== X-Google-Smtp-Source: AH8x227WYuLgE321IvFIExbU66JDs4s1svPBzIDGwbVc4bT2Ze88KMuTPC4F3VO+x2avp3wfUCO51QFt33QdtWms+QQ= X-Received: by 10.36.146.196 with SMTP id l187mr26846821itd.115.1517208847331; Sun, 28 Jan 2018 22:54:07 -0800 (PST) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 10.79.201.67 with HTTP; Sun, 28 Jan 2018 22:54:06 -0800 (PST) X-Originating-IP: [50.253.99.174] In-Reply-To: <20180129063950.GA59901@bluezbox.com> 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> <13025.1517179897@critter.freebsd.dk> <20180129063950.GA59901@bluezbox.com> From: Warner Losh Date: Sun, 28 Jan 2018 23:54:06 -0700 X-Google-Sender-Auth: oMRvkGxHB6s4WWIQBEnLKMDp-Zw Message-ID: Subject: Re: svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules To: Oleksandr Tymoshenko Cc: Poul-Henning Kamp , Emmanuel Vadot , 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: Mon, 29 Jan 2018 06:54:08 -0000 On Sun, Jan 28, 2018 at 11:39 PM, Oleksandr Tymoshenko wrote: > > Ignoring value of ofw_bus_status_okay(dev) in probe method is > wrong for number of reasons and pinctl and clocks were brought up > as an example why "status" property is more than just > attach/dont-attach flag. So there is strictly technical reason > why it should be checked. Once this boilerplate code moves to > simplebus bus probe method as I believe Warner proposed drivers > will not be able to control this aspect any more. > I have patches that does exactly this. I have a few more that will allow drivers some control over the situation planned, but they require hammering out some things in newbus first, which I'll write up once I have something workable and it's been blessed by a couple people I've asked to give feedback. The writeup will just be the first round, not the final destination, btw. We need to work out several tricky issues that are interlocking to ensure that we can enable drivers selectively after boot time. That's tricky for a lot of reasons, but the tl;dr version is that embedded is different because you don't just enable a device, but all the dependencies which, unlike x86, are often 'non-linear' and possibly non-orthogonal to other devices (eg, you may either have A B and C enabled, or D E and F, but never any other combination, how do you transition from state to state sanely). It's a tricky ball of wax that we've neglected for too long. Warner