All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 memstick: support for legacy sony memsticks
@ 2012-09-26  9:48 Maxim Levitsky
  2012-09-26  9:49 ` [PATCH 1/2] scatterlist: add sg_nents Maxim Levitsky
  2012-09-26  9:49 ` [PATCH 2/2] memstick: add support for legacy memorysticks Maxim Levitsky
  0 siblings, 2 replies; 15+ messages in thread
From: Maxim Levitsky @ 2012-09-26  9:48 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Alex Dubov, linux-kernel, Tejun Heo

Hi,
I am sending an updated version of the driver, this time
addressing comments by Tejun Heo


Changes since V2:

* Removed the mutex kludge - ordered worqueue indeed works well
* Removed manul major/minor handling
* Gave more specific names to two sg helpers.

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.



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

end of thread, other threads:[~2012-10-11 10:29 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-26  9:48 [PATCH v3 memstick: support for legacy sony memsticks Maxim Levitsky
2012-09-26  9:49 ` [PATCH 1/2] scatterlist: add sg_nents Maxim Levitsky
2012-09-26 16:38   ` Tejun Heo
2012-09-27 10:47     ` Jens Axboe
2012-09-27 14:28       ` [PATCH] scatterlist: refactor the sg_nents Maxim Levitsky
2012-09-28  8:37         ` Jens Axboe
2012-10-11 10:27   ` [PATCH 1/2] scatterlist: add sg_nents Geert Uytterhoeven
2012-10-11 10:28     ` Geert Uytterhoeven
2012-09-26  9:49 ` [PATCH 2/2] memstick: add support for legacy memorysticks Maxim Levitsky
2012-09-26 16:41   ` Tejun Heo
2012-09-29 17:20   ` Geert Uytterhoeven
2012-10-01 20:24     ` Maxim Levitsky
2012-10-08 15:36     ` Fixes for ms_block Maxim Levitsky
2012-10-08 15:36       ` [PATCH 1/2] memstick: remove unused field from state struct Maxim Levitsky
2012-10-08 15:36       ` [PATCH 2/2] memstick: ms_block: fix complile issue Maxim Levitsky

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.