Received-SPF: pass (google.com: domain of linux-kernel+bounces-19453-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Mon, 8 Jan 2024 12:46:29 +0100
From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To: Philipp Stanner <pstanner@redhat.com>
Cc: dri-devel@lists.freedesktop.org, Fabio Estevam <festevam@gmail.com>, 
	Daniel Vetter <daniel@ffwll.ch>, "Rafael J. Wysocki" <rafael@kernel.org>, 
	Takashi Iwai <tiwai@suse.de>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, 
	Sascha Hauer <s.hauer@pengutronix.de>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, 
	linux-kernel@vger.kernel.org, Maxime Ripard <mripard@kernel.org>, 
	Mark Brown <broonie@kernel.org>, NXP Linux Team <linux-imx@nxp.com>, 
	Thomas Zimmermann <tzimmermann@suse.de>, Laurentiu Palcu <laurentiu.palcu@oss.nxp.com>, 
	David Gow <davidgow@google.com>, Shawn Guo <shawnguo@kernel.org>, 
	David Airlie <airlied@gmail.com>, Pengutronix Kernel Team <kernel@pengutronix.de>, 
	linux-arm-kernel@lists.infradead.org, Lucas Stach <l.stach@pengutronix.de>
Subject: Re: [PATCH v2 1/2] platform_device: add devres function region-reqs
Message-ID: <ixywpvuwlhdpv6szvssipy2ygjhzdvt6nrbcppy4yx5ix5b3is@pq7s6hpse2ni>
References: <20240108092042.16949-2-pstanner@redhat.com>
 <20240108092042.16949-3-pstanner@redhat.com>
 <hywkbwwwkddbd5vye366bhz64dlpet4chv3kzwfu5dx6rvvix6@2jnk3xx6vfiy>
 <404aea6b7bb7874064153044f04f3b8f6fccb97b.camel@redhat.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="4r5afev3r6p7skdi"
Content-Disposition: inline
In-Reply-To: <404aea6b7bb7874064153044f04f3b8f6fccb97b.camel@redhat.com>


--4r5afev3r6p7skdi
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Jan 08, 2024 at 10:45:31AM +0100, Philipp Stanner wrote:
> On Mon, 2024-01-08 at 10:37 +0100, Uwe Kleine-K=F6nig wrote:
> > Other than that I indifferent if this is a good idea. There are so many
> > helpers around these functions ...
>=20
> Around which, the devres functions in general? There are, but that's
> kind of the point, unless we'd want everyone to call into the lowest
> level region-request functions with their own devres callbacks.
>=20
> In any case: What would your suggestion be, should parties who can't
> (without restructuring very large parts of their code) ioremap() and
> request() simultaneously just not use devres? See my patch #2
> Or is there another way to reach that goal that I'm not aware of?

This wasn't a constructive feedback unfortunately and more a feeling
than a measurable criticism. To actually improve the state, maybe first
check what helpers are actually there, how they are used and if they are
suitable to what they are used for.

Having many helpers is a hint that the usage is complicated. Is that
because the situation is complicated, or is this just a big pile of
inconsistency that can be simplified and consolidated?

Also I think there are helpers that take a resource type parameter (as
your function) and others hard code it in the function name. Maybe
unifying that would be nice, too.

Best regards
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=F6nig            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

--4r5afev3r6p7skdi
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmWb4I8ACgkQj4D7WH0S
/k6x9wf/RNngXMX4s0qPs1qC/g9icFOmmH2DMWt97cJ3/OzyVs7x1qTteymH2ANR
NGt8HE68vKkFyM6QoZz1z/1qAcMZLHWSO44ok7+cYVXfROHiVKe5/13ofkObfyMh
8aUYI/XXd/PXmXrWuxiCyCH8IdWoiBryBqd9DOWrssjqOb/QZyC0y4Mj34gDGWF2
dC/M0HByG8TnH/KPydYys5sHf68xdVTsDhABJtxNo/vDMc0yBdB8dzmb02GjM8Qy
kC6odjtco3TBr5tPAJENW1F2pn7Rm3bMBeLEjwo68oAHTKrqpzkUo/v2rbo11MOf
+NFUHJXT9gfOOY1B6rdzcT2WPmu+TQ==
=38uX
-----END PGP SIGNATURE-----

--4r5afev3r6p7skdi--