Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 19 Dec 2022 18:16:10 +0000
From:   Mark Brown <broonie@kernel.org>
To:     Witold Sadowski <wsadowski@marvell.com>
Cc:     linux-spi@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, jpawar@cadence.com,
        pthombar@cadence.com, konrad@cadence.com, wbartczak@marvell.com,
        wzmuda@marvell.com
Subject: Re: [PATCH 5/7] spi: cadence: Add read access size switch
Message-ID: <Y6Cqavt/wT2sQfRU@sirena.org.uk>
References: <20221219144254.20883-1-wsadowski@marvell.com>
 <20221219144254.20883-6-wsadowski@marvell.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="fUY/ItggUwzxJAnk"
Content-Disposition: inline
In-Reply-To: <20221219144254.20883-6-wsadowski@marvell.com>
Precedence: bulk


--fUY/ItggUwzxJAnk
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Mon, Dec 19, 2022 at 06:42:52AM -0800, Witold Sadowski wrote:
> Allow to use different SDMA read size.
> In Marvell implementation of that IP each SDMA
> access will read 8 bytes at once, and is not
> configurable.

If this isn't configurable then shouldn't we just key off the compatible
rather than having a separate property?

--fUY/ItggUwzxJAnk
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmOgqmoACgkQJNaLcl1U
h9DQwwgAgdo84Nf5Ics4LK4oxae6TizIsBnruqnm3WPEvcjnNlSWq1bejox99uH5
3hPpyBH8pxhAJIwkUHukxVTsZmjdd7x7ZeNUWqi0nV+EFbEWZkpPsR2wb4vEuZY0
KtB9mufJFdBmPmseqLGuSObzLtJoh9ERjrY/GtQil77IO5Zd/8NsFFId4PibehAQ
2oK0xezwLv7N9Jd82b/Ai6RmE35sg50Rf0UlP7v7NnNTnbOyvpyTkD7SYzEFeWQu
jO2B3dorzTljzIkAmnvuq5yMkPVBeO85mK60+gNaJvAg4WeuC4R5IgrwqJGLxSmJ
Bwx1geciQym3FAbIb7kJjLQIRKv9nw==
=Sai/
-----END PGP SIGNATURE-----

--fUY/ItggUwzxJAnk--