From: Conor Dooley <conor@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Conor Dooley <conor.dooley@microchip.com>,
Daire McNamara <daire.mcnamara@microchip.com>,
Cyril Jean <cyril.jean@microchip.com>,
Valentina.FernandezAlanis@microchip.com,
linux-riscv@lists.infradead.org, linux-spi@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 0/5] microchip core-qspi gpio-cs fixes + cleanup
Date: Wed, 22 Apr 2026 17:12:36 +0100 [thread overview]
Message-ID: <20260422-refocus-massive-c3eea64ee60c@spud> (raw)
In-Reply-To: <8cb49ccb-5a12-4770-9c91-982d78b5f962@sirena.org.uk>
[-- Attachment #1.1: Type: text/plain, Size: 1258 bytes --]
On Wed, Apr 22, 2026 at 04:25:43PM +0100, Mark Brown wrote:
> On Wed, Apr 22, 2026 at 04:15:41PM +0100, Conor Dooley wrote:
>
> > Got a couple of patches here, the first two deal with an issue that a
> > customer reported where using a mixed configuration of the built-in chip
> > select and a gpio lead to data corruption. Turned out, the built-in chip
> > select always gets activated at the moment even if a gpio-cs is being
> > used. With that fixed, I noticed that any read with multiple data lines
> > was not working properly, so patch two changes how the driver deals with
> > emulating mem-ops. The remaining patches are things I consider fixes but
> > not meaningful enough to get a Fixes tag. LMK if you want em split out
> > to be put on for-next rather than for-current.
>
> I don't know what for-current is but they certainly don't apply against
> for-7.1 which is the only active branch given that v7.0 released (and he
> same content as for-linus and for-next as a result).
Ah I forgot about that widespread devm_ removal series.
> It would be
> convenient to split the fixes and cleanups, unless there's actul
> dependencies.
Sure. I'll rebase, split and resend. There shouldn't be any meaningful
dependencies.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
[-- Attachment #2: Type: text/plain, Size: 161 bytes --]
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
prev parent reply other threads:[~2026-04-22 16:12 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-22 15:15 [PATCH v1 0/5] microchip core-qspi gpio-cs fixes + cleanup Conor Dooley
2026-04-22 15:15 ` [PATCH v1 1/5] spi: microchip-core-qspi: control built-in cs manually Conor Dooley
2026-04-22 15:15 ` [PATCH v1 2/5] spi: microchip-core-qspi: don't attempt to transmit during emulated read-only dual/quad operations Conor Dooley
2026-04-22 15:15 ` [PATCH v1 3/5] spi: microchip-core-qspi: report device on which timeout occured instead of which controller Conor Dooley
2026-04-22 15:15 ` [PATCH v1 4/5] spi: microchip-core-qspi: remove an unused define Conor Dooley
2026-04-22 15:15 ` [PATCH v1 5/5] spi: microchip-core-qspi: remove some inline markings Conor Dooley
2026-04-22 15:25 ` [PATCH v1 0/5] microchip core-qspi gpio-cs fixes + cleanup Mark Brown
2026-04-22 16:12 ` Conor Dooley [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=20260422-refocus-massive-c3eea64ee60c@spud \
--to=conor@kernel.org \
--cc=Valentina.FernandezAlanis@microchip.com \
--cc=broonie@kernel.org \
--cc=conor.dooley@microchip.com \
--cc=cyril.jean@microchip.com \
--cc=daire.mcnamara@microchip.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=linux-spi@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox