public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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


  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