All of lore.kernel.org
 help / color / mirror / Atom feed
From: John Wang <wangzq.jn@gmail.com>
To: "Cédric Le Goater" <clg@kaod.org>
Cc: OpenBMC Maillist <openbmc@lists.ozlabs.org>,
	Lei Yu <yulei.sh@bytedance.com>,
	Ryan Chen <ryan_chen@aspeedtech.com>,
	Joel Stanley <joel@jms.id.au>
Subject: Re: Call for testing: spi-mem driver for Aspeed SMC controllers
Date: Wed, 2 Mar 2022 10:47:16 +0800	[thread overview]
Message-ID: <CAHwNHZWFZj+Bg6VLKpdp0PuwO2-gR46vzs28dm0HZmCEyTNuyA@mail.gmail.com> (raw)
In-Reply-To: <b99ecbd2-caa4-c7b3-518c-45477e062d53@kaod.org>

Cédric Le Goater <clg@kaod.org> 于2022年3月1日周二 18:25写道:
>
> On 3/1/22 10:57, John Wang wrote:
> > Joel Stanley <joel@jms.id.au> 于2022年2月25日周五 13:30写道:
> >>
> >> Cedric has authored a new drive for the SPI NOR devices on the ASPEED
> >> SoCs. It supports 2400, 2500 and 2600. It is written using the spi-mem
> >> subsystem, which should mean it can go upstream.
> >>
> >> This is great news, as our current driver is half upstream, half in
> >> the openbmc tree, due to some issues getting it merged as part of the
> >> spi-nor subsystem.
> >>
> >> It would be great to have testing. I've created a commit that patches
> >> in support to whatever kernel tree you're using. Hopefully this makes
> >> it easier to test atop your internal branches:
> >>
> >> https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/51551
> >
> > tested on a fp5280g2 (ast2500), it looks okay.
>
> Thanks !
>
> > root@fp5280g2:~# dmesg |grep spi
> > [    1.594677] spi-nor spi0.0: mx66l51235f (65536 Kbytes)
> > [    1.728194] spi-aspeed-smc 1e620000.spi: CE0 read buswidth:2 [0x203c0641]
> > [    1.992733] spi-nor spi1.0: mx66l51235f (65536 Kbytes)
> > [    2.211741] spi-aspeed-smc 1e630000.spi: CE0 read buswidth:2 [0x203c0741]
>
> Training was done on both controllers. It's a little faster on SPI1 because
> the clock is set at KCLK/2 whereas FMC uses HCLK/4. Dual IO 4B read command
> is the default setting for both.
>
> I test writing with :
>
>      dd if=/dev/random of=$tmp bs=1M count=$count
>      md5sum $tmp
>
>      time flashcp -v $tmp $mtd  # ~3min for 32MB
>      time md5sum $mtd # ~3s for 32MB
>
> but not on /dev/mtd0 !
>

test writing on a 2500 :)

root@fp5280g2:/tmp# time dd if=/dev/random of=/tmp/test bs=1M count=32
32+0 records in
32+0 records out

real 0m3.144s
user 0m0.002s
sys 0m2.278s
root@fp5280g2:/tmp# md5sum /tmp/test
bed6912a0b257cb7acb79f1c7f99cf6e  /tmp/test

root@fp5280g2:/tmp# time flashcp -v /tmp/test /dev/mtd4  # 32M
Erasing block: 512/512 (100%)
Writing kb: 32768/32768 (100%)
Verifying kb: 32768/32768 (100%)

real 4m12.713s
user 0m1.878s
sys 2m42.690s
root@fp5280g2:/tmp# time md5sum /dev/mtd4
bed6912a0b257cb7acb79f1c7f99cf6e  /dev/mtd4

real 0m7.668s
user 0m0.832s
sys 0m3.524s

  reply	other threads:[~2022-03-02  2:48 UTC|newest]

Thread overview: 115+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-14  9:42 [PATCH 00/10] spi: spi-mem: Add driver for Aspeed SMC controllers Cédric Le Goater
2022-02-14  9:42 ` Cédric Le Goater
2022-02-14  9:42 ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 01/10] mtd: spi-nor: aspeed: Rename Kconfig option Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-25  7:31   ` Pratyush Yadav
2022-02-25  7:31     ` Pratyush Yadav
2022-02-25  7:31     ` Pratyush Yadav
2022-02-27 18:50     ` Cédric Le Goater
2022-02-27 18:50       ` Cédric Le Goater
2022-02-27 18:50       ` Cédric Le Goater
2022-02-28  6:07       ` Joel Stanley
2022-02-28  6:07         ` Joel Stanley
2022-02-28  6:07         ` Joel Stanley
2022-02-28 14:37         ` Cédric Le Goater
2022-02-28 14:37           ` Cédric Le Goater
2022-02-28 14:37           ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 02/10] dt-bindings: spi: Add Aspeed SMC controllers device tree binding Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-15 19:49   ` Rob Herring
2022-02-15 19:49     ` Rob Herring
2022-02-15 19:49     ` Rob Herring
2022-02-15 21:06   ` Rob Herring
2022-02-15 21:06     ` Rob Herring
2022-02-15 21:06     ` Rob Herring
2022-02-17  8:37     ` Cédric Le Goater
2022-02-17  8:37       ` Cédric Le Goater
2022-02-17  8:37       ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 03/10] spi: spi-mem: Add driver for Aspeed SMC controllers Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-15  6:27   ` Lukas Wunner
2022-02-15  9:07     ` Cédric Le Goater
2022-02-15  9:07       ` Cédric Le Goater
2022-02-15  9:07       ` Cédric Le Goater
2022-02-25  7:50   ` Pratyush Yadav
2022-02-25  7:50     ` Pratyush Yadav
2022-02-25  7:50     ` Pratyush Yadav
2022-02-27 18:46     ` Cédric Le Goater
2022-02-27 18:46       ` Cédric Le Goater
2022-02-27 18:46       ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 04/10] spi: aspeed: Add support for direct mapping Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-25  9:12   ` Pratyush Yadav
2022-02-25  9:12     ` Pratyush Yadav
2022-02-25  9:12     ` Pratyush Yadav
2022-02-27 21:06     ` Cédric Le Goater
2022-02-27 21:06       ` Cédric Le Goater
2022-02-27 21:06       ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 05/10] spi: aspeed: Adjust direct mapping to device size Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 06/10] spi: aspeed: Workaround AST2500 limitations Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 07/10] spi: aspeed: Add support for the AST2400 SPI controller Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 08/10] spi: aspeed: Calibrate read timings Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-25  9:18   ` Pratyush Yadav
2022-02-25  9:18     ` Pratyush Yadav
2022-02-25  9:18     ` Pratyush Yadav
2022-02-27 21:27     ` Cédric Le Goater
2022-02-27 21:27       ` Cédric Le Goater
2022-02-27 21:27       ` Cédric Le Goater
2022-02-14  9:42 ` [PATCH 09/10] ARM: dts: aspeed: Enable Dual SPI RX transfers Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-16  7:21   ` Joel Stanley
2022-02-16  7:21     ` Joel Stanley
2022-02-16  7:21     ` Joel Stanley
2022-02-14  9:42 ` [PATCH 10/10] spi: aspeed: Activate new spi-mem driver Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-14  9:42   ` Cédric Le Goater
2022-02-16  7:02   ` Joel Stanley
2022-02-16  7:02     ` Joel Stanley
2022-02-16  7:02     ` Joel Stanley
2022-02-16  8:12     ` Cédric Le Goater
2022-02-16  8:12       ` Cédric Le Goater
2022-02-16  8:12       ` Cédric Le Goater
2022-02-16  7:21 ` [PATCH 00/10] spi: spi-mem: Add driver for Aspeed SMC controllers Joel Stanley
2022-02-16  7:21   ` Joel Stanley
2022-02-16  7:21   ` Joel Stanley
2022-02-25  5:29 ` Call for testing: spi-mem " Joel Stanley
2022-02-27 18:53   ` Cédric Le Goater
2022-02-28  6:30     ` Tao Ren
2022-03-02  6:05     ` Tao Ren
2022-03-02  6:23       ` Cédric Le Goater
2022-03-02  6:47         ` Tao Ren
2022-03-03  6:03         ` Tao Ren
2022-03-03  7:02           ` Cédric Le Goater
2022-03-04  7:53             ` Tao Ren
2022-03-01  9:57   ` John Wang
2022-03-01 10:25     ` Cédric Le Goater
2022-03-02  2:47       ` John Wang [this message]
2022-03-01 12:20     ` Joel Stanley
2022-03-02  2:28       ` [External] " Lei Yu
2022-03-07  9:41         ` Lei Yu
2022-03-07 13:06           ` Cédric Le Goater
2022-03-08  1:53             ` Lei Yu
2022-03-08  9:30               ` Cédric Le Goater
2022-03-08  9:47                 ` Lei Yu
2022-03-08 14:57                   ` Cédric Le Goater
2022-03-09  1:58                     ` Lei Yu
2022-03-09  7:40                       ` Cédric Le Goater
2022-03-09  7:50                         ` Cédric Le Goater
2022-03-02  5:45       ` Cédric Le Goater
2022-03-03  9:45         ` Joel Stanley
2022-03-03  9:51           ` Cédric Le Goater
2022-03-03  9:57             ` Joel Stanley

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=CAHwNHZWFZj+Bg6VLKpdp0PuwO2-gR46vzs28dm0HZmCEyTNuyA@mail.gmail.com \
    --to=wangzq.jn@gmail.com \
    --cc=clg@kaod.org \
    --cc=joel@jms.id.au \
    --cc=openbmc@lists.ozlabs.org \
    --cc=ryan_chen@aspeedtech.com \
    --cc=yulei.sh@bytedance.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.