Bartlomiej Zolnierkiewicz wrote:
Index: b/drivers/ide/ide-disk.c
===================================================================
--- a/drivers/ide/ide-disk.c
+++ b/drivers/ide/ide-disk.c
[...]
+ if ((tf->command & 0x01) == 0) {
+ u32 high, low;
Isn't newline needed after declarations?
Well, it is not stricly needed, this code should compile and work without it.
;)
fixed (FWIW the original code also lacked newline)
Yeah, I saw but that's hardly worth imitating. :-)
[...]
New revision of the patch (it also uses the suggestion about using unnamed
unions for error/feature and status/command, thanks!):
[PATCH] ide: add struct ide_taskfile (take 2)
* Don't set write-only ide_task_t.hobRegister[6] and ide_task_t.hobRegister[7]
in idedisk_set_max_address_ext().
* Add struct ide_taskfile and use it in ide_task_t instead of tfRegister[]
and hobRegister[].
* Remove no longer needed IDE_CONTROL_OFFSET_HOB define.
* Add #ifndef/#endif __KERNEL__ around definitions of {task,hob}_struct_t.
While at it:
* Use ATA_LBA define for LBA bit (0x40) as suggested by Tejun Heo.
v2:
* Add missing newlines. (Noticed by Sergei)
* Use ~ATA_LBA instead of 0xBF. (Noticed by Sergei)
* Use unnamed unions for error/feature and status/command.
(Suggested by Sergei).
There should be no functionality changes caused by this patch.
Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]>
---
drivers/ide/ide-disk.c | 156 ++++++++++++++++++++++++---------------------
drivers/ide/ide-io.c | 80 +++++++++++++----------
drivers/ide/ide-iops.c | 12 +--
drivers/ide/ide-lib.c | 3
drivers/ide/ide-acpi.c | 8 ---
drivers/ide/ide-disk.c | 120 +++++++++++++++++++++------------------------
drivers/ide/ide-io.c | 61 ++++++++++++----------
drivers/ide/ide-iops.c | 12 ++--
drivers/ide/ide-lib.c | 3 -
Don't know how but ide-disk.c, ide-io[ps].c and ide-lib.c appear twice in
the diffstat output... 8-)
MBR, Sergei
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
