linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brian Norris <computersforpeace@gmail.com>
To: Javier Martinez Canillas <javier@osg.samsung.com>
Cc: linux-kernel@vger.kernel.org,
	"Andrzej Hajda" <a.hajda@samsung.com>,
	linux-fbdev@vger.kernel.org,
	"Michael Hennerich" <Michael.Hennerich@analog.com>,
	linux-iio@vger.kernel.org, linux-wireless@vger.kernel.org,
	"Lee Jones" <lee.jones@linaro.org>,
	"Jean-Christophe Plagniol-Villard" <plagnioj@jcrosoft.com>,
	linux-mtd@lists.infradead.org,
	"Lauro Ramos Venancio" <lauro.venancio@openbossa.org>,
	"Søren Andersen" <san@rosetechnology.dk>,
	devel@driverdev.osuosl.org,
	"Randy Dunlap" <rdunlap@infradead.org>,
	"Masanari Iida" <standby24x7@gmail.com>,
	"Jiri Kosina" <jkosina@suse.cz>,
	"Devendra Naga" <devendra.aaru@gmail.com>,
	"Tomi Valkeinen" <tomi.valkeinen@ti.com>,
	"Kyungmin Park" <kyungmin.park@samsung.com>,
	"Krzysztof Kozlowski" <k.kozlowski@samsung.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Aloisio Almeida Jr" <aloisio.almeida@openbossa.org>,
	"Jonathan Cameron" <jic23@kernel>
Subject: Re: [PATCH 00/18] Export SPI and OF module aliases in missing drivers
Date: Thu, 20 Aug 2015 14:11:52 -0700	[thread overview]
Message-ID: <20150820211152.GI74600@google.com> (raw)
In-Reply-To: <1440054451-1223-1-git-send-email-javier@osg.samsung.com>

On Thu, Aug 20, 2015 at 09:07:13AM +0200, Javier Martinez Canillas wrote:
> Patches #1 and #2 solves a), patches #3 to #8 solves b) and patches

^^^ I'm dying to know how this sentence ends :)

> Patch #18 changes the logic of spi_uevent() to report an OF modalias if
> the device was registered using OF. But this patch is included in the
> series only as an RFC for illustration purposes since changing that
> without first applying all the other patches in this series, will break
> module autoloading for the drivers of devices registered using OF but
> that lacks an of_match_table. I'll repost patch #18 once all the patches
> in this series have landed.

On a more productive note, the patches I've looked at look good to me.
The missing aliases are a problem enough that should be fixed (i.e.,
part (b)). I'll leave the SPI framework changes to others to comment on.

Brian

  parent reply	other threads:[~2015-08-20 21:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-20  7:07 [PATCH 00/18] Export SPI and OF module aliases in missing drivers Javier Martinez Canillas
2015-08-20  7:07 ` [PATCH 18/18] spi: (RFC, don't apply) report OF style modalias when probing using DT Javier Martinez Canillas
     [not found]   ` <1440054451-1223-19-git-send-email-javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-08-20 18:36     ` Mark Brown
     [not found]       ` <20150820183634.GN12027-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2015-08-20 21:08         ` Brian Norris
     [not found]           ` <20150820210822.GH74600-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
2015-08-20 21:45             ` Javier Martinez Canillas
     [not found]               ` <55D64A65.7030905-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-08-20 23:25                 ` Mark Brown
2015-08-20 23:47                   ` Javier Martinez Canillas
     [not found]                     ` <55D666F4.2080801-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-08-21 17:29                       ` Mark Brown
     [not found]                         ` <20150821172931.GK12027-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2015-08-24  8:00                           ` Javier Martinez Canillas
2015-08-20 21:11 ` Brian Norris [this message]
2015-08-20 21:50   ` [PATCH 00/18] Export SPI and OF module aliases in missing drivers Javier Martinez Canillas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150820211152.GI74600@google.com \
    --to=computersforpeace@gmail.com \
    --cc=Michael.Hennerich@analog.com \
    --cc=a.hajda@samsung.com \
    --cc=aloisio.almeida@openbossa.org \
    --cc=devel@driverdev.osuosl.org \
    --cc=devendra.aaru@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=javier@osg.samsung.com \
    --cc=jic23@kernel \
    --cc=jkosina@suse.cz \
    --cc=k.kozlowski@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=lauro.venancio@openbossa.org \
    --cc=lee.jones@linaro.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=plagnioj@jcrosoft.com \
    --cc=rdunlap@infradead.org \
    --cc=san@rosetechnology.dk \
    --cc=standby24x7@gmail.com \
    --cc=tomi.valkeinen@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).