On Sat, Sep 30, 2017 at 1:35 PM, Julian Calaby wrote:
> Hi Chen-Yu,
>
> On Fri, Sep 29, 2017 at 8:22 PM, Chen-Yu Tsai wrote:
>> On Fri, Sep 29, 2017 at 6:20 PM, Maxime Ripard
>> wrote:
>>> On Fri, Sep 29, 2017 at 08:22:56AM +, Chen-Yu Tsai wrote:
On systems with 2 TCONs such as the A31,
Hi Lucas,
[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Lucas-Stach/drm-etnaviv-remove-iommu
Hi Meghana,
[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm
Yes, caused by the commit e37a7b4088da
("drm/amd/powerplay: tidy up ret checks in amd_powerplay.c")
Replace error when split patches.
Have sent the fix patch.
Please review.
Best Regards
Rex
-Original Message-
From: Alex Deucher [mailto:alexdeuc...@gmail.com]
Sent: Friday, September
My mean is like the attached, I revert part of yours.
Regards,
David zhou
On 2017年09月29日 22:15, Marek Olšák wrote:
On Fri, Sep 29, 2017 at 4:13 PM, Marek Olšák wrote:
On Fri, Sep 29, 2017 at 4:44 AM, Chunming Zhou wrote:
On 2017年09月13日 04:42, Marek Olšák wrote:
From: Marek Olšák
For a
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm
From: Andrey Grodzovsky
Adding initial tests for locks detection when SW
scheduler FIFO is full.
The test works by submitting a batch of identical commands which make the CP
stall waiting for condition to become true. The condition is later satisfied
form a helper thread. Other events that happe
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-Move
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-Move
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Meghana-Madhyastha/drm-tinydrm-Move
On Thu, Sep 28, 2017 at 9:46 AM, Colin King wrote:
> From: Colin Ian King
>
> The functions alloc_pasid and free_pasid are local to the
> source and do not need to be in global scope, so make them static.
>
> Cleans up sparse warnings:
> warning: symbol 'alloc_pasid' was not declared. Should it
https://bugs.freedesktop.org/show_bug.cgi?id=92248
--- Comment #43 from Hector Velazquez
---
This test are failing on CFL-S-1 QA
Tests List:
igt@kms_plane_scaling
Output
. . .
DEBUG
https://bugs.freedesktop.org/show_bug.cgi?id=99553
Bug 99553 depends on bug 99856, which changed state.
Bug 99856 Summary: OpenCL Hello world returns "unsupported call to function
get_local_size"
https://bugs.freedesktop.org/show_bug.cgi?id=99856
What|Removed |
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 99f6288635ceb793ea79c787d83ee64a95b46cd4
commit: 6a1f70848ca224e9fa74dfc72c05a0f9c8711b98 [1152/1167] drm/amd/powerplay:
refine code in amd_powerplay.c
config: ia64-allmodconfig (attached as .config)
compiler: ia64-
On 26/09/17 10:31 PM, Alex Deucher wrote:
> Hi Dave,
>
> First feature pull for 4.15. Highlights:
> - Per VM BO support
> - Lots of powerplay cleanups
> - Powerplay support for CI
> - pasid mgr for kfd
> - interrupt infrastructure for recoverable page faults
> - SR-IOV fixes
> - initial GPU reset
Den 29.09.2017 16.13, skrev Meghana Madhyastha:
On Fri, Sep 29, 2017 at 02:33:12PM +0200, Noralf Trønnes wrote:
Den 29.09.2017 14.20, skrev Meghana Madhyastha:
On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trønnes wrote:
Den 29.09.2017 05.22, skrev Meghana Madhyastha:
On Thu, Sep 28, 2017
Rename tinydrm_of_find_backlight to drm_of_find_backlight
and move it into drm_of.c from tinydrm-helpers.c. This is
because other drivers in the drm subsystem might need to call
this function. In that case and otherwise, it is better from
an organizational point of view to move it into drm_of.c alo
Add the dummy versions (function definition returning -EINVAL) of
drm_of_find_backlight and devm_drm_of_find_backlight in the #else
part of the conditional directive in drm_of.h. This is needed
for drivers where CONFIG_OF is optional.
Signed-off-by: Meghana Madhyastha
---
Changes in v4:
-This com
Add devm_drm_of_find_backlight and the corresponding release
function because some drivers such as tinydrm use devres versions
of functions for requiring device resources.
Signed-off-by: Meghana Madhyastha
---
Changes in v4:
-Made the release function an internal static function.
-Removed specifi
This patchset introduces some changes such as move
tinydrm_of_find_backlight to drm_of.c and rename it to
drm_of_find_backlight for better organizational structure.
Changes in v4:
- Remove "devices like tinydrm" from the function comments.
- Made devm_drm_of_find_backlight_release an internal func
Hi Srishti,
On Fri, Sep 29, 2017 at 03:30:40PM +0530, Srishti Sharma wrote:
> Replace drm_dev_unref with drm_dev_put as it is more consistent
> with kernel coding style. Done using the following semantic
> patch by coccinelle.
Thanks for the patch!
Acked-by: Liviu Dudau
I'm going to pull this
Hi Benjamin,
and many thanks for this cleanup patchset.
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
Philippe :-)
On 09/29/2017 02:59 PM, Benjamin Gaignard wrote:
> The goal of this series is to simplify driver code when they need to clean up
> a previously allocated panel bridge.
> F
The reset path wants to initialize the clock control register regardless
of the DYNAMIC_FREQUENCY_SCALING feature, so don't call clock update, but
explicitly load the register.
Also disabling of the debug registers is moved into the reset function,
so we always get to the same state after a GPU re
This comment is outdated as the driver is taking care about clock
gating and the pulse eater for quite some time already.
Signed-off-by: Lucas Stach
---
drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
b/driver
They are not used in any way, so can go away.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
drivers/gpu/drm/etnaviv/etnaviv_iommu.c| 21 -
drivers/gpu/drm/etnaviv/etnaviv_iommu_v2.c | 14 --
2 files changed,
We only ever do GEM object creation by handle, as there is no kernel
internal use of GEM objects.
Signed-off-by: Lucas Stach
---
drivers/gpu/drm/etnaviv/etnaviv_drv.h | 4
drivers/gpu/drm/etnaviv/etnaviv_gem.c | 19 ---
2 files changed, 23 deletions(-)
diff --git a/drivers
A function doing a single assignment is not really helping the
code flow.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
drivers/gpu/drm/etnaviv/etnaviv_iommu.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git
This is a preparation to remove the etnaviv dependency on the IOMMU
subsystem by importing the relevant parts of the iommu map/unamp
functions into the driver.
Signed-off-by: Lucas Stach
Reviewed-By: Wladimir J. van der Laan
---
v2: change return type of etnaviv_domain_unmap to void
---
drivers
The handler has never been used, so it's really just dead code.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
drivers/gpu/drm/etnaviv/etnaviv_mmu.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/etnaviv/etnaviv_mmu.
And clean up the header file a bit.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
drivers/gpu/drm/etnaviv/etnaviv_mmu.c | 8
drivers/gpu/drm/etnaviv/etnaviv_mmu.h | 8 +---
2 files changed, 5 insertions(+), 11 deletions(-)
diff --
Those functions are simple enough to fold them into the calling
function. This also fixes a correctness issue, as the alloc/free
functions didn't specifiy the device the memory was allocated for.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
dr
Using the IOMMU API to manage the internal GPU MMU has been an
historical accident and it keeps getting in the way, as well as
entangling the driver with the inner workings of the IOMMU
subsystem.
Clean this up by removing the usage of iommu_domain, which is the
last piece linking etnaviv to the I
It wasn't protecting anything, as the single word writes used to
set up or tear down a translation are already inherently atomic,
so the spinlock is pure overhead.
Signed-off-by: Lucas Stach
Reviewed-by: Philipp Zabel
Reviewed-By: Wladimir J. van der Laan
---
drivers/gpu/drm/etnaviv/etnaviv_io
On Fri, Sep 29, 2017 at 4:13 PM, Marek Olšák wrote:
> On Fri, Sep 29, 2017 at 4:44 AM, Chunming Zhou wrote:
>>
>>
>> On 2017年09月13日 04:42, Marek Olšák wrote:
>>>
>>> From: Marek Olšák
>>>
>>> For amdgpu.
>>>
>>> drm_syncobj_create is renamed to drm_syncobj_create_as_handle, and new
>>> helpers d
On Fri, Sep 29, 2017 at 4:44 AM, Chunming Zhou wrote:
>
>
> On 2017年09月13日 04:42, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> For amdgpu.
>>
>> drm_syncobj_create is renamed to drm_syncobj_create_as_handle, and new
>> helpers drm_syncobj_create and drm_syncobj_get_handle are added.
>>
>> Sig
On Fri, Sep 29, 2017 at 02:33:12PM +0200, Noralf Trønnes wrote:
>
> Den 29.09.2017 14.20, skrev Meghana Madhyastha:
> >On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trønnes wrote:
> >>Den 29.09.2017 05.22, skrev Meghana Madhyastha:
> >>>On Thu, Sep 28, 2017 at 06:02:27PM +0200, Noralf Trønnes w
Rex, probably related to the recent cleanups in powerplay.
On Fri, Sep 29, 2017 at 10:09 AM, Dieter Nützel wrote:
> Hello all,
>
> since latest update
>
> 1d7da702e70d3c27408a3bb312c71d6be9f7bebe
> drm/amd/powerplay: fix spelling mistake: "dividable" -> "divisible"
>
> I didn't get fan info with
Hello all,
since latest update
1d7da702e70d3c27408a3bb312c71d6be9f7bebe
drm/amd/powerplay: fix spelling mistake: "dividable" -> "divisible"
I didn't get fan info with my RX580 (Polaris21) any longer.
Worked with this commit:
786df0b89fe5a0b405d4de0a1ce03003c0743ec3
drm/amd/display: fix pflip
Quoting Alex Deucher (2017-09-27 02:36:07)
> Hi Dave,
>
> Initial pull request for DC support. We've completed a substantial amount of
> the cleanup and restructuring in our TODO. There are a few additional
> cleanups that we are continuing to work on, but I don't think there are any
> showstopp
Hello Marek,
I just tested this series, and I noticed a lot of these lines:
> exynos-sysmmu 11a4.sysmmu: restoring state
> exynos-sysmmu 11a4.sysmmu: saving state
I guess it would make sense to enable autosuspend for runtime PM in each of the
hw drivers.
I've just send a patch that does
Signed-off-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_drm_fimc.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c
b/drivers/gpu/drm/exynos/exynos_drm_fimc.c
index 1c48c57381f1..bffb4b513453 100644
--- a/drivers/gpu/d
On Fri, 2017-09-29 at 17:07 +0800, Chih-Wei Huang wrote:
> 2017-09-29 16:44 GMT+08:00 Robert Foss :
> > On Fri, 2017-09-29 at 13:49 +0800, Chih-Wei Huang wrote:
> > > 2017-09-29 5:29 GMT+08:00 Rob Herring :
> > > > Perhaps just leave the current state as a separate branch.
> > >
> > > Did you mean
Make sure that bridge parameter is not NULL and can be safely
cast into a panel_bridge structure.
Signed-off-by: Benjamin Gaignard
---
drivers/gpu/drm/bridge/panel.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/panel.c b/drivers/gpu/drm/bri
With a call to drm_of_panel_bridge_remove() we could remove the bridge
without store it in ldtc internal driver structure.
Signed-off-by: Benjamin Gaignard
---
drivers/gpu/drm/stm/ltdc.c | 16 +---
drivers/gpu/drm/stm/ltdc.h | 2 --
2 files changed, 5 insertions(+), 13 deletions(-)
This function is the pendant of drm_of_find_panel_or_bridge()
to remove a previously allocated panel_bridge.
Given a specific port and endpoint it remove the panel bridge.
Since drm_panel_bridge_remove() will check that bridge parameter
is not NULL and is a real drm_panel_bridge and no a simple bri
The goal of this series is to simplify driver code when they need to clean up
a previously allocated panel bridge.
Few drivers have "is_panel_bridge" flag to be able to distinguish a
drm_panel_bridge from "simple" drm_bridge.
To remove this flag I propose to
- let drm_panel_bridge_remove() check if
On Fri, Sep 29, 2017 at 1:42 AM, Dave Airlie wrote:
> On 29 September 2017 at 06:41, Marek Olšák wrote:
>> Can I get Rb for this series?
>>
>
> For the series,
>
> Reviewed-by: Dave Airlie
>
> Alex, please merge the two drm core precursor with patch 3.
Alex, this is for drm-next, where I can't
On Fri, Sep 29, 2017 at 02:20:16PM +0200, Noralf Trønnes wrote:
>
> Den 29.09.2017 05.17, skrev Meghana Madhyastha:
> >On Thu, Sep 28, 2017 at 06:19:35PM +0200, Noralf Trønnes wrote:
> >>Den 28.09.2017 11.15, skrev Meghana Madhyastha:
> >>>Add devm_drm_of_find_backlight and the corresponding relea
On Wed, 2017-09-20 at 20:20 +0200, Daniel Vetter wrote:
> On Mon, Sep 11, 2017 at 01:06:32PM +0200, Christian König wrote:
> > Am 11.09.2017 um 12:01 schrieb Chris Wilson:
> > > [SNIP]
> > > > Yeah, but that is illegal with a fence objects.
> > > >
> > > > When anybody allocates fences this way it
Den 29.09.2017 14.20, skrev Meghana Madhyastha:
On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trønnes wrote:
Den 29.09.2017 05.22, skrev Meghana Madhyastha:
On Thu, Sep 28, 2017 at 06:02:27PM +0200, Noralf Trønnes wrote:
Den 28.09.2017 16.08, skrev Daniel Vetter:
On Thu, Sep 28, 2017 at 0
Den 29.09.2017 09.25, skrev Daniel Vetter:
On Wed, Sep 27, 2017 at 01:38:46AM -0600, Haneen Mohammed wrote:
Remove drmP.h as it is not needed anymore since nothing it
defines is used in these files.
Signed-off-by: Haneen Mohammed
Applied, thanks.
drivers/gpu/drm/armada/armada_fb.c uses DRM
Hi Laurent,
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 2016-02-23 23:57, Laurent Pinchart wrote:
> Hi Tomi,
>
> Thank you for the patch.
>
> On Friday 19 February 2016 11:47:40 Tomi Valkeinen wrote:
>> Err
On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trønnes wrote:
>
> Den 29.09.2017 05.22, skrev Meghana Madhyastha:
> >On Thu, Sep 28, 2017 at 06:02:27PM +0200, Noralf Trønnes wrote:
> >>Den 28.09.2017 16.08, skrev Daniel Vetter:
> >>>On Thu, Sep 28, 2017 at 02:44:34PM +0530, Meghana Madhyastha wr
Den 29.09.2017 05.17, skrev Meghana Madhyastha:
On Thu, Sep 28, 2017 at 06:19:35PM +0200, Noralf Trønnes wrote:
Den 28.09.2017 11.15, skrev Meghana Madhyastha:
Add devm_drm_of_find_backlight and the corresponding release
function because some drivers such as tinydrm use devres versions
of func
Den 29.09.2017 05.22, skrev Meghana Madhyastha:
On Thu, Sep 28, 2017 at 06:02:27PM +0200, Noralf Trønnes wrote:
Den 28.09.2017 16.08, skrev Daniel Vetter:
On Thu, Sep 28, 2017 at 02:44:34PM +0530, Meghana Madhyastha wrote:
Rename tinydrm_of_find_backlight to drm_of_find_backlight
and move it
The error bit definitions are typoed in DMM_IRQSTAT_ERR_MASK which went
unnoticed since the DMM_IRQSTAT_ERR_MASK was not used.
Change the bit definitions to the correct ones.
Signed-off-by: Peter Ujfalusi
---
drivers/gpu/drm/omapdrm/omap_dmm_priv.h | 12 ++--
1 file changed, 6 insertio
To help diagnose DMM errors, print out information if any of the error bits
are set in the interrupt status register.
Signed-off-by: Peter Ujfalusi
---
drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
b/d
Check the status of the DMM engine after it is reported that the
transaction was completed as in rare cases the engine might not reached a
working state.
The wait_status() will print information in case the DMM is not reached the
expected state and the dmm_txn_commit() will return with an error c
If the wait_status() fails either because of an error reported in the
STATUS register or because of a timeout waiting for the wait_mask, print
information which might help diagnose the reason.
Signed-off-by: Peter Ujfalusi
---
drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 12 ++--
1 file c
Hi,
the following series adds basic error handling and reporting for the dmm/tiler
driver.
With the error handling in place we can catch the problems early to avoid more
serious consequences.
Regards,
Peter
---
Peter Ujfalusi (4):
drm/omap: DMM: Fix DMM_IRQSTAT_ERR_MASK definition
drm/omap:
On Thu, 28 Sep 2017, Rodrigo Vivi wrote:
> Merged both patches to dinq. Thanks for the patches.
While patch 1 was a simple addition of a few DP macros, we need to get
ack from Dave or (preferrably non-Intel) drm-misc maintainers before
queuing non-i915 patches through drm-intel.
Dave, Sean, ack
On Mon, 2017-09-04 at 18:16 +0300, Marius Vlad wrote:
> From: Marius Vlad
>
> Signed-off-by: Marius Vlad
> Signed-off-by: Marius-Adrian Negreanu
> @@ -1183,6 +1183,71 @@ static void i915_driver_cleanup_hw(struct
> drm_i915_private *dev_priv)
> i915_ggtt_cleanup_hw(dev_priv);
> }
>
On Fri, Sep 29, 2017 at 6:20 PM, Maxime Ripard
wrote:
> On Fri, Sep 29, 2017 at 08:22:56AM +, Chen-Yu Tsai wrote:
>> On systems with 2 TCONs such as the A31, it is possible to demux the
>> output of the TCONs to one encoder.
>>
>> Add support for this for the A31.
>>
>> Signed-off-by: Chen-Yu
On Fri, Sep 29, 2017 at 08:22:56AM +, Chen-Yu Tsai wrote:
> On systems with 2 TCONs such as the A31, it is possible to demux the
> output of the TCONs to one encoder.
>
> Add support for this for the A31.
>
> Signed-off-by: Chen-Yu Tsai
> ---
> drivers/gpu/drm/sun4i/sun4i_tcon.c | 38
> +++
On Fri, Sep 29, 2017 at 08:22:55AM +, Chen-Yu Tsai wrote:
> static const struct sun4i_tcon_quirks sun5i_a13_quirks = {
> - .has_unknown_mux = true,
> - .has_channel_1 = true,
> + .has_unknown_mux= true,
> + .has_channel_1 = true,
> + .set_mux
MIXER in SoCs prior to Exynos5420 supports only 4 video modes:
720x480, 720x576, 1280x720, 1920x1080. Support for other modes
can be enabled by manipulating timings of HDMI. To do it
adjusted_mode should contain actual mode set on crtc.
With this patch it is possible to enable 1024x768 and 1280x102
crtc::mode_fixup callback is required by crtcs which internally use
a different mode than requested by user - case of Exynos Mixer.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_drm_crtc.c | 15 +++
drivers/gpu/drm/exynos/exynos_drm_drv.h
From: Daniel Drake
Configuration details from Samsung. This enables 1366x768@60Hz,
which also needs the 256px timing hack to work around a mixer
limitation.
Signed-off-by: Daniel Drake
Signed-off-by: Tobias Jakobi
[a.ha...@samsung.com: rebased onto proposed patchset]
Signed-off-by: Andrzej Haj
Screen resolution configuration depends on HW version, let's put it into
single function to make it consistent and simplify the code.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 17 +++--
1 file changed, 7 insertions(+), 10 del
Since HDMI can handle these modes despite of MIXER limitations let's
enable them.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c
b/drivers/gpu/drm/exy
mixer_resources adds only unnecessary redirection, removing it makes the
code shorter and cleaner.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 323 --
1 file changed, 147 insertions(+), 176 deletions(-)
dif
MIXER in SoCs prior to Exynos5420 supports only 4 video modes:
720x480, 720x576, 1280x720, 1920x1080. Support for other modes can be
enabled by manipulating timings of HDMI. To allow it MIXER must pass
actual video mode to HDMI, the proper way to do it is to modify
adjusted_mode property in crtc::m
Display mode is preserved in CRTC state, there is no need to keep local
copy of it. Moreover since HDMI should configure registers according to
requested mode, use it instead of adjusted_mode, which should contain
mode produced by CRTC - functionally it does not change anything, but
subsequent patc
Mode limitation checked in mixer driver affects only older HW.
Mixer in Exynos542x has no such limitations. While at it patch changes
validation callback to recently introduced mode_valid which is more
suitable for the check. Additionally little cleanup is performed.
Signed-off-by: Andrzej Hajda
Mode setup code is called from video plane update and mixer plane update.
Let's group it together in mixer_commit function like in case of other
Exynos CRTCs.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakobi
---
drivers/gpu/drm/exynos/exynos_mixer.c | 33 ++---
Mode commit should not be called for every plane separately. It is enough
to call it once in enable callback. The change also requires that
the interlace check is moved to mixer_commit. It should be done in
the same patch to avoid regression.
Signed-off-by: Andrzej Hajda
Reviewed-by: Tobias Jakob
Hi all,
This patchset does two main things:
- removes mode limitation for Exynos542x chips, multiple modes were filtered
out due to lack of HW version checking code,
- enables two modes on older chips, thanks to quirk found by Daniel Drake,
and published by Tobias Jakobi [1][2].
Beside this it
2017-09-29 16:44 GMT+08:00 Robert Foss :
> On Fri, 2017-09-29 at 13:49 +0800, Chih-Wei Huang wrote:
>> 2017-09-29 5:29 GMT+08:00 Rob Herring :
>> > Perhaps just leave the current state as a separate branch.
>>
>> Did you mean we maintain the branch in our repo?
>> (that's what we do now, but I hope
On Fri, Sep 29, 2017 at 08:22:54AM +, Chen-Yu Tsai wrote:
> The HDMI DDC clock found in the CCU is the parent of the actual DDC
> clock within the HDMI controller. That clock is also named "hdmi-ddc".
>
> Rename the one in the CCU to "ddc". This makes more sense than renaming
> the one in the
On Fri, Sep 29, 2017 at 08:22:53AM +, Chen-Yu Tsai wrote:
> The 2x outputs of the 2 video PLL clocks are directly used by the
> HDMI controller block.
>
> Export them so they can be referenced in the device tree.
>
> Fixes: c6e6c96d8fa6 ("clk: sunxi-ng: Add A31/A31s clocks")
> Signed-off-by:
On Fri, 2017-09-29 at 13:49 +0800, Chih-Wei Huang wrote:
> 2017-09-29 5:29 GMT+08:00 Rob Herring :
> > On Thu, Sep 28, 2017 at 11:43 AM, Chih-Wei Huang
> > wrote:
> > > 2017-09-27 19:58 GMT+08:00 Robert Foss > > >:
> > > > From: Sean Paul
> > > >
> > > > Since HWC2 doesn't require the use of th
On 09/29/2017 09:32 AM, Marek Szyprowski wrote:
> From: Andrzej Pietrasiewicz
>
> TOP "aclk400_mscl" clock should be kept enabled all the time to allow
> proper access to power management control for MSC power domain and
> devices that are a part of it. This change is required for scaler to
> wor
Different SoCs have different muxing options and values for the TCON
outputs. Instead of stuffing every possibility in sun4i_tcon_set_mux(),
add a callback pointer to sun4i_tcon_quirks that each TCON variant
can use to provide muxing support.
The current muxing options in sun4i_tcon_set_mux() for
This patch adds a macro regmap_field_read_poll_timeout that works
similar to the readx_poll_timeout defined in linux/iopoll.h, except
that this can also return the error value returned by a failed
regmap_field_read.
Signed-off-by: Chen-Yu Tsai
---
include/linux/regmap.h | 39
Now that we support the HDMI controller on the A31 SoC, we can add it
to the device tree.
This adds a device node for the HDMI controller, and the of_graph nodes
connecting it to the 2 TCONs.
Signed-off-by: Chen-Yu Tsai
---
arch/arm/boot/dts/sun6i-a31.dtsi | 55 +
The HDMI driver is written with readl/writel I/O to the registers.
However, to support the A31 variant, which has a different layout
for the DDC registers, it was recommended to use regfields to have
a cleaner implementation. To use regfields, we need to create an
underlying regmap.
This patch onl
The HDMI controller found in earlier Allwinner SoCs have slight
differences between the A10, A10s, and the A31:
- Need different initial values for the PLL related registers
- Different behavior of the DDC and TMDS clocks
- Different register layout for the DDC portion
- Separate DDC pa
On SoCs with two display pipelines, it is possible that the two
pipelines are active at the same time, with potentially incompatible
dot clocks.
Let the HDMI encoder's TMDS clock go through all of its parents when
calculating possible clock rates. This allows usage of the second video
PLL as its p
The 2x outputs of the 2 video PLL clocks are directly used by the
HDMI controller block.
Export them so they can be referenced in the device tree.
Fixes: c6e6c96d8fa6 ("clk: sunxi-ng: Add A31/A31s clocks")
Signed-off-by: Chen-Yu Tsai
Acked-by: Maxime Ripard
---
drivers/clk/sunxi-ng/ccu-sun6i-a
The HDMI controller in the A31 SoC is slightly different from the
earlier version. In addition to the TMDS clock and DDC controls,
this version now takes a second DDC clock input.
Add a compatible string for it, and add the DDC clock input to the
list of clocks required.
Signed-off-by: Chen-Yu Ts
On systems with 2 TCONs such as the A31, it is possible to demux the
output of the TCONs to one encoder.
Add support for this for the A31.
Signed-off-by: Chen-Yu Tsai
---
drivers/gpu/drm/sun4i/sun4i_tcon.c | 38 ++
1 file changed, 38 insertions(+)
diff --git
The HDMI DDC clock found in the CCU is the parent of the actual DDC
clock within the HDMI controller. That clock is also named "hdmi-ddc".
Rename the one in the CCU to "ddc". This makes more sense than renaming
the one in the HDMI controller to something else.
Fixes: c6e6c96d8fa6 ("clk: sunxi-ng:
The HDMI controller found in the A31 SoCs is slightly different
from the one already supported, which is found in the A10s:
- Need different initial values for the PLL related registers
- Different behavior of the DDC and TMDS clocks
- Different register layout for the DDC portion
- Sep
The HDMI driver enables the bus and mod clocks in the bind function, but
does not disable them if it then bails our due to any errors. Neither
does it disable the clocks in the unbind function.
Fix this by adding a proper error path to the bind function, and
clk_disable_unprepare calls to the unbi
All the A31/A31s devices I own have some kind of HDMI connector wired
to the dedicated HDMI pins on the SoC:
- A31 Hummingbird (standard HDMI connector, display already enabled)
- Sinlinx SinA31s (standard HDMI connector)
- MSI Primo81 tablet (micro HDMI connector)
Enable the display pipeli
Hi everyone,
This is v3 of my A31 HDMI support series.
Changes since v2:
- TCON muxing moved into functions for each platform, with pointers
to them in the TCON quirks structure.
- CCU "hdmi-ddc" clock renamed to "ddc".
- Added Maxime's acks.
Changes since v1:
- Core ch
The DDC block for the HDMI controller is different on the A31.
This patch adds the register definitions.
Signed-off-by: Chen-Yu Tsai
Acked-by: Maxime Ripard
---
drivers/gpu/drm/sun4i/sun4i_hdmi.h | 31 +++
1 file changed, 31 insertions(+)
diff --git a/drivers/gpu/d
Some hardware modules, like FIMC in Exynos4 series are shared between
V4L2 (camera support) and DRM (memory-to-memory processing) subsystems.
This patch provides a simple check to let such drivers to be used in the
driver components framework.
Signed-off-by: Marek Szyprowski
---
drivers/gpu/drm/
From: Andrzej Pietrasiewicz
TOP "aclk400_mscl" clock should be kept enabled all the time to allow
proper access to power management control for MSC power domain and
devices that are a part of it. This change is required for scaler to
work properly after domain power on/off sequence.
Fixes: 318fa
1 - 100 of 117 matches
Mail list logo
