All of lore.kernel.org
 help / color / mirror / Atom feed
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Josh Wu <josh.wu@atmel.com>
Cc: hongxu.cn@gmail.com, jack.lan@freescale.com,
	nick.spence@freescale.com, tie-fei.zang@freescale.com,
	nicolas.ferre@atmel.com, dwmw2@infradead.org,
	linux-mtd@lists.infradead.org, scottwood@freescale.com,
	Dipen.Dudhat@freescale.com, ivan.djelic@parrot.com,
	computersforpeace@gmail.com, plagnioj@jcrosoft.com,
	tglx@linutronix.de, linux-arm-kernel@lists.infradead.org,
	bryan.wu@analog.com
Subject: Re: [PATCH v11 0/4] MTD: at91: Add PMECC support for at91 nand flash driver
Date: Wed, 27 Jun 2012 07:28:40 +0300	[thread overview]
Message-ID: <1340771320.2317.13.camel@koala> (raw)
In-Reply-To: <4FEA87E5.9010604@atmel.com>

[-- Attachment #1: Type: text/plain, Size: 1627 bytes --]

On Wed, 2012-06-27 at 12:11 +0800, Josh Wu wrote:
> Hi, Artem
> 
> On 6/27/2012 11:49 AM, Artem Bityutskiy wrote:
> > On Mon, 2012-06-25 at 18:07 +0800, Josh Wu wrote:
> >> Those patches is based on v3.5-rc4
> >>
> >> Changes since v10,
> >> 	add one more patch in this patch set, which add 'int' return value for write_page()/write_page_raw() functions in structure of nand_ecc_ctrl.
> >> 	Instead of calling BUG(), atmel_nand_pmecc_write_page() will return -EIO when time out to read the pmecc status register.
> >> 	increase the time-out duration to 100ms, which has more toleration.
> >> 	add oob_required argument for pmecc read/write functions to align with v3.5-rc4.
> > I have issues compiling l2-mtd.git tree (based on 3.5-rc5) with the
> > attached defconfig. Probably not your fault, but may be you have a
> > fix/suggestion? I did not have time to look closer. Aiaiai reports:
> 
> I think the name of kernel config for AT91 is changed, so you can try 
> add following two line in your original config file.
> 
> CONFIG_SOC_AT91SAM9=y
> CONFIG_SOC_AT91RM9200=y
> 
> run the make menuconfig will get new config, that can pass the compile.

I get "arm-unknown-linux-gnueabi-ld: no machine record defined". I think
it wants me to select the board. I've tried one and get another
compilation issue:

arch/arm/mach-at91/built-in.o:(.arch.info.init+0xbc): undefined
reference to `at91sam926x_timer'

It looks like AT91 stuff is not looked after carefully.

Can you please send me a working defconfig based on the one I sent you -
just modify it.

-- 
Best Regards,
Artem Bityutskiy

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: dedekind1@gmail.com (Artem Bityutskiy)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v11 0/4] MTD: at91: Add PMECC support for at91 nand flash driver
Date: Wed, 27 Jun 2012 07:28:40 +0300	[thread overview]
Message-ID: <1340771320.2317.13.camel@koala> (raw)
In-Reply-To: <4FEA87E5.9010604@atmel.com>

On Wed, 2012-06-27 at 12:11 +0800, Josh Wu wrote:
> Hi, Artem
> 
> On 6/27/2012 11:49 AM, Artem Bityutskiy wrote:
> > On Mon, 2012-06-25 at 18:07 +0800, Josh Wu wrote:
> >> Those patches is based on v3.5-rc4
> >>
> >> Changes since v10,
> >> 	add one more patch in this patch set, which add 'int' return value for write_page()/write_page_raw() functions in structure of nand_ecc_ctrl.
> >> 	Instead of calling BUG(), atmel_nand_pmecc_write_page() will return -EIO when time out to read the pmecc status register.
> >> 	increase the time-out duration to 100ms, which has more toleration.
> >> 	add oob_required argument for pmecc read/write functions to align with v3.5-rc4.
> > I have issues compiling l2-mtd.git tree (based on 3.5-rc5) with the
> > attached defconfig. Probably not your fault, but may be you have a
> > fix/suggestion? I did not have time to look closer. Aiaiai reports:
> 
> I think the name of kernel config for AT91 is changed, so you can try 
> add following two line in your original config file.
> 
> CONFIG_SOC_AT91SAM9=y
> CONFIG_SOC_AT91RM9200=y
> 
> run the make menuconfig will get new config, that can pass the compile.

I get "arm-unknown-linux-gnueabi-ld: no machine record defined". I think
it wants me to select the board. I've tried one and get another
compilation issue:

arch/arm/mach-at91/built-in.o:(.arch.info.init+0xbc): undefined
reference to `at91sam926x_timer'

It looks like AT91 stuff is not looked after carefully.

Can you please send me a working defconfig based on the one I sent you -
just modify it.

-- 
Best Regards,
Artem Bityutskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120627/eba33f8d/attachment-0001.sig>

  reply	other threads:[~2012-06-27  4:28 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-25 10:07 [PATCH v11 0/4] MTD: at91: Add PMECC support for at91 nand flash driver Josh Wu
2012-06-25 10:07 ` Josh Wu
2012-06-25 10:07 ` [PATCH v11 1/4] MTD: at91: extract hw ecc initialization to one function Josh Wu
2012-06-25 10:07   ` Josh Wu
2012-06-27  9:26   ` Artem Bityutskiy
2012-06-27  9:26     ` Artem Bityutskiy
2012-06-25 10:07 ` [PATCH v11 2/4] MTD: at91: add dt parameters for Atmel PMECC Josh Wu
2012-06-25 10:07   ` Josh Wu
2012-06-25 10:35   ` Richard Genoud
2012-06-25 10:35     ` Richard Genoud
2012-06-26  6:05     ` Josh Wu
2012-06-26  6:05       ` Josh Wu
2012-06-25 10:07 ` [PATCH v11 3/4] MTD: nand: add return value for write_page()/write_page_raw() functions in structure of nand_ecc_ctrl Josh Wu
2012-06-25 10:07   ` Josh Wu
2012-06-27  9:26   ` Artem Bityutskiy
2012-06-27  9:26     ` Artem Bityutskiy
2012-06-25 10:07 ` [PATCH v11 4/4] MTD: at91: atmel_nand: Update driver to support Programmable Multibit ECC controller Josh Wu
2012-06-25 10:07   ` Josh Wu
2012-06-25 10:27   ` Richard Genoud
2012-06-25 10:27     ` Richard Genoud
2012-06-26  6:03     ` Josh Wu
2012-06-26  6:03       ` Josh Wu
2012-06-26 14:15   ` Richard Genoud
2012-06-26 14:15     ` Richard Genoud
2012-06-26 14:37     ` [PATCH v11 4/4] MTD: at91: atmel_nand: Update driver to support Programmable Multibit ECC controller (REPORT SPAM) William F.
2012-06-26 14:37       ` William F.
2012-06-27 11:22     ` [PATCH v11 4/4] MTD: at91: atmel_nand: Update driver to support Programmable Multibit ECC controller Josh Wu
2012-06-27 11:22       ` Josh Wu
2012-06-27  3:45 ` [PATCH v11 0/4] MTD: at91: Add PMECC support for at91 nand flash driver Artem Bityutskiy
2012-06-27  3:45   ` Artem Bityutskiy
2012-06-27  3:49 ` Artem Bityutskiy
2012-06-27  3:49   ` Artem Bityutskiy
2012-06-27  4:11   ` Josh Wu
2012-06-27  4:11     ` Josh Wu
2012-06-27  4:28     ` Artem Bityutskiy [this message]
2012-06-27  4:28       ` Artem Bityutskiy
2012-06-27  5:34       ` Josh Wu
2012-06-27  5:34         ` Josh Wu
2012-06-27  8:39       ` Nicolas Ferre
2012-06-27  8:39         ` Nicolas Ferre
2012-06-27  9:14         ` Artem Bityutskiy
2012-06-27  9:14           ` Artem Bityutskiy
2012-06-27  9:28 ` Artem Bityutskiy
2012-06-27  9:28   ` Artem Bityutskiy

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=1340771320.2317.13.camel@koala \
    --to=dedekind1@gmail.com \
    --cc=Dipen.Dudhat@freescale.com \
    --cc=bryan.wu@analog.com \
    --cc=computersforpeace@gmail.com \
    --cc=dwmw2@infradead.org \
    --cc=hongxu.cn@gmail.com \
    --cc=ivan.djelic@parrot.com \
    --cc=jack.lan@freescale.com \
    --cc=josh.wu@atmel.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=nick.spence@freescale.com \
    --cc=nicolas.ferre@atmel.com \
    --cc=plagnioj@jcrosoft.com \
    --cc=scottwood@freescale.com \
    --cc=tglx@linutronix.de \
    --cc=tie-fei.zang@freescale.com \
    /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.