Skip to content

ide-cd: first conversion batch

Borislav Petkov linux-ide lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 10 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8ide-atapi: compute cmd_len based on device type in ide_transfer_pcdrivers/ide/ide-atapi.c+13 -10 replies
2/8ide-atapi: assign expiry and timeout based on device typedrivers/ide/ide-atapi.c+20 -151 reply
3/8ide-atapi: split drive-specific functionality in ide_issue_pcdrivers/ide/ide-atapi.c+20 -160 replies
4/8ide-cd: remove xferlen arg to cdrom_start_packet_commanddrivers/ide/ide-cd.c+5 -50 replies
5/8ide-cd: remove handler wrappersdrivers/ide/ide-cd.c+48 -731 reply
6/8ide-atapi: remove timeout arg to ide_issue_pcdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-tape.c +1 more+9 -40 replies
7/8ide-atapi: put the rest of non-ide-cd code into the else-clause of ide_transfer_pcdrivers/ide/ide-atapi.c+8 -80 replies
8/8ide-atapi: start dma in a drive-specific waydrivers/ide/ide-atapi.c+11 -40 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/8] ide-cd: first conversion batch

Hi Bart,

here's the first batch of conversion patches. Now ide_issue_pc/ide_transfer_pc are almost completely aligned to their ide-cd counterparts cdrom_start_packet_command/cdrom_transfer_packet_command. The only thing missing is the setting of the irq handler through ide_set_handler but this is not that trivial since if i'd moved ide-cd's irq handler to ide-atapi that would suck in a bunch of other ide-cd functions which is not what we want, (yes/no)? and I'm going to have to ponder a little bit more on that.

Those have been lightly tested with ide-cd and ide-floppy.

 drivers/ide/ide-atapi.c  |  115 ++++++++++++++++----------------------------
 drivers/ide/ide-cd.c     |  121 ++++++++++++++++++++++++++++------------------
 drivers/ide/ide-floppy.c |    2 +-
 drivers/ide/ide-tape.c   |    2 +-
 include/linux/ide.h      |    2 +-
 5 files changed, 117 insertions(+), 125 deletions(-)