Subject: Re: CVS commit: src/sys/dev/pci
To: None <gavan@netbsd.org>
From: Jason Thorpe <thorpej@shagadelic.org>
List: source-changes
Date: 02/15/2006 16:15:23
On Feb 15, 2006, at 4:02 PM, Gavan Fantom wrote:

>
> Module Name:	src
> Committed By:	gavan
> Date:		Thu Feb 16 00:02:00 UTC 2006
>
> Modified Files:
> 	src/sys/dev/pci: if_wm.c
> Added Files:
> 	src/sys/dev/pci: if_wmvar.h
>
> Log Message:
> Compute the EEPROM checksum to determine the presence of valid  
> EEPROM data.
> If EEPROM data is invalid, allow a MD hook to supply EEPROM data  
> instead.

An MD hook is wrong.  You should be using properties for this (you  
can set the property in your platform's device_register() routine).   
(Yes, I know that there are currently other drivers that do this  
incorrectly, but those came before properties existed in our kernel.)

In general, changes like this should be reviewed...

-- thorpej