From patchwork Sat Nov 25 17:33:39 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans de Goede X-Patchwork-Id: 10074899 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 27E61602BC for ; Sat, 25 Nov 2017 17:34:01 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1505E29013 for ; Sat, 25 Nov 2017 17:34:01 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 0A23D29012; Sat, 25 Nov 2017 17:34:01 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A2B1329016 for ; Sat, 25 Nov 2017 17:34:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751865AbdKYReA (ORCPT ); Sat, 25 Nov 2017 12:34:00 -0500 Received: from mail-wr0-f193.google.com ([209.85.128.193]:41903 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751658AbdKYRd7 (ORCPT ); Sat, 25 Nov 2017 12:33:59 -0500 Received: by mail-wr0-f193.google.com with SMTP id z14so22749032wrb.8 for ; Sat, 25 Nov 2017 09:33:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=j0y0ymsuEQ+LUL4/PhFYVg9DsQIewB8dHcxOobTdBJ0=; b=MR/hBuOGH1zyLe+8a6fKDDI+VcvXx95uNoXgt74r1x8mbRXoU7dWfdXiEhdcQFR9eG ZFOfGvnmXeEzdiO/LcW7LwcYGMUbbMTZBxG3Gz1KNVUQg7tsUEisAWGtTOkjzo6GMG3a AIKo9ipzqa7JudLAIjCHtgYx0alh1DwHBkQuWpQWQipW+nvZO2zfmztJqGED3meQ6NYo 59tQJFgJQG5DL3sDFRw5bwlqxL9iTL0uzZfmX/lg3/uoTk8w2mwgJeMaa9Jhkrb9gKzZ uZ/52BC3kCqf/e5Dl1c2pqLNDO6BBPOdZY2G088XXiHjDSOmerI2akBgec1lNJSNj1Sc y2IQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=j0y0ymsuEQ+LUL4/PhFYVg9DsQIewB8dHcxOobTdBJ0=; b=Dc81+J5XunUfgjX3alh5WlF/O1fDqYvEzf1AAiuvY5l8YpwF3m/gcmXwv4NoRC0YJv 96XYKf01eHICnh1usWxL3GeJT3aZ+x8V/GzE8wOykSar51NJnBJvxieUpVZgM7Za43Mk JJIj8pdcG4kJPce7k2oL9oIL/0Ucq9kQ3HMP4DjG52sx9AitIy4X23fVqfl0nYwHDEVi BltFBbneRqQ9Z2NsL/9PZzlVuZfr2a+W1Lfsl2Uxy9aQfDvzpGyONCBAKVCEtCiA1dkq hHJYoGIX68HkUyKXeNZU4iVK+dZ4+eFw4+kmaT7eZxxhsvhEyZnUzdZlNcH50fYdwQxO tRbg== X-Gm-Message-State: AJaThX5BviZSw1Hhmlq4KrZmMszTgYDJP5jp2ddpyrLDyeBkkHtNdTob upmn7MRYKO/fSnOCc9wo8tE= X-Google-Smtp-Source: AGs4zMbMq3D8oTCiJ6QSAC4p2ENNhCSi96ir+xu6Ap0wSE3qKLiv/VFmwm9ZK5g/qS0/FQaDKID64A== X-Received: by 10.223.188.141 with SMTP id g13mr7636175wrh.169.1511631238350; Sat, 25 Nov 2017 09:33:58 -0800 (PST) Received: from localhost.localdomain.com ([2001:470:7a95:4242:2c1c:b9eb:f8cb:6ccb]) by smtp.gmail.com with ESMTPSA id o135sm8439658wmg.1.2017.11.25.09.33.57 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sat, 25 Nov 2017 09:33:57 -0800 (PST) From: Hans de Goede X-Google-Original-From: Hans de Goede To: Daniel Vetter , Jani Nikula , Sean Paul , Dave Airlie , Bartlomiej Zolnierkiewicz Cc: Hans de Goede , intel-gfx , dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org Subject: [PATCH v6 5/7] drm/i915: Add "panel orientation" property to the panel connector, v6. Date: Sat, 25 Nov 2017 18:33:39 +0100 Message-Id: <20171125173341.14162-6-hdegoede@redhat.com> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20171125173341.14162-1-hdegoede@redhat.com> References: <20171125173341.14162-1-hdegoede@redhat.com> Sender: linux-fbdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Ideally we could use the VBT for this, that would be simple, in intel_dsi_init() check dev_priv->vbt.dsi.config->rotation, set connector->display_info.panel_orientation accordingly and call drm_connector_init_panel_orientation_property(), done. Unfortunately vbt.dsi.config->rotation is always 0 even on tablets with an upside down LCD and where the GOP is properly rotating the EFI fb in hardware. So instead we end up reading the rotation from the primary plane. This commit only implements the panel orientation property for DSI panels on BYT / CHT hardware, as all known non normal oriented panels sofar are only found on this hardware. Changes in v2: -Read back the rotation applied by the GOP from the primary plane instead of relying on dev_priv->vbt.dsi.config->rotation, because it seems that the VBT rotation filed is always 0 even on devices where the GOP does apply a rotation Changes in v3: -Rewrite the code to read back the orientation from the primary plane to contain all of this in intel_dsi.c instead of poking a bunch of holes between all the different layers Changes in v6: -Move hardware readout to intel_dsi_init() Signed-off-by: Hans de Goede Reviewed-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_dsi.c | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c index f09474b0c4d3..f67d321376e4 100644 --- a/drivers/gpu/drm/i915/intel_dsi.c +++ b/drivers/gpu/drm/i915/intel_dsi.c @@ -1666,6 +1666,27 @@ static const struct drm_connector_funcs intel_dsi_connector_funcs = { .atomic_duplicate_state = intel_digital_connector_duplicate_state, }; +static int intel_dsi_get_panel_orientation(struct intel_connector *connector) +{ + struct drm_i915_private *dev_priv = to_i915(connector->base.dev); + int orientation = DRM_MODE_PANEL_ORIENTATION_NORMAL; + enum i9xx_plane_id plane; + u32 val; + + if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) { + if (connector->encoder->crtc_mask == BIT(PIPE_B)) + plane = PLANE_B; + else + plane = PLANE_A; + + val = I915_READ(DSPCNTR(plane)); + if (val & DISPPLANE_ROTATE_180) + orientation = DRM_MODE_PANEL_ORIENTATION_BOTTOM_UP; + } + + return orientation; +} + static void intel_dsi_add_properties(struct intel_connector *connector) { struct drm_i915_private *dev_priv = to_i915(connector->base.dev); @@ -1681,6 +1702,13 @@ static void intel_dsi_add_properties(struct intel_connector *connector) allowed_scalers); connector->base.state->scaling_mode = DRM_MODE_SCALE_ASPECT; + + connector->base.display_info.panel_orientation = + intel_dsi_get_panel_orientation(connector); + drm_connector_init_panel_orientation_property( + &connector->base, + connector->panel.fixed_mode->hdisplay, + connector->panel.fixed_mode->vdisplay); } }