Date: Sat, 24 Jan 2009 18:56:12 +0100
From: Pierre Ossman <drzeus-mmc@drzeus.cx>
To: Andy Whitcroft <apw@canonical.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/1] mmc: add MODALIAS linkage for MMC/SD devices
Message-ID: <20090124185612.5afddbb0@mjolnir.drzeus.cx>
In-Reply-To: <20090115150003.GB6896@shadowen.org>
References: <1231268198-12556-1-git-send-email-apw@canonical.com>
	<20090112160519.3ba21848@mjolnir.drzeus.cx>
	<20090115150003.GB6896@shadowen.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1434
Lines: 36

On Thu, 15 Jan 2009 15:00:03 +0000
Andy Whitcroft <apw@canonical.com> wrote:

> 
> Would it make more sense to simply use mmc: with no data as the MODALIAS
> tag here.  And make mmc-block have mmc:* as an alias.  That way the
> actual (useless) numeric data is not exposed but we still will load the
> mmc-block module for everything exactly as we do with the udev rule.
> This should still make things work from a distro point of view and not
> expose anything which might then get relied on.  If we do it so extra
> data can still be added later we won't prevent sane data being exposed
> should we ever have any.
> 
> If you are happier with that I can re-spin the patches that way?
> 

Well, as long as we're on the track of temporary hack, we might as well
just export "mmc_block" as the modalias. Or would there be any
side-effects to that?

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  rdesktop, core developer          http://www.rdesktop.org

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/