linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: Christoph Hellwig <hch@lst.de>
Cc: "David S. Miller" <davem@davemloft.net>,
	Jens Axboe <axboe@kernel.dk>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Richard Henderson <rth@twiddle.net>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Matt Turner <mattst88@gmail.com>,
	Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	Michael Ellerman <mpe@ellerman.id.au>,
	linux-ide@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org,
	linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH 02/10] ARM: disable CONFIG_IDE in footbridge_defconfig
Date: Mon, 22 Mar 2021 15:15:03 +0000	[thread overview]
Message-ID: <20210322151503.GX1463@shell.armlinux.org.uk> (raw)
In-Reply-To: <20210322145403.GA30942@lst.de>

On Mon, Mar 22, 2021 at 03:54:03PM +0100, Christoph Hellwig wrote:
> On Fri, Mar 19, 2021 at 05:53:12PM +0000, Russell King - ARM Linux admin wrote:
> > If I extend the arch/arm/kernel/bios32.c code to kill BARs 2/3 (which
> > actually are not present on the CY82C693) then the IDE driver works
> > for me, but the PATA driver does not:
> > 
> > cy82c693 0000:00:06.1: IDE controller (0x1080:0xc693 rev 0x00)
> > cy82c693 0000:00:06.1: not 100% native mode: will probe irqs later
> > legacy IDE will be removed in 2021, please switch to libata
> > Report any missing HW support to linux-ide@vger.kernel.org
> >     ide0: BM-DMA at 0x1080-0x1087
> >     ide1: BM-DMA at 0x1088-0x108f
> > Probing IDE interface ide0...
> > hda: PIONEER DVD-RW DVR-105, ATAPI CD/DVD-ROM drive
> > hda: host max PIO4 wanted PIO255(auto-tune) selected PIO4
> > ...
> > 
> > (unbind Cypress_IDE and try binding pata_cypress)
> > 
> > pata_cypress 0000:00:06.1: no available native port
> 
> This comes from ata_pci_sff_init_host when it tails to initialize
> a port.  There are three cases why it can't initialize the port:
> 
>  1) because it is marked as dummy, which is the case for the second
>     port of the cypress controller, but you're not using that even
>     with the old ide driver, and we'd still not get that message just
>     because of that second port.
>  2) when ata_resources_present returns false because the BAR has
>     a zero start or length
>  3) because pcim_iomap_regions() fails.  This prints a warning to the
>     log ("failed to request/iomap BARs for port %d (errno=%d)") that you
>     should have seen
> 
> So the problem here has to be number two.  The legacy ide driver OTOH
> seems to lack a lot of these checks, although I'm not sure how it
> manages to actually work without those.
> 
> Can you show how the BAR assignment for the device looks using lscpi
> or a tool of your choice?

There's a big problem here. I have to explicitly zero the resources
(getting rid of the legacy ones assigned by the PCI probe code)
because they are in fact _wrong_ for the CY82C693. The PCI code
assumes that PCI function 1 (primary port) and PCI function 2
(secondary port) are two independent dual-channel IDE ports, and
as the PROG-IF of the class code indicates that all ports are in
legacy mode, the PCI code assigns the legacy ioport resources to
_both_ PCI functions. Essentially, the CY82C693 is a bit of an odd-ball
because it splits the two IDE ports across two functions rather than a
single function.

It gets worse than that though - due to a change to remove
pcibios_min_io from the generic code, moving it into the ARM
architecture code, this has caused a regression that prevents the
legacy resources being registered against the bus resource. So even
if they are there, they cause probe failures. I haven't found a
reasonable way to solve this yet, but until there is, there is no
way that the PATA driver can be used as the "legacy mode" support
is effectively done via the PCI code assigning virtual IO port
resources.

I'm quite surprised that the CY82C693 even works on Alpha - I've
asked for a lspci for that last week but nothing has yet been
forthcoming from whoever responded to your patch for Alpha - so I
can't compare what I'm seeing with what's happening with Alpha.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!

  reply	other threads:[~2021-03-22 15:16 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-18  4:56 remove the legacy ide driver Christoph Hellwig
2021-03-18  4:56 ` [PATCH 01/10] alpha: use libata instead of " Christoph Hellwig
2021-03-18  5:54   ` Al Viro
2021-03-18  6:07     ` Christoph Hellwig
2021-03-18 17:09       ` Måns Rullgård
2021-03-18 18:03         ` Måns Rullgård
2021-03-18  7:54     ` John Paul Adrian Glaubitz
2021-03-19 16:23     ` Serge Belyshev
2021-03-18  4:56 ` [PATCH 02/10] ARM: disable CONFIG_IDE in footbridge_defconfig Christoph Hellwig
2021-03-19 17:07   ` Russell King - ARM Linux admin
2021-03-19 17:53     ` Russell King - ARM Linux admin
2021-03-22 14:54       ` Christoph Hellwig
2021-03-22 15:15         ` Russell King - ARM Linux admin [this message]
2021-03-22 15:18           ` Christoph Hellwig
2021-03-22 15:33             ` Christoph Hellwig
2021-03-23 14:03               ` Russell King - ARM Linux admin
2021-03-22 16:59             ` Russell King - ARM Linux admin
2021-03-22 16:09           ` John Paul Adrian Glaubitz
2021-03-22 16:47             ` Cye Borg
2021-03-22 17:03             ` Russell King - ARM Linux admin
2021-03-22 17:10               ` Cye Borg
2021-03-23 18:43                 ` Russell King - ARM Linux admin
2021-03-24  5:47                   ` Cye Borg
2021-03-18  4:56 ` [PATCH 03/10] ARM: disable CONFIG_IDE in pxa_defconfig Christoph Hellwig
2021-03-18  4:57 ` [PATCH 04/10] MIPS: disable CONFIG_IDE in sb1250_swarm_defconfig Christoph Hellwig
2021-03-19  0:12   ` Maciej W. Rozycki
2021-03-24  9:59   ` Thomas Bogendoerfer
2021-03-18  4:57 ` [PATCH 05/10] MIPS: switch workpad_defconfig from legacy IDE to libata Christoph Hellwig
2021-03-24 10:00   ` Thomas Bogendoerfer
2021-03-18  4:57 ` [PATCH 06/10] MIPS: disable CONFIG_IDE in rbtx49xx_defconfig Christoph Hellwig
2021-03-24 10:00   ` Thomas Bogendoerfer
2021-03-18  4:57 ` [PATCH 07/10] MIPS: disable CONFIG_IDE in bigsur_defconfig Christoph Hellwig
2021-03-18 10:08   ` Sergei Shtylyov
2021-03-24 10:00   ` Thomas Bogendoerfer
2021-03-18  4:57 ` [PATCH 08/10] MIPS: disable CONFIG_IDE in malta*_defconfig Christoph Hellwig
2021-03-18 10:12   ` Sergei Shtylyov
2021-03-18 14:19   ` Thomas Bogendoerfer
2021-03-19 13:03     ` Christoph Hellwig
2021-03-24 10:01   ` Thomas Bogendoerfer
2021-03-18  4:57 ` [PATCH 09/10] m68k: use libata instead of the legacy ide driver Christoph Hellwig
2021-03-18  4:57 ` [PATCH 10/10] ide: remove " Christoph Hellwig
2021-03-20  3:44   ` Maciej W. Rozycki
2021-03-20  4:04     ` Maciej W. Rozycki
2021-03-19  1:43 ` Finn Thain
2021-03-19  5:43   ` Christoph Hellwig
2021-03-19 16:48 ` Maciej W. Rozycki
2021-03-21  9:20 ` John Paul Adrian Glaubitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210322151503.GX1463@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=axboe@kernel.dk \
    --cc=davem@davemloft.net \
    --cc=geert@linux-m68k.org \
    --cc=hch@lst.de \
    --cc=ink@jurassic.park.msu.ru \
    --cc=linux-alpha@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-m68k@lists.linux-m68k.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mattst88@gmail.com \
    --cc=mpe@ellerman.id.au \
    --cc=rth@twiddle.net \
    --cc=tsbogend@alpha.franken.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).