[PATCH v2] atmel/spi: fix missing probe during the switch to module_platform_driver

Russell King - ARM Linux linux at arm.linux.org.uk
Thu Nov 3 14:08:21 EDT 2011


On Thu, Nov 03, 2011 at 07:03:38PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 17:43 Thu 03 Nov     , Russell King - ARM Linux wrote:
> > On Thu, Nov 03, 2011 at 05:51:13PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > this is an other fix I do not want to fix 2 issue in one commit and I do care
> > > of mismatch erros I fix them all the time I send patch for some rm9200 few
> > > weeks ago
> > 
> > It should not be a separate patch - it's all to do with fixing the
> > original problem.
> 
> yes when we was supposed to swtch to module_platform_driver we use the .probe
> which is suposed to be __devinit in this case but I was prefering to do it in
> 2 steps

Then do the __devinit transformation _first_ so you don't introduce a new
bug in the middle of your 'fix'.

But I personally think you're just creating needless churn by splitting
this one logical change.



More information about the linux-arm-kernel mailing list