public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 memstick: support for legacy sony memsticks
@ 2012-09-25  8:38 Maxim Levitsky
  2012-09-25  8:38 ` [PATCH 1/2] scatterlist: add sg_nents Maxim Levitsky
                   ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: Maxim Levitsky @ 2012-09-25  8:38 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Alex Dubov, linux-kernel, Tejun Heo


Hi,

I believe that I addressed most(all?) issues raised in the review, thus I
am resending the driver.


Changes from V1:

* Replaced explicit state numbers with enum items.
  I agree that code is cleaner this way.

* Swithced to using a workqueue.
  Unfortunelly, I still see that workqueue items are executed in parallel
  I suspect that this happens if one work item sleeps. In this case I
  don't want other work items to run too. I fixed this with a mutex, and anyway
  it nice to have it to guaranteee this.

* moved sg_nents into scatterlist.c/h and put that in separate patch as
  requested

* Made functions static, fixed most whitespace issues (I hope all),
  and I hope addressed all other comments by Andrew Morton.

Best regards,
	Maxim Levitsky

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2012-09-25 20:24 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-25  8:38 [PATCH v2 memstick: support for legacy sony memsticks Maxim Levitsky
2012-09-25  8:38 ` [PATCH 1/2] scatterlist: add sg_nents Maxim Levitsky
2012-09-25  8:38 ` [PATCH 2/2] memstick: add support for legacy memorysticks Maxim Levitsky
2012-09-25 18:25   ` Tejun Heo
2012-09-25 19:26     ` Maxim Levitsky
2012-09-25 19:38       ` Tejun Heo
2012-09-25 20:24         ` Maxim Levitsky
2012-09-25 18:02 ` [PATCH v2 memstick: support for legacy sony memsticks Tejun Heo
2012-09-25 19:34   ` Maxim Levitsky
2012-09-25 19:40     ` Tejun Heo
2012-09-25 20:13       ` Maxim Levitsky

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox