Hello Brian,

On 08/20/2015 11:11 PM, Brian Norris wrote:

On Thu, Aug 20, 2015 at 09:07:13AM +0200, Javier Martinez Canillas wrote:

quoted

Patches #1 and #2 solves a), patches #3 to #8 solves b) and patches

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

Sigh, I did some last minute restructuring of the cover letter and
seems I missed a sentence. I meant to said:

"and patches #9 to #17 solves c)."
 

quoted

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.

Great, thanks a lot for your feedback.
 

Brian

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America