[PATCH v5 0/7] drm/panel: simple: Add mode support to devicetree

Douglas Anderson
2019-04-01T17:17:59Z

I'm reviving Sean Paul's old patchset to get mode support in device
tree. The cover letter for his v3 is at:
https://lists.freedesktop.org/archives/dri-devel/2018-February/165162.html

No code is different between v4 and v5, just commit messages and text
in the bindings.

I've pulled together the patches that didn't land in v3, addressed
outstanding feedback, and reposted. Atop them I've added patches for
rk3288-veyron-chromebook (used for jaq, jerry, mighty, speedy) and
rk3288-veryon-minnie.

Please let me know how they look.

In general I have added people to the whole series who I think would
like the whole series and then let get_maintainer pick extra people it
thinks are relevant to each individual patch. If I see you respond to
any of the patches in the series, though, I'll add you to the whole
series Cc list next time.

Changes in v5:
- Removed bit about OS may ignore (Rob/Ezequiel)
- Added Heiko's Tested-by
- It's not just jerry, it's most rk3288 Chromebooks (Heiko)

Changes in v4:
- Simplify desc. for when override should be used (Thierry/Laurent)
- Removed Rob H review since it's been a year and wording changed
- Don't add mode from timing if override was specified (Thierry)
- Add warning if timing and fixed mode was specified (Thierry)
- Don't add fixed mode if timing was specified (Thierry)
- Refactor/rename a bit to avoid extra indentation from "if" tests
- i should be unsigned (Thierry)
- Add annoying WARN_ONs for some cases (Thierry)
- Simplify 'No display_timing found' handling (Thierry)
- Rename to panel_simple_parse_override_mode() (Thierry)
- Rebase to top of Heiko's tree
- Converted changelog to after-the-cut for non-DRM change.
- display_timing for Innolux n116bge new for v4.
- display_timing for AUO b101ean01 new for v4.
- rk3288-veyron-jerry patch new for v4.
- rk3288-veyron-minnie patch new for v4.

Changes in v3:
- Go back to using the timing subnode directly, but rename to
panel-timing (Rob)
- No longer parse display-timings subnode, use panel-timing (Rob)
- Unwrap the timing from display-timings and rename panel-timing (Rob)

Changes in v2:
- Split out the binding into a new patch (Rob)
- display-timings is a new section (Rob)
- Use the full display-timings subnode instead of picking the timing
out (Rob/Thierry)
- Parse the full display-timings node (using the native-mode) (Rob)
- Wrap the timing in display-timings node to match binding (Rob/Thierry)

Douglas Anderson (4):
drm/panel: simple: Use display_timing for Innolux n116bge
drm/panel: simple: Use display_timing for AUO b101ean01
ARM: dts: rockchip: Specify rk3288-veyron-chromebook's display timings
ARM: dts: rockchip: Specify rk3288-veyron-minnie's display timings

Sean Paul (3):
dt-bindings: Add panel-timing subnode to simple-panel
drm/panel: simple: Add ability to override typical timing
arm64: dts: rockchip: Specify override mode for kevin panel

.../bindings/display/panel/simple-panel.txt | 22 +++
.../boot/dts/rk3288-veyron-chromebook.dtsi | 14 ++
arch/arm/boot/dts/rk3288-veyron-minnie.dts | 14 ++
.../boot/dts/rockchip/rk3399-gru-kevin.dts | 14 ++
drivers/gpu/drm/panel/panel-simple.c | 171 ++++++++++++++----
5 files changed, 203 insertions(+), 32 deletions(-)

--
2.21.0.392.gf8f6787159e-goog