Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[York Sun]

On 02/02/2018 04:07 AM, Rajat Srivastava wrote:
> Currently in LS1088A, XIP mode in QSPI works up to 16 MB
> addresses. This patch enables QSPI support in XIP mode for
> addresses above 16 MB as well.
> 
> Signed-off-by: Rajat Srivastava 
> ---

Applied to fsl-qoriq master, awaiting upstream. Thanks

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[Rajat Srivastava]

> -Original Message-
> From: York Sun
> Sent: Tuesday, February 06, 2018 10:38 PM
> To: Rajat Srivastava ; u-boot@lists.denx.de
> Subject: Re: [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses
> 
> On 02/06/2018 02:59 AM, Rajat Srivastava wrote:
> >
> >
> 
> 
> >> How do you put the image into it to begin with? Don't tell me you
> >> were using an external tool or a hacked version of older U-Boot.
> >>
> > This patch enables data read above 16MB using AHB (on Uboot prompt)
> > and does not involve our QSPI driver. It is equivalent to reading from DDR 
> > via
> cp or md commands.
> > Whereas writing to flash is possible via IPS mode which involves our
> > driver (using sf commands).
> >
> OK. So we are still far away to have the full flash supported.
> 
AHB Write is not supported by QSPI hardware.

Rajat
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[York Sun]

On 02/06/2018 02:59 AM, Rajat Srivastava wrote:
> 
> 


>> How do you put the image into it to begin with? Don't tell me you were using 
>> an
>> external tool or a hacked version of older U-Boot.
>>
> This patch enables data read above 16MB using AHB (on Uboot prompt) and does 
> not involve our QSPI driver. It is equivalent to reading from DDR via cp or 
> md commands.
> Whereas writing to flash is possible via IPS mode which involves our driver 
> (using sf 
> commands).
> 

OK. So we are still far away to have the full flash supported.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[Rajat Srivastava]

> -Original Message-
> From: York Sun
> Sent: Monday, February 05, 2018 10:20 PM
> To: Rajat Srivastava ; u-boot@lists.denx.de
> Subject: Re: [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses
> 
> On 02/04/2018 09:37 PM, Rajat Srivastava wrote:
> >
> >
> >> -Original Message-
> >> From: York Sun
> >> Sent: Friday, February 02, 2018 9:29 PM
> >> To: Rajat Srivastava ; u-boot@lists.denx.de
> >> Subject: Re: [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB
> >> addresses
> >>
> >> On 02/02/2018 04:07 AM, Rajat Srivastava wrote:
> >>> Currently in LS1088A, XIP mode in QSPI works up to 16 MB addresses.
> >>> This patch enables QSPI support in XIP mode for addresses above 16
> >>> MB as well.
> >>
> >> Can you write to QSPI above 16MB address?
> >
> > No, I think execute in place (XIP) itself means we can only read.
> > Please suggest if I shall change the patch description.
> >
> 
> How do you put the image into it to begin with? Don't tell me you were using 
> an
> external tool or a hacked version of older U-Boot.
> 
This patch enables data read above 16MB using AHB (on Uboot prompt) and does 
not involve our QSPI driver. It is equivalent to reading from DDR via cp or md 
commands.
Whereas writing to flash is possible via IPS mode which involves our driver 
(using sf 
commands).
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[York Sun]

On 02/04/2018 09:37 PM, Rajat Srivastava wrote:
> 
> 
>> -Original Message-
>> From: York Sun
>> Sent: Friday, February 02, 2018 9:29 PM
>> To: Rajat Srivastava ; u-boot@lists.denx.de
>> Subject: Re: [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses
>>
>> On 02/02/2018 04:07 AM, Rajat Srivastava wrote:
>>> Currently in LS1088A, XIP mode in QSPI works up to 16 MB addresses.
>>> This patch enables QSPI support in XIP mode for addresses above 16 MB
>>> as well.
>>
>> Can you write to QSPI above 16MB address?
>  
> No, I think execute in place (XIP) itself means we can only read.
> Please suggest if I shall change the patch description.
> 

How do you put the image into it to begin with? Don't tell me you were
using an external tool or a hacked version of older U-Boot.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[Rajat Srivastava]

> -Original Message-
> From: York Sun
> Sent: Friday, February 02, 2018 9:29 PM
> To: Rajat Srivastava ; u-boot@lists.denx.de
> Subject: Re: [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses
> 
> On 02/02/2018 04:07 AM, Rajat Srivastava wrote:
> > Currently in LS1088A, XIP mode in QSPI works up to 16 MB addresses.
> > This patch enables QSPI support in XIP mode for addresses above 16 MB
> > as well.
> 
> Can you write to QSPI above 16MB address?
 
No, I think execute in place (XIP) itself means we can only read.
Please suggest if I shall change the patch description.

Rajat
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] ls1088a: qspi: Enable XIP mode above 16 MB addresses

[York Sun]

On 02/02/2018 04:07 AM, Rajat Srivastava wrote:
> Currently in LS1088A, XIP mode in QSPI works up to 16 MB
> addresses. This patch enables QSPI support in XIP mode for
> addresses above 16 MB as well.

Can you write to QSPI above 16MB address?

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot