Re: [PATCH 03/16] trm290: use ide_execute_command()

2008-01-22 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 15/16] ide: remove ide_auto_reduce_xfer()

2008-01-22 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: While at it: * Remove needless '!drive->crc_count' check. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line

Re: [PATCH 14/16] ide: move drive->crc_count check out from check_dma_crc()

2008-01-22 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 13/16] ide: remove ide_ata66_check()

2008-01-22 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 11/16] ide: unexport SELECT_DRIVE()

2008-01-22 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PRO

Re: [PATCH 4/4] hpt366: MWDMA filter for SATA cards (take 2)

2007-10-02 Thread Sergei Shtylyov
igned-off-by: Sergei Shtylyov <[EMAIL PROTECTED]> applied Thanks, Sergei - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ a

Re: [PATCH 01/63] ide-cd: fix SAMSUNG CD-ROM SCR-3231 quirk

2007-12-21 Thread Sergei Shtylyov
lomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info

Re: [PATCH 04/63] ide-cd: fix error messages in cdrom_{read,write}_check_ireason()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 05/63] ide-cd: add missing 'ireason' masking to cdrom_write_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Mask 'ireason' variable with 0x3 so the valid interrupt reason value is passed to cdrom_write_check_ireason() for checking. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]&

Re: [PATCH 02/63] ide-cd: fix ACER/AOpen 24X CDROM speed reporting on big-endian machines

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: * Fix ACER/AOpen 24X CDROM speed reporting on big-endian machines by adding missing le16_to_cpu() calls. While at it: * Replace ntohs() by be16_to_cpu(). Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov &

Re: [PATCH 08/63] ide-cd: fix error message in cdrom_pc_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 09/63] ide-cd: fix 'ireason' reporting in cdrom_pc_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Mask 'ireason' variable so only the valid interrupt reason bits will be reported on "drive appears confused" error. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]>

Re: [PATCH 13/63] ide-cd: fix DMA error handling in cdrom_newpc_intr()

2007-12-21 Thread Sergei Shtylyov
TED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 16/63] ide-cd: remove stale cdrom_transfer_packet_command() comment

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PRO

Re: [PATCH 59/63] ide-cd: call blk_dump_rq_flags() on "missing data" in cdrom_rw_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 63/63] ide-cd: update driver version, comments and copyrights

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: * Bump driver version. * Remove filename and stale TODO from comments. * Add my copyrights. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from thi

Re: [PATCH 21/63] ide-cd: remove unused and write-only struct ide_cd_config_flags fields

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: unused fields: * ->writing * ->reserved write-only fields: * ->playmsf_as_bcd * ->subchan_as_bcd * ->test_write * ->supp_disc_present Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMA

Re: [PATCH 06/63] ide-cd: fix error messages in cdrom_write_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PRO

Re: [PATCH 07/63] ide-cd: add error message for DMA error to cdrom_read_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 42/63] ide-cd: coding style fixes for cdrom_get_toc_entry()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: This is a preparation to move code handling cdrom.c IOCTLs out of ide-cd.c. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line

Re: [PATCH 14/63] ide-cd: fix trailing whitespaces in changelog

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 17/63] ide-cd: remove unused defines from ide-cd.h

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 46/63] ide-cd: call blk_dump_rq_flags() on "missing data" in cdrom_newpc_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 22/63] ide-cd: remove struct atapi_{mechstat_header,changer_info,slot}

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: * Remove ->changer_info from struct cdrom_info (it is never allocated). * Remove unused struct atapi_{mechstat_header,changer_info,slot}. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROT

Re: [PATCH 23/63] ide-cd: cleanup ide_cdrom_update_speed()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 19/63] ide-cd: remove unused struct atapi_cdrom_subchnl

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PRO

Re: [PATCH 20/63] ide-cd: remove needless zeroing of 'info' fields from ide_cdrom_setup()

2007-12-21 Thread Sergei Shtylyov
d in 'info'). Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More

Re: [PATCH 30/63] ide-cd: remove NO_DOOR_LOCKING define

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 34/63] ide-cd: remove duplicate sense keys definitions from ide-cd.h

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: ide-cd.c already uses sense keys definitions from . Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe lin

Re: [PATCH 18/63] ide-cd: remove dead code from cdrom_pc_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTE

Re: [PATCH 12/63] ide-cd: fix "missing data" handling in cdrom_pc_intr()

2007-12-21 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: If drive still wants to transfer the data we need to pad the transfer instead of just finishing the request. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscri

Re: [PATCH 1/2] ide: add ide_dump_identify() debug helper

2007-12-25 Thread Sergei Shtylyov
is enabled. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More m

Re: [MIPS] MEM_SDREFCFG is not defined for Alchemy DB1550 (compile fail)

2007-12-25 Thread Sergei Shtylyov
Hello. Alon Bar-Lev wrote: When I have: CONFIG_MIPS_DB1550 CONFIG_SOC_AU1550 CONFIG_SOC_AU1X00 CONFIG_PM MEM_SDREFCFG is used at: arch/mips/au1000/common/power.c::pm_do_freq() PM code is generally broken and unmaintained, so no wonder. I don't remember if anyone has fixed CPU context r

Re: [MIPS] MEM_SDREFCFG is not defined for Alchemy DB1550 (compile fail)

2007-12-25 Thread Sergei Shtylyov
Alon Bar-Lev wrote: PM code is generally broken and unmaintained, so no wonder. I don't remember if anyone has fixed CPU context restoration code (it uses a "skewed" stack frame). So suspend modes on these boards are not supported? Only "Always On" configuration is supported? Sleep mo

Re: [PATCH 2/9] ide: fix ide_intr() for non-PCI devices and CONFIG_BLK_DEV_IDEPCI=y

2007-12-28 Thread Sergei Shtylyov
Bartlomiej Zolnierkiewicz wrote: 'hwif->pci_dev && !hwif->pci_dev->vendor' condition is never true, check for 'hwif->chipset != ide_pci' instead. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAI

Re: [PATCH 1/9] piix: remove stale comments

2007-12-28 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> Index: b/drivers/ide/pci/piix.c === --- a/drivers/ide/pci/piix.c +

Re: [PATCH 7/9] icside: use hwif->dev

2007-12-28 Thread Sergei Shtylyov
ussell King <[EMAIL PROTECTED]> Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei PS: Russel, that was only an eyes check, the last word is yours. :-) -- To unsubscribe from this list: send the line "

Re: [PATCH 4/9] ide: use ide_destroy_dmatable() instead of pci_unmap_sg()

2007-12-29 Thread Sergei Shtylyov
<[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please

Re: [PATCH 7/9] icside: use hwif->dev

2007-12-29 Thread Sergei Shtylyov
ussell King <[EMAIL PROTECTED]> Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH 7/9] icside: use hwif->dev

2007-12-29 Thread Sergei Shtylyov
Sergei Shtylyov wrote: * Setup hwif->dev in icside_setup(). * Use hwif->dev instead of state->dev in icside_build_sglist(), icside_dma_end(), icside_dma_start() and icside_dma_setup(). * Remove no longer needed 'dev' field from struct icside_state. Cc: Russell Kin

Re: [PATCH 3/3] ide: add IDE_HFLAG_NO_DSC host flag

2007-12-29 Thread Sergei Shtylyov
ewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> Index: b/include/linux/ide.h === --- a/include/linux/ide.h +++ b/include/linux/ide.h @@ -1098,6 +1098,8 @@ enum { IDE_HFLAG_CY82C693

Re: [PATCH 26/28] blk_end_request: changing ide-cd (take 3)

2007-12-03 Thread Sergei Shtylyov
m Kiyoshi Ueda. Cc: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED] Cc: Kiyoshi Ueda <[EMAIL PROTECTED] Cc: Jun'ichi Nomura <[EMAIL PROTECTED]> Cc: Tejun Heo <[EMAIL PROTECTED]> Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECT

Re: [PATCH 8/8] ide: add ->cable_detect method to ide_hwif_t

2008-01-12 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: * Add ->cable_detect method to ide_hwif_t. * Call the new method in ide_init_port() if: - the host supports UDMA modes > UDMA2 ('hwif->ultra_mask & 78') - DMA initialization was successful (if hwif->dma_base is not set ide_init_port() sets hwif

Re: PCI resource unavailable on mips

2007-12-17 Thread Sergei Shtylyov
Hello. Jon Dufresne wrote: I did a bit more work and investigation on this and it turns out I could not read the mmio in kernel space because I had not done a pci_enable_device_bars() on the device. I had never done this on x86 so I didn't realize it was necessary. The virtual address 0xc030

Re: [PATCH] include/asm-mips/: Spelling fixes

2007-12-17 Thread Sergei Shtylyov
Hello. Joe Perches wrote: Signed-off-by: Joe Perches <[EMAIL PROTECTED]> diff --git a/include/asm-mips/mach-wrppmc/mach-gt64120.h b/include/asm-mips/mach-wrppmc/mach-gt64120.h index 00d8bf6..465234a 100644 --- a/include/asm-mips/mach-wrppmc/mach-gt64120.h +++ b/include/asm-mips/mach-wrppmc/

Re: [PATCH 7/8] ide: add struct ide_port_info instances to legacy host drivers

2008-01-28 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: * Remove 'struct pci_dev *dev' argument from ide_hwif_setup_dma(). * Un-static ide_hwif_setup_dma() and add CONFIG_BLK_DEV_IDEDMA_PCI=n version. * Add 'const struct ide_port_info *d' argument to ide_device_add[_all](). * Factor out generic ports

Re: [PATCH 5/8] ide: always set DMA masks in ide_pci_setup_ports()

2008-01-29 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: Always set DMA masks in ide_pci_setup_ports() to make sure that the valid masks for a host are set. Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei -- To unsubscri

Re: [PATCH][RESUBMIT] drivers/ide/ide-probe.c, kernel 2.6.23.1

2007-11-16 Thread Sergei Shtylyov
Jonas Stare wrote: diff -u linux-2.6.23.1-orig/drivers/ide/ide-probe.c linux-2.6.23.1/drivers/ide/ide-probe.c --- linux-2.6.23.1-orig/drivers/ide/ide-probe.c 2007-10-12 18:43:44.0 +0200 +++ linux-2.6.23.1/drivers/ide/ide-probe.c 2007-11-16 10:26:23.0 +0100 @@ -643,6 +643,7

Re: Regression: commit "ide: constify struct ide_port_info" causes breakage

2007-10-26 Thread Sergei Shtylyov
Hello. Linus Torvalds wrote: We can just convert cariables declared as 'struct hpt_info' to const too -- they are read-only templates. Generally "__devinitdata" is better than "const", since it means that the data may be thrown away. Agreed. So if you drop one over the other, N

Re: Regression: commit "ide: constify struct ide_port_info" causes breakage

2007-10-26 Thread Sergei Shtylyov
Hello. Russell King wrote: commit 8562043606430185cad26d085d46adcc7ad67fd1 is broken, causing: CC drivers/ide/pci/cmd64x.o CC drivers/ide/pci/hpt366.o drivers/ide/pci/hpt366.c:1428: error: hpt366_chipsets causes a section type conflict and therefore should be reverted. The

Re: [PATCH] ucc_geth: add support for netpoll

2007-10-27 Thread Sergei Shtylyov
Hello. Anton Vorontsov wrote: This patch adds netpoll support for the QE UCC Gigabit Ethernet driver. The approach is very similar to the gianfar driver. It's rather contrarywise -- this is standard approach and gianfar with its 3 TSEC IRQs has a quite non-standard poll_controller() imple

Re: [patch 3/3] scsi: wd33c93 needs

2007-07-20 Thread Sergei Shtylyov
Hello Christoph: +#include These days that should probably be . Not at all, linux/irq.h is something entirely different. Actually, WBR, Sergei - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: [PATCH] pata_hpt37x: Fix 2.6.22 clock PLL regression

2007-07-24 Thread Sergei Shtylyov
Hello. Alan Cox wrote: Just one version of Linux ago The PLL code broke - oh no! But set the right mode And fix up the code Makes the PLL timing sync go /me applauds :-) Closes-bug: #8791 Signed-off-by: Alan Cox <[EMAIL PROTECTED]> Acked-by:

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-25 Thread Sergei Shtylyov
Hello. Vitaly Bordug wrote: This updates relevant platform code (freescale mpc8349itx target) to make the CompactFlash work in TrueIDE mode. Erm, I'm not sure it's worth submitting the platform device driver for PowerPC at this point, but well... Signed-off-by: Anton Vorontsov <[EMAIL

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-25 Thread Sergei Shtylyov
Hello, I wrote: diff --git a/arch/powerpc/boot/dts/mpc8349emitx.dts b/arch/powerpc/boot/dts/mpc8349emitx.dts index db0d003..f8f0e8a 100644 --- a/arch/powerpc/boot/dts/mpc8349emitx.dts +++ b/arch/powerpc/boot/dts/mpc8349emitx.dts @@ -37,6 +37,15 @@ reg = < 1000>;

Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)

2007-07-25 Thread Sergei Shtylyov
Vitaly Bordug wrote: This is now very similar to pata_platform.c, they both use same platform data structure and same resources. To achieve that, byte_lanes_swapping platform data variable and platform specified iops removed from that driver. It's fine, since those were never used anyway.

Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)

2007-07-25 Thread Sergei Shtylyov
Hello. Scott Wood wrote: +hwif->hw.io_ports[IDE_DATA_OFFSET] = port; + +port += (1 << pdata->ioport_shift); +for (i = IDE_ERROR_OFFSET; i <= IDE_STATUS_OFFSET; + i++, port += (1 << pdata->ioport_shift)) Looks like shift doesn't buy as anything, why not just use strid

Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)

2007-07-25 Thread Sergei Shtylyov
Guennadi Liakhovetski wrote: This is now very similar to pata_platform.c, they both use same platform data structure and same resources. To achieve that, byte_lanes_swapping platform data variable and platform specified iops removed from that driver. It's fine, since those were never used any

Re: [PATCH 1/2] [IDE] Platform IDE driver

2007-07-25 Thread Sergei Shtylyov
Scott Wood wrote: It doesn't buy us anything in here, but it's conceivable that someone may want to write a driver that uses a shift in the I/O accessor rather than an array of port offsets, It wouldn't be IDE driver then, and neither it would be libata which also does this another way th

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-26 Thread Sergei Shtylyov
Hello, I wrote: Also, what mmio-ide in the compat properly means in the context of ide_platform which is able to handle both port and memory mapped IDE. I/O-space is only valid in the context of PCI, ISA, or similar buses, and the bus-specific reg format indicates whether it's mmio-space

Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)

2007-07-26 Thread Sergei Shtylyov
Hello. Guennadi Liakhovetski wrote: driver to using platform-device. I got a reply, that it's not worth it now that IDE is slowly becoming obsolete, and the pata_platform serves the perpose perfectly well. I found this argument reasonable, I had the same doubt, just wanted to double-check. So

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-26 Thread Sergei Shtylyov
Hello. Vitaly Bordug wrote: I acn undertand your complaint in the context of an OF driver (which we don't have yet) but "mmio-ide" just means nothing to the current driver, and it doesn't convery enough info on the programming interface for the conceivable OF driver, it also does need to know

Re: [PATCH 2/4] hpt366: UltraDMA filter for SATA cards (take 3)

2007-08-31 Thread Sergei Shtylyov
tra_mask as a default mask for the ide_dma_filter() method to behave correctly; - move the HPT370[A] cases below the HPT36x case for consistency... Signed-off-by: Sergei Shtylyov <[EMAIL PROTECTED]> --- Argh! I've managed to put = instead of &= here and there, so

Re: hda: set_drive_speed_status: status=0x51 { DriveReady SeekComplete Error }

2007-08-31 Thread Sergei Shtylyov
Alan Cox wrote: Standards: Likely used: 1 Prehistory LBA, IORDY not likely No DMA, nothing above PIO2 Cool! :-) Buffer type: 0002: dual port, multi-sector Buffer size: 1.0kB bytes avail on r/w long: 4 Cannot perform double-word IO Ca

Re: PROBLEM: kernel 2.6.22.6 pata_pdc202xx_old.c limiting to UDMA/33 instead of UDMA/100 (UPDATED 2.6.22.6)

2007-09-01 Thread Sergei Shtylyov
Dear n: Please do *not* follow-up to a completely inderended thread. Start your own, please. :-/ Update with kernel 2.6.22.6 i am getting this error now ata2.00: ATA-6: ST3120026A, 3.06, max UDMA/100 Gathering courage and reply to libata bug report -- I'm more of the drivers/ide/ guy

Re: hda: set_drive_speed_status: status=0x51 { DriveReady SeekComplete Error }

2007-09-01 Thread Sergei Shtylyov
Hello. John Sigler wrote: When my system boots, I get several set_drive_speed_status errors. (Please see attached dmesg output.) Can someone explain what they mean? How do I get rid of them? IDE code attempts to autotune PIO mode and fails at that because your device is too old (or its

Re: hda: set_drive_speed_status: status=0x51 { DriveReady SeekComplete Error }

2007-09-01 Thread Sergei Shtylyov
John Sigler wrote: What do the warnings mean? :-) That your drive does not support set transfer mode/speed command at Which is perfectly valid in the original ATA spec. all, or that value which kernel tried is not supported by the drive... They just should skip programming the dri

[PATCH 3/4] ide: make ide_rate_filter() also respect PIO and SW/MW DMA mode masks

2007-09-02 Thread Sergei Shtylyov
Make ide_rate_filter() also respect PIO/SWDMA/MWDMA mode masks. While at it, make the udma_filter() method calls take precedence over using the mode masks. --- This version doesn't use explicit UltraDMA masks, so converting them to the ATA_UDMA* is left for another, global patch. This patch agai

Re: [PATCH 3/4] ide: make ide_rate_filter() also respect PIO and SW/MW DMA mode masks

2007-09-02 Thread Sergei Shtylyov
Hello, I wrote: Make ide_rate_filter() also respect PIO/SWDMA/MWDMA mode masks. While at it, make the udma_filter() method calls take precedence over using the mode masks. --- This version doesn't use explicit UltraDMA masks, so converting them to the ATA_UDMA* is left for another, global patc

[PATCH 3/4] ide: make ide_rate_filter() also respect PIO and SW/MW DMA mode masks (take 2)

2007-09-02 Thread Sergei Shtylyov
Make ide_rate_filter() also respect PIO/SWDMA/MWDMA mode masks. While at it, make the udma_filter() method calls take precedence over using the mode masks. Signed-off-by: Sergei Shtylyov <[EMAIL PROTECTED]> --- Alas, the previous version of the patch wasn't even worth signing it off

[PATCH 2/4] hpt366: MWDMA filter for SATA cards

2007-09-02 Thread Sergei Shtylyov
with his consent, also adding the method callout to ide_rate_filter(); - installed the method for all chips to only return empty mask if a SATA drive is detected onHPT372{AN]/374 chips... Signed-off-by: Sergei Shtylyov <[EMAIL PROTECTED]> --- This patch against the current Linus

Re: [PATCH 2/4] hpt366: MWDMA filter for SATA cards

2007-09-02 Thread Sergei Shtylyov
ips... Signed-off-by: Sergei Shtylyov <[EMAIL PROTECTED]> --- This patch against the current Linus' tree and unfortunately I was able to only compile test it since that tree gives MODPOST warning and dies early on bootup. Will hopefully try to test if/when I have time... :-) In

Re: hda: set_drive_speed_status: status=0x51 { DriveReady SeekComplete Error }

2007-09-03 Thread Sergei Shtylyov
Hello. John Sigler wrote: When my system boots, I get several set_drive_speed_status errors. (Please see attached dmesg output.) Can someone explain what they mean? How do I get rid of them? IDE code attempts to autotune PIO mode and fails at that because your device is too old (or its

Re: libata git tree, mbox queue status and contents

2007-08-05 Thread Sergei Shtylyov
Alan Cox wrote: * Alan: IORDY handling -- upstream whenever Alan is happy I'm happy with it from testing. Just a little worried about it going upstream mid -rc as it could have a weird side effect somewhere. I've verified an original pre ATA IDE drive with it too now 8) * Alan: ACPI checks

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-25 Thread Sergei Shtylyov
Scott Wood wrote: Scott Wood wrote: Also, what mmio-ide in the compat properly means in the context of ide_platform which is able to handle both port and memory mapped IDE. I/O-space is only valid in the context of PCI, ISA, or similar buses, and the bus-specific reg format indicates

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-25 Thread Sergei Shtylyov
Scott Wood wrote: Also, what mmio-ide in the compat properly means in the context of ide_platform which is able to handle both port and memory mapped IDE. I think we must get rid with this crap, and since this IDE register mapping is pretty much board specific, call it something like "mpc834

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-07-25 Thread Sergei Shtylyov
Hello. Scott Wood wrote: Also, what mmio-ide in the compat properly means in the context of ide_platform which is able to handle both port and memory mapped IDE. I/O-space is only valid in the context of PCI, ISA, or similar buses, and the bus-specific reg format indicates whether it's mm

Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)

2007-07-25 Thread Sergei Shtylyov
Guennadi Liakhovetski wrote: This is now very similar to pata_platform.c, they both use same platform data structure and same resources. To achieve that, byte_lanes_swapping platform data variable and platform specified iops removed from that driver. It's fine, since those were never used any

Re: [PATCH] Fix compilation for non CONFIG_HOTPLUG case

2007-07-29 Thread Sergei Shtylyov
Hello. Niklaus Giger wrote: Fixes compilation issues for embedded boards which do not support HOTPLUG Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> diff --git a/drivers/base/core.c b/drivers/base/core.c index 3599ab2..a09bfc8 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @

Re: [PATCH] libata: Correct IORDY handling

2007-07-31 Thread Sergei Shtylyov
t;[EMAIL PROTECTED]> Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.23rc1-mm1/drivers/ata/libata-core.c linux-2.6.23rc1-mm1/drivers/ata/libata-core.c --- linux.vanilla-2.6.23rc1-mm1/drivers/ata/libata-core.c

Re: [PATCH 1/2] [IDE] Platform IDE driver

2007-08-01 Thread Sergei Shtylyov
Hello. Segher Boessenkool wrote: This doesn't mean that shift is better anyway. If everyone considers it better, I give up. But be warned that shift (stride) is not the only property characterizing register accesses -- the regs might be only accessible as 16/32-bit quantities, for example

Re: [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface

2007-08-01 Thread Sergei Shtylyov
Hello. Segher Boessenkool wrote: +[EMAIL PROTECTED] { +compatible = "mmio-ide"; +device_type = "ide"; Why not "ata"? The hardware is called (E)IDE, the protocol is called ATA. Sorry for not denouncing this earlier. :-) ATA is the name of ANSI standard descr

Re: [PATCH] [428/2many] MAINTAINERS - SERIAL ATA (SATA) SUBSYSTEM:

2007-08-17 Thread Sergei Shtylyov
Hello. [EMAIL PROTECTED] wrote: Add file pattern to MAINTAINER entry Signed-off-by: Joe Perches <[EMAIL PROTECTED]> diff --git a/MAINTAINERS b/MAINTAINERS index 04c969c..016cd0c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4077,6 +4077,7 @@ M:[EMAIL PROTECTED] L: [EMAIL PRO

Re: [PATCH 0/3] ide: Fix use of paired device

2007-10-18 Thread Sergei Shtylyov
Hello. Benjamin Herrenschmidt wrote: At least 2 drivers (siimage and cs5535) have a bug where they use the construct: ide_drive_t *pair = &hwif->drives[drive->dn ^ 1]; To access the other drive in a master/slave pair. This is bogus because drive->dn is not the unit number, b

Re: [PATCH 0/3] ide: Fix use of paired device

2007-10-18 Thread Sergei Shtylyov
Hello, I wrote: At least 2 drivers (siimage and cs5535) have a bug where they use the construct: ide_drive_t *pair= &hwif->drives[drive->dn ^ 1]; To access the other drive in a master/slave pair. This is bogus because drive->dn is not the unit number, but the global drive number, t

Re: [PATCH 1/3] ide: Add ide_get_paired_drive() helper

2007-10-18 Thread Sergei Shtylyov
Benjamin Herrenschmidt wrote: This adds a helper to get to the "other" drive on a pair connected to a given hwif. Maibe "mate drive" would've been a better name that "paired"... Signed-off-by: Benjamin Herrenschmidt <[EMAIL PROTECTED]> Acked-by:

Re: [PATCH 2/3] ide: Fix siimage driver accessing beyond array boundary

2007-10-18 Thread Sergei Shtylyov
AIL PROTECTED]> Please update the driver version in the driver's heading comment. Otherwise Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] M

Re: [PATCH 3/3] ide: Fix cs5535 driver accessing beyond array boundary

2007-10-18 Thread Sergei Shtylyov
AIL PROTECTED]> Please update the driver version in the driver's heading comment. It also would be preferrable to fold this patch together with siimage path, or even fold all three together. Otherwise Acked-by: Sergei Shtylyov <[EMAIL PROTECTED]> MBR, Sergei - To unsubscribe fro

Re: [PATCH 2.6.21-rt2] PowerPC: decrementer clockevent driver

2007-10-18 Thread Sergei Shtylyov
Sergei Shtylyov wrote: Add PowerPC decrementer clock event driver. Every effort has been made to support the different implementations of the decrementer: the classic one (with 970 series variation), 40x and Book E specific ones. I had to make CONFIG_GENERIC_CLOCKEVENTS option selectable

Re: [PATCH 1/19] ide: update MAINTAINERS entry

2007-01-19 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: update MAINTAINERS entry Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> --- MAINTAINERS |7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) Index: my-2.6/MAINTAINERS

Re: [PATCH 8/15] ide: disable DMA in ->ide_dma_check for "no IORDY" case

2007-01-19 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: disable DMA in ->ide_dma_check for "no IORDY" case I've looked thru the code, and found more issues with the PIO fallback there. Will try to cook up patches for at least some drivers... Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL

Re: [PATCH 8/15] ide: disable DMA in ->ide_dma_check for "no IORDY" case

2007-01-19 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: disable DMA in ->ide_dma_check for "no IORDY" case I've looked thru the code, and found more issues with the PIO fallback there. Will try to cook up patches for at least some drivers... Great, if possible please base them on top of th

Re: [PATCH 8/15] ide: disable DMA in ->ide_dma_check for "no IORDY" case

2007-01-20 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: I've looked thru the code, and found more issues with the PIO fallback there. Will try to cook up patches for at least some drivers... Great, if possible please base them on top of the IDE tree... Erm, I had doubts about it (having in mind that a

Re: [PATCH 8/15] ide: disable DMA in ->ide_dma_check for "no IORDY" case

2007-01-20 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: The other advantage of doing cleanups is that code becomes cleaner/simpler which matters a lot for this codebase, i.e. ide-dma-off-void.patch exposed (yet to be fixed) bug in set_using_dma() (->ide_dma_off_quietly always returns 0 which is passed by ->ide

Re: [PATCH 10/15] ide: add ide_set_dma() helper

2007-01-20 Thread Sergei Shtylyov
Hello again. :-) Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: add ide_set_dma() helper * add ide_set_dma() helper and make ide_hwif_t.ide_dma_check return -1 when DMA needs to be disabled (== need to call ->ide_dma_off_quietly) 0 when DMA needs to be enabled (== need to call ->ide_dma_

Re: [PATCH 12/15] ide: make ide_hwif_t.ide_dma_host_on void

2007-01-20 Thread Sergei Shtylyov
Hello again. :-) Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: make ide_hwif_t.ide_dma_host_on void * since ide_hwif_t.ide_dma_host_on is called either when drive->using_dma == 1 or when return value is discarded make it void, also drop "ide_" prefix * make __ide_dma_host_on() void and dro

Re: [PATCH 11/15] ide: make ide_hwif_t.ide_dma_{host_off,off_quietly} void

2007-01-20 Thread Sergei Shtylyov
Hello again. :-) Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: make ide_hwif_t.ide_dma_{host_off,off_quietly} void Below are my nits on the patch itself, and the code it changes. Index: b/drivers/ide/pci/atiixp.c === --- a/d

Re: [PATCH] Make CARDBUS_MEM_SIZE and CARDBUS_IO_SIZE customizable

2007-01-22 Thread Sergei Shtylyov
Hello. Atsushi Nemoto wrote: Subject: [PATCH] Make CARDBUS_MEM_SIZE and CARDBUS_IO_SIZE customizable CARDBUS_MEM_SIZE was increased to 64MB on 2.6.20-rc2, but larger size might result in allocation failure for the reserving itself on some platforms (for example typical 32bit MIPS). Make it (a

Re: [PATCH 13/15] ide: fix UDMA/MWDMA/SWDMA masks

2007-01-22 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: fix UDMA/MWDMA/SWDMA masks * use 0x00 instead of 0x80 to disable ->{ultra,mwdma,swdma}_mask * add udma_mask field to ide_pci_device_t and use it to initialize ->ultra_mask in aec62xx, pdc202xx_new and pdc202xx_old drivers * fix UDMA mask

Re: [PATCH 13/15] ide: fix UDMA/MWDMA/SWDMA masks

2007-01-22 Thread Sergei Shtylyov
Hello. Bartlomiej Zolnierkiewicz wrote: [PATCH] ide: fix UDMA/MWDMA/SWDMA masks * use 0x00 instead of 0x80 to disable ->{ultra,mwdma,swdma}_mask * add udma_mask field to ide_pci_device_t and use it to initialize ->ultra_mask in aec62xx, pdc202xx_new and pdc202xx_old drivers * fix UDMA mask

<    1   2   3   4   5   6   7   8   9   10   >