devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>
To: Vignesh R <vigneshr-l0cyMroinI0@public.gmane.org>,
	"linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Cc: Graham Moore
	<grmoore-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>,
	Alan Tull
	<atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>,
	Brian Norris
	<computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	David Woodhouse <dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	Dinh Nguyen
	<dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>,
	Yves Vandervennet
	<yvanderv-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH V12 2/2] mtd: spi-nor: Add driver for Cadence Quad SPI Flash Controller
Date: Fri, 17 Jun 2016 11:09:27 +0200	[thread overview]
Message-ID: <5763BE47.8040209@denx.de> (raw)
In-Reply-To: <57637FF1.9030802-l0cyMroinI0@public.gmane.org>

On 06/17/2016 06:43 AM, Vignesh R wrote:
> 
> 
> On Thursday 16 June 2016 06:51 PM, Marek Vasut wrote:
>> On 06/16/2016 08:43 AM, Vignesh R wrote:
> [...]
>>>> - I didn't find any way to find when all the data in the current 1 MiB
>>>>   block were written and you can remap another 1 MiB block in place.
>>>
>>> I believe this constraint only applies if enahbremap bit is set in cfg
>>> register, if not, then the entire memory map can be accessed.
>>
>> And where is that memory window accessible then, at which address ?
>> The SoCFPGA peripherals are stuffed in some 12 MiB of the address
>> space, the rest is bootrom/sram/ram and the FPGA bridges, so I find
>> it hard to believe you can place ie. 128 MiB SPI NOR mapping somewhere
>> in there.
>>
>> My impression is that in direct mode, the qspi will always overlay the
>> address 0x0 on socfpga , but that might be configurable , I'm not sure.
>>
> 
> Ah, socfgpa provides just 1MB window to access QSPI_DATA area. So direct
> mode may not be practical as it will require remapping.
> But, on TI EVM, QSPI_DATA area is 64MB in size, so looks like direct
> mode(when added) needs to be tied to TI specific compatible.
> 

Ah, so that explains it. Thanks for looking into it.

Still, keep in mind there are 128 MiB devices (N25Q00AA for example), so
even with the 64 MiB data area, it won't be entirely seamless.

-- 
Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2016-06-17  9:09 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-04  0:39 [PATCH V8 1/2] mtd: spi-nor: Bindings for Cadence Quad SPI Flash Controller driver Marek Vasut
     [not found] ` <1465000774-7762-1-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
2016-06-04  0:39   ` [PATCH V12 2/2] mtd: spi-nor: Add driver for Cadence Quad SPI Flash Controller Marek Vasut
     [not found]     ` <1465000774-7762-2-git-send-email-marex-ynQEQJNshbs@public.gmane.org>
2016-06-14  5:10       ` Vignesh R
2016-06-14 12:59         ` Marek Vasut
     [not found]           ` <575FFF97.7040104-ynQEQJNshbs@public.gmane.org>
2016-06-16  6:43             ` Vignesh R
     [not found]               ` <57624A98.4060308-l0cyMroinI0@public.gmane.org>
2016-06-16 13:21                 ` Marek Vasut
     [not found]                   ` <5762A7E6.8000109-ynQEQJNshbs@public.gmane.org>
2016-06-17  4:43                     ` Vignesh R
     [not found]                       ` <57637FF1.9030802-l0cyMroinI0@public.gmane.org>
2016-06-17  9:09                         ` Marek Vasut [this message]
2016-07-18  0:52       ` Brian Norris
     [not found]         ` <20160718005238.GA80196-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
2016-07-18  9:35           ` Marek Vasut
     [not found]             ` <be1a637c-7d86-df4e-0d47-8516ec639974-ynQEQJNshbs@public.gmane.org>
2016-07-18 16:58               ` Brian Norris
2016-07-18 17:02           ` Brian Norris
2016-06-07 14:00   ` [PATCH V8 1/2] mtd: spi-nor: Bindings for Cadence Quad SPI Flash Controller driver Rob Herring
2016-07-18 17:00     ` Brian Norris

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=5763BE47.8040209@denx.de \
    --to=marex-ynqeqjnshbs@public.gmane.org \
    --cc=atull-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org \
    --cc=computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dinguyen-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org \
    --cc=dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    --cc=grmoore-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org \
    --cc=linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=vigneshr-l0cyMroinI0@public.gmane.org \
    --cc=yvanderv-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.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;
as well as URLs for NNTP newsgroup(s).