From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH v3 02/14] drivers: soc: sunxi: Add dedicated compatibles for the A13, A20 and A33 Date: Fri, 11 May 2018 12:20:33 +0200 Message-ID: <20180511102033.i6jtzljbvkg47wz3@flea> References: <20180507124500.20434-1-paul.kocialkowski@bootlin.com> <20180507124500.20434-3-paul.kocialkowski@bootlin.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ropnmmpiqwyci4ux" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Chen-Yu Tsai Cc: Paul Kocialkowski , Linux Media Mailing List , devicetree , linux-arm-kernel , linux-kernel , Mauro Carvalho Chehab , Rob Herring , Mark Rutland , Greg Kroah-Hartman , "David S . Miller" , Andrew Morton , Linus Walleij , Randy Dunlap , Hans Verkuil , Arnd Bergmann , Stanimir Varbanov , Sakari Ailus Philipp Zabel

List-Id: devicetree@vger.kernel.org --ropnmmpiqwyci4ux Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 10, 2018 at 10:05:33PM -0700, Chen-Yu Tsai wrote: > On Mon, May 7, 2018 at 5:44 AM, Paul Kocialkowski > wrote: > > This introduces platform-specific compatibles for the A13, A20 and A33 > > SRAM driver. No particular adaptation for these platforms is required at > > this point, although this might become the case in the future. > > > > Signed-off-by: Paul Kocialkowski > > --- > > drivers/soc/sunxi/sunxi_sram.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/drivers/soc/sunxi/sunxi_sram.c b/drivers/soc/sunxi/sunxi_s= ram.c > > index 74cb81f37bd6..43ebc3bd33f2 100644 > > --- a/drivers/soc/sunxi/sunxi_sram.c > > +++ b/drivers/soc/sunxi/sunxi_sram.c > > @@ -315,6 +315,9 @@ static int sunxi_sram_probe(struct platform_device = *pdev) > > > > static const struct of_device_id sunxi_sram_dt_match[] =3D { > > { .compatible =3D "allwinner,sun4i-a10-sram-controller" }, > > + { .compatible =3D "allwinner,sun5i-a13-sram-controller" }, > > + { .compatible =3D "allwinner,sun7i-a20-sram-controller" }, > > + { .compatible =3D "allwinner,sun8i-a33-sram-controller" }, >=20 > We should probably name these "system-controller". Maxime? This would make sense yes, but we don't really need to add the A20 one to the driver, it's exactly the same than the A10. Maxime --=20 Maxime Ripard, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com --ropnmmpiqwyci4ux Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEE0VqZU19dR2zEVaqr0rTAlCFNr3QFAlr1bnAACgkQ0rTAlCFN r3RGSxAAioGxVMYYKKP7EpsOvbcLiyJu90l6vMcfmB01xdWSxl+0TlQ713l3g6cU gcFQgm6W5tdJ0i3ps84bGi3H6SqZJhc/S6so/zgS/sI9FNLSq5ztjLw0gAgaaJL5 /pc3Z5HR5hKKWZ3Knho1NG1GYnpTPR2YcN1Wa0GZfrgMgsZkeduW3iJTSC/wOnWF 8+c512DMqND1tq2TRgIAG99pisRh3mHFfKxaTOy7xI5eufZNSqn/an5Doz25wbIZ HoE53kw1+ALSvxoQwHGtfbiyAf4GjK/qQygiFvb3veFYDyIQpq8FOYjjDbaE/LLr KOIGIcHmPoQzo3o8CEupTW98volKFo33WY/Q/+FJn2RXoT8+ZCdObEIBJA3Za8+T wEG5EfuKmw2zUSqiALZlDTz3W7gHYQKlOLmZRNje4TDoeNIOu63D/xTB+2OhRWix bj7FOvRoO5n/ERwMNJgfZmic2xh7mBD2+x7vvYib2+cRguEl6bEqtpP5i4rcCKH3 l5gBKmS0+hr39Qasi5NbiRHK4IVceZ8WgVHLLUuhSQXRoGijZMZygiITE+Ql4wop WR8yhAjwlIReRwkpDe46iU8ep51A0sTIyQ3zfGwv6pv+xLmW+lC7wdXIjKuU19DL EB9+WAZc/J3M92BSw8KZ+SFWL35/orGB5s24sgKegg7GzMI8oR0= =SzVX -----END PGP SIGNATURE----- --ropnmmpiqwyci4ux--