From: Chen-Yu Tsai <wens@csie.org>
To: Maxime Ripard <maxime.ripard@free-electrons.com>,
David Airlie <airlied@linux.ie>
Cc: Chen-Yu Tsai <wens@csie.org>,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
linux-sunxi@googlegroups.com
Subject: [PATCH 0/7] drm/sun4i: More cleanups
Date: Sat, 14 Oct 2017 12:02:45 +0800 [thread overview]
Message-ID: <20171014040252.9621-1-wens@csie.org> (raw)
Hi,
Here's another bunch of cleanups for sun4i-drm. Most of these were
found while working on A10/A20 DRM and HDMI support. To be clear,
nothing was broken before these patches.
Patch 1 trims the sun4i-drm probe sequence by not adding repeating
components. The component can deal with duplicates, but it leads
to excessive debug logs which is confusing for developers not in
the know.
Patch 2 moves regmap creation to a point where access is actually
possible, i.e. clocks enabled and reset controls deasserted.
Patch 3 moves to the new drm_fb_cma_get_gem_addr() helper instead
of open coding the same thing.
Patch 4 expands the comment explaining why we clear the backend
registers explicitly.
Patch 5 fixes the buffer address programmed into the backend layer
registers. This issue only hits devices with more than 1GB RAM.
Patch 6 documents some newly discovered but unused register bits
in the HDMI controller.
Patch 7 delays setting of the PAD_CTRL1 register in the HDMI controller
to the mode_set function. The main reason to do this is that between
probe time and when the display is actually setup, the controller itself
toggles some of the bits in this register. In particular, on the A10
it toggles the invert bits documented in the previous patch. This
causes the color to be completed inverted.
Please have a look.
Regards
ChenYu
Chen-Yu Tsai (7):
drm/sun4i: don't add components that are already in the queue
drm/sun4i: backend: Create regmap after access is possible
drm/sun4i: backend: Use drm_fb_cma_get_gem_addr() to get display
memory
drm/sun4i: backend: Add comment explaining why registers are cleared
drm/sun4i: backend: Offset layer buffer address by DRAM starting
address
drm/sun4i: hdmi: Document PAD_CTRL1 output invert bits
drm/sun4i: hdmi: Move PAD_CTRL1 setting to mode_set function
drivers/gpu/drm/sun4i/sun4i_backend.c | 45 ++++++++++++++++++----------------
drivers/gpu/drm/sun4i/sun4i_drv.c | 16 ++++++++++++
drivers/gpu/drm/sun4i/sun4i_hdmi.h | 5 ++++
drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c | 26 ++++++++++++--------
4 files changed, 61 insertions(+), 31 deletions(-)
--
2.14.2
next reply other threads:[~2017-10-14 4:04 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-14 4:02 Chen-Yu Tsai [this message]
2017-10-14 4:02 ` [PATCH 1/7] drm/sun4i: don't add components that are already in the queue Chen-Yu Tsai
2017-10-14 4:02 ` [PATCH 2/7] drm/sun4i: backend: Create regmap after access is possible Chen-Yu Tsai
2017-10-14 4:02 ` [PATCH 3/7] drm/sun4i: backend: Use drm_fb_cma_get_gem_addr() to get display memory Chen-Yu Tsai
2017-10-14 4:02 ` [PATCH 4/7] drm/sun4i: backend: Add comment explaining why registers are cleared Chen-Yu Tsai
2017-10-14 4:02 ` [PATCH 5/7] drm/sun4i: backend: Offset layer buffer address by DRAM starting address Chen-Yu Tsai
2017-10-16 8:00 ` Maxime Ripard
2017-10-16 8:20 ` [linux-sunxi] " Chen-Yu Tsai
2017-10-16 20:15 ` Maxime Ripard
2017-10-17 3:40 ` Chen-Yu Tsai
2017-10-16 9:42 ` icenowy
2017-10-14 4:02 ` [PATCH 6/7] drm/sun4i: hdmi: Document PAD_CTRL1 output invert bits Chen-Yu Tsai
2017-10-14 4:02 ` [PATCH 7/7] drm/sun4i: hdmi: Move PAD_CTRL1 setting to mode_set function Chen-Yu Tsai
2017-10-17 4:23 [PATCH 0/7] drm/sun4i: More cleanups Chen-Yu Tsai
2017-10-17 4:25 ` Chen-Yu Tsai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171014040252.9621-1-wens@csie.org \
--to=wens@csie.org \
--cc=airlied@linux.ie \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=maxime.ripard@free-electrons.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®