All of lore.kernel.org
 help / color / mirror / Atom feed
From: miltonm@bga.com (Milton Miller)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] Stop ARM boards crashing when CUPS is loaded - 2.6.35-rc5
Date: Fri, 16 Jul 2010 11:23:55 -0500	[thread overview]
Message-ID: <miltonm-arm-parport-1@bga.com> (raw)
In-Reply-To: <4C402746.6080309@canonical.com>

On Fri Jul 16 2010 about 05:33:06 EST,  Lee Jones wrote:
> > The best solution is probably for the parport code to go through a
> > modernisation cycle like the serial code did, essentially using
> > platform devices to pass the base addresses. This would make the
> > driver more portable, and eliminates this problem entirely (because
> > platforms which don't have parports won't register the platform device(s)
> > necessary for parport to even probe illegal addresses.)
> 
> This sounds brilliant - when are you going to start? </kidding>

It has a long time ago ...

drivers/parport/parport_pc.c calls parport_pc_find_nonpci_ports,
which is in asm/parport.h

> 
> In all seriousness, do you think anyone is likely to undertake this
> work anytime soon? I am seeing this problem in a distribution which
> is due for release in October. I have no problem implementing a config
> change in the meantime, but as you say, a more _correct_ and portable
> solution should be sought.

Why not replace the arm asm/parport.h with asm-generic/parport.h which
already has a check for CONFIG_ISA, which appears to only be selected
on a few ARM platforms?

milton

WARNING: multiple messages have this Message-ID (diff)
From: Milton Miller <miltonm@bga.com>
To: Lee Jones <lee.jones@canonical.com>
Cc: linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Russell King <rmk@arm.linux.org.uk>,
	Martin Michlmayr <tbm@cyrius.com>,
	Woody Suwalski <terraluna977@gmail.com>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] Stop ARM boards crashing when CUPS is loaded - 2.6.35-rc5
Date: Fri, 16 Jul 2010 11:23:55 -0500	[thread overview]
Message-ID: <miltonm-arm-parport-1@bga.com> (raw)
In-Reply-To: <4C402746.6080309@canonical.com>

On Fri Jul 16 2010 about 05:33:06 EST,  Lee Jones wrote:
> > The best solution is probably for the parport code to go through a
> > modernisation cycle like the serial code did, essentially using
> > platform devices to pass the base addresses. This would make the
> > driver more portable, and eliminates this problem entirely (because
> > platforms which don't have parports won't register the platform device(s)
> > necessary for parport to even probe illegal addresses.)
> 
> This sounds brilliant - when are you going to start? </kidding>

It has a long time ago ...

drivers/parport/parport_pc.c calls parport_pc_find_nonpci_ports,
which is in asm/parport.h

> 
> In all seriousness, do you think anyone is likely to undertake this
> work anytime soon? I am seeing this problem in a distribution which
> is due for release in October. I have no problem implementing a config
> change in the meantime, but as you say, a more _correct_ and portable
> solution should be sought.

Why not replace the arm asm/parport.h with asm-generic/parport.h which
already has a check for CONFIG_ISA, which appears to only be selected
on a few ARM platforms?

milton

  parent reply	other threads:[~2010-07-16 16:23 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-13 14:05 [PATCH] Stop ARM boards crashing when CUPS is loaded - 2.6.35-rc5 Lee Jones
2010-07-15 20:02 ` Andrew Morton
2010-07-15 20:02   ` Andrew Morton
2010-07-15 20:06   ` Russell King - ARM Linux
2010-07-15 20:06     ` Russell King - ARM Linux
2010-07-16  9:08     ` Lee Jones
2010-07-16  9:08       ` Lee Jones
2010-07-16  9:20       ` Russell King - ARM Linux
2010-07-16  9:20         ` Russell King - ARM Linux
2010-07-16  9:32         ` Lee Jones
2010-07-16  9:32           ` Lee Jones
2010-07-16 11:12           ` Lee Jones
2010-07-16 11:12             ` Lee Jones
2010-07-26 22:26             ` Greg KH
2010-07-26 22:26               ` Greg KH
2010-07-16 16:23           ` Milton Miller [this message]
2010-07-16 16:23             ` Milton Miller
2010-07-16 16:42             ` Lee Jones
2010-07-16 16:42               ` Lee Jones
2010-07-16 19:38               ` Milton Miller
2010-07-16 19:38                 ` Milton Miller
2010-07-16 20:54                 ` Russell King - ARM Linux
2010-07-16 20:54                   ` Russell King - ARM Linux
2010-07-16  9:37       ` Martin Michlmayr
2010-07-16  9:37         ` Martin Michlmayr
2010-07-16 13:27         ` Woody Suwalski
2010-07-16 13:27           ` Woody Suwalski

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=miltonm-arm-parport-1@bga.com \
    --to=miltonm@bga.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.