Hi, Mark Brown wrote: > On Fri, Jul 18, 2014 at 01:30:53PM +0200, Lothar Waßmann wrote: > > Aua. This really hurts. I wonder how this could ever be admitted to > > the Linux kernel... > > Further comments suppressed because the would most likely violate the > > CDA. > > > > If someone should not grasp what this patch does, they should consider > > what happens upon unloading/reloading the kernel module. > > > > Signed-off-by: Lothar Waßmann <LW@xxxxxxxxxxxxxxxxxxx> > > I'm not going to apply this with a commit message such as the above. > Quite aside from the tone the fact that it doesn't describe the issue > is not helpful for review, one of the things done in review is to try to > check that the change has the intended effect. > Maybe the original author or the maintainer who accepted this can come up with a decent fix for this. Lothar Waßmann -- ___________________________________________________________ Ka-Ro electronics GmbH | Pascalstraße 22 | D - 52076 Aachen Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10 Geschäftsführer: Matthias Kaussen Handelsregistereintrag: Amtsgericht Aachen, HRB 4996 www.karo-electronics.de | info@xxxxxxxxxxxxxxxxxxx ___________________________________________________________ -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html