[Top] [All Lists]

Re: [PATCH V8] MIPS: lantiq: add NOR flash support

To: John Crispin <>
Subject: Re: [PATCH V8] MIPS: lantiq: add NOR flash support
From: Artem Bityutskiy <>
Date: Tue, 05 Apr 2011 17:26:38 +0300
Cc: Ralf Baechle <>,, Ralph Hempel <>,, Daniel Schwierzeck <>, David Woodhouse <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=domainkey-signature:subject:from:reply-to:to:cc:in-reply-to :references:content-type:date:message-id:mime-version:x-mailer :content-transfer-encoding; bh=XykkLfmIU2yUBfwPEX0XastlRZqvYOwUwDPpRCnKKws=; b=PjnmjHjG+zc+VwPZJohbALnQgwIanJp+44R7SJpwFFVPUX9v+abwmf6z44fz4ox3Mq yWKAGIC68JpZpAFdKQ9FMEiFDATu52ZqwqUwEpV2lGQa3YXGL+IU9GXSdSB8iZpFExPu GdXjqOuERPc54GSmGzpaPQblZHzkcMr0iAfSU=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=gamma; h=subject:from:reply-to:to:cc:in-reply-to:references:content-type :date:message-id:mime-version:x-mailer:content-transfer-encoding; b=cnuqJzggZGHVmJie1UsHOGum3lTLRA7SQJwsx/DDYOzlX3kF8FYsGuIcaZFwue2aEg M3o7KlHVNlCdH3DlvAtEV530hdysLE9OYWsxFU0J2SpT/1BL7+qb4kTtGMVL5+UoqB6r KToAn+0bitWCjGTltHnp8gOojE0u4+N984UuU=
In-reply-to: <>
Original-recipient: rfc822;
References: <>
On Tue, 2011-04-05 at 16:19 +0200, John Crispin wrote:

> +/* 
> + * The NOR flash is connected to the same external bus unit (EBU) as PCI.
> + * To make PCI work we need to enable the endianess swapping for the address
> + * written to the EBU. This endianess swapping works for PCI correctly but
> + * fails for attached NOR devices. To workaround this we need to use a 
> complex
> + * map. The workaround involves swapping all addresses whilste probing the 
> chip.
> + * Once probing is complete we stop swapping the addresses but swizzle the
> + * unlock addresses to ensure that access to the NOR device works correctly.
> + */
> +
> +enum ltq_nor_state {
> +};

You do not have to re-send because of this, just a note that in this
case it makes more sense to use anonymous enum. Indeed, you do not need
this 'ltq_nor_state' name at all, and C enums are not proper types
anyway (no real type-checking), so it is just a tiny bit nicer to do:

enum {

But this is not important at all, just a side note :-)

Best Regards,
Artem Bityutskiy (Артём Битюцкий)

<Prev in Thread] Current Thread [Next in Thread>