linux-kernel - Re: [01/22] Cyclades PC300 driver: merge in Cyclades driver version 4.1.0

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 linux-hardening linux-cve-announce 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]

Message-ID: <20120130143708.GB13388@suse.de>
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 linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/