Re: [RFC v4]update broken web addresses in the kernel.

Re: [RFC v4]update broken web addresses in the kernel.

[John W. Linville]

On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
> Below is an updated patch from the original fixing broken web addresses in the kernel.
> Thanks for all the help and info on this to everybody..
> Hopefully I didnt miss any of them(if so let me know, and I'll resend).

Changing a URL for a relocated page is one thing, but removing links
isn't necessarily a great idea.  Even if the site is technically
gone, it may be possible to find information e.g through the Internet
Archive Wayback Machine.

> diff --git a/drivers/net/wireless/orinoco/main.c b/drivers/net/wireless/orinoco/main.c
> index e8e2d0f..ba10b07 100644
> --- a/drivers/net/wireless/orinoco/main.c
> +++ b/drivers/net/wireless/orinoco/main.c
> @@ -17,7 +17,6 @@
>   *
>   * Portions based on wvlan_cs.c 1.0.6, Copyright Andreas Neuhaus <andy
>   * AT fasta.fh-dortmund.de>
> - *      http://www.stud.fh-dortmund.de/~andy/wvlan/
>   *
>   * The contents of this file are subject to the Mozilla Public License
>   * Version 1.1 (the "License"); you may not use this file except in

http://web.archive.org/web/20070716051348/http://www.stud.fh-dortmund.de/~andy/wvlan/

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

Re: [RFC v4]update broken web addresses in the kernel.

[Joe Perches]

On Mon, 2010-09-27 at 11:10 -0400, John W. Linville wrote:
> On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
> > Below is an updated patch from the original fixing broken web addresses in the kernel.
> > Thanks for all the help and info on this to everybody..
> > Hopefully I didnt miss any of them(if so let me know, and I'll resend).
> Changing a URL for a relocated page is one thing, but removing links
> isn't necessarily a great idea.  Even if the site is technically
> gone, it may be possible to find information e.g through the Internet
> Archive Wayback Machine.

Perhaps it'd be better to scrape the contents of the various
web pages, collect them somewhere like wiki.kernel.org and
encourage others to put new contributions in that site.

Re: [RFC v4]update broken web addresses in the kernel.

[Justin P. Mattock]

On 09/27/2010 09:03 AM, Joe Perches wrote:
> On Mon, 2010-09-27 at 11:10 -0400, John W. Linville wrote:
>> On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
>>> Below is an updated patch from the original fixing broken web addresses in the kernel.
>>> Thanks for all the help and info on this to everybody..
>>> Hopefully I didnt miss any of them(if so let me know, and I'll resend).
>> Changing a URL for a relocated page is one thing, but removing links
>> isn't necessarily a great idea.  Even if the site is technically
>> gone, it may be possible to find information e.g through the Internet
>> Archive Wayback Machine.
>
> Perhaps it'd be better to scrape the contents of the various
> web pages, collect them somewhere like wiki.kernel.org and
> encourage others to put new contributions in that site.
>
>
>
>


yeah I think somebody was saying something about having a separate file, 
with all the web addresses in them or something...In any case, up to you 
guys..

Justin P. Mattock

Re: [RFC v4]update broken web addresses in the kernel.

[Finn Thain]

On Mon, 27 Sep 2010, Justin P. Mattock wrote:

> On 09/27/2010 09:03 AM, Joe Perches wrote:
> > On Mon, 2010-09-27 at 11:10 -0400, John W. Linville wrote:
> > > On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
> > > > Below is an updated patch from the original fixing broken web 
> > > > addresses in the kernel. Thanks for all the help and info on this 
> > > > to everybody.. Hopefully I didnt miss any of them(if so let me 
> > > > know, and I'll resend).
> > > Changing a URL for a relocated page is one thing, but removing links 
> > > isn't necessarily a great idea.  Even if the site is technically 
> > > gone, it may be possible to find information e.g through the 
> > > Internet Archive Wayback Machine.
> > 
> > Perhaps it'd be better to scrape the contents of the various web 
> > pages, collect them somewhere like wiki.kernel.org and encourage 
> > others to put new contributions in that site.

The copyright problem aside, this might be a good idea for material not 
already archived but I don't think it makes sense to start a new 
archive when archive.org (or other) has the information.

And which version(s) do you scrape? I discussed some problems with 
changing URLs in another thread: http://lkml.org/lkml/2010/9/22/22

Anyway, without knowing what future archive(s) would be available or 
relevant to any given URL in the future, I think the best we might do is a 
"Retrieved on YYYY-MM-DD" qualification for new URLs.

> 
> 
> yeah I think somebody was saying something about having a separate file, 
> with all the web addresses in them or something...In any case, up to you 
> guys..

I don't see how moving the addresses would help.
And would it not make the information harder to find?

Finn

> 
> Justin P. Mattock
> 

Re: [RFC v4]update broken web addresses in the kernel.

[Justin P. Mattock]

On 09/27/2010 06:37 PM, Finn Thain wrote:
>
> On Mon, 27 Sep 2010, Justin P. Mattock wrote:
>
>> On 09/27/2010 09:03 AM, Joe Perches wrote:
>>> On Mon, 2010-09-27 at 11:10 -0400, John W. Linville wrote:
>>>> On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
>>>>> Below is an updated patch from the original fixing broken web
>>>>> addresses in the kernel. Thanks for all the help and info on this
>>>>> to everybody.. Hopefully I didnt miss any of them(if so let me
>>>>> know, and I'll resend).
>>>> Changing a URL for a relocated page is one thing, but removing links
>>>> isn't necessarily a great idea.  Even if the site is technically
>>>> gone, it may be possible to find information e.g through the
>>>> Internet Archive Wayback Machine.
>>>
>>> Perhaps it'd be better to scrape the contents of the various web
>>> pages, collect them somewhere like wiki.kernel.org and encourage
>>> others to put new contributions in that site.
>
> The copyright problem aside, this might be a good idea for material not
> already archived but I don't think it makes sense to start a new
> archive when archive.org (or other) has the information.
>
> And which version(s) do you scrape? I discussed some problems with
> changing URLs in another thread: http://lkml.org/lkml/2010/9/22/22
>
> Anyway, without knowing what future archive(s) would be available or
> relevant to any given URL in the future, I think the best we might do is a
> "Retrieved on YYYY-MM-DD" qualification for new URLs.
>
>>
>>
>> yeah I think somebody was saying something about having a separate file,
>> with all the web addresses in them or something...In any case, up to you
>> guys..
>
> I don't see how moving the addresses would help.
> And would it not make the information harder to find?
>
> Finn
>

depends on how the setup is.. I was thinking of having some kind of 
letter/number scheme i.e. (A-13) in the comment, then in the file with 
all the addresses under article "A" 13th address down you would have the 
web address.(but keep in mind that's just speculation..)

Justin P. Mattock

Re: [RFC v4]update broken web addresses in the kernel.

[Justin P. Mattock]

On 09/27/2010 08:10 AM, John W. Linville wrote:
> On Sun, Sep 26, 2010 at 11:31:15AM -0700, Justin P. Mattock wrote:
>> Below is an updated patch from the original fixing broken web addresses in the kernel.
>> Thanks for all the help and info on this to everybody..
>> Hopefully I didnt miss any of them(if so let me know, and I'll resend).
>
> Changing a URL for a relocated page is one thing, but removing links
> isn't necessarily a great idea.  Even if the site is technically
> gone, it may be possible to find information e.g through the Internet
> Archive Wayback Machine.
>
>> diff --git a/drivers/net/wireless/orinoco/main.c b/drivers/net/wireless/orinoco/main.c
>> index e8e2d0f..ba10b07 100644
>> --- a/drivers/net/wireless/orinoco/main.c
>> +++ b/drivers/net/wireless/orinoco/main.c
>> @@ -17,7 +17,6 @@
>>    *
>>    * Portions based on wvlan_cs.c 1.0.6, Copyright Andreas Neuhaus<andy
>>    * AT fasta.fh-dortmund.de>
>> - *      http://www.stud.fh-dortmund.de/~andy/wvlan/
>>    *
>>    * The contents of this file are subject to the Mozilla Public License
>>    * Version 1.1 (the "License"); you may not use this file except in
>
> http://web.archive.org/web/20070716051348/http://www.stud.fh-dortmund.de/~andy/wvlan/
>

most of the urls that are not removed that are still broken work with 
the wayback machine..(from what I remember this one would give me a 
server error(reason for removing), but it doesn't look so).

Thanks for this, I'll change that up.

Justin P. Mattock

Re: [RFC v4]update broken web addresses in the kernel.

[Finn Thain]

On Sun, 26 Sep 2010, Justin P. Mattock wrote:

> diff --git a/drivers/char/apm-emulation.c b/drivers/char/apm-emulation.c
> index 033e150..d7d9a78 100644
> --- a/drivers/char/apm-emulation.c
> +++ b/drivers/char/apm-emulation.c
> @@ -8,7 +8,7 @@
>   *   (APM) BIOS Interface Specification, Revision 1.2, February 1996.
>   *
>   * [This document is available from Microsoft at:
> - *    http://www.microsoft.com/hwdev/busbios/amp_12.htm]
> + *    http://www.microsoft.com/whdc/archive/amp_12.mspx]
>   */


Can we lose the brackets?


>  #include <linux/module.h>
>  #include <linux/poll.h>

> diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
> index 280c9b5..b2bd5fb 100644
> --- a/drivers/firmware/Kconfig
> +++ b/drivers/firmware/Kconfig
> @@ -73,8 +73,8 @@ config EFI_PCDP
>  	  on how the driver discovers devices.
>  
>  	  You must also enable the appropriate drivers (serial, VGA, etc.)
> -
> -	  See <http://www.dig64.org/specifications/DIG64_HCDPv20_042804.pdf>
> +	  See [DIG64_HCDPv20_042804.pdf] available from


What are the square brackets for?


> +	  <http://www.dig64.org/specifications/>
>  
>  config DELL_RBU
>  	tristate "BIOS update support for DELL systems via sysfs"

> diff --git a/drivers/firmware/pcdp.h b/drivers/firmware/pcdp.h
> index ce910d6..777d928 100644
> --- a/drivers/firmware/pcdp.h
> +++ b/drivers/firmware/pcdp.h
> @@ -1,8 +1,8 @@
>  /*
>   * Definitions for PCDP-defined console devices
>   *
> - * v1.0a: http://www.dig64.org/specifications/DIG64_HCDPv10a_01.pdf
> - * v2.0:  http://www.dig64.org/specifications/DIG64_PCDPv20.pdf
> + * For [v1.0a: DIG64_HCDPv10a_01.pdf] [v2.0: DIG64_PCDPv20.pdf] 


For DIG64_HCDPv10a_01.pdf and DIG64_PCDPv20.pdf (v10.a and v2.0 resp.), 


> + * Please see <http://www.dig64.org/specifications/>


please see <http://www.dig64.org/specifications/>


>   *
>   * (c) Copyright 2002, 2004 Hewlett-Packard Development Company, L.P.
>   *	Khalid Aziz <khalid.aziz@hp.com>
> diff --git a/drivers/gpu/drm/README.drm b/drivers/gpu/drm/README.drm
> index b5b3327..bf29068 100644
> --- a/drivers/gpu/drm/README.drm
> +++ b/drivers/gpu/drm/README.drm
> @@ -28,7 +28,7 @@ ways:
>  Documentation on the DRI is available from:
>      http://dri.freedesktop.org/wiki/Documentation
>      http://sourceforge.net/project/showfiles.php?group_id87
> -    http://dri.sourceforge.net/doc/
> +    http://dri.sourceforge.net/doc/DRIuserguide.html 


http://dri.freedesktop.org/wiki/Documentation


>  
>  For specific information about kernel-level support, see:
>  

> diff --git a/drivers/input/mouse/touchkit_ps2.c b/drivers/input/mouse/touchkit_ps2.c
> index 88121c5..e74753e 100644
> --- a/drivers/input/mouse/touchkit_ps2.c
> +++ b/drivers/input/mouse/touchkit_ps2.c
> @@ -22,7 +22,7 @@
>   * Based upon touchkitusb.c
>   *
>   * Vendor documentation is available in support section of:
> - * http://www.egalax.com.tw/
> + * http://home.eeti.com.tw/web20/drivers/Software%20Programming%20Guide_v2.0.pdf 


The comment text needs to be fixed also.


>   */
>  
>  #include <linux/kernel.h>

> diff --git a/drivers/media/dvb/frontends/mt312.c b/drivers/media/dvb/frontends/mt312.c
> index 472907d..08023f1 100644
> --- a/drivers/media/dvb/frontends/mt312.c
> +++ b/drivers/media/dvb/frontends/mt312.c
> @@ -20,8 +20,9 @@
>      Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>  
>      References:
> -    http://products.zarlink.com/product_profiles/MT312.htm
> -    http://products.zarlink.com/product_profiles/SL1935.htm
> +    http://www.zarlink.com/zarlink/hs/71.htm
> +    http://www.chipcatalog.com/Zarlink/MT312.htm
> +    http://www.chipcatalog.com/Zarlink/SL1935.htm


There's no need for links to chipcatalog.com. As I said in the previous 
review, we have search engines for finding datasheets. Besides, the old 
links are in the web archive (complete with PDFs) so there's no need to 
remove them.


>  */
>  
>  #include <linux/delay.h>
> diff --git a/drivers/media/dvb/frontends/mt312.h b/drivers/media/dvb/frontends/mt312.h
> index 29e3bb5..6d32e3f 100644
> --- a/drivers/media/dvb/frontends/mt312.h
> +++ b/drivers/media/dvb/frontends/mt312.h
> @@ -19,8 +19,9 @@
>      Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>  
>      References:
> -    http://products.zarlink.com/product_profiles/MT312.htm
> -    http://products.zarlink.com/product_profiles/SL1935.htm
> +    http://www.zarlink.com/zarlink/hs/71.htm
> +    http://www.chipcatalog.com/Zarlink/MT312.htm
> +    http://www.chipcatalog.com/Zarlink/SL1935.htm


Same here.


>  */
>  
>  #ifndef MT312_H

> --- a/drivers/media/video/gspca/gspca.c
> +++ b/drivers/media/video/gspca/gspca.c
> @@ -2338,7 +2338,7 @@ EXPORT_SYMBOL(gspca_resume);
>  
>  /* auto gain and exposure algorithm based on the knee algorithm described here:
>     http://ytse.tricolour.net/docs/LowLightOptimization.html
> -
> +   http://81.209.78.62:8080/docs/LowLightOptimization.html


This is in the web archive.


>     Returns 0 if no changes were made, 1 if the gain and or exposure settings
>     where changed. */
>  int gspca_auto_gain_n_exposure(struct gspca_dev *gspca_dev, int avg_lum,

> index 701d942..142efd2 100644
> --- a/drivers/mtd/maps/Kconfig
> +++ b/drivers/mtd/maps/Kconfig
> @@ -256,8 +256,7 @@ config MTD_DILNETPC
>  	depends on X86 && MTD_CONCAT && MTD_PARTITIONS && MTD_CFI_INTELEXT && BROKEN
>  	help
>  	  MTD map driver for SSV DIL/Net PC Boards "DNP" and "ADNP".
> -	  For details, see <http://www.ssv-embedded.de/ssv/pc104/p169.htm>
> - 	  and <http://www.ssv-embedded.de/ssv/pc104/p170.htm>
> +	  For details, see <http://www.dilnetpc.com/>


This is in the web archive. This removes information.


>  
>  config MTD_DILNETPC_BOOTSIZE
>  	hex "Size of DIL/Net PC flash boot partition"

> diff --git a/drivers/mtd/maps/dilnetpc.c b/drivers/mtd/maps/dilnetpc.c
> index 0713e3a..114899b 100644
> --- a/drivers/mtd/maps/dilnetpc.c
> +++ b/drivers/mtd/maps/dilnetpc.c
> @@ -19,8 +19,7 @@
>   * board: DNP/1486 and ADNP/1486. The DNP version has 2 megs of flash
>   * ROM (Intel 28F016S3) and 8 megs of DRAM, the ADNP version has 4 megs
>   * flash and 16 megs of RAM.
> - * For details, see http://www.ssv-embedded.de/ssv/pc104/p169.htm
> - * and http://www.ssv-embedded.de/ssv/pc104/p170.htm
> + * For details, see http://www.dilnetpc.com/ 


Same.


>   */
>  
>  #include <linux/module.h>

> index 49aac70..f4523cd 100644
> --- a/drivers/net/hamachi.c
> +++ b/drivers/net/hamachi.c
> @@ -20,9 +20,6 @@
>  
>  	Support and updates available at
>  	http://www.scyld.com/network/hamachi.html
> -	[link no longer provides useful info -jgarzik]
> -	or
> -	http://www.parl.clemson.edu/~keithu/hamachi.html


Why?


>  
>  */
>  

> index 6af6c83..0723c9f 100644
> --- a/drivers/staging/comedi/drivers/daqboard2000.c
> +++ b/drivers/staging/comedi/drivers/daqboard2000.c
> @@ -51,7 +51,7 @@ Configuration options:
>     for the card, and here are the findings so far.
>  
>     1. A good document that describes the PCI interface chip is found at:
> -      http://plx.plxtech.com/download/9080/databook/9080db-106.pdf
> +      [9080db-106.pdf] http://www.plxtech.com/products/io/pci9080 


Comments are supposed to be human readable text. Square brackets don't 
quote filenames by any convention I'm aware of (?)
I'd suggest:

      A good document that describes the PCI interface chip is 9080db-106.pdf
      available from http://www.plxtech.com/products/io/pci9080


>  
>     2. The initialization done so far is:
>          a. program the FPGA (windows code sans a lot of error messages)

> diff --git a/drivers/video/vga16fb.c b/drivers/video/vga16fb.c
> index 28ccab4..fc853a5 100644
> --- a/drivers/video/vga16fb.c
> +++ b/drivers/video/vga16fb.c
> @@ -2,7 +2,7 @@
>   * linux/drivers/video/vga16.c -- VGA 16-color framebuffer driver
>   * 
>   * Copyright 1999 Ben Pfaff <pfaffben@debian.org> and Petr Vandrovec <VANDROVE@vc.cvut.cz>
> - * Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
> + * Based on VGA info at http://www.osdever.net/FreeVGA/home.htm 


The old link is in the web archive, you needn't remove it. Add the new one 
if you wish.


>   * Based on VESA framebuffer (c) 1998 Gerd Knorr <kraxel@goldbach.in-berlin.de>
>   *
>   * This file is subject to the terms and conditions of the GNU General
> diff --git a/drivers/video/vgastate.c b/drivers/video/vgastate.c
> index b91c466..f4043f8 100644
> --- a/drivers/video/vgastate.c
> +++ b/drivers/video/vgastate.c
> @@ -5,7 +5,7 @@
>   * 
>   * Copyright history from vga16fb.c:
>   *	Copyright 1999 Ben Pfaff and Petr Vandrovec
> - *	Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
> + *	Based on VGA info at http://www.osdever.net/FreeVGA/home.htm 


Same.


>   *	Based on VESA framebuffer (c) 1998 Gerd Knorr
>   *
>   * This file is subject to the terms and conditions of the GNU General
> diff --git a/include/linux/ixjuser.h b/include/linux/ixjuser.h
> index 88b4589..39ef699 100644
> --- a/include/linux/ixjuser.h
> +++ b/include/linux/ixjuser.h
> @@ -25,7 +25,7 @@
>   *                  Mike Preston, <mpreston@quicknet.net>
>   *
>   * More information about the hardware related to this driver can be found
> - * at our website:    http://www.quicknet.net
> + * at our website:   http://blemings.org/hugh/keyspan.html 


This appears to have been fat-fingered. No patch needed.


>   *
>   * Fixes:
>   *

> diff --git a/include/video/vga.h b/include/video/vga.h
> index b49a512..2b8691f 100644
> --- a/include/video/vga.h
> +++ b/include/video/vga.h
> @@ -5,7 +5,7 @@
>   * 
>   * Copyright history from vga16fb.c:
>   *	Copyright 1999 Ben Pfaff and Petr Vandrovec
> - *	Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
> + *	Based on VGA info at http://www.osdever.net/FreeVGA/home.htm 


See above.

Thanks.

Finn

Re: [RFC v4]update broken web addresses in the kernel.

[Justin P. Mattock]

On 09/27/2010 07:59 PM, Finn Thain wrote:
>
> On Sun, 26 Sep 2010, Justin P. Mattock wrote:
>
>> diff --git a/drivers/char/apm-emulation.c b/drivers/char/apm-emulation.c
>> index 033e150..d7d9a78 100644
>> --- a/drivers/char/apm-emulation.c
>> +++ b/drivers/char/apm-emulation.c
>> @@ -8,7 +8,7 @@
>>    *   (APM) BIOS Interface Specification, Revision 1.2, February 1996.
>>    *
>>    * [This document is available from Microsoft at:
>> - *    http://www.microsoft.com/hwdev/busbios/amp_12.htm]
>> + *    http://www.microsoft.com/whdc/archive/amp_12.mspx]
>>    */
>
>
> Can we lose the brackets?
>
>
>>   #include<linux/module.h>
>>   #include<linux/poll.h>
>
>> diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
>> index 280c9b5..b2bd5fb 100644
>> --- a/drivers/firmware/Kconfig
>> +++ b/drivers/firmware/Kconfig
>> @@ -73,8 +73,8 @@ config EFI_PCDP
>>   	  on how the driver discovers devices.
>>
>>   	  You must also enable the appropriate drivers (serial, VGA, etc.)
>> -
>> -	  See<http://www.dig64.org/specifications/DIG64_HCDPv20_042804.pdf>
>> +	  See [DIG64_HCDPv20_042804.pdf] available from
>
>
> What are the square brackets for?
>
>
>> +	<http://www.dig64.org/specifications/>
>>
>>   config DELL_RBU
>>   	tristate "BIOS update support for DELL systems via sysfs"
>
>> diff --git a/drivers/firmware/pcdp.h b/drivers/firmware/pcdp.h
>> index ce910d6..777d928 100644
>> --- a/drivers/firmware/pcdp.h
>> +++ b/drivers/firmware/pcdp.h
>> @@ -1,8 +1,8 @@
>>   /*
>>    * Definitions for PCDP-defined console devices
>>    *
>> - * v1.0a: http://www.dig64.org/specifications/DIG64_HCDPv10a_01.pdf
>> - * v2.0:  http://www.dig64.org/specifications/DIG64_PCDPv20.pdf
>> + * For [v1.0a: DIG64_HCDPv10a_01.pdf] [v2.0: DIG64_PCDPv20.pdf]
>
>
> For DIG64_HCDPv10a_01.pdf and DIG64_PCDPv20.pdf (v10.a and v2.0 resp.),
>
>
>> + * Please see<http://www.dig64.org/specifications/>
>
>
> please see<http://www.dig64.org/specifications/>
>
>
>>    *
>>    * (c) Copyright 2002, 2004 Hewlett-Packard Development Company, L.P.
>>    *	Khalid Aziz<khalid.aziz@hp.com>
>> diff --git a/drivers/gpu/drm/README.drm b/drivers/gpu/drm/README.drm
>> index b5b3327..bf29068 100644
>> --- a/drivers/gpu/drm/README.drm
>> +++ b/drivers/gpu/drm/README.drm
>> @@ -28,7 +28,7 @@ ways:
>>   Documentation on the DRI is available from:
>>       http://dri.freedesktop.org/wiki/Documentation
>>       http://sourceforge.net/project/showfiles.php?group_id87
>> -    http://dri.sourceforge.net/doc/
>> +    http://dri.sourceforge.net/doc/DRIuserguide.html
>
>
> http://dri.freedesktop.org/wiki/Documentation
>
>
>>
>>   For specific information about kernel-level support, see:
>>
>
>> diff --git a/drivers/input/mouse/touchkit_ps2.c b/drivers/input/mouse/touchkit_ps2.c
>> index 88121c5..e74753e 100644
>> --- a/drivers/input/mouse/touchkit_ps2.c
>> +++ b/drivers/input/mouse/touchkit_ps2.c
>> @@ -22,7 +22,7 @@
>>    * Based upon touchkitusb.c
>>    *
>>    * Vendor documentation is available in support section of:
>> - * http://www.egalax.com.tw/
>> + * http://home.eeti.com.tw/web20/drivers/Software%20Programming%20Guide_v2.0.pdf
>
>
> The comment text needs to be fixed also.
>
>
>>    */
>>
>>   #include<linux/kernel.h>
>
>> diff --git a/drivers/media/dvb/frontends/mt312.c b/drivers/media/dvb/frontends/mt312.c
>> index 472907d..08023f1 100644
>> --- a/drivers/media/dvb/frontends/mt312.c
>> +++ b/drivers/media/dvb/frontends/mt312.c
>> @@ -20,8 +20,9 @@
>>       Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>>
>>       References:
>> -    http://products.zarlink.com/product_profiles/MT312.htm
>> -    http://products.zarlink.com/product_profiles/SL1935.htm
>> +    http://www.zarlink.com/zarlink/hs/71.htm
>> +    http://www.chipcatalog.com/Zarlink/MT312.htm
>> +    http://www.chipcatalog.com/Zarlink/SL1935.htm
>
>
> There's no need for links to chipcatalog.com. As I said in the previous
> review, we have search engines for finding datasheets. Besides, the old
> links are in the web archive (complete with PDFs) so there's no need to
> remove them.
>
>
>>   */
>>
>>   #include<linux/delay.h>
>> diff --git a/drivers/media/dvb/frontends/mt312.h b/drivers/media/dvb/frontends/mt312.h
>> index 29e3bb5..6d32e3f 100644
>> --- a/drivers/media/dvb/frontends/mt312.h
>> +++ b/drivers/media/dvb/frontends/mt312.h
>> @@ -19,8 +19,9 @@
>>       Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>>
>>       References:
>> -    http://products.zarlink.com/product_profiles/MT312.htm
>> -    http://products.zarlink.com/product_profiles/SL1935.htm
>> +    http://www.zarlink.com/zarlink/hs/71.htm
>> +    http://www.chipcatalog.com/Zarlink/MT312.htm
>> +    http://www.chipcatalog.com/Zarlink/SL1935.htm
>
>
> Same here.
>
>
>>   */
>>
>>   #ifndef MT312_H
>
>> --- a/drivers/media/video/gspca/gspca.c
>> +++ b/drivers/media/video/gspca/gspca.c
>> @@ -2338,7 +2338,7 @@ EXPORT_SYMBOL(gspca_resume);
>>
>>   /* auto gain and exposure algorithm based on the knee algorithm described here:
>>      http://ytse.tricolour.net/docs/LowLightOptimization.html
>> -
>> +   http://81.209.78.62:8080/docs/LowLightOptimization.html
>
>
> This is in the web archive.
>
>
>>      Returns 0 if no changes were made, 1 if the gain and or exposure settings
>>      where changed. */
>>   int gspca_auto_gain_n_exposure(struct gspca_dev *gspca_dev, int avg_lum,
>
>> index 701d942..142efd2 100644
>> --- a/drivers/mtd/maps/Kconfig
>> +++ b/drivers/mtd/maps/Kconfig
>> @@ -256,8 +256,7 @@ config MTD_DILNETPC
>>   	depends on X86&&  MTD_CONCAT&&  MTD_PARTITIONS&&  MTD_CFI_INTELEXT&&  BROKEN
>>   	help
>>   	  MTD map driver for SSV DIL/Net PC Boards "DNP" and "ADNP".
>> -	  For details, see<http://www.ssv-embedded.de/ssv/pc104/p169.htm>
>> - 	  and<http://www.ssv-embedded.de/ssv/pc104/p170.htm>
>> +	  For details, see<http://www.dilnetpc.com/>
>
>
> This is in the web archive. This removes information.
>
>
>>
>>   config MTD_DILNETPC_BOOTSIZE
>>   	hex "Size of DIL/Net PC flash boot partition"
>
>> diff --git a/drivers/mtd/maps/dilnetpc.c b/drivers/mtd/maps/dilnetpc.c
>> index 0713e3a..114899b 100644
>> --- a/drivers/mtd/maps/dilnetpc.c
>> +++ b/drivers/mtd/maps/dilnetpc.c
>> @@ -19,8 +19,7 @@
>>    * board: DNP/1486 and ADNP/1486. The DNP version has 2 megs of flash
>>    * ROM (Intel 28F016S3) and 8 megs of DRAM, the ADNP version has 4 megs
>>    * flash and 16 megs of RAM.
>> - * For details, see http://www.ssv-embedded.de/ssv/pc104/p169.htm
>> - * and http://www.ssv-embedded.de/ssv/pc104/p170.htm
>> + * For details, see http://www.dilnetpc.com/
>
>
> Same.
>
>
>>    */
>>
>>   #include<linux/module.h>
>
>> index 49aac70..f4523cd 100644
>> --- a/drivers/net/hamachi.c
>> +++ b/drivers/net/hamachi.c
>> @@ -20,9 +20,6 @@
>>
>>   	Support and updates available at
>>   	http://www.scyld.com/network/hamachi.html
>> -	[link no longer provides useful info -jgarzik]
>> -	or
>> -	http://www.parl.clemson.edu/~keithu/hamachi.html
>
>
> Why?
>
>
>>
>>   */
>>
>
>> index 6af6c83..0723c9f 100644
>> --- a/drivers/staging/comedi/drivers/daqboard2000.c
>> +++ b/drivers/staging/comedi/drivers/daqboard2000.c
>> @@ -51,7 +51,7 @@ Configuration options:
>>      for the card, and here are the findings so far.
>>
>>      1. A good document that describes the PCI interface chip is found at:
>> -      http://plx.plxtech.com/download/9080/databook/9080db-106.pdf
>> +      [9080db-106.pdf] http://www.plxtech.com/products/io/pci9080
>
>
> Comments are supposed to be human readable text. Square brackets don't
> quote filenames by any convention I'm aware of (?)
> I'd suggest:
>
>        A good document that describes the PCI interface chip is 9080db-106.pdf
>        available from http://www.plxtech.com/products/io/pci9080
>
>
>>
>>      2. The initialization done so far is:
>>           a. program the FPGA (windows code sans a lot of error messages)
>
>> diff --git a/drivers/video/vga16fb.c b/drivers/video/vga16fb.c
>> index 28ccab4..fc853a5 100644
>> --- a/drivers/video/vga16fb.c
>> +++ b/drivers/video/vga16fb.c
>> @@ -2,7 +2,7 @@
>>    * linux/drivers/video/vga16.c -- VGA 16-color framebuffer driver
>>    *
>>    * Copyright 1999 Ben Pfaff<pfaffben@debian.org>  and Petr Vandrovec<VANDROVE@vc.cvut.cz>
>> - * Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
>> + * Based on VGA info at http://www.osdever.net/FreeVGA/home.htm
>
>
> The old link is in the web archive, you needn't remove it. Add the new one
> if you wish.
>
>
>>    * Based on VESA framebuffer (c) 1998 Gerd Knorr<kraxel@goldbach.in-berlin.de>
>>    *
>>    * This file is subject to the terms and conditions of the GNU General
>> diff --git a/drivers/video/vgastate.c b/drivers/video/vgastate.c
>> index b91c466..f4043f8 100644
>> --- a/drivers/video/vgastate.c
>> +++ b/drivers/video/vgastate.c
>> @@ -5,7 +5,7 @@
>>    *
>>    * Copyright history from vga16fb.c:
>>    *	Copyright 1999 Ben Pfaff and Petr Vandrovec
>> - *	Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
>> + *	Based on VGA info at http://www.osdever.net/FreeVGA/home.htm
>
>
> Same.
>
>
>>    *	Based on VESA framebuffer (c) 1998 Gerd Knorr
>>    *
>>    * This file is subject to the terms and conditions of the GNU General
>> diff --git a/include/linux/ixjuser.h b/include/linux/ixjuser.h
>> index 88b4589..39ef699 100644
>> --- a/include/linux/ixjuser.h
>> +++ b/include/linux/ixjuser.h
>> @@ -25,7 +25,7 @@
>>    *                  Mike Preston,<mpreston@quicknet.net>
>>    *
>>    * More information about the hardware related to this driver can be found
>> - * at our website:    http://www.quicknet.net
>> + * at our website:   http://blemings.org/hugh/keyspan.html
>
>
> This appears to have been fat-fingered. No patch needed.
>
>
>>    *
>>    * Fixes:
>>    *
>
>> diff --git a/include/video/vga.h b/include/video/vga.h
>> index b49a512..2b8691f 100644
>> --- a/include/video/vga.h
>> +++ b/include/video/vga.h
>> @@ -5,7 +5,7 @@
>>    *
>>    * Copyright history from vga16fb.c:
>>    *	Copyright 1999 Ben Pfaff and Petr Vandrovec
>> - *	Based on VGA info at http://www.goodnet.com/~tinara/FreeVGA/home.htm
>> + *	Based on VGA info at http://www.osdever.net/FreeVGA/home.htm
>
>
> See above.
>
> Thanks.
>
> Finn
>

alright.. I'll redu this... remove the brackets, and fix up the rest..
one note is:
http://dri.freedesktop.org/wiki/Documentation
this address is already there.. probably best to just leave the 
sourceforge address and use the webarchive and/or if theirs a
proper address for the sourceforge dri..

Justin P. Mattock

Re: [RFC v4]update broken web addresses in the kernel.

[Finn Thain]

On Mon, 27 Sep 2010, Justin P. Mattock wrote:

> alright.. I'll redu this... remove the brackets, and fix up the rest..
> one note is:
> http://dri.freedesktop.org/wiki/Documentation
> this address is already there..

Right you are.

> probably best to just leave the sourceforge
> address and use the webarchive and/or if theirs a
> proper address for the sourceforge dri..

I didn't find a sensible address for the sourceforge stuff now that 
indexes are turned off for /doc/.

I agree, best just to leave it as is, since it is in the archive.

Finn

> 
> Justin P. Mattock
> 
> 

Re: [RFC v4]update broken web addresses in the kernel.

[Justin P. Mattock]

On 09/27/2010 09:15 PM, Finn Thain wrote:
>
> On Mon, 27 Sep 2010, Justin P. Mattock wrote:
>
>> alright.. I'll redu this... remove the brackets, and fix up the rest..
>> one note is:
>> http://dri.freedesktop.org/wiki/Documentation
>> this address is already there..
>
> Right you are.
>
>> probably best to just leave the sourceforge
>> address and use the webarchive and/or if theirs a
>> proper address for the sourceforge dri..
>
> I didn't find a sensible address for the sourceforge stuff now that
> indexes are turned off for /doc/.
>
> I agree, best just to leave it as is, since it is in the archive.
>
> Finn
>
>>
>> Justin P. Mattock
>>
>>
>


alright... resent, hopefully I didnt miss any of them...
eyes are going out of whack over here..

Justin P. Mattock