lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Mon, 30 Jan 2012 06:37:08 -0800
From:	Greg KH <gregkh@...e.de>
To:	Andrea Shepard <andrea@...sephoneslair.org>
Cc:	linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
	khc@...waw.pl, davem@...emloft.net, mmarek@...e.cz,
	jkosina@...e.cz, joe@...ches.com, justinmattock@...il.com,
	alan@...ux.intel.com, jdmason@...zu.us
Subject: Re: [01/22] Cyclades PC300 driver: merge in Cyclades driver version
 4.1.0

On Sun, Jan 29, 2012 at 06:43:54PM -0800, Andrea Shepard wrote:
>  struct RUNTIME_9050 {
> -	u32 loc_addr_range[4];	/* 00-0Ch : Local Address Ranges */
> -	u32 loc_rom_range;	/* 10h : Local ROM Range */
> -	u32 loc_addr_base[4];	/* 14-20h : Local Address Base Addrs */
> -	u32 loc_rom_base;	/* 24h : Local ROM Base */
> -	u32 loc_bus_descr[4];	/* 28-34h : Local Bus Descriptors */
> -	u32 rom_bus_descr;	/* 38h : ROM Bus Descriptor */
> -	u32 cs_base[4];		/* 3C-48h : Chip Select Base Addrs */
> -	u32 intr_ctrl_stat;	/* 4Ch : Interrupt Control/Status */
> -	u32 init_ctrl;		/* 50h : EEPROM ctrl, Init Ctrl, etc */
> +	uclong	loc_addr_range[4];	/* 00-0Ch : Local Address Ranges */
> +	uclong	loc_rom_range;		/* 10h : Local ROM Range */
> +	uclong	loc_addr_base[4];	/* 14-20h : Local Address Base Addrs */
> +	uclong	loc_rom_base;		/* 24h : Local ROM Base */
> +	uclong	loc_bus_descr[4];	/* 28-34h : Local Bus Descriptors */
> +	uclong	rom_bus_descr;		/* 38h : ROM Bus Descriptor */
> +	uclong	cs_base[4];		/* 3C-48h : Chip Select Base Addrs */
> +	uclong	intr_ctrl_stat;		/* 4Ch : Interrupt Control/Status */
> +	uclong	init_ctrl;		/* 50h : EEPROM ctrl, Init Ctrl, etc */

Now I'm not the network driver maintainer, so he might take this type of
thing, but I sure wouldn't.

I understand your want to dump the updated version of the driver in, and
then clean it up, but come on, allowing changes like this, that you can
see are obviously wrong (same goes for the return lines), and then
cleaning it up (hopefully) later on?

You can edit the diff by hand to remove this type of thing, so the churn
for it is not needed at all, which makes your overall diff smaller and
series easier to read and understand.

but that's just my opinion...

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists