From owner-freebsd-embedded@FreeBSD.ORG Wed Mar 5 21:56:05 2014 Return-Path: Delivered-To: freebsd-embedded@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 6C2858AA; Wed, 5 Mar 2014 21:56:05 +0000 (UTC) Received: from mail-la0-x232.google.com (mail-la0-x232.google.com [IPv6:2a00:1450:4010:c03::232]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 4B7E02BE; Wed, 5 Mar 2014 21:56:04 +0000 (UTC) Received: by mail-la0-f50.google.com with SMTP id y1so1139360lam.37 for ; Wed, 05 Mar 2014 13:56:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=ng7adWkPMKnLijqoT28RKh0vpUtyT+ZOZ/fwDaFiiGM=; b=xkEheOH7OouvT4awjNT3rOGk7usqHOrAqPQ4ij42FOg16Ssk9VwxpPPnwcocJ+R8yD F94Ry4jA3aQFPToQl8Sq0W1y310ljn4qc4y3w21FFA1lpHv75HmEdC9ft5O0UtON5Cc0 9V23CrDsS9w9eXFXLGOc1hrjs7HjLgQhReuheLwbSo+tIJr5j+IXWRPiJKTB7EySs8an MOnxJSmwyUda0lmfRK9nXY+6eHglqbAQ1/Qlk35taWNizHafeCeqbdFjB0W+nCVxRa7U hwwqavb0ZggV4hJ2ed664Lj9MX/28u91HsXVXDsxb2t3FuxtgbghOWCdfMPM09Q6k2Rv adgA== MIME-Version: 1.0 X-Received: by 10.152.181.37 with SMTP id dt5mr3242115lac.43.1394056562325; Wed, 05 Mar 2014 13:56:02 -0800 (PST) Sender: pkelsey@gmail.com Received: by 10.112.142.5 with HTTP; Wed, 5 Mar 2014 13:56:02 -0800 (PST) In-Reply-To: <16A5203B-B06D-4129-A54F-F34D5FA28D2B@bsdimp.com> References: <7C2B7036-51CC-4C97-80C4-0A439874357D@bsdimp.com> <1393939277.1149.300.camel@revolution.hippie.lan> <438620C4-7712-4B01-A46F-CB57946A30BF@bsdimp.com> <16A5203B-B06D-4129-A54F-F34D5FA28D2B@bsdimp.com> Date: Wed, 5 Mar 2014 16:56:02 -0500 X-Google-Sender-Auth: ZCJ9SdXeFPz5hc2HF0Hels8UoLQ Message-ID: Subject: Re: [PATCH] simplebus child device probe order control via FDT (motivated by BeagleBone Black) From: Patrick Kelsey To: Warner Losh Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.17 Cc: "freebsd-hackers@freebsd.org" , freebsd-arm , freebsd-embedded@freebsd.org, Ian Lepore X-BeenThere: freebsd-embedded@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: Dedicated and Embedded Systems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Mar 2014 21:56:05 -0000 On Wed, Mar 5, 2014 at 4:24 PM, Warner Losh wrote: > > On Mar 5, 2014, at 11:55 AM, Patrick Kelsey wrote: > > > > > > > > > On Wed, Mar 5, 2014 at 8:31 AM, Warner Losh wrote: > > > > On Mar 4, 2014, at 11:53 PM, Patrick Kelsey wrote: > > > > > > > > > > > > > > On Tue, Mar 4, 2014 at 8:21 AM, Ian Lepore wrote: > > > > > > There's a standard property for mmc/sd devices, "non-removable" whose > > > presence denotes things like soldered-on eMMC parts. Only one of our > > > many sdhci drivers supports it right now (imx). In general the core > > > part of the driver (dev/sdhci) doesn't have good support for > > > insert/remove/presence detection that's handled off to the side > (whether > > > it's non-removable or a gpio pin). > > > > > > That alone doesn't solve the wider problem, though, which I think > breaks > > > down into two pieces. Let's say you've got two slots, call them left > > > and right. You end up with these two problems... > > > > > > * Sometimes the right slot is mmcsd0, but it turns into mmcsd1 if > > > there's a card in the left slot when I boot; I don't want it to > > > change. > > > > > > And not just a boot-time issue, of course. If you were to remove > those two cards and then reinsert them in the opposite time-order, their > device names would swap. > > > > > > * I want the slot on the left to be named '1' and the right to be > > > '0'. > > > > > > The first problem is easily solved without impacting dts in any way. > We > > > just wire down the relationship controllerN -> mmcN -> mmcsdN. This is > > > exactly equivelent to the old ATA_STATIC_ID option in its effect -- you > > > don't get to choose the names, but you know they won't change based on > > > which devices are present. It could be controlled with a tunable. > > > > > > It's harder to envision the fix for the second part without adding an > > > ad-hoc property for the devices. Even with a property I'm not sure how > > > easy it would be. > > > > > > We should be able to assign a geographic address (controllerX:slotY) > to each mmcsd device in a given system (let's ignore for now the > theoretical possibility of multiple cards on one bus). The 'controllerX' > part of the address could be the controller's pathname from the devicetree, > or an index assigned by mmcbr machinery at attach time. The "slotY" part > of the address is assigned by the specific controller device driver using > some internally-determined fixed mapping between the assigned values and > its physical slots. This geographic address could be used to create an > additional set of /dev entries with stable names. There could be a > mechanism for user-configurable aliases for the geographical addresses. > > > > There's already a chance to run a script when a device is attached that > can create /dev/slot0 or /dev/slot1 that has geographical information > available to it. People use ddvd for this in the USB world all the time to > make sure that tty devices get a symlink based on their location or serial > number. > > > > Why is mmc so different it needs its own mechanism? > > > > I'm laying out my view of the information that would be needed and the > types of actions that would have to be taken based on that information to > solve the issue. I'm not saying devd can't be the piece that is used to > implement the actions (indeed, I noted devd as a potential building block > for a solution in my initial response). I'm also not saying mmc needs its > own mechanism, I'm saying it needs /a/ mechanism, and so far there still > seems to be something missing (because it's not there, or I'm still > ignorant of it). > > > > What seems to be missing is geographical addressing for mmc devices. > > > > I think what you might be saying is that the existing mmcsd add/remove > code could be augmented to send devctl notifications, along the lines of: > > > > devctl_notify("MMC", "DEVICE", "ATTACH"|"DETACH", "... > controller= > slot= rca=") > > > > and then I and the fine author of devctl and devd would be pleased. > > MMC doesn't need anything special here. That's already present. Looking at > the device tree we see on one of the platforms that's handy for me to > access: > > at91_mci0 > mmc0 > mmcsd0 at rca=0xb368 > > So you know that mmcsd0 is on mmc0 bus attached to at91_mci0, which is > ultimately the FDT node where things came from. There's not a user-defined > slot associated with this (and we should have a SLOT A vs SLOT B as a > location info for this platform, because we can have two cards on the one > bus in the MMC case), true, but for your use case I don't think that you > need it. We should be attaching the host controller regardless of whether > the or not there's a card in there, so it will be fixed. While some > additional information would be useful to publish, I don't think your use > case requires it... > > The reason you need something extra here is that what is there now breaks down whenever you don't have a one-to-one mapping between controllers and buses. Any controller with more than one slot can yield something of the form: sdhci0 mmc0 mmcsd0 at rca=0xabcd mmc1 mmcsd1 at rca=0x1234 and you have no idea what physical slot in the system mmcsd0 and mmcsd1 are in. For my immediate use case, sure, I can rely on the one-to-one relationship between controllers and buses. At this point, though, rather than skate by on that happy coincidence, I'd rather invest what now seems to be a rather small amount of effort adding mmcsd devctl notifications that would cover the multiple-slots-per-controller case as well, and then build the rest of what I want on top of that information coming out of devd. On the at91 platform cited above, that allows you to connect two MMC cards to the same bus (i.e., multidrop configuration), is there any way to distinguish which card is in which physical slot? I'm still under the impression that this is the one case where we aren't going to be able to determine the physical location of every mmcsd device. -Patrick