public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: "M8 (Servaes Joordens)" <Servaes.Joordens@m8.nl>
To: David Woodhouse <dwmw2@infradead.org>,
	mtd-forum <linux-mtd@lists.infradead.org>
Subject: Re: address lines
Date: Sat, 03 Nov 2007 13:07:15 +0100	[thread overview]
Message-ID: <472C6473.8020502@m8.nl> (raw)
In-Reply-To: <1194090374.9405.42.camel@pmac.infradead.org>

David,

Yes you did. I did the following:

- I changed the call to simple_map_init(xxx) in physmap.c to
my_simple_map_init(xxx)
- I add the bank-switching routines in physmap.c and link them via
my_simple_map_init(xxx)

I put an printk in every bank-switching routine. I does work properly in the begining, but then I get the following prints: (I added a printk in every function to check the caling order of things)

cfi_cmdset_001.c: cfi_intelext_point
cfi_cmdset_001.c: do_point_onechip
cfi_cmdset_001.c: get_chip
jffs2_scan_eraseblock(): Magic bitmask 0x1985 not found at 0x00000000: 0x890a in
stead

I expected that prior to error-report a read or write action to the flash is done. In that case I should have seen an printk saying physmap.c:bank_switching_map_read or physmap.c:bank_switching_map_write.

Servaes




David Woodhouse wrote:
> On Sat, 2007-11-03 at 11:54 +0100, M8 (Servaes Joordens) wrote:
>   
>> I took a closer look at the bank-switching problem. The situation is:
>>
>> - I have an 4Mb Flash chip MT28F320J3FS-11
>> - A0-A19 are connected to the data-bus
>> - A20 is connected to GPIO PF4
>>
>> So the hardware probe detects 4MB flash in 1 chip and tries to access
>> it 
>> in a normal manner. How should I solve this? I got a patch from the 
>> manufacturer of the hardware for the file cfi_cmdset_0001.c, but I do 
>> wonder if that is the correct way to do it?
>>     
>
> No, it isn't. The correct way to do it is a fairly trivial 'complex
> mapping' driver, which ensures that the GPIO is set appropriately for
> the high bit of the address, before each access. Didn't I already point
> you to examples?
>
>   


-- 

Servaes Joordens

-------------------------------------------
M8
Tappersweg 29
2031ET	Haarlem
The Netherlands
Tel:	+31 23 5311122
Mob:	+31 6 51183379
Fax:	+31 23 5322388

VAT-ID: nl-813029090B01
kvk:	34131430 Haarlem

  reply	other threads:[~2007-11-03 12:07 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-03 10:54 address lines M8 (Servaes Joordens)
2007-11-03 11:46 ` David Woodhouse
2007-11-03 12:07   ` M8 (Servaes Joordens) [this message]
2007-11-03 12:13     ` David Woodhouse
2007-11-03 12:27       ` M8 (Servaes Joordens)
2007-11-03 12:29         ` David Woodhouse
2007-11-03 13:30           ` M8 (Servaes Joordens)
2007-11-03 14:35           ` M8 (Servaes Joordens)
2007-11-04  9:30           ` M8 (Servaes Joordens)
2007-11-04 11:23             ` David Woodhouse

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=472C6473.8020502@m8.nl \
    --to=servaes.joordens@m8.nl \
    --cc=dwmw2@infradead.org \
    --cc=linux-mtd@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox