Groups > linux.kernel > #1207854

Re: [PATCH] arm: footbridge_defconfig: disable IDE subsystem

From	Russell King - ARM Linux <linux@arm.linux.org.uk>
Newsgroups	linux.kernel
Subject	Re: [PATCH] arm: footbridge_defconfig: disable IDE subsystem
Date	2015-08-14 20:50 +0200
Message-ID	<pXsdb-206-9@gated-at.bofh.it> (permalink)
References	<pXqEq-8kc-7@gated-at.bofh.it> <pXqO5-8vw-3@gated-at.bofh.it> <pXrhb-Cx-77@gated-at.bofh.it> <pXrAv-1fH-57@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Fri, Aug 14, 2015 at 08:08:13PM +0200, Bartlomiej Zolnierkiewicz wrote:
> On Friday, August 14, 2015 06:43:30 PM Russell King - ARM Linux wrote:
> > On Fri, Aug 14, 2015 at 07:04:23PM +0200, Bartlomiej Zolnierkiewicz wrote:
> > > This patch disables deprecated IDE subsystem in footbridge_defconfig
> > > (no IDE host drivers are selected in this config so there is no valid
> > > reason to enable IDE subsystem itself).
> > 
> > That's actually incorrect.  I use IDE on Footbridge because it works
> > loads with my CY82C693 board.  I never submitted the patches which make
> > CY82C693 work on ARM though...
> > 
> > However, the CY82C693 must not be allowed to use DMA (which is why the
> > ATA driver can't be used) - if it does, it locks the PCI bus.  It's
> > a hardware incompatibility between some PCI devices and the Footbridge.
> 
> PATA driver is the same in this regard as IDE one.  They both try to
> use DMA if supported by a device.  Would be great if you could fix
> them (or at least PATA one) to limit your setup to PIO automatically
> (please also note that you can limit libata to PIO manually these
> days).
> 
> When it comes to CY82C693 there is also a leftover issue with PATA
> driver that it only supports primary interface currently (BTW this is
> why I left Alpha's defconfig alone for now - it uses CY82C693).  I can
> look into fixing this (or assist in this work) if you are interested
> in it.

Well, it looks like the PCI layer has been screwed for this for some time:

cy82c693 0000:00:06.1: IDE controller (0x1080:0xc693 rev 0x00)
Cypress_IDE 0000:00:06.1: device not available (can't reserve [io  0x0170-0x0177])
Cypress_IDE 0000:00:06.1: device not available (can't reserve [io  0x0170-0x0177])
cy82c693 0000:00:06.1: couldn't enable device
Cypress_IDE: probe of 0000:00:06.1 failed with error -22

The PCI layer forces the standard legacy ports onto the Cypress device,
and then the "primary" resources get null'd out by the long-standing
CY82C693 quirk in the ARM bios32.c.

However - the ARM quick is not what causes the above - the above looks
to be caused by the generic PCI code going wrong - it adds those
legacy resources, and nothing ever connects them to the resource tree:

000000a0-000000bf : pic2
000000c0-000000df : dma2
00000213-00000213 : ISAPnP
000002f8-000002ff : serial8250.0

which then causes pci_enable_device() to fail.

I doubt CY82C693 even works on Alpha... or any IDE interface using the
legacy IO ports...  So it may take quite a bit of work to get going
again - not something I'm going to attempt prior to the 4.3 merge
window.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[RFT PATCH] arm: am200epdkit_defconfig: convert to use libata PATA  drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [RFT PATCH] arm: netwinder_defconfig: convert to use libata PATA  drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [RFT PATCH] arm: h3600_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [RFT PATCH] arm: omap1_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
    Re: [RFT PATCH] arm: omap1_defconfig: convert to use libata PATA  drivers Tony Lindgren <tony@atomide.com> - 2015-08-17 08:40 +0200
  [PATCH] arm: trizeps4_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [RFT PATCH] arm: spitz_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: mainstone_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
    Re: [PATCH] arm: mainstone_defconfig: disable IDE subsystem Robert Jarzmik <robert.jarzmik@free.fr> - 2015-08-14 21:20 +0200
  [PATCH] arm: badge4_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [RFT PATCH] arm: corgi_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: pxa255-idp_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: colibri_pxa270_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: shannon_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: lart_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:10 +0200
  [PATCH] arm: footbridge_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
    Re: [PATCH] arm: footbridge_defconfig: disable IDE subsystem Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-08-14 19:50 +0200
      Re: [PATCH] arm: footbridge_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 20:10 +0200
        Re: [PATCH] arm: footbridge_defconfig: disable IDE subsystem Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-08-14 20:50 +0200
  [RFT PATCH] arm: jornada720_defconfig: convert to use libata PATA  drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
  [PATCH] arm: cerfcube_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
  [PATCH] arm: lpda270_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
  [RFT PATCH] arm: ixp4xx_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
  [PATCH] arm: assabet_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
  [RFT PATCH] arm: collie_defconfig: convert to use libata PATA drivers Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2015-08-14 19:20 +0200
    Re: [RFT PATCH] arm: collie_defconfig: convert to use libata PATA drivers Dmitry Eremin-Solenikov <dbaryshkov@gmail.com> - 2015-08-14 19:30 +0200

csiph-web