From: Michael Walle <michael@walle.cc>
To: Biju Das <biju.das.jz@bp.renesas.com>
Cc: Mark Brown <broonie@kernel.org>,
Miquel Raynal <miquel.raynal@bootlin.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
linux-spi@vger.kernel.org, linux-mtd@lists.infradead.org,
Geert Uytterhoeven <geert+renesas@glider.be>,
Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>,
"biju.das.au" <biju.das.au@gmail.com>,
linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH RFC 0/4] Add set_iofv() callback
Date: Mon, 13 Nov 2023 15:47:40 +0100 [thread overview]
Message-ID: <e8ed909b39b5a7de46a07d2b57c0393a@walle.cc> (raw)
In-Reply-To: <4c6674a6717152fc1a244d123a2db896@walle.cc>
> Maybe
> it will already drive IO3 high by default?! (and esp. what the bootROM
> is doing
> if that SoC is able to load the first stage bootloader from NOR).
I just had another look at your Renesas SoC and indeed, the register
default is
to set IO3 high if not used. Mh. I still think 3,3,3,3 is the saner
default.
But I might be wrong. Hard to tell, as the sample size is just Micron
and Atmel
for now. And it's still unclear to me why the Atmel isn't working with
3,3,3,1.
-michael
WARNING: multiple messages have this Message-ID (diff)
From: Michael Walle <michael@walle.cc>
To: Biju Das <biju.das.jz@bp.renesas.com>
Cc: Mark Brown <broonie@kernel.org>,
Miquel Raynal <miquel.raynal@bootlin.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
linux-spi@vger.kernel.org, linux-mtd@lists.infradead.org,
Geert Uytterhoeven <geert+renesas@glider.be>,
Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>,
"biju.das.au" <biju.das.au@gmail.com>,
linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH RFC 0/4] Add set_iofv() callback
Date: Mon, 13 Nov 2023 15:47:40 +0100 [thread overview]
Message-ID: <e8ed909b39b5a7de46a07d2b57c0393a@walle.cc> (raw)
In-Reply-To: <4c6674a6717152fc1a244d123a2db896@walle.cc>
> Maybe
> it will already drive IO3 high by default?! (and esp. what the bootROM
> is doing
> if that SoC is able to load the first stage bootloader from NOR).
I just had another look at your Renesas SoC and indeed, the register
default is
to set IO3 high if not used. Mh. I still think 3,3,3,3 is the saner
default.
But I might be wrong. Hard to tell, as the sample size is just Micron
and Atmel
for now. And it's still unclear to me why the Atmel isn't working with
3,3,3,1.
-michael
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2023-11-13 14:47 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-08 17:11 [PATCH RFC 0/4] Add set_iofv() callback Biju Das
2023-11-08 17:11 ` Biju Das
2023-11-08 17:11 ` [PATCH RFC 1/4] spi: spi-mem: " Biju Das
2023-11-08 17:11 ` Biju Das
2023-11-09 7:56 ` Geert Uytterhoeven
2023-11-09 7:56 ` Geert Uytterhoeven
2023-11-09 8:28 ` Biju Das
2023-11-08 17:11 ` [PATCH RFC 2/4] mtd: spi-nor: Add post_sfdp() callback Biju Das
2023-11-08 17:11 ` Biju Das
2023-11-08 17:11 ` [PATCH RFC 3/4] memory: renesas-rpc-if: Add support for overriding IO fixed values Biju Das
2023-11-21 9:08 ` Krzysztof Kozlowski
2023-11-08 17:11 ` [PATCH RFC 4/4] spi: rpc-if: Add set_iofv() callback Biju Das
2023-11-09 9:01 ` [PATCH RFC 0/4] " Michael Walle
2023-11-09 9:01 ` Michael Walle
2023-11-09 10:04 ` Biju Das
2023-11-09 10:48 ` Michael Walle
2023-11-09 10:48 ` Michael Walle
2023-11-09 11:48 ` Biju Das
2023-11-09 12:40 ` Michael Walle
2023-11-09 12:40 ` Michael Walle
2023-11-09 18:02 ` Biju Das
2023-11-10 10:11 ` Michael Walle
2023-11-10 10:11 ` Michael Walle
2023-11-10 11:35 ` Biju Das
2023-11-11 12:26 ` Biju Das
2023-11-11 13:08 ` Biju Das
2023-11-13 14:04 ` Michael Walle
2023-11-13 14:04 ` Michael Walle
2023-11-13 14:27 ` Biju Das
2023-11-13 14:48 ` Michael Walle
2023-11-13 14:48 ` Michael Walle
2023-11-13 14:59 ` Biju Das
2023-11-13 15:10 ` Michael Walle
2023-11-13 15:10 ` Michael Walle
2023-11-13 15:55 ` Biju Das
2023-11-14 10:05 ` Michael Walle
2023-11-14 10:05 ` Michael Walle
2023-11-12 20:24 ` Biju Das
2023-11-13 14:37 ` Michael Walle
2023-11-13 14:37 ` Michael Walle
2023-11-13 14:47 ` Michael Walle [this message]
2023-11-13 14:47 ` Michael Walle
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=e8ed909b39b5a7de46a07d2b57c0393a@walle.cc \
--to=michael@walle.cc \
--cc=biju.das.au@gmail.com \
--cc=biju.das.jz@bp.renesas.com \
--cc=broonie@kernel.org \
--cc=geert+renesas@glider.be \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=miquel.raynal@bootlin.com \
--cc=prabhakar.mahadev-lad.rj@bp.renesas.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.