DragonFly BSD
DragonFly commits List (threaded) for 2005-05
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: cvs commit: src/sys/dev/netif/de if_de.c if_devar.h


From: Joerg Sonnenberger <joerg@xxxxxxxxxxxxxxxxx>
Date: Thu, 12 May 2005 17:45:46 +0200
Mail-followup-to: commits@crater.dragonflybsd.org

On Thu, May 12, 2005 at 08:36:00AM +0800, Sepherosa Ziehau wrote:
> @@ -4184,16 +4172,14 @@ tulip_pci_attach(device_t dev)
> ......
> +	device_printf("address unknown\n");
> ......
> 
> Will it be `device_printf(dev, "address unknown\n");'?

Yeah.

> There are still quite a lot of netif driver using printf, I may help
> to change them into device_printf()/if_printf().

Please do so, send me the patches directly.

Rule of thumb is that device_printf is prefered for the initial part
before ether_ifattach and depending on the context for low-level
reports. The normal case should be if_printf (consistency with interface
renaming capability).

Joerg



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]