From: Adrian Bunk <bunk@fs.tum.de>
To: Linus Torvalds <torvalds@transmeta.com>,
Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Linux v2.5.50
Date: Sat, 30 Nov 2002 21:06:35 +0100 [thread overview]
Message-ID: <20021130200635.GE20774@fs.tum.de> (raw)
In-Reply-To: <Pine.LNX.4.44.0211271456160.18214-100000@penguin.transmeta.com>
On Wed, Nov 27, 2002 at 03:07:38PM -0800, Linus Torvalds wrote:
>...
> Summary of changes from v2.5.49 to v2.5.50
> ============================================
2.5.50 includes two changes that have the following in common:
- Alan forwarded them to you
- they were originally sent by me
- they are buggy
Below are explanations and fixes.
>...
> Alan Cox <alan@lxorguk.ukuu.org.uk>:
>...
> o remove junk from vlsi_ir
>...
1. the maintainer prefers to leave the LINUX_VERSION_CODE und to add an
#include <linux/version.h> to make it easier to have the same driver
in both 2.4 and 2.5
2. my removal of pci_dma_prep_single was complete nonsense (no, it's not
part of the comment...)
The following patch fixes it:
--- linux-2.5.50/include/net/irda/vlsi_ir.h.old 2002-11-30 20:50:10.000000000 +0100
+++ linux-2.5.50/include/net/irda/vlsi_ir.h 2002-11-30 20:50:20.000000000 +0100
@@ -27,6 +27,26 @@
#ifndef IRDA_VLSI_FIR_H
#define IRDA_VLSI_FIR_H
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,4)
+#ifdef CONFIG_PROC_FS
+/* PDE() introduced in 2.5.4 */
+#define PDE(inode) ((inode)->u.generic_ip)
+#endif
+#endif
+
+/*
+ * #if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,xx)
+ *
+ * missing pci-dma api call to give streaming dma buffer back to hw
+ * patch floating on lkml - probably present in 2.5.26 or later
+ * otherwise defining it as noop is ok, since the vlsi-ir is only
+ * used on two oldish x86-based notebooks which are cache-coherent
+ */
+#define pci_dma_prep_single(dev, addr, size, direction) /* nothing */
+/*
+ * #endif
+ */
+
/* ================================================================ */
/* non-standard PCI registers */
--- linux-2.5.50/drivers/net/irda/vlsi_ir.c.old 2002-11-30 20:51:12.000000000 +0100
+++ linux-2.5.50/drivers/net/irda/vlsi_ir.c 2002-11-30 20:51:39.000000000 +0100
@@ -44,6 +44,7 @@
#include <linux/time.h>
#include <linux/proc_fs.h>
#include <linux/smp_lock.h>
+#include <linux/version.h>
#include <asm/uaccess.h>
#include <net/irda/irda.h>
> o fix sound kconfig file locations
>...
The changed file locations are OK, wrond is my addition of the question
strings to MSNDCLAS_HAVE_BOOT are MSNDPIN_HAVE_BOOT. As Roman Zippel
told me in another thread they were define_bool in the old kconfig.
The following patch corrects this:
--- linux-2.5.50/sound/oss/Kconfig.old 2002-11-30 20:47:01.000000000 +0100
+++ linux-2.5.50/sound/oss/Kconfig 2002-11-30 20:47:25.000000000 +0100
@@ -293,7 +293,7 @@
depends on SOUND_PRIME && SOUND_MSNDCLAS=y
config MSNDCLAS_HAVE_BOOT
- bool "Have MSNDINIT.BIN firmware file"
+ bool
depends on SOUND_MSNDCLAS=y
default y
@@ -355,7 +355,7 @@
depends on SOUND_PRIME && SOUND_MSNDPIN=y
config MSNDPIN_HAVE_BOOT
- bool "Have PNDSPINI.BIN firmware file"
+ bool
depends on SOUND_MSNDPIN=y
default y
Sorry
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
next prev parent reply other threads:[~2002-11-30 19:59 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-11-27 23:07 Linux v2.5.50 Linus Torvalds
2002-11-28 0:49 ` Udo A. Steinberg
2002-11-28 9:10 ` Gerd Knorr
2002-11-28 11:35 ` Gerd Knorr
2002-11-28 2:43 ` [uPATCH] NCR5380.c compile fix " Paul
2002-12-04 10:52 ` Geert Uytterhoeven
2002-11-28 4:55 ` [oops] BUG at mm/highmem.c:455 " Paul
2002-11-28 4:57 ` Nathan Walp
2002-11-28 10:15 ` drivers/pci/quirks.c / " Sebastian Benoit
2002-11-28 16:25 ` Alan Cox
2002-11-28 17:02 ` Ivan Kokshaysky
2002-11-28 17:30 ` Dave Jones
2002-11-28 18:54 ` Alan Cox
2002-11-28 18:25 ` Dave Jones
2002-11-28 18:30 ` Mike Dresser
2002-11-28 17:41 ` Alan Cox
2002-11-28 17:31 ` Dave Jones
2002-11-28 17:41 ` Ivan Kokshaysky
2002-11-28 13:17 ` Geert Uytterhoeven
2002-11-30 20:06 ` Adrian Bunk [this message]
2002-12-01 14:41 ` Adrian Bunk
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=20021130200635.GE20774@fs.tum.de \
--to=bunk@fs.tum.de \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.com \
/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