linux-kernel - Re: [PATCH][Fix] New Unified AVR32/AT91 MCI Driver that supports both MCI slots used at the same time

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: <c8284b5b0906121118r413bff3frd4a1d9e12e76bfca@mail.gmail.com>
Date:	Fri, 12 Jun 2009 11:18:10 -0700
From:	Rob Emanuele <poorarm@...reis.com>
To:	Nicolas Ferre <nicolas.ferre@...el.com>
Cc:	Andrew Victor <avictor.za@...il.com>,
	Haavard Skinnemoen <haavard.skinnemoen@...el.com>,
	Joey Oravec <joravec@...wtech.com>,
	linux-arm-kernel@...ts.arm.linux.org.uk,
	linux-kernel@...r.kernel.org, drzeus-mmc@...eus.cx
Subject: Re: [PATCH][Fix] New Unified AVR32/AT91 MCI Driver that supports both 
	MCI slots used at the same time

Hi Nicolas & Haavard,

>> Well, what is the best way to differentiate it from the at91_mci
>> driver and keep users from trying to use both drivers?
>
> I propose that we setup a kind of choice sub menu in the Kconfig for
> those two drivers when they are both supported.
>

I'll try Haavard's suggestion of making the 2 drivers mutually
exclusive and I'll remove "Unified" verbiage.

>> Who would that be as I haven't seen anyone who maintains any of those
>> boards other than the at91rm8200 (at least nothing listed in the
>> MAINTAINERS file)?  The board-sam9g20ek.c platform only shows Atmel as
>> the most recent copyright.
>
> Hey, AT91 are very well maintained, SAM9 as well as at91rm9200.
> So, AT91 specific bits should be sent to this mailing-list with Andrew
> Victor in copy.

I never meant to imply that they were not maintained or even not
maintained well, only that the maintainer's contact information was
not obvious.  Below is a patch to correct that listing for Andrew
Victor.

> I will also certainly add comments on this code.
>

I appreciate any comments you have.

--Rob

diff --git a/MAINTAINERS b/MAINTAINERS
index 5d460c9..a3bc13b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -472,7 +472,7 @@ M:	kernel@...tstofly.org
 L:	linux-arm-kernel@...ts.arm.linux.org.uk (subscribers-only)
 S:	Maintained

-ARM/ATMEL AT91RM9200 ARM ARCHITECTURE
+ARM/ATMEL AT91RM9200 & AT91SAM9* ARM ARCHITECTURES
 P:	Andrew Victor
 M:	linux@...im.org.za
 L:	linux-arm-kernel@...ts.arm.linux.org.uk (subscribers-only)
--
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/