yu kuai
97e5ea1de9
video: fbdev: radeonfb: remove set but not used variable 'bytpp'
...
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/video/fbdev/aty/radeon_base.c: In function
‘radeonfb_set_par’:
drivers/video/fbdev/aty/radeon_base.c:1660:32: warning:
variable ‘bytpp’ set but not used [-Wunused-but-set-variable]
It is never used, and so can be removed.
Signed-off-by: yu kuai <yukuai3@huawei.com >
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org >
Cc: zhengbin <zhengbin13@huawei.com >
Cc: yi.zhang <yi.zhang@huawei.com >
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-5-yukuai3@huawei.com
2020-03-02 16:32:10 +01:00
yu kuai
66cc77edb6
video: fbdev: radeonfb: remove set but not used variable '‘cSync’'
...
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/video/fbdev/aty/radeon_base.c: In function
‘radeonfb_set_par’:
drivers/video/fbdev/aty/radeon_base.c:1653:48: warning: variable
‘cSync’ set but not used [-Wunused-but-set-variable]
It is never used, and so can be removed.
Signed-off-by: yu kuai <yukuai3@huawei.com >
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org >
Cc: zhengbin <zhengbin13@huawei.com >
Cc: yi.zhang <yi.zhang@huawei.com >
[b.zolnierkie: patch summary fixups]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-4-yukuai3@huawei.com
2020-03-02 16:32:09 +01:00
yu kuai
eea60c30e7
video: fbdev: radeonfb: remove set but not used variable 'vSyncPol'
...
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/video/fbdev/aty/radeon_base.c: In function
‘radeonfb_set_par’:
drivers/video/fbdev/aty/radeon_base.c:1653:38: warning: variable
‘vSyncPol’ set but not used [-Wunused-but-set-variable]
It is never used, and so can be removed.
Signed-off-by: yu kuai <yukuai3@huawei.com >
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org >
Cc: zhengbin <zhengbin13@huawei.com >
Cc: yi.zhang <yi.zhang@huawei.com >
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-3-yukuai3@huawei.com
2020-03-02 16:32:08 +01:00
yu kuai
977750e84f
video: fbdev: radeonfb: remove set but not used variable 'hSyncPol'
...
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/video/fbdev/aty/radeon_base.c: In function
‘radeonfb_set_par’:
drivers/video/fbdev/aty/radeon_base.c:1653:6: warning: variable
‘hSyncPol’ set but not used [-Wunused-but-set-variable]
It is never used, and so can be removed.
Signed-off-by: yu kuai <yukuai3@huawei.com >
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org >
Cc: zhengbin <zhengbin13@huawei.com >
Cc: yi.zhang <yi.zhang@huawei.com >
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-2-yukuai3@huawei.com
2020-03-02 16:32:07 +01:00
Bartlomiej Zolnierkiewicz
9e24d63b05
video: fbdev: wm8505fb: add COMPILE_TEST support
...
Add COMPILE_TEST support to wm8505fb driver for better compile
testing coverage.
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/900c16b3-9306-7d17-f467-0f98bc95416a@samsung.com
2020-03-02 16:32:05 +01:00
Bartlomiej Zolnierkiewicz
459b0fa711
video: fbdev: wm8505fb: fix sparse warnings about using incorrect types
...
Use ->screen_buffer instead of ->screen_base to fix sparse warnings.
[ Please see commit 17a7b0b4d9 ("fb.h: Provide alternate screen_base
pointer") for details. ]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/567cba81-5fec-4d91-f711-c0bdbfe5b513@samsung.com
2020-03-02 16:32:04 +01:00
Bartlomiej Zolnierkiewicz
ee5aca2e71
video: fbdev: w100fb: add COMPILE_TEST support
...
Add COMPILE_TEST support to w100fb driver for better compile
testing coverage.
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/a929db5e-d373-7b09-ae2b-efec227f7e85@samsung.com
2020-03-02 16:32:03 +01:00
Bartlomiej Zolnierkiewicz
df3381071c
video: fbdev: w100fb: fix sparse warnings
...
* Add missing __iomem annotations where needed.
* Make w100fb_probe() static.
* Return NULL pointer (instead of using plain integer) in
w100_get_xtal_tabl().
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/d438108a-e569-a14a-a9b1-3fefd88fcadc@samsung.com
2020-03-02 16:31:51 +01:00
Bartlomiej Zolnierkiewicz
3d3287e609
video: fbdev: arcfb: add COMPILE_TEST support
...
Add COMPILE_TEST support to arcfb driver for better compile
testing coverage.
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/acf2cc2e-614d-f0fb-ce40-cee62bfcde4c@samsung.com
2020-03-02 16:31:50 +01:00
Bartlomiej Zolnierkiewicz
6df8627609
video: fbdev: sh_mobile_lcdcfb: add COMPILE_TEST support
...
Add COMPILE_TEST support to sh_mobile_lcdcfb driver for better compile
testing coverage.
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be >
Link: https://patchwork.freedesktop.org/patch/msgid/d98fea18-b72e-6d0f-33ac-1421738bd12b@samsung.com
2020-03-02 16:31:49 +01:00
Bartlomiej Zolnierkiewicz
5d30146f94
video: fbdev: sh_mobile_lcdcfb: fix sparse warnings about using incorrect types
...
Use ->screen_buffer instead of ->screen_base to fix sparse warnings.
[ Please see commit 17a7b0b4d9 ("fb.h: Provide alternate screen_base
pointer") for details. ]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be >
Link: https://patchwork.freedesktop.org/patch/msgid/c687dbc5-cf5a-9508-2a61-e757a1a14568@samsung.com
2020-03-02 16:31:48 +01:00
YueHaibing
3e26e0c47e
video: fbdev: pxa168fb: remove unnecessary platform_get_irq
...
commit 640ba2444f ("drivers/video/pxa168fb.c: use devm_ functions")
left behind this, it can be removed.
Signed-off-by: YueHaibing <yuehaibing@huawei.com >
Cc: Arnd Bergmann <arnd@arndb.de >
Cc: Daniel Vetter <daniel.vetter@ffwll.ch >
Cc: Christophe JAILLET <christophe.jaillet@wanadoo.fr >
Cc: Jani Nikula <jani.nikula@intel.com >
Cc: Chuhong Yuan <hslester96@gmail.com >
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200117032241.59148-1-yuehaibing@huawei.com
2020-03-02 16:31:47 +01:00
Thomas Zimmermann
6f2bb119da
drm/qxl: Use simple encoder
...
The qxl driver uses an empty implementation for its encoder. Replace
the code with the generic simple encoder.
v4:
* handle errors returned from drm_simple_encoder_init()
v2:
* rebase onto new simple-encoder interface
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de >
Reviewed-by: Sam Ravnborg <sam@ravnborg.org >
Acked-by: Gerd Hoffmann <kraxel@redhat.com >
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de >
Link: https://patchwork.freedesktop.org/patch/msgid/20200228081828.18463-5-tzimmermann@suse.de
2020-03-02 09:22:56 +01:00
Thomas Zimmermann
03e44ad19b
drm/mgag200: Use simple encoder
...
The mgag200 driver uses an empty implementation for its encoder. Replace
the code with the generic simple encoder.
v4:
* print error message with drm_err()
v3:
* init pre-allocated encoder with drm_simple_encoder_init()
v2:
* rebase onto new simple-encoder interface
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de >
Reviewed-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200228081828.18463-4-tzimmermann@suse.de
2020-03-02 09:22:49 +01:00
Thomas Zimmermann
4220fdf027
drm/ast: Use simple encoder
...
The ast driver uses an empty implementation for its encoder. Replace
the code with the generic simple encoder.
v2:
* rebase onto new simple-encoder interface
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200228081828.18463-3-tzimmermann@suse.de
2020-03-02 09:22:43 +01:00
Thomas Zimmermann
63170ac6f2
drm/simple-kms: Add drm_simple_encoder_{init,create}()
...
This patch makes the internal encoder implementation of the simple
KMS helpers available to drivers.
These simple-encoder helpers initialize an encoder with an empty
implementation. This covers the requirements of most of the existing
DRM drivers. A call to drm_simple_encoder_create() allocates and
initializes an encoder instance, a call to drm_simple_encoder_init()
initializes a pre-allocated instance.
v3:
* remove drm_simple_encoder_create(); not required yet
* provide more precise documentation
v2:
* move simple encoder to KMS helpers
* remove name argument; simplifies implementation
* don't allocate with devm_ interfaces; unsafe with DRM
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de >
Reviewed-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200228081828.18463-2-tzimmermann@suse.de
2020-03-02 09:22:35 +01:00
Sam Ravnborg
4a1d0dbc83
drm/panel: simple: add panel-dpi support
...
The panel-dpi compatible is a fallback that
allows the DT to specify the timing.
When matching panel-dpi expect the device tree to include the
timing information for the display-panel.
Background for this change:
There are a lot of panels and new models hits the market very often.
It is a lost cause trying to chase them all and users of new panels
will often find them in situations that the panel they ues are not
supported by the kernel.
On top of this a lot of panels are customized based on customer
specifications.
Including the panel timing in the device tree allows for a simple
way to describe the actual HW and use this description in a generic
way in the kernel.
This allows uses of proprietary panels, or panels which are not
included in the kernel, to specify the timing in the device tree
together with all the other HW descriptions.
And thus, using the device tree it is then easy to add support
for an otherwise unknown panel.
The current support expect panels that do not require any
delays for prepare/enable/disable/unprepare.
Oleksandr Suvorov replied:
I've just tested this patch on Apalis iMX6Q and Colibri iMX7D using
panel settings from the following patch:
https://lore.kernel.org/linux-arm-kernel/20200115123401.2264293-4-oleksandr.suvorov@toradex.com/
It works for me, thanks!
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Reviewed-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com >
Tested-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com >
Acked-by: Maxime Ripard <mripard@kernel.org >
Cc: Thierry Reding <thierry.reding@gmail.com >
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Cc: Maxime Ripard <mripard@kernel.org >
Cc: Oleksandr Suvorov <oleksandr.suvorov@toradex.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200216181513.28109-6-sam@ravnborg.org
2020-02-29 19:11:51 +01:00
Sam Ravnborg
fa10224087
dt-bindings: display: add data-mapping to panel-dpi
...
Add data-mapping property that can be used to specify
the media format used for the connection betwwen the
display controller (connector) and the panel.
v2:
- drop lvds666 (Rob)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Reviewed-by: Rob Herring <robh@kernel.org >
Acked-by: Maxime Ripard <mripard@kernel.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200216181513.28109-5-sam@ravnborg.org
2020-02-29 19:10:29 +01:00
Sam Ravnborg
4e840bea0c
dt-bindings: display: convert panel-dpi to DT schema
...
With panel-timing converted, now convert the single
remaining .txt user in panel/ of panel-timing to DT schema.
v2:
- Drop Thierry as maintainer, as this is not a general panel binding
and I have no acks.
- Drop requirement for a panel- specific binding - "panel-dpi" is enough
- Updated example
v3:
- added yaml document terminator "..."
- always require a specific binding - panel-dpi (based on feedback from Rob)
- use "power-supply" for the supply property, and made it mandatory
"power-supply" is the standard property for panels
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Reviewed-by: Rob Herring <robh@kernel.org >
Acked-by: Maxime Ripard <mripard@kernel.org >
Cc: Rob Herring <robh@kernel.org >
Cc: Thierry Reding <thierry.reding@gmail.com >
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Cc: Maxime Ripard <mripard@kernel.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200216181513.28109-4-sam@ravnborg.org
2020-02-29 19:08:10 +01:00
Sam Ravnborg
4dd23a47d6
dt-bindings: display: convert display-timings to DT schema
...
Add display-timings.yaml - that references panel-timings.yaml.
display-timings.yaml will be used for display bindings
when they are converted to meta-schema format.
For now the old display-timing.txt points to the new
display-timings.yaml - and all users are left as-is.
v2:
- Updated native-mode description
v3:
- Simpler "^timing" pattern (Rob)
- timing node is of type object (Rob)
- added display-timings to panel-common.yaml
- added yaml document terminator "..."
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Reviewed-by: Rob Herring <robh@kernel.org >
Acked-by: Maxime Ripard <mripard@kernel.org >
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Cc: Thierry Reding <thierry.reding@gmail.com >
Cc: Oleksandr Suvorov <oleksandr.suvorov@toradex.com >
Cc: devicetree@vger.kernel.org
Link: https://patchwork.freedesktop.org/patch/msgid/20200216181513.28109-3-sam@ravnborg.org
2020-02-29 19:08:04 +01:00
Sam Ravnborg
3aa31ced2f
dt-bindings: display: add panel-timing.yaml
...
Add meta-schema variant of panel-timing and
reference it from panel-common.yaml.
Part of this came form other files with other
licenses - original commits:
commit cc3f414cf2 ("video: add of helper for display timings/videomode")
commit 86f46565df ("dt-bindings: display: display-timing: Add property to configure sync drive edge")
commit 9cad9c95d7 ("Documentation: DocBook DRM framework documentation")
The original authors acked the license change to:
(GPL-2.0-only OR BSD-2-Clause)
v2:
- Got OK from original authors for re-license
Huge thanks for the quick replies!
- Typo fixes (Oleksandr)
- Drop -array variant when not needed (Maxime)
- Replace oneOf:... with enum (Maxime)
- Drop type from clock-frequency (Rob)
- Drop "|" when not needed (Rob)
v3:
- Added comment to acks that are only for the license change
- Add yaml document terminator "..."
- Updated description (removed reference to native-mode)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com > [license change]
Acked-by: Peter Ujfalusi <peter.ujfalusi@ti.com > [license change]
Acked-by: Steffen Trumtrar <s.trumtrar@pengutronix.de > [license change]
Acked-by: Philipp Zabel <p.zabel@pengutronix.de > [license change]
Reviewed-by: Rob Herring <robh@kernel.org >
Acked-by: Maxime Ripard <mripard@kernel.org >
Cc: Thierry Reding <thierry.reding@gmail.com >
Cc: Oleksandr Suvorov <oleksandr.suvorov@toradex.com >
Cc: Maxime Ripard <mripard@kernel.org >
Cc: devicetree@vger.kernel.org
Link: https://patchwork.freedesktop.org/patch/msgid/20200216181513.28109-2-sam@ravnborg.org
2020-02-29 19:07:52 +01:00
Linus Walleij
899f24ed8d
drm/panel: Add driver for Novatek NT35510-based panels
...
This adds a driver for panels based on the Novatek NT35510
display driver IC, such as the Hydis HVA40WV1 panel found
in the Samsung GT-S7710.
The NT35510 can be used with both internal and external
backlight (such as GPIO backlight) so we support both:
if no external backlight is found, we register a subdriver
for the internal backlight.
Cc: Stephan Gerhold <stephan@gerhold.net >
Signed-off-by: Linus Walleij <linus.walleij@linaro.org >
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200223121841.26836-3-linus.walleij@linaro.org
2020-02-29 14:02:46 +01:00
Linus Walleij
5ce96fe032
drm/panel: Add DT bindings for Novatek NT35510-based panels
...
This adds device tree bindings for the Novatek NT35510-based
family of panels. Since several such panels are in existence
we define bindings common for all, and define the compatible
string for one certain panel (Hydis HVA40WV1).
As other panels are discovered and investigated, we can add
more compatibles to the binding using oneOf constructions.
Cc: Stephan Gerhold <stephan@gerhold.net >
Cc: devicetree@vger.kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org >
Reviewed-by: Rob Herring <robh@kernel.org >
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200223121841.26836-2-linus.walleij@linaro.org
2020-02-29 14:02:26 +01:00
Linus Walleij
dc05809265
dt-bindings: Add vendor prefix for Hydis technologies
...
This vendor has produced a number of display panels,
including HVA40WV1.
Cc: devicetree@vger.kernel.org
Cc: Stephan Gerhold <stephan@gerhold.net >
Signed-off-by: Linus Walleij <linus.walleij@linaro.org >
Acked-by: Rob Herring <robh@kernel.org >
Signed-off-by: Sam Ravnborg <sam@ravnborg.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200223121841.26836-1-linus.walleij@linaro.org
2020-02-29 14:02:04 +01:00
Dan Carpenter
abdd768e9c
drm: prevent a harmless integer overflow in drm_legacy_sg_alloc()
...
There is an integer overflow when we round up to PAGE_SIZE, but it's
harmless because we never re-use "request->size" for anything meaningful.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com >
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/msgid/20200228092321.axulddmkxrujkmas@kili.mountain
2020-02-29 00:16:12 +01:00
Emmanuel Vadot
df0e7f7e03
drm/client: Dual licence the file in GPL-2 and MIT
...
Contributors for this file are :
Chris Wilson <chris@chris-wilson.co.uk >
Denis Efremov <efremov@linux.com >
Jani Nikula <jani.nikula@intel.com >
Maxime Ripard <mripard@kernel.org >
Noralf Trønnes <noralf@tronnes.org >
Sam Ravnborg <sam@ravnborg.org >
Thomas Zimmermann <tzimmermann@suse.de >
Acked-by: Jani Nikula <jani.nikula@intel.com >
Acked-by: Chris Wilson <chris@chris-wilson.co.uk >
Acked-by: Noralf Trønnes <noralf@tronnes.org >
Acked-by: Denis Efremov <efremov@linux.com >
Acked-by: Sam Ravnborg <sam@ravnborg.org >
Acked-by: Maxime Ripard <mripard@kernel.org >
Acked-by: Thomas Zimmermann <tzimmermann@suse.de >
Signed-off-by: Emmanuel Vadot <manu@FreeBSD.org >
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/msgid/20200215180911.18299-2-manu@FreeBSD.org
2020-02-29 00:16:12 +01:00
YueHaibing
46cf7d98a0
drm/tidss: Drop pointless static qualifier in dispc_find_csc()
...
There is no need to have the 'const struct dispc_csc_coef *coef'
variable static since new value always be assigned before use it.
Signed-off-by: YueHaibing <yuehaibing@huawei.com >
Reviewed-by: Jyri Sarha <jsarha@ti.com >
Tested-by: Jyri Sarha <jsarha@ti.com >
Signed-off-by: Jyri Sarha <jsarha@ti.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200227065057.92766-1-yuehaibing@huawei.com
2020-02-28 14:48:58 +02:00
Colin Ian King
652f67fd25
drm/tidss: fix spelling mistakes "bufer" and "requsted"
...
There are two spelling mistakes in warning and debug messages.
Fix them.
Signed-off-by: Colin Ian King <colin.king@canonical.com >
Reviewed-by: Jyri Sarha <jsarha@ti.com >
Tested-by: Jyri Sarha <jsarha@ti.com >
Signed-off-by: Jyri Sarha <jsarha@ti.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200224174226.387874-1-colin.king@canonical.com
2020-02-28 14:48:58 +02:00
Laurent Pinchart
905ad78028
drm/tidss: Use drm_for_each_bridge_in_chain()
...
Replace the manual encoder->bridge_chain walk with the
drm_for_each_bridge_in_chain() macro. Drivers should not touch the
bridge_chain field directly.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Reviewed-by: Jyri Sarha <jsarha@ti.com >
Tested-by: Jyri Sarha <jsarha@ti.com >
Signed-off-by: Jyri Sarha <jsarha@ti.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200222110718.26272-1-laurent.pinchart@ideasonboard.com
2020-02-28 14:48:58 +02:00
Jyri Sarha
b33b547488
drm/tidss: dispc: Fix broken plane positioning code
...
The old implementation of placing planes on the CRTC while configuring
the planes was naive and relied on the order in which the planes were
configured, enabled, and disabled. The situation where a plane's zpos
was changed on the fly was completely broken. The usual symptoms of
this problem was scrambled display and a flood of sync lost errors,
when a plane was active in two layers at the same time, or a missing
plane, in case when a layer was accidentally disabled.
The rewrite takes a more straight forward approach when HW is
concerned. The plane positioning registers are in the CRTC (or
actually OVR) register space and it is more natural to configure them
in a one go when configuring the CRTC. To do this we need make sure we
have all the planes on the updated CRTCs in the new atomic state. The
untouched planes on CRTCs that need plane position update are added to
the atomic state in tidss_atomic_check().
Signed-off-by: Jyri Sarha <jsarha@ti.com >
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200227120052.23168-1-jsarha@ti.com
2020-02-28 14:48:58 +02:00
Benjamin Gaignard
d6b8bbca6b
drm/dp_mst: Check crc4 value while building sideband message
...
Check that computed crc value is matching the one encoded in the message.
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com >
Reviewed-by: Lyude Paul <lyude@redhat.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200203121620.9002-1-benjamin.gaignard@st.com
2020-02-28 13:08:00 +01:00
Benjamin Gaignard
cb897542c6
drm/dp_mst: Fix W=1 warnings
...
Fix the warnings that show up with W=1.
They are all about unused but set variables.
If functions returns are not used anymore make them void.
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com >
Reviewed-by: Lyude Paul <lyude@redhat.com >
Link: https://patchwork.freedesktop.org/patch/msgid/20200205084842.5642-1-benjamin.gaignard@st.com
2020-02-28 13:07:22 +01:00
kbuild test robot
987b90d34f
drm/panfrost: default_supplies[] can be static
...
Fixes: 3e1399bccf ("drm/panfrost: Add support for multiple regulators")
Signed-off-by: kbuild test robot <lkp@intel.com >
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org >
Signed-off-by: Rob Herring <robh@kernel.org >
Link: https://patchwork.freedesktop.org/patch/msgid/20200227014100.GA61938@e50d7db646c3
2020-02-27 09:26:18 -06:00
Christian König
4981cdb063
dma-buf: make move_notify mandatory if importer_ops are provided
...
This makes the move_notify callback mandatory when the importer_ops are
provided. Since amdgpu is now migrated it doesn't make much sense
anymore to allow this.
Signed-off-by: Christian König <christian.koenig@amd.com >
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353995/?series=73646&rev=1
2020-02-27 14:58:01 +01:00
Christian König
bd2275eeed
dma-buf: drop dynamic_mapping flag
...
Instead use the pin() callback to detect dynamic DMA-buf handling.
Since amdgpu is now migrated it doesn't make much sense to keep
the extra flag.
Signed-off-by: Christian König <christian.koenig@amd.com >
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353997/?series=73646&rev=1
2020-02-27 14:58:01 +01:00
Christian König
a448cb003e
drm/amdgpu: implement amdgpu_gem_prime_move_notify v2
...
Implement the importer side of unpinned DMA-buf handling.
v2: update page tables immediately
Signed-off-by: Christian König <christian.koenig@amd.com >
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353998/?series=73646&rev=1
2020-02-27 14:58:01 +01:00
Christian König
2d4dad2734
drm/amdgpu: add amdgpu_dma_buf_pin/unpin v2
...
This implements the exporter side of unpinned DMA-buf handling.
v2: fix minor coding style issues
Signed-off-by: Christian König <christian.koenig@amd.com >
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353999/?series=73646&rev=1
2020-02-27 14:58:01 +01:00
Christian König
4993ba0263
drm/amdgpu: use allowed_domains for exported DMA-bufs
...
Avoid that we ping/pong the buffers when we stop to pin DMA-buf
exports by using the allowed domains for exported buffers.
Signed-off-by: Christian König <christian.koenig@amd.com >
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353996/?series=73646&rev=1
2020-02-27 14:58:01 +01:00
Christian König
d2588d2ded
drm/ttm: remove the backing store if no placement is given
...
Pipeline removal of the BOs backing store when no placement is given
during validation.
Signed-off-by: Christian König <christian.koenig@amd.com >
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353994/?series=73646&rev=1
2020-02-27 14:58:00 +01:00
Christian König
bb42df4662
dma-buf: add dynamic DMA-buf handling v15
...
On the exporter side we add optional explicit pinning callbacks. Which are
called when the importer doesn't implement dynamic handling, move notification
or need the DMA-buf locked in place for its use case.
On the importer side we add an optional move_notify callback. This callback is
used by the exporter to inform the importers that their mappings should be
destroyed as soon as possible.
This allows the exporter to provide the mappings without the need to pin
the backing store.
v2: don't try to invalidate mappings when the callback is NULL,
lock the reservation obj while using the attachments,
add helper to set the callback
v3: move flag for invalidation support into the DMA-buf,
use new attach_info structure to set the callback
v4: use importer_priv field instead of mangling exporter priv.
v5: drop invalidation_supported flag
v6: squash together with pin/unpin changes
v7: pin/unpin takes an attachment now
v8: nuke dma_buf_attachment_(map|unmap)_locked,
everything is now handled backward compatible
v9: always cache when export/importer don't agree on dynamic handling
v10: minimal style cleanup
v11: drop automatically re-entry avoidance
v12: rename callback to move_notify
v13: add might_lock in appropriate places
v14: rebase on separated locking change
v15: add EXPERIMENTAL flag, some more code comments
Signed-off-by: Christian König <christian.koenig@amd.com >
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch >
Link: https://patchwork.freedesktop.org/patch/353993/?series=73646&rev=1
2020-02-27 14:58:00 +01:00
Vasily Khoruzhick
258145ea35
drm/panel: simple: Add NewEast Optoelectronics CO., LTD WJFH116008A panel support
...
This commit adds support for the NewEast Optoelectronics CO., LTD
WJFH116008A 11.6" 1920x1080 TFT LCD panel.
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com >
Signed-off-by: Maxime Ripard <maxime@cerno.tech >
Link: https://patchwork.freedesktop.org/patch/msgid/20200226081011.1347245-6-anarsoul@gmail.com
2020-02-27 13:52:27 +01:00
Vasily Khoruzhick
0c10640959
dt-bindings: display: simple: Add NewEast Optoelectronics WJFH116008A compatible
...
This commit adds compatible for NewEast Optoelectronics WJFH116008A panel
to panel-simple binding
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com >
Signed-off-by: Maxime Ripard <maxime@cerno.tech >
Link: https://patchwork.freedesktop.org/patch/msgid/20200226081011.1347245-5-anarsoul@gmail.com
2020-02-27 13:52:08 +01:00
Vasily Khoruzhick
c153a892f6
dt-bindings: Add Guangdong Neweast Optoelectronics CO. LTD vendor prefix
...
Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD
Acked-by: Rob Herring <robh@kernel.org >
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com >
Signed-off-by: Maxime Ripard <maxime@cerno.tech >
Link: https://patchwork.freedesktop.org/patch/msgid/20200226081011.1347245-4-anarsoul@gmail.com
2020-02-27 13:49:43 +01:00
Vasily Khoruzhick
6f13d2ce05
drm/bridge: anx6345: don't print error message if regulator is not ready
...
We don't want to print scary message if devm_regulator_get() returns
-EPROBE_DEFER
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com >
Signed-off-by: Maxime Ripard <maxime@cerno.tech >
Link: https://patchwork.freedesktop.org/patch/msgid/20200226081011.1347245-3-anarsoul@gmail.com
2020-02-27 13:49:27 +01:00
Gurchetan Singh
18b39fb975
drm/virtio: add virtio_gpu_is_shmem helper
...
The plan is use have both shmem and virtual "vram" running
side-by-side in virtio-gpu. It looks like we'll eventually use
struct drm_gem_object as a base class, and we'll need to convert
to shmem and vram objects on the fly. As a first step, add a
virtio_gpu_is_shmem helper. Thanks to kraxel for suggesting this
approach on Gitlab.
Suggested-by: Gerd Hoffman <kraxel@redhat.com >
Signed-off-by: Gurchetan Singh <gurchetansingh@chromium.org >
Link: http://patchwork.freedesktop.org/patch/msgid/20200227002601.745-3-gurchetansingh@chromium.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2020-02-27 11:31:34 +01:00
Gurchetan Singh
068a8fea37
drm/virtio: make mmap callback consistent with callbacks
...
This is a very, very minor cleanup.
Signed-off-by: Gurchetan Singh <gurchetansingh@chromium.org >
Link: http://patchwork.freedesktop.org/patch/msgid/20200227002601.745-2-gurchetansingh@chromium.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2020-02-27 11:31:34 +01:00
Tian Tao
c3480301d7
drm/hisilicon: Fixed pcie resource conflict between drm and firmware
...
use the drm_fb_helper_remove_conflicting_pci_framebuffer to remove
the framebuffer initialized by fireware/bootloader to avoid resource
conflict.
Signed-off-by: Tian Tao <tiantao6@hisilicon.com >
Acked-by: Thomas Zimmermann <tzimmermann@suse.de >
Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org >
Link: https://patchwork.freedesktop.org/patch/msgid/1582524112-5628-1-git-send-email-tiantao6@hisilicon.com
2020-02-27 04:34:32 +00:00
Tian Tao
29b7216b57
drm/hisilicon: Set preferred mode resolution and maximum resolution
...
set the preferred mode resolution to 1024 * 768 and maximum
resolution to 1920 * 1200.
Signed-off-by: Tian Tao <tiantao6@hisilicon.com >
Signed-off-by: Gong junjie <gongjunjie2@huawei.com >
Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org >
Link: https://patchwork.freedesktop.org/patch/msgid/1582080707-18825-1-git-send-email-tiantao6@hisilicon.com
2020-02-27 04:34:32 +00:00
Tian Tao
bc17e55749
drm/hisilicon: Add the mode_valid function
...
add mode_valid function, we can make sure the resolution is valid.
Signed-off-by: Tian Tao <tiantao6@hisilicon.com >
Signed-off-by: Gong junjie <gongjunjie2@huawei.com >
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de >
Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org >
Link: https://patchwork.freedesktop.org/patch/msgid/1578476501-45807-1-git-send-email-tiantao6@hisilicon.com
2020-02-27 04:34:32 +00:00
Tian Tao
de3c3ed4c8
drm/hisilicon: fixed the wrong resolution configurations
...
The maximum resolution supported by hibmc is 1920 * 1200 instead of
1920 * 1440, this patch fixed this problem
Signed-off-by: Tian Tao <tiantao6@hisilicon.com >
Signed-off-by: Gong junjie <gongjunjie2@huawei.com >
Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org >
Link: https://patchwork.freedesktop.org/patch/msgid/1578023756-46567-1-git-send-email-tiantao6@hisilicon.com
2020-02-27 04:34:31 +00:00