From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH v2 04/11] drm: sun4i: add support for H3's TCON0/1 Date: Wed, 7 Jun 2017 16:19:57 +0200 Message-ID: <20170607141957.akonr7hmuwpt7hlt@flea.lan> References: <20170604160149.30230-1-icenowy@aosc.io> <20170604160149.30230-5-icenowy@aosc.io> <3642057.oKT9kIlu8M@jernej-laptop> <493AC6F2-3CB3-48F5-9568-A34AA927238F@aosc.io> <20170607094343.qqiwnu2mhmoi6jn7@flea.lan> <52F9F9C7-9A9C-435D-A0AF-FDD4317DAD69@aosc.io> Reply-To: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="kthojeegoisboeih" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <52F9F9C7-9A9C-435D-A0AF-FDD4317DAD69-h8G6r0blFSE@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Icenowy Zheng Cc: Jernej =?utf-8?Q?=C5=A0krabec?= , Rob Herring , Chen-Yu Tsai , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Icenowy Zheng List-Id: devicetree@vger.kernel.org --kthojeegoisboeih Content-Type: text/plain; charset="UTF-8" Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 07, 2017 at 05:44:56PM +0800, Icenowy Zheng wrote: > =E4=BA=8E 2017=E5=B9=B46=E6=9C=887=E6=97=A5 GMT+08:00 =E4=B8=8B=E5=8D=885= :43:43, Maxime Ripard =E5=86=99=E5=88=B0= : > >On Mon, Jun 05, 2017 at 03:03:47AM +0800, Icenowy Zheng wrote: > >> >You should also expand function sun4i_drv_node_is_tcon() at > >sun4i_drv.c > >> >with=20 > >> >new entries, but I'm not sure if this fits in this patch. > >>=20 > >> Instead I think it should be renamed to something like > >> "sun4i_drv_node_is_tcon_with_ch0". > > > >I'm not sure, or at least, it shouldn't make any difference, since > >TCON without a channel 0 will not have an endpoint 0, so this will be > >dealt with already. >=20 > But that will prevent new coders from add CH1-less TCON > compatibles to this function. Why? We already have such TCONs (like the A33's, or V3S') in that function. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout. --kthojeegoisboeih Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBAgAGBQJZOAuNAAoJEBx+YmzsjxAgSo0P/0hoSsKFQvbaW7yFKg12VPkj bUgzJcndyoL13J3BBZsMa8jHvWvlGYBP5VwTfLLxlsGxxauaDNOmvgh/v2hAckoh 8fmD7piZeb5rc0nPdnhCgTLYuZ2w4w+7xRtLG9Ey5LYAECfClFUaDWbDShypRJLB VmzBF+5MTlY9VvSU/B/ZsZPGWeVyVNBz1ykJav9cBEMf503biyEDLli9/s4kH/xd +ev0hE0CdP5U/AzeCofbvmf0wxt5C91nBNgNBx7fdzZcwmwzNrimU1ze0bkBkPgu 2/CToSI7//jTkxBGwb1jYaBzt93ewohfGbLxd4TDWUlYFLAQoKKbjbQcEJjouwqr +6p812H0fKkIjmNy9t6f0mBepHYUewbhGhxlFNufX24u28rTfCUTSEK/6fh5iFxE gEIl4F+C8VoqgTcLVkzdC2/AKwAqDjOquO63Nt/CwBp7HVLB5XDBKolmYLk2ZZWo kIa1dcrxLmxGynO/o01IekgIlwr2CjWg0Lq+wLlV0vr61blFQeSZDiHYSP3eIK1V D9KuGI19FpCH+msiXWheJL8rejIUjcKWUznFYa6zd13lObYfntyRBGbU9qqfsrIa fnXQpnUJacLfju4Lfz7Iw1kGzILetJ1bB3cfv3MMtXVSh/UA+jYdmrAHrz9OVB2l TGatpdCwrIuo4RxydG4X =df3h -----END PGP SIGNATURE----- --kthojeegoisboeih--