From patchwork Sun May 20 06:08:01 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Grant Likely X-Patchwork-Id: 160241 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from ozlabs.org (localhost [IPv6:::1]) by ozlabs.org (Postfix) with ESMTP id 98EABB707F for ; Sun, 20 May 2012 16:08:50 +1000 (EST) Received: from mail-pz0-f51.google.com (mail-pz0-f51.google.com [209.85.210.51]) (using TLSv1 with cipher RC4-MD5 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority" (not verified)) by ozlabs.org (Postfix) with ESMTPS id C0ED9B6FAB for ; Sun, 20 May 2012 16:08:05 +1000 (EST) Received: by dajt11 with SMTP id t11so6917095daj.38 for ; Sat, 19 May 2012 23:08:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=sender:from:subject:to:cc:in-reply-to:references:date:message-id :x-gm-message-state; bh=/h5uIm4kc5mZVZUNzpnVm9bFN5A9sojgcxUBOtYAbqg=; b=RrXtCISsQU/hZ7UiJaPJgxsXsKrykavNibFx3FziozCakusVTKwaEQukPKV7h6UgJ/ zE1iF8dt9ySeP9BFNOHkToxMxOLJmDkUP6tF/4Rs84aDt2ZKToY2baGjGO5IjSLjc7W2 2Y3tbJxnQm2vB8rJ300HuNQZ5s/o+zpumMBuOqsyOgYMxDnZui+v77A6MSxx5695zhNN EXhDFyxtvC6qmdpo4bSHY2U8uXBSVnFadOSb0NkVNL9gG6wRlX5gnf9L17V+htUxtVv7 D3agz2x368lLmvooBfHzwU5lGYaWExZHQpcuALB9SKNkYMvvcayUFCsvrAi2jM6EcxGa aE3A== Received: by 10.68.200.193 with SMTP id ju1mr52235368pbc.90.1337494083721; Sat, 19 May 2012 23:08:03 -0700 (PDT) Received: from localhost (S0106d8b37715ee14.cg.shawcable.net. [68.146.14.168]) by mx.google.com with ESMTPS id rv8sm13214960pbc.64.2012.05.19.23.08.02 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 19 May 2012 23:08:03 -0700 (PDT) Received: by localhost (Postfix, from userid 1000) id 03CE73E03B8; Sun, 20 May 2012 00:08:01 -0600 (MDT) From: Grant Likely Subject: Re: [PATCH 1/3] of: Add prefix parameter to of_modalias_node(). To: David Daney , devicetree-discuss@lists.ozlabs.org, Rob Herring , spi-devel-general@lists.sourceforge.net In-Reply-To: <20120520055436.13AF03E03B8@localhost> References: <1336773923-17866-1-git-send-email-ddaney.cavm@gmail.com> <1336773923-17866-2-git-send-email-ddaney.cavm@gmail.com> <20120520055436.13AF03E03B8@localhost> Date: Sun, 20 May 2012 00:08:01 -0600 Message-Id: <20120520060802.03CE73E03B8@localhost> X-Gm-Message-State: ALoCoQlMn5N9Kx36O7pX/L18bGJudluTfGB2qxIPhhcBDSebH7RSNMGnKxltRh5pFJ4FgmCLU09M Cc: linux-mips@linux-mips.org, alsa-devel@alsa-project.org, Mark Brown , David Daney , Takashi Iwai , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Liam Girdwood , Jaroslav Kysela , linuxppc-dev@lists.ozlabs.org, Timur Tabi X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org On Sat, 19 May 2012 23:54:36 -0600, Grant Likely wrote: > On Fri, 11 May 2012 15:05:21 -0700, David Daney wrote: > > From: David Daney > > > > When generating MODALIASes, it is convenient to add things like "spi:" > > or "i2c:" to the front of the strings. This allows the standard > > modprobe to find the right driver when automatically populating bus > > children from the device tree structure. > > > > Add a prefix parameter, and adjust callers. For > > of_register_spi_devices() use the "spi:" prefix. > > > > Signed-off-by: David Daney > > Applied, thanks. Some notes below... Wait... why is this necessary? The module type prefix isn't stored in the modalias value for any other bus type as far as I can see, and with this series it appears that the "spi:" prefix may or may not be present in the modalias. That doesn't look right. Why isn't prefixing spi: at uevent time sufficient? IIUC, modprobe depends on either UEVENT or the modalias attribute to know which driver to probe. It does look like the attribute is missing the spi: prefix though. Does the following change work instead of these two patches? for you then I'll push it out. g. diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index 3d8f662..da8aac7 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c @@ -51,7 +51,7 @@ modalias_show(struct device *dev, struct device_attribute *a, char *buf) { const struct spi_device *spi = to_spi_device(dev); - return sprintf(buf, "%s\n", spi->modalias); + return sprintf(buf, "%s%s\n", SPI_MODULE_PREFIX, spi->modalias); } So, I've dropped this patch from my tree. If the change above works