mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "David S. Miller" <davem@redhat.com>
To: Abramo Bagnara <abramo@alsa-project.org>
Cc: David Woodhouse <dwmw2@infradead.org>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Linux Kernel Development <linux-kernel@vger.kernel.org>
Subject: Re: unsigned long ioremap()?
Date: Thu, 3 May 2001 17:22:20 -0700 (PDT)	[thread overview]
Message-ID: <15089.63036.52229.489681@pizda.ninka.net> (raw)
In-Reply-To: <3AF12B94.60083603@alsa-project.org>
In-Reply-To: <3AF10E80.63727970@alsa-project.org> <Pine.LNX.4.05.10105030852330.9438-100000@callisto.of.borg> <15089.979.650927.634060@pizda.ninka.net> <11718.988883128@redhat.com> <3AF12B94.60083603@alsa-project.org>


Abramo Bagnara writes:
 > IMO this is a far less effective debugging strategy.

I agree with you.

But guess what driver authors are going to do?  They are going to cast
the thing left and right.  And sure you can then search for that, but
it isn't likely to make people fix this from the start.

I suppose the point is that there is a fine line wrt. using APIs to
influence people to "do the right thing", and this has been
exemplified in several threads I've been involved in wrt. PCI dma
and other topics. :-)

One final point, I want to reiterate that I believe:

	foo = readl(&regs->bar);

is perfectly legal and should not be discouraged and in particular,
not made painful to do.

Later,
David S. Miller
davem@redhat.com


  parent reply	other threads:[~2001-05-04  0:22 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-05-03  6:55 Geert Uytterhoeven
2001-05-03  7:08 ` David S. Miller
2001-05-03  7:18   ` Jeff Garzik
2001-05-03  7:29     ` C.Praveen
2001-05-03  7:46   ` Jonathan Lundell
2001-05-03  7:57     ` Geert Uytterhoeven
2001-05-03  7:53   ` Abramo Bagnara
2001-05-03  8:08     ` Jeff Garzik
2001-05-03  8:26       ` Geert Uytterhoeven
2001-05-03  8:39       ` Abramo Bagnara
2001-05-03  8:44         ` Jeff Garzik
2001-05-03  8:53           ` Abramo Bagnara
2001-05-03  9:45   ` David Woodhouse
2001-05-03  9:57     ` Abramo Bagnara
2001-05-03 10:02     ` David Woodhouse
2001-05-04  0:22     ` David S. Miller [this message]
2001-05-04  7:15       ` Abramo Bagnara
2001-05-04 13:53         ` Jonathan Lundell
2001-05-13 14:00         ` Jes Sorensen
2001-05-13 14:38           ` Abramo Bagnara
2001-05-04  7:30       ` David S. Miller
2001-05-04 11:07         ` Abramo Bagnara
2001-05-03  7:33 ` Jonathan Lundell

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=15089.63036.52229.489681@pizda.ninka.net \
    --to=davem@redhat.com \
    --cc=abramo@alsa-project.org \
    --cc=dwmw2@infradead.org \
    --cc=geert@linux-m68k.org \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®