Message-ID: <20151109144932.GE22591@linux-mips.org>
Date:	Mon, 9 Nov 2015 15:49:32 +0100
From:	Ralf Baechle <ralf@...ux-mips.org>
To:	"Maciej W. Rozycki" <macro@...ux-mips.org>
Cc:	Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
	linux-mips@...ux-mips.org, linux-ide@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mips: sb1250_swarm_defconfig: disable IDE subsystem

On Mon, Nov 09, 2015 at 02:33:19PM +0000, Maciej W. Rozycki wrote:

> On Mon, 14 Sep 2015, Bartlomiej Zolnierkiewicz wrote:
> 
> > This patch disables deprecated IDE subsystem in sb1250_swarm_defconfig
> > (no IDE host drivers are selected in this config so there is no valid
> > reason to enable IDE subsystem itself).
> 
>  Someone forgot to enable BLK_DEV_PLATFORM, it would seem, at the time the 
> host driver was converted to a PATA platform device (back in 2008, commit 
> 2fef357c).  Frankly I don't think this config file has been recently (as 
> in "since 2008") properly maintained, the last time I revalidated it was 
> shortly before the conversion and nobody else bothered since then.  I'll 
> give it a shot to modernise it at the earliest opportunity, I have other 
> stuff to sort out with this system outstanding.
> 
>  For the record, I have this in .config I've most recently used (with
> 3.19.0):
> 
> CONFIG_HAVE_IDE=y
> CONFIG_IDE=y
> CONFIG_IDE_ATAPI=y
> CONFIG_IDE_GD=y
> CONFIG_IDE_GD_ATA=y
> CONFIG_BLK_DEV_IDECD=y
> CONFIG_BLK_DEV_IDECD_VERBOSE_ERRORS=y
> CONFIG_IDE_TASK_IOCTL=y
> CONFIG_IDE_PROC_FS=y
> CONFIG_BLK_DEV_PLATFORM=y
> CONFIG_HAVE_PATA_PLATFORM=y
> 
> and the driver does register and sees the devices there:
> 
> [...]
> pata-swarm: PATA interface at GenBus slot 4
> futex hash table entries: 512 (order: 1, 49152 bytes)
> Block layer SCSI generic (bsg) driver version 0.4 loaded (major 253)
> io scheduler noop registered
> io scheduler deadline registered
> io scheduler cfq registered (default)
> duart0 at MMIO 0x10060100 (irq = 8, base_baud = 5000000) is a SB1250 DUART
> duart1 at MMIO 0x10060200 (irq = 9, base_baud = 5000000) is a SB1250 DUART
> brd: module loaded
> loop: module loaded
> Uniform Multi-Platform E-IDE driver
> ide-gd driver 1.18
> ide-cd driver 5.00
> Probing IDE interface ide0...
> hda: TOSHIBA MK2016GAP, ATA DISK drive
> hdb: ST380011A, ATA DISK drive
> ide0 at 0x90000000100b3e00-0x90000000100b3ee0,0x90000000100b7ec0 on irq 36
> hda: max request size: 128KiB
> hda: 39070080 sectors (20003 MB), CHS=38760/16/63
> hda: cache flushes not supported
>  hda: hda1 hda2 hda3 < hda5 hda6 hda7 hda8 hda9 hda10 >
> hdb: max request size: 1024KiB
> hdb: 156301488 sectors (80026 MB) w/2048KiB Cache, CHS=16383/255/63
> hdb: cache flushes supported
>  hdb: hdb1 hdb2 hdb3 < hdb5 hdb6 hdb7 hdb8 hdb9 > hdb4
> [...]
> 
> I can check if things still work correctly when routed through libata, 
> although it'll have to wait a couple of weeks yet at the least as I have 
> wired my SWARM for hardware debugging, making it not immediately bootable 
> and I'll be departing soon (i.e. I have no time for complicated fiddling).  
> The host driver itself is actually in arch/mips/sibyte/swarm/platform.c 
> BTW.
> 
>  Note to self: it would be nice if physical rather than virtual MMIO 
> addresses were reported too.

Part of the problem is that everybody who is serious about using a Swarm
is using PCI PATA/SATA card, so this part receives very little TLC.  I
btw. can't test because the controller on my Pass 2 board is broken ...

  Ralf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists