All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [EXT] Re: [PATCH 5/5] arm64: mvebu: a8k: Add NAND configuration parameters
Date: Wed, 19 Apr 2017 10:55:59 +0200	[thread overview]
Message-ID: <f70afd56-38be-8dcb-59f5-460e0f483895@denx.de> (raw)
In-Reply-To: <d0318323e582416dae25091294e59e4a@IL-EXCH02.marvell.com>

Hi Kosta,

On 19.04.2017 10:50, Kostya Porotchkin wrote:
> Hi, Stefan,
>
>> -----Original Message-----
>> From: Stefan Roese [mailto:sr at denx.de]
>> Sent: Wednesday, April 19, 2017 11:45
>> To: Kostya Porotchkin; u-boot at lists.denx.de
>> Cc: Igal Liberman; Nadav Haklai
>> Subject: [EXT] Re: [PATCH 5/5] arm64: mvebu: a8k: Add NAND configuration
>> parameters
>>
>> External Email
>>
>> ----------------------------------------------------------------------
>> Hi Kosta,
>>
>> On 28.03.2017 17:16, kostap at marvell.com wrote:
>>> From: Konstantin Porotchkin <kostap@marvell.com>
>>>
>>> Add NAND configuration parameters to A8K shared config file.
>>> Add defconfig for db-88f7040 board with boot from NAND setup.
>>>
>>> Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
>>> Cc: Stefan Roese <sr@denx.de>
>>> Cc: Igal Liberman <igall@marvell.com>
>>> Cc: Nadav Haklai <nadavh@marvell.com>
>>> ---
>>>  configs/mvebu_db-88f7040-nand_defconfig | 72
>> +++++++++++++++++++++++++++++++++
>>>  include/configs/mvebu_armada-8k.h       | 11 +++++
>>>  2 files changed, 83 insertions(+)
>>>  create mode 100644 configs/mvebu_db-88f7040-nand_defconfig
>>
>> I'm currently testing this patchset and get this compilation error via
>> Travis:
>>
>>    aarch64:  +   mvebu_db-88f7040-nand
>> +drivers/mtd/nand/pxa3xx_nand.c: In function alloc_nand_resource:
>> +drivers/mtd/nand/pxa3xx_nand.c:1513:36: error: MVEBU_NAND_BASE
>> +undeclared (first use in this function)
>> +  info->mmio_base = (void __iomem *)MVEBU_NAND_BASE;
>>
>> Did I miss a patch that adds this macro for A7/8k?
> [Konstantin Porotchkin]
> The hardcoded values should be changed in the following patch:
> [PATCH 1/5] fix: nand: pxa3xx: Remove hardcode values from the driver
> However I do not see it on patchworks site.
> Should I re-send it?

No need. I've found it in my inbox. Travis build is already restarted...

Thanks,
Stefan

      reply	other threads:[~2017-04-19  8:55 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-28 15:16 [U-Boot] [PATCH 0/5] Add support for NAND flash to A7K/A8K SoC family kostap at marvell.com
2017-03-28 15:16 ` [U-Boot] [PATCH 1/5] fix: nand: pxa3xx: Remove hardcode values from the driver kostap at marvell.com
2017-05-08 12:28   ` Stefan Roese
2017-05-09 14:40   ` Stefan Roese
2017-03-28 15:16 ` [U-Boot] [PATCH 2/5] arm64: mvebu: a8k: Add support for NAND clock get kostap at marvell.com
2017-03-30 13:15   ` Stefan Roese
2017-03-30 15:46     ` Konstantin Porotchkin
2017-04-03  8:51       ` Stefan Roese
2017-03-28 15:16 ` [U-Boot] [PATCH 3/5] arm64: mvebu: Fix the bubt comamnd NAND device support kostap at marvell.com
2017-05-08 12:28   ` Stefan Roese
2017-05-09 14:41   ` Stefan Roese
2017-03-28 15:16 ` [U-Boot] [PATCH 4/5] arm64: a8k: dts: Add support for NAND devices on A8K platform kostap at marvell.com
2017-03-30 13:21   ` Stefan Roese
2017-03-28 15:16 ` [U-Boot] [PATCH 5/5] arm64: mvebu: a8k: Add NAND configuration parameters kostap at marvell.com
2017-04-19  8:44   ` Stefan Roese
2017-04-19  8:50     ` [U-Boot] [EXT] " Kostya Porotchkin
2017-04-19  8:55       ` Stefan Roese [this message]

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=f70afd56-38be-8dcb-59f5-460e0f483895@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.