All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: "Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: Bjorn Helgaas <helgaas@kernel.org>,
	 Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	 Bjorn Helgaas <bhelgaas@google.com>,
	linux-mips@vger.kernel.org,  LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/2] MIPS: Malta: Fix PCI southbridge legacy resource clashes
Date: Wed, 22 Oct 2025 14:08:43 +0300 (EEST)	[thread overview]
Message-ID: <93ffa5d8-61eb-abdf-8271-8f676d36fe56@linux.intel.com> (raw)
In-Reply-To: <alpine.DEB.2.21.2510211901530.8377@angie.orcam.me.uk>

[-- Attachment #1: Type: text/plain, Size: 2213 bytes --]

On Tue, 21 Oct 2025, Maciej W. Rozycki wrote:

> Hi,
> 
>  This mini patch series sorts out issues with southbridge legacy resource 
> management on the MIPS Malta platform.  Two changes turned out required, 
> because merely removing the clash would regress the PS/2 interfaces, fixed 
> by accident with the PCIBIOS_MIN_IO fix.
> 
>  This does prove nobody has used these interfaces since forever, or most 
> likely since the move to the new serio driver.  Things most likely worked 
> fine with 2.4 and I still have such old Malta kernel builds lying around 
> (though sadly no hardware to try with), although I do know I've never used 
> the PS/2 stuff with this platform, e.g. quoting an arbitrarily picked 
> 2.4.19-rc1 bootstrap log:
> 
> CPU revision is: 00018101
> Primary instruction cache 16kb, linesize 32 bytes(4 ways)
> Primary data cache 16kb, linesize 32 bytes (4 ways)
> Linux version 2.4.19-rc1 (macro@macro.ds2.pg.gda.pl) (gcc version 2.95.4 20010319 (prerelease)) #1 Fri Aug 23 02:55:02 CEST 2002
> [...]
> parport0: PC-style at 0x378 [PCSPP,EPP]
> initialize_kbd: Keyboard reset failed, no ACK
> Detected PS/2 Mouse Port.
> pty: 256 Unix98 ptys configured
> keyboard: Timeout - AT keyboard not present?(ed)
> keyboard: Timeout - AT keyboard not present?(f4)
> Serial driver version 5.05c (2001-07-08) with MANY_PORTS SHARE_IRQ DETECT_IRQ SERIAL_PCI enabled
> [...]
> 
> However to prevent the PS/2 interfaces from getting fixed and then broken 
> again with backports in a random fashion I have marked both changes for 
> backporting as appropriate.
> 
>  Bjorn, may I request that these changes be placed, with Thomas's ack of 
> course (hopefully a formality), ahead of Ilpo's commit 16fbaba2b78f 
> ("MIPS: Malta: Use pcibios_align_resource() to block io range") (or 
> whatever the latest version is, as said commit seems to be missing tags 
> updates you mentioned), and then merged via your tree?  That will prevent 
> things from breaking just to be fixed again shortly, and overall getting 
> out of sync.
> 
>  See individual commit descriptions for details.

FWIW,

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

-- 
 i.

      parent reply	other threads:[~2025-10-22 11:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-21 19:38 [PATCH 0/2] MIPS: Malta: Fix PCI southbridge legacy resource clashes Maciej W. Rozycki
2025-10-21 19:38 ` [PATCH 1/2] MIPS: Malta: Fix keyboard resource preventing i8042 driver from registering Maciej W. Rozycki
2025-10-21 19:45   ` Thomas Bogendoerfer
2025-10-21 19:38 ` [PATCH 2/2] MIPS: Malta: Fix PCI southbridge legacy resource reservations Maciej W. Rozycki
2025-10-21 19:45   ` Thomas Bogendoerfer
2025-10-22 11:42   ` Jonas Gorski
2025-10-22 12:48     ` Maciej W. Rozycki
2025-10-21 20:59 ` [PATCH 0/2] MIPS: Malta: Fix PCI southbridge legacy resource clashes Bjorn Helgaas
2025-10-22 11:08 ` Ilpo Järvinen [this message]

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=93ffa5d8-61eb-abdf-8271-8f676d36fe56@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=bhelgaas@google.com \
    --cc=helgaas@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=macro@orcam.me.uk \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.