linux-kernel - Re: [git patches] Staging: rtxxx0: cleanup and "dumb" merge

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090426183954.GA18449@kroah.com>
Date:	Sun, 26 Apr 2009 11:39:54 -0700
From:	Greg KH <greg@...ah.com>
To:	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [git patches] Staging: rtxxx0: cleanup and "dumb" merge

On Sun, Apr 26, 2009 at 05:59:20PM +0200, Bartlomiej Zolnierkiewicz wrote:
> 
> Hi,
> 
> This patchset cleans up rt{286,287,307}0 drivers (66 KLOC removed) and then
> "dumb" (i.e. rt3070 shares code with rt2870 by including rt2870's .c files
> and depends on ifdefs to get the correct end result) merges them (142 KLOC
> removed).  All in all 208 KLOC are gone and we now use the common code for
> all three Ralink drivers.

Wonderful!

Were you able to test that this still works afterward?

> "dumb" merge is of course not the ideal solution but allows relatively easy
> verification, makes the eventual real merge much easier and in the meantime
> allows us to work with the common code for all drivers (since the initial
> merge these drivers already became slightly out-of-sync with each other).
> 
> [ I know about rt2x00 project (http://rt2x00.serialmonkey.com/) and I praise
>   their efforts.  However I still wanted to skim through the original code,
>   mainly because I heard a lot of "interesting" things about it. ;)  Then I
>   started to clean it to make reading easier and before I knew it I ended up
>   with this monster patchset... ]
> 
> 
> All changes are available in the following git tree (which is based on top
> of next-20090424):
> 
> git://git.kernel.org:/pub/scm/linux/kernel/git/bart/misc.git rtsta-for-next
> 
> [ I'm not sending individual patches to LKML (they are too big and there
>   is too many of them) but if somebody prefers to get them by mail instead
>   of grabbing them from git tree please just ping me. ]

I'll suck them in from the git tree, that's fine with me.

thanks again for doing this work, it is much appreciated.

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/