From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [PATCH v6 1/3] drm/vc4: Move the DSI clock divider workaround closer to the clock call. Date: Thu, 28 Sep 2017 10:20:08 -0700 Message-ID: <871smqhh9z.fsf@anholt.net> References: <20170927193654.12609-1-eric@anholt.net> <20170927193654.12609-2-eric@anholt.net> <20170928125449.GA28022@ulmo> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0801081354==" Return-path: Received: from anholt.net (anholt.net [50.246.234.109]) by gabe.freedesktop.org (Postfix) with ESMTP id A53BD6E9B1 for ; Thu, 28 Sep 2017 17:20:10 +0000 (UTC) In-Reply-To: <20170928125449.GA28022@ulmo> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Thierry Reding Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org --===============0801081354== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Thierry Reding writes: > [ Unknown signature status ] > On Wed, Sep 27, 2017 at 12:36:52PM -0700, Eric Anholt wrote: >> We want the adjusted_mode->clock to be the actual clock we're >> expecting to program, so that consumers see the right values for clock >> and vrefresh. >>=20 >> Signed-off-by: Eric Anholt >> --- >> drivers/gpu/drm/vc4/vc4_dsi.c | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) > > Do you want me to apply this to drm-misc-next along with the touchscreen > panel driver? drm-misc-next procedure trends toward the submitter applies the patch once they get review, if they have the commit rights. However, I'm happy either way as long as I can get the code in. --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAlnNL0gACgkQtdYpNtH8 nujZmw//bFqo2BuXFEeQdNlr0DWFrkZ2yl6968tCU88m/9ImQxJlRNaVAEyyIWfi U7FzQ+SDcQFZbpZdwLhMVB3h6JN4ydB053yS61zdbdN4anam1RUnj6jBb/s2qEVZ WQn2DFoFraYx3qEPfNTY1ybZCgCma0s/G7YfNWUdx4k6qEnhGVweSj0VQlQC/1rk GKl6IqlpDTMQ3cz8C6L5Qnh/8e1cbOvqAvOsbg3QXUfSjXtYMUWgK4fYh/qFxjmq Q+UQAPaXl4vZXOudvSqFwg9TqmyKiszZPfjT9ovkmbl0X+akdYYOXqbpLpHX4Dp1 m7Cc+aco+tv12CjaJ8zFPH619lzbOsJN8VR5nQl9/fXTS3UlnqMaFZfQDayB/CP4 Q87o34I4a97OZ33duE+2vYPqjyrZA8KMERU3bcx+UQwooS8QjX7o4NleYHTrDBSW f3V3AJLFZGsz3KzKBpibKG8k0tIoIphg47vwgYH7leGTdVdQ2gh15BGaq9s2R503 cNhydYqW0vZSito8j3kwpggBLD1VMjzzXvchugHKDPNKvULxhI2L4YlyGa8C0Vhx KLIBQ9YSNBQxM6YylK7UWH2x9C7MkGYWfHs478ark1d3F094KlnqoBdYh03TkY1U RWTxlnYOWu96AIQPxF0R30HcPTawj+gLYUMIoyVXnVpHor26nkI= =ZxC9 -----END PGP SIGNATURE----- --=-=-=-- --===============0801081354== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0801081354==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752093AbdI1RUM (ORCPT ); Thu, 28 Sep 2017 13:20:12 -0400 Received: from anholt.net ([50.246.234.109]:55726 "EHLO anholt.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751339AbdI1RUL (ORCPT ); Thu, 28 Sep 2017 13:20:11 -0400 From: Eric Anholt To: Thierry Reding Cc: dri-devel@lists.freedesktop.org, Archit Taneja , Andrzej Hajda , Laurent Pinchart , linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 1/3] drm/vc4: Move the DSI clock divider workaround closer to the clock call. In-Reply-To: <20170928125449.GA28022@ulmo> References: <20170927193654.12609-1-eric@anholt.net> <20170927193654.12609-2-eric@anholt.net> <20170928125449.GA28022@ulmo> User-Agent: Notmuch/0.22.2+1~gb0bcfaa (http://notmuchmail.org) Emacs/25.2.2 (x86_64-pc-linux-gnu) Date: Thu, 28 Sep 2017 10:20:08 -0700 Message-ID: <871smqhh9z.fsf@anholt.net> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Thierry Reding writes: > [ Unknown signature status ] > On Wed, Sep 27, 2017 at 12:36:52PM -0700, Eric Anholt wrote: >> We want the adjusted_mode->clock to be the actual clock we're >> expecting to program, so that consumers see the right values for clock >> and vrefresh. >>=20 >> Signed-off-by: Eric Anholt >> --- >> drivers/gpu/drm/vc4/vc4_dsi.c | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) > > Do you want me to apply this to drm-misc-next along with the touchscreen > panel driver? drm-misc-next procedure trends toward the submitter applies the patch once they get review, if they have the commit rights. However, I'm happy either way as long as I can get the code in. --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAlnNL0gACgkQtdYpNtH8 nujZmw//bFqo2BuXFEeQdNlr0DWFrkZ2yl6968tCU88m/9ImQxJlRNaVAEyyIWfi U7FzQ+SDcQFZbpZdwLhMVB3h6JN4ydB053yS61zdbdN4anam1RUnj6jBb/s2qEVZ WQn2DFoFraYx3qEPfNTY1ybZCgCma0s/G7YfNWUdx4k6qEnhGVweSj0VQlQC/1rk GKl6IqlpDTMQ3cz8C6L5Qnh/8e1cbOvqAvOsbg3QXUfSjXtYMUWgK4fYh/qFxjmq Q+UQAPaXl4vZXOudvSqFwg9TqmyKiszZPfjT9ovkmbl0X+akdYYOXqbpLpHX4Dp1 m7Cc+aco+tv12CjaJ8zFPH619lzbOsJN8VR5nQl9/fXTS3UlnqMaFZfQDayB/CP4 Q87o34I4a97OZ33duE+2vYPqjyrZA8KMERU3bcx+UQwooS8QjX7o4NleYHTrDBSW f3V3AJLFZGsz3KzKBpibKG8k0tIoIphg47vwgYH7leGTdVdQ2gh15BGaq9s2R503 cNhydYqW0vZSito8j3kwpggBLD1VMjzzXvchugHKDPNKvULxhI2L4YlyGa8C0Vhx KLIBQ9YSNBQxM6YylK7UWH2x9C7MkGYWfHs478ark1d3F094KlnqoBdYh03TkY1U RWTxlnYOWu96AIQPxF0R30HcPTawj+gLYUMIoyVXnVpHor26nkI= =ZxC9 -----END PGP SIGNATURE----- --=-=-=--