2016ë
09ì 28ì¼ 09:12ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Hello Inki,
>
>
> Inki Dae wrote:
>>
>>
>> 2016ë
09ì 28ì¼ 08:31ì Inki Dae ì´(ê°) ì´ ê¸:
>>>
>>>
>>> 2016ë
09ì 28ì¼ 01:52ì Tobias Jakobi ì´(ê°) ì´ ê¸
2016ë
09ì 28ì¼ 09:03ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Hey Inki,
>
>
> Inki Dae wrote:
>>
>>
>> 2016ë
09ì 28ì¼ 01:52ì Tobias Jakobi ì´(ê°) ì´ ê¸:
>>> Hello Andrzej,
>>>
>>>
>>> Andrzej Hajda wrot
2016ë
09ì 28ì¼ 08:31ì Inki Dae ì´(ê°) ì´ ê¸:
>
>
> 2016ë
09ì 28ì¼ 01:52ì Tobias Jakobi ì´(ê°) ì´ ê¸:
>> Hello Andrzej,
>>
>>
>> Andrzej Hajda wrote:
>>> On 27.09.2016 13:22, Tobias Jakobi wrote:
>>>> Hello Inki,
>&
We would need to review this patch series for -next in a hurry.
I have to request git-pull soon.
2016ë
09ì 28ì¼ 00:50ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Hello everyone,
>
> as discussed with Marek I have broken down my initial patch into smaller
> piecer.
>
> Anyway, this series fixes a
2016ë
09ì 27ì¼ 20:22ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Hey Inki,
>
>
> Inki Dae wrote:
>> 2016ë
09ì 27ì¼ 14:40ì Tobias Jakobi ì´(ê°) ì´ ê¸:
>>> Inki Dae wrote:
>>>>
>>>>
>>>> 2016ë
09ì 26ì¼ 20:36ì To
2016ë
09ì 28ì¼ 01:52ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Hello Andrzej,
>
>
> Andrzej Hajda wrote:
>> On 27.09.2016 13:22, Tobias Jakobi wrote:
>>> Hello Inki,
>>>
>>>
>>> Inki Dae wrote:
>>>> 2016ë
09ì 26ì¼ 20:36ì
2016ë
09ì 27ì¼ 14:40ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Inki Dae wrote:
>>
>>
>> 2016ë
09ì 26ì¼ 20:36ì Tobias Jakobi ì´(ê°) ì´ ê¸:
>>> Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
>>> in mixer_cfg_layer().
>>> Trig
2016ë
09ì 26ì¼ 20:36ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
> in mixer_cfg_layer().
> Trigger this via atomic flush.
>
> Changes in v2:
> - issue mixer_cfg_layer() in mixer_disable()
> - rename fields as suggested by Andrzej
> -
2016ë
09ì 26ì¼ 20:36ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
> in mixer_cfg_layer().
> Trigger this via atomic flush.
>
> Changes in v2:
> - issue mixer_cfg_layer() in mixer_disable()
> - rename fields as suggested by Andrzej
> -
2016ë
09ì 22ì¼ 23:57ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Apply some 'make-up' in g2d_probe().
Just clean up. Picked it up.
>
> Signed-off-by: Tobias Jakobi
> ---
> drivers/gpu/drm/exynos/exynos_drm_g2d.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
2016ë
09ì 22ì¼ 23:57ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
> in mixer_cfg_layer().
> Trigger this via atomic flush.
>
> Changes in v2:
> - issue mixer_cfg_layer() in mixer_disable()
> - rename fields as suggested by Andrzej
> -
Picked this one up.
Thanks,
Inki Dae
2016ë
09ì 22ì¼ 23:57ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> The mixer context struct already has a 'flags' field, so
> we can use it to store the 'interlace', 'vp_enabled' and
> 'has_sclk' booleans.
> We use the non-atomic helper functio
Hi Dave,
There was my mistake reported below when I merged one fixup manually.
http://www.spinics.net/lists/dri-devel/msg118283.html
Fixed.
Thanks,
Inki Dae
2016ë
09ì 18ì¼ 23:12ì Inki Dae ì´(ê°) ì´ ê¸:
> Hi Dave,
>
>Just fixup to runtime pm usage and some
Hi Dave,
Just fixup to runtime pm usage and some cleanups.
Please kindly let me know if there is any problem.
Ps. We will request git-pull for -next soon if no critical issue,
which includes only code refactoring on hdmi ddc and phy.
Thanks,
Inki Dae
The following changes
Hi,
Thanks for your contribution.
Seems good to me. Will picked them up soon if no critical issues.
Thanks,
Inki Dae
2016-08-31 15:14 GMT+09:00 Milo Kim :
> v4:
> Only DRM patchset is sent, DTS patch was sent separately.
>
> Milo Kim (3):
> gpu: drm: exynos_hdmi:
2016ë
08ì 25ì¼ 21:14ì Daniel Vetter ì´(ê°) ì´ ê¸:
> On Thu, Aug 25, 2016 at 08:45:25PM +0900, Inki Dae wrote:
>>
>>
>> 2016ë
08ì 25ì¼ 17:42ì Daniel Vetter ì´(ê°) ì´ ê¸:
>>> On Thu, Aug 25, 2016 at 05:06:55PM +0900, Inki Dae wrote:
>>>
2016ë
08ì 25ì¼ 17:42ì Daniel Vetter ì´(ê°) ì´ ê¸:
> On Thu, Aug 25, 2016 at 05:06:55PM +0900, Inki Dae wrote:
>>
>>
>> 2016ë
08ì 24ì¼ 20:57ì Daniel Vetter ì´(ê°) ì´ ê¸:
>>> On Wed, Aug 24, 2016 at 08:44:24PM +0900, Inki Dae wrote:
>>>
2016ë
08ì 24ì¼ 20:57ì Daniel Vetter ì´(ê°) ì´ ê¸:
> On Wed, Aug 24, 2016 at 08:44:24PM +0900, Inki Dae wrote:
>> Hi,
>>
>> 2016ë
08ì 23ì¼ 18:41ì Daniel Stone ì´(ê°) ì´ ê¸:
>>> Hi,
>>>
>>> On 22 August 2016 at 16:23, Rob Cla
seems other maintainers leave NAK for this patch series
because this trying had failed already long time ago - they had learned such
thing didn't work well. So seems we have to keep this in only Exynos DRM.
[1] https://lwn.net/Articles/640290/
Thanks,
Inki Dae
>
> Cheers,
> D
Hi Shuah,
2016ë
08ì 13ì¼ 02:52ì Shuah Khan ì´(ê°) ì´ ê¸:
> On 08/12/2016 11:28 AM, Shuah Khan wrote:
>> On 08/10/2016 05:05 PM, Shuah Khan wrote:
>>> On 08/10/2016 04:59 PM, Inki Dae wrote:
>>>> Hi Shuah,
>>>>
>>>> 2016ë
08ì
imedia
device and other DMA devices.
Even though IOMMU support is disabled, other framework based DMA drivers can
use IOMMU - i.e., GPU driver -
and they can use non-contiguous GEM buffer through UMM. (DMABUF)
So GEM allocation type is not dependent on IOMMU.
Thanks,
Inki Dae
> allocate non-c
Picked it up. :)
Thanks,
Inki Dae
2016-08-09 8:48 GMT+09:00 Shuah Khan :
> Fix unsupported GEM memory type error message to include the memory type
> information.
>
> Signed-off-by: Shuah Khan
> ---
> Changes since v1:
> -- Comment changed to read clearly
>
&
buf_fence __user *arg)
> +{
> + struct reservation_object *resv = dmabuf->resv;
> + struct dma_buf_fence cmd;
> + struct fence *fence;
> + int ret;
> +
> + if (copy_from_user(, arg, sizeof(cmd)))
> + return -EFAULT;
> +
> + fence = NULL;
> + //fence = sync_file_get_fence(cmd.fd);
Your mistake?
Thanks,
Inki Dae
> + if (!fence)
> + return -EINVAL;
> +
as-is.
Was there any changes for Android sync driver to be dependent on DMA buffer?
Thanks,
Inki Dae
>
> with the greatest victim being amdgpu.
>
> Just the highlights shown below.
> -Chris
>
> ---
> drivers/base/Kconfig| 6 +-
> drive
in exynos_drm_iommu.h
for ARM32/64 supports because these two architectures are not compatible
yet so the conditional codes will be removed later with architectures
unified.
Please kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit
t; structure, because it will be freed on driver remove not on device
> release. This means in case of Exynos DRM being compiled as module and
> loaded 2 times, a user-after-free issue will happen.
Picked this patch series up.
Thanks,
Inki Dae
>
> Signed-off-by: Marek Szyprowski
before.
However, we will need to consider PSR(Panel Self Reflash) mode to resolve
this issue fundamentally later.
Please kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit 0ab15bdeb2943bd6491a35ec4eeb53a9a4436525:
Merge branch 'drm-fixes
2016ë
06ì 10ì¼ 09:24ì Javier Martinez Canillas ì´(ê°) ì´ ê¸:
> Hello Inki,
>
> On Thu, Jun 9, 2016 at 6:35 PM, Inki Dae wrote:
>
> [snip]
>
>>> I know that removing .trg_type is enough but I also removed those lines
>>> because the fields are
Picked it up.
Thanks,
Inki Dae
2016ë
06ì 10ì¼ 19:12ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Ping!
>
> - Tobias
>
> Tobias Jakobi wrote:
>> This makes the defines consistent with the rest.
>>
>> Signed-off-by: Tobias Jakobi
>> ---
>>
Picked it up.
Thanks,
Inki Dae
2016ë
06ì 10ì¼ 19:12ì Tobias Jakobi ì´(ê°) ì´ ê¸:
>> 3 files changed, 3 deletions(-)
Picked it up.
Thanks,
Inki Dae
2016ë
06ì 10ì¼ 19:12ì Tobias Jakobi ì´(ê°) ì´ ê¸:
> Ping!
>
> - Tobias
>
> Tobias Jakobi wrote:
>> The current bitwise AND should result in the same assembler
>> but this is what the code is actually supposed to do.
>>
2016ë
05ì 30ì¼ 03:35ì Daniel Vetter ì´(ê°) ì´ ê¸:
> We want to hide drm_atomic_state internals better.
Acked-by: Inki Dae
Thanks,
Inki Dae
>
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 8
&g
this function
with mutex lock?
If there is such case then,
some_function()
{
mutex_lock();
...
mutex_unlock();
drm_fb_helper_add_one_connector();
...
}
So in this case, other users should consider to make sure to unlock before
calling this function. That
Hi Javier,
2016ë
06ì 09ì¼ 09:17ì Javier Martinez Canillas ì´(ê°) ì´ ê¸:
> Hello Inki,
>
> Thanks for your feedback.
>
> On 06/08/2016 07:09 PM, Inki Dae wrote:
>> Hi Javier,
>>
>> 2016ë
06ì 02ì¼ 23:20ì Javier Martinez Canillas ì´(ê°) ì´ ê
shift = 11,
> - .trg_type = I80_HW_TRG,
> .has_shadowcon = 1,
> .has_vidoutcon = 1,
> .has_vtsel = 1,
> .has_mic_bypass = 1,
> .has_dp_clk = 1,
> - .has_hw_trigger = 1,
> - .has_trigger_per_te = 1,
We don't need to remove above
Hi Javier,
2016ë
05ì 31ì¼ 07:58ì Javier Martinez Canillas ì´(ê°) ì´ ê¸:
> Hello Inki,
>
> On 04/05/2016 04:27 AM, Inki Dae wrote:
>> This patch adds HW trigger support on i80 mode.
>>
>> Until now, Exynos DRM only supported SW trigger which was set
>>
mode of Panel device.
So this patch makes sure to change the trigger mode after power off
and on again. Later we need to add PSR relevant codes instead.
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35
1 file changed, 35 insertions
Hi Javier,
2016ë
05ì 31ì¼ 07:58ì Javier Martinez Canillas ì´(ê°) ì´ ê¸:
> Hello Inki,
>
> On 04/05/2016 04:27 AM, Inki Dae wrote:
>> This patch adds HW trigger support on i80 mode.
>>
>> Until now, Exynos DRM only supported SW trigger which was set
>>
2016-05-30 3:35 GMT+09:00 Daniel Vetter :
> We want to hide drm_atomic_state internals better.
>
Acked-by: Inki Dae
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 8
> 1 file changed, 4 insertions(+), 4 deleti
if there is any problem.
Thanks,
Inki Dae
The following changes since commit 2e726dc4b4e2dd3ae3fe675f9d3af88a2d593ee1:
Merge tag 'mediatek-drm-2016-05-09' of git://git.pengutronix.de/git/pza/linux
into drm-next (2016-05-10 15:01:47 +1000)
are available in the git repository at:
git
2016ë
05ì 10ì¼ 16:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> On 05/10/2016 08:24 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
05ì 10ì¼ 15:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>>> Hi Inki,
>>>
>>>
>>> On 05/10/2016 07:31 AM, Ink
This patch changes GPL license to X11/MIT.
Signed-off-by: Inki Dae
---
exynos/exynos_fimg2d.c | 21 +
exynos/exynos_fimg2d.h | 21 +
exynos/fimg2d_reg.h| 21 +
tests/exynos/exynos_fimg2d_event.c
2016ë
04ì 27ì¼ 02:29ì Daniel Vetter ì´(ê°) ì´ ê¸:
> No dev->struct_mutex anywhere to be seen.
Acked-by: Inki Dae
Thanks,
Inki Dae
>
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +-
> 1 file ch
t not now yet. Picked it up.
Thanks,
Inki Dae
>
> Cc: Inki Dae
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/exynos/exynos_drm_fb.c | 11 ---
> 1 file changed, 11 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fb.c
> b/drivers/gp
Hi Andrzej,
2016ë
05ì 10ì¼ 15:08ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> Hi Inki,
>
>
> On 05/10/2016 07:31 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
04ì 29ì¼ 22:42ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>>> It seems trigger cannot be configure
Shouldn't it be configured in case of using SW trigger mode also?
Thanks,
Inki Dae
> +
> /* lcd on and use command if */
> val = VIDOUT_LCD_ON;
> if (ctx->out_type & IFTYPE_I80) {
> val |= VIDOUT_COMMAND_IF;
> - decon_setup_trigger(c
t;> write directly without using the shadow registers.
>
> My knowledge about fimd is also limited, but it seems it is not possible
Right, not possible to update registers directly in Exynos SoC.
> to update registers directly - it looks for me reasonable - update on
> vsync is a cheap way to make
ld you queue it to next pull request?
>
Got it.
Thanks,
Inki Dae
> Regards
> Andrzej
>
> On 03/23/2016 02:25 PM, Andrzej Hajda wrote:
>> HDMI-PHY clock should be accessible from other components in the pipeline.
>>
>> Signed-off-by: Andrze
trigger
mode so this patch makes trigger mode - HW or SW trigger - to be
set according to SoC version properly.
- And some cleanups and regression fixups.
Please, kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since
;> to "exynos5420-fimd". To support mic bypass from display
>>>> path, the new compatible is introduced for exynos5420.
>>>>
>>>> Cc: Inki Dae
>>>> Cc: Kukjin Kim
>>>> Cc: Krzysztof Kozlowski
>>>> Signed-off-by:
= of_device_get_match_data(dev);
drivers/gpu/drm/exynos/exynos_drm_dsi.c: In function 'exynos_dsi_probe':
drivers/gpu/drm/exynos/exynos_drm_dsi.c:1827:19: warning: assignment discards
'const' qualifier from pointer target type
Looks you missed const prefix and for patch 3 also but I can fix them.
Thanks,
Inki Dae
>
> ret = exynos_dsi_parse_dt(dsi);
> if (ret)
>
HI Dave,
This pull request includes several regression fixups.
Please kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit 928815245cbdaa611873424759d5e7a7293dd18b:
Merge tag 'drm-intel-fixes-2016-04-07' of
git://anongit.freedesktop.org/drm
2016ë
04ì 12ì¼ 16:33ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> On 04/12/2016 04:25 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
03ì 23ì¼ 22:15ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>>> DECON should be updated after un-protecting windows and after changing
&g
because HDMI Timing Generator generates VSYNC signal
which works as a hardware trigger.
Changelog v2.
- If interface type is HDMI then set out_type to I80.
- fix compile warning.
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 57 ++-
1 file changed
2016ë
04ì 12ì¼ 14:55ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> Hi Inki,
>
> On 04/12/2016 02:40 AM, Inki Dae wrote:
>> Hi Andrzej,
>>
>> 2016ë
04ì 05ì¼ 21:52ì Inki Dae ì´(ê°) ì´ ê¸:
>>> Hi Andrzej,
>>>
>>> 2016-04-05 20:07 GMT+
output?
In addition, we would need another patch which updates TV relevant registers
only in case of DECON-TV. DECON_UPDATE is a register for DECON-TV.
Thanks,
Inki Dae
>
> Signed-off-by: Andrzej Hajda
> ---
> drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 15 ---
&g
This patch removes suffixes from I80 relevant register definitions,
which are misleading.
This is based on top of below patch set,
http://www.spinics.net/lists/dri-devel/msg104057.html
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 2 +-
drivers/gpu/drm
Hi Andrzej,
2016ë
04ì 05ì¼ 21:52ì Inki Dae ì´(ê°) ì´ ê¸:
> Hi Andrzej,
>
> 2016-04-05 20:07 GMT+09:00 Andrzej Hajda :
>> Hi Inki,
>>
>> On 04/05/2016 10:27 AM, Inki Dae wrote:
>>> This patch cleans up interface type relevant codes.
>>>
>
Hi Andrzej,
2016-04-05 20:07 GMT+09:00 Andrzej Hajda :
> Hi Inki,
>
> On 04/05/2016 10:27 AM, Inki Dae wrote:
>> This patch cleans up interface type relevant codes.
>>
>> Trigger mode is determinded only by i80 mode, which isn't
>> related to Display types -
This patch cleans up interface type relevant codes.
Trigger mode is determinded only by i80 mode, which isn't
related to Display types - HDMI or Display controller.
So this patch makes the trigger mode to be set only in case of
i80 mode.
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos
by FIMD controller.
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 47
1 file changed, 47 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index 752c6b0..c4cd16a 100644
page fault issue so keep it.
Signed-off-by: Inki Dae
---
drivers/gpu/drm/exynos/exynos7_drm_decon.c | 1 -
drivers/gpu/drm/exynos/exynos_drm_drv.h| 3 ---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 1 -
drivers/gpu/drm/exynos/exynos_mixer.c | 39 --
4 files
, HW trigger mode will be used in default if
SoC suppots HW trigger - Exynos3250, 3472, 4415, 5260 5410, 5420 and 5422.
P.s. HW trigger support for Exynos 7 series will be added later.
Thanks,
Inki Dae
Inki Dae (3):
drm/exynos: clean up wait_for_vblank
drm/exynos: fimd: add HW trigger support
2016ë
03ì 31ì¼ 23:10ì Rob Clark ì´(ê°) ì´ ê¸:
> On Thu, Mar 31, 2016 at 7:26 AM, Inki Dae wrote:
>> Hi Daniel,
>>
>> 2016-03-31 19:56 GMT+09:00 Daniel Stone :
>>> Hi Inki,
>>>
>>> On 31 March 2016 at 11:05, Inki Dae wrote:
>>>&
only be used for shared buffers, e.g.
> explicit syncing in an android client running on top of implicitly synced
> ozone/kms.
Good idea. I hope fence things of mainline would be more discussed so
could be considered for many cases.
Thanks,
Inki Dae
> -Daniel
> --
> Daniel V
Hi Daniel,
2016-03-31 19:56 GMT+09:00 Daniel Stone :
> Hi Inki,
>
> On 31 March 2016 at 11:05, Inki Dae wrote:
>> 2016ë
03ì 31ì¼ 18:35ì Daniel Stone ì´(ê°) ì´ ê¸:
>>> On 31 March 2016 at 08:45, Inki Dae wrote:
>>>> As of now, it seems that thi
Hi Daniel,
2016ë
03ì 31ì¼ 18:35ì Daniel Stone ì´(ê°) ì´ ê¸:
> Hi Inki,
>
> On 31 March 2016 at 08:45, Inki Dae wrote:
>> 2016ë
03ì 29ì¼ 22:23ì Rob Clark ì´(ê°) ì´ ê¸:
>>> On Mon, Mar 28, 2016 at 10:18 PM, Inki Dae wrote:
>>>>
2016ë
03ì 29ì¼ 22:23ì Rob Clark ì´(ê°) ì´ ê¸:
> On Mon, Mar 28, 2016 at 10:18 PM, Inki Dae wrote:
>>
>> In addition, I wonder how explicit and implicit fences could coexist
>> together.
>> Rob said,
>> "Implicit sync ofc remain
Hi Daniel,
2016ë
03ì 28ì¼ 22:26ì Daniel Stone ì´(ê°) ì´ ê¸:
> Hi Inki,
>
> On 28 March 2016 at 02:26, Inki Dae wrote:
>> 2016ë
03ì 25ì¼ 21:10ì Daniel Stone ì´(ê°) ì´ ê¸:
>>> Second, really. Vulkan avoids implicit sync entirely, and exposes
&
Hi Javier,
Thanks for your patch set.
Will merge them if there is no issue.
Thanks,
Inki Dae
2016ë
03ì 29ì¼ 10:28ì Javier Martinez Canillas ì´(ê°) ì´ ê¸:
> Hello Inki,
>
> This patch series contains some fixes for the Kconfig symbol dependencies
> of the Exyno
Hi Rob and Daniel,
2016ë
03ì 25ì¼ 21:10ì Daniel Stone ì´(ê°) ì´ ê¸:
> Hi all,
>
> On 25 March 2016 at 11:58, Rob Clark wrote:
>> On Thu, Mar 24, 2016 at 7:49 PM, Inki Dae wrote:
>>> It's definitely different case. This tries to add new user-space interfac
al &= ~(MIC0_RGB_MUX | MIC0_I80_MUX | MIC0_ON_MUX);
>
> regmap_write(mic->sysreg, DSD_CFG_MUX, val);
> - if (ret)
> - DRM_ERROR("mic: Failed to read system register\n");
I think we missed to keep return value from regmap_write function,
ret = regma
2016ë
03ì 25ì¼ 00:40ì Rob Clark ì´(ê°) ì´ ê¸:
> On Thu, Mar 24, 2016 at 4:18 AM, Inki Dae wrote:
>> Hi,
>>
>> 2016ë
03ì 24ì¼ 03:47ì Gustavo Padovan ì´(ê°) ì´ ê¸:
>>> From: Gustavo Padovan
>>>
>>> Hi,
>>>
>>
Hi Guestavo,
2016ë
03ì 24ì¼ 23:39ì Gustavo Padovan ì´(ê°) ì´ ê¸:
> Hi Inki,
>
> 2016-03-24 Inki Dae :
>
>> Hi,
>>
>> 2016ë
03ì 24ì¼ 03:47ì Gustavo Padovan ì´(ê°) ì´ ê¸:
>>> From: Gustavo Padovan
>>>
>>> Hi,
nyone who can explan why Android specific thing is tried to spread
into Linux DRM? Was there any consensus to use Android sync driver - which uses
explicit sync interfaces - as Linux standard?
Thanks,
Inki Dae
>
> /**
> * struct fence_collection - aggregate fences together
interface used for DMA device as a Linux standard?
I don't see why Android specific things are spreaded into Linux DRM.
Thanks,
Inki Dae
>
> Signed-off-by: Gustavo Padovan
> ---
> drivers/gpu/drm/drm_atomic.c| 4
> drivers/gpu/drm/drm_atomic_helper.c | 1 +
2016ë
03ì 24ì¼ 10:07ì Inki Dae ì´(ê°) ì´ ê¸:
> Hi Andrzej,
>
> 2016ë
03ì 23ì¼ 22:15ì Andrzej Hajda ì´(ê°) ì´ ê¸:
>> Hi Inki,
>>
>> This set of patches provides set of different fixes and enhancements
>> for DECON -> HDMI path. It is
by the way, they were posted about 5 months ago :)
Oops, really sorry. Will have a review ASAP.
Thanks,
Inki Dae
> - IOMMU patches by Marek (for some mysterious reason HDMI path on 5433
> works only with IOMMU enabled),
> - latest exynos-drm-next patches.
>
2016ë
03ì 23ì¼ 08:39ì Russell King - ARM Linux ì´(ê°) ì´ ê¸:
> On Wed, Mar 23, 2016 at 08:09:33AM +0900, Inki Dae wrote:
>> In this case, someone else may send an email again like you "who is going to
>> merge?"
>> That would be why we need a maintai
2016ë
03ì 23ì¼ 07:52ì Heiko Stübner ì´(ê°) ì´ ê¸:
> Am Mittwoch, 23. März 2016, 07:44:59 schrieb Inki Dae:
>> + Ajay kumar with Samsung email
>>
>> Hi,
>>
>> 2016ë
03ì 23ì¼ 07:12ì Heiko Stübner ì´(ê°) ì´ ê¸:
>>> Hi,
>>>
late. Dave had already pull-request.
To Ajay,
How about adding you as a drm bridge maintainer? DRM SoC driver maintainers
would need a person who can manage the drm bridge relevant pathes.
Thanks,
Inki Dae
>
> Thanks
> Heiko
>
>
>
> Tested-by: Javier Martinez Canillas
> Signed-off-by: Yakir Yang
Acked-by: Inki Dae
Thanks,
Inki Dae
> ---
> Changes in v8: None
> Changes in v7: None
> Changes in v6:
> - Fix the wrong code in previous series, and test on Samsung snow Chromebook
> su
for Tizen platform.
All applications - who want to use dma buffer - use this module
to allocate dma buffer through DRM GEM.
Kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit 550e3b23a53c88adfa46e64f9d442743e65d47da:
Merge branch 'drm
by removing #ifdef ~ #endif things.
- Configure DMA-mapping address space common to Exynos DRM devices in more
generic
without any hacks.
- some fixups and cleanups.
Please kindly let me know if there is any problem.
Thanks,
Inki Dae
The following changes since commit
)
> return;
>
> xfer->result = 0;
> @@ -1189,10 +1186,11 @@ static bool exynos_dsi_transfer_finish(struct
> exynos_dsi *dsi)
> spin_unlock_irqrestore(>transfer_lock, flags);
>
> dev_dbg(dsi->dev,
> - "> xfer %p, tx_len %u, tx_done %u, rx_len %u, rx_done %u\n",
> - xfer, xfer->tx_len, xfer->tx_done, xfer->rx_len,
> xfer->rx_done);
> + "> xfer %p, tx_len %lu, tx_done %u, rx_len %u, rx_done %u\n",
> + xfer, xfer->packet.payload_length, xfer->tx_done,
> xfer->rx_len,
%zu would be required for xfer->packet.payload_length which has size_t type.
Thanks,
Inki Dae
Picked it up.
Thanks,
Inki Dae
2016ë
02ì 17ì¼ 22:33ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> Core provides generic helper to create DSI packet, use it instead of
> custom code.
>
> Signed-off-by: Andrzej Hajda
> ---
> drivers/gpu/drm/exynos/exy
For 1 and 2, picked it up.
Thanks,
Inki Dae
2016ë
02ì 18ì¼ 22:34ì Marek Szyprowski ì´(ê°) ì´ ê¸:
> This patch refactors driver and device registration by moving all drivers
> to the common array. This way additional flags can be added later for
> new features. #ifdef-based
2016ë
01ì 27ì¼ 23:44ì Marek Szyprowski ì´(ê°) ì´ ê¸:
> This patch adds support for blending related properties to Exynos DRM
> core and Exynos Mixer CRTC device.
To drm-misc.
Acked-by : Inki Dae
Thanks,
Inki Dae
>
> Signed-off-by: Marek Szyprowski
> ---
>
2016ë
01ì 27ì¼ 23:44ì Marek Szyprowski ì´(ê°) ì´ ê¸:
> This patch replaces zpos property handling custom code in Exynos DRM
> driver with calls to generic DRM code.
It'd be better to go to drm-misc.
Acked-by: Inki Dae
Thanks,
Inki Dae
>
> Signed-off-by: Mare
Picked it up.
Thanks,
Inki Dae
2016ë
02ì 12ì¼ 22:31ì Chanho Park ì´(ê°) ì´ ê¸:
> This patch adds a exynos5420 driver data to support mic_bypass
> option to bypass the mic from display out path.
> The mic(Mobile image compressor) compresses RGB data from fimd
> and send th
Hi Chanho,
2016ë
02ì 12ì¼ 16:39ì Chanho Park ì´(ê°) ì´ ê¸:
> Hi,
>
> On Fri, Feb 12, 2016 at 1:56 PM, Inki Dae wrote:
>> Hi Chanho,
>>
>> There was a missing one so below is my review again.
>>
>> 2016ë
02ì 11ì¼ 22:59ì Chanho Park ì´(ê°
Picked it up.
Thanks,
Inki Dae
2016ë
02ì 11ì¼ 20:32ì Marek Szyprowski ì´(ê°) ì´ ê¸:
> dma_mmap_attrs() should be called with cpu address returned by
> dma_alloc_attrs(). Existing code however passed pages array base as cpu
> address. This worked only by a pure luck on ARM arc
Picked it up.
Thanks,
Inki Dae
2016ë
02ì 11ì¼ 20:25ì Andrzej Hajda ì´(ê°) ì´ ê¸:
> Decon requires that clocks should be disabled in reverse order. Otherwise
> system hangs.
>
> Signed-off-by: Andrzej Hajda
> ---
> drivers/gpu/drm/exynos/exynos5433_drm_decon.c
could fix it easily like below but I'm not sure that it's reasonable.
+extern void arch_setup_dma_ops(struct device *dev, u64 dma_base,
+ u64 size, struct iommu_ops *iommu,
+ bool coherent) { };
Thanks,
Inki Dae
> dev_set_drvdata(dev->dev, dev);
> dev->dev_private = (void *)private;
>
>
Hi Chanho,
There was a missing one so below is my review again.
2016ë
02ì 11ì¼ 22:59ì Chanho Park ì´(ê°) ì´ ê¸:
> Hi Inki,
>
> On Thu, Feb 11, 2016 at 7:37 PM, Inki Dae wrote:
>> Hi Chanho,
>>
>> 2016ë
01ì 30ì¼ 22:58ì Chanho Park ì´(ê°) ì´
Hi Marek,
For all patches, picked them up.
Thanks,
Inki Dae
2016ë
02ì 03ì¼ 21:42ì Marek Szyprowski ì´(ê°) ì´ ê¸:
> From: Joonyoung Shim
>
> Because PLAT_SAMSUNG isn't include exynos SoCs for arm64, but
> ARCH_EXYNOS can do it. And it also needs to add ARCH_S3C
he
> exynos5433. For example, the exynos5422 uses sw reset like
> previous chips.
Picked it up.
Thanks,
Inki Dae
>
> Cc: Inki Dae
> Cc: Joonyoung Shim
> Cc: Seung-Woo Kim
> Signed-off-by: Chanho Park
> ---
> .../bindings/display/exynos/exynos_dsim.txt| 1 +
to the mipi dsi.
> The bypass option can be founded from system register and the bit
> of the option is 11.
>
> Cc: Inki Dae
> Cc: Joonyoung Shim
> Cc: Seung-Woo Kim
> Signed-off-by: Chanho Park
> ---
> .../devicetree/bindings/display/exynos/samsung-fimd.txt| 2 ++
&
Hi Dave,
2016ë
02ì 05ì¼ 13:46ì Dave Airlie ì´(ê°) ì´ ê¸:
> On 1 February 2016 at 16:42, Inki Dae wrote:
>> Hi Dave,
>>
>>Just regression fixes.
>>- Fix build warning and error without PM configuration
>>- Fix no display issue on Snow
nd - why the ippdrv has to be printed? Is it useful for debugging?
Marek fixed just compilation warnings on ARM64 so yes it wouldn't need to print
out ippdrv address but as other cleanup patch.
Thanks,
Inki Dae
>
> Best regards,
> Krzysztof
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc"
> in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
501 - 600 of 2980 matches
Mail list logo