[PATCH 0/8] drm/bochs: convert bochs to atomic mode-setting

2016-03-04 Thread John Hunter
Hi Gerd, Sorry about that, but I am now busy with my master's degree stuffs. Really can't spare any time on this until maybe July. On Thu, Mar 3, 2016 at 5:28 PM, Gerd Hoffmann wrote: > Hi, > > > > Did testing a while back & reported back to John (not sure this was in > > > public on the lis

[Action requested] GSoC 2015: Thanks for your participation!

2015-08-23 Thread John Hunter
Hi Martin, Here is my blog post of my current situation http://zhjwpku.blogspot.com/ Cheers, Zhao On Sat, Aug 22, 2015 at 3:52 AM, Martin Peres wrote: > Hello everyone, > > Today is the official pencil down date so it is a good time for me to > congratulate everyone for their participation! > >

[PATCH 7/7] drm/cirrus: add DRIVER_ATOMIC to .driver_features

2015-08-05 Thread John Hunter
From: Zhao Junwang Since driver is now fully atomic, we should add DRIVER_ATOMIC to .driver_features Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthew Garrett Cc: Dave Airlie Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_drv.c |2 +- 1 file changed, 1 insertion(+), 1 del

[PATCH 6/7] drm/cirrus: atomic dpms support

2015-08-05 Thread John Hunter
From: Zhao Junwang Run dpms operations through the atomic interfaces. This basically removes the .dpms() callback from encoders and crtcs and use .disable and .enable to turn the crtc on and off. use drm_atomic_helper_connector_dpms for connector Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthe

[PATCH 5/7] drm/cirrus: phase 3: use atomic .set_config helper

2015-08-05 Thread John Hunter
From: Zhao Junwang Now that phase 1 and phase 2 are completed, switch .set_config helper to use the atomic one. -stop looking legacy crtc->primary->fb, instead we should use crtc->primary->state->fb .prepare() calls are no more needed, remove them .mode_set() and .mode_set_base are no longer ne

[PATCH 4/7] drm/cirrus: add return 0 to make sure if (bo->pin_count) early_exit

2015-08-05 Thread John Hunter
From: Zhao Junwang when the first modeset calls prepare_fb, bo->pin_count change from 0 to 1, then the second modeset with the same fb, that should set bo->pin_count to 2, and then when cleanup_fb was called, bo->pin_count should be 2 to 1. But in the cirrus_bo_pin, it will set bo->pin_count = 1

[PATCH 3/7] drm/cirrus: phase 3: atomic updates of planes

2015-08-05 Thread John Hunter
From: Zhao Junwang Now that phase 1 and 2 are complete we can switch the update/disable_plane callbacks to their atomic version Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthew Garrett Cc: Dave Airlie Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_main.c |3 +++ drivers/g

[PATCH 2/7] drm/cirrus: phase 2: wire up state reset, duplicate and destroy

2015-08-05 Thread John Hunter
From: Zhao Junwang Set CRTC, planes and connectors to use the default implementations from the atomic helper library. Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthew Garrett Cc: Dave Airlie Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_mode.c | 12 1 file cha

[PATCH 1/7] drm/cirrus: phase 1 - use the transitional helpers

2015-08-05 Thread John Hunter
From: Zhao Junwang -register driver's own primary plane -use drm_crtc_init_with_planes instead drm_crtc_init -the new atomic_infrastructure needs ->mode_set_nofb callback Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthew Garrett Cc: Dave Airlie Signed-off-by: Zhao Junwang --- drivers/gpu/dr

[PATCH 0/7] drm/cirrus: convert cirrus to atomic mode-setting

2015-08-05 Thread John Hunter
From: Zhao Junwang This patch series aim to convert DRM_CIRRUS to atomic mode-setting. This mostly reference my previous patch series on DRM_BOCHS and Gustavo Padovan;s patch series on drm/exynos. Zhao Junwang (7): drm/cirrus: phase 1 - use the transitional helpers drm/cirrus: phase 2: wire

[PATCH 7/7] drm/cirrus: add DRIVER_ATOMIC to .driver_features

2015-08-05 Thread John Hunter
From: Zhao Junwang Since driver is now fully atomic, we should add DRIVER_ATOMIC to .driver_features Cc: Daniel Vetter Cc: Gerd Hoffmann Cc: Matthew Garrett Cc: Dave Airlie Cc: Zhao Junwang Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_drv.c |2 +- 1 file changed, 1 i

[PATCH 7/7] drm/cirrus: add DRIVER_ATOMIC to .driver_features

2015-07-30 Thread John Hunter
From: Zhao Junwang Since driver is now fully atomic, we should add DRIVER_ATOMIC to .driver_features Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_drv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.c b/drivers/gpu/dr

[PATCH 6/7] drm/cirrus: atomic dpms support

2015-07-30 Thread John Hunter
From: Zhao Junwang Run dpms operations through the atomic interfaces. This basically removes the .dpms() callback from encoders and crtcs and use .disable and .enable to turn the crtc on and off. use drm_atomic_helper_connector_dpms for connector Signed-off-by: Zhao Junwang --- drivers/gpu/dr

[PATCH 5/7] drm/cirrus: phase 3: use atomic .set_config helper

2015-07-30 Thread John Hunter
From: Zhao Junwang Now that phase 1 and phase 2 are completed, switch .set_config helper to use the atomic one. -stop looking legacy crtc->primary->fb, instead we should use crtc->primary->state->fb .prepare() calls are no more needed, remove them .mode_set() and .mode_set_base are no longer ne

[PATCH 4/7] drm/cirrus: add return 0 to make sure if (bo->pin_count) early_exit

2015-07-30 Thread John Hunter
From: Zhao Junwang when the first modeset calls prepare_fb, bo->pin_count change from 0 to 1, then the second modeset with the same fb, that should set bo->pin_count to 2, and then when cleanup_fb was called, bo->pin_count should be 2 to 1. But in the cirrus_bo_pin, it will set bo->pin_count = 1

[PATCH 3/7] drm/cirrus: phase 3: atomic updates of planes

2015-07-30 Thread John Hunter
From: Zhao Junwang Now that phase 1 and 2 are complete we can switch the update/disable_plane callbacks to their atomic version Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_main.c |3 +++ drivers/gpu/drm/cirrus/cirrus_mode.c |4 ++-- 2 files changed, 5 insertions(+), 2

[PATCH 2/7] drm/cirrus: phase 2: wire up state reset, duplicate and destroy

2015-07-30 Thread John Hunter
From: Zhao Junwang Set CRTC, planes and connectors to use the default implementations from the atomic helper library. Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_mode.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c b

[PATCH 1/7] drm/cirrus: phase 1 - use the transitional helpers

2015-07-30 Thread John Hunter
From: Zhao Junwang -register driver's own primary plane -use drm_crtc_init_with_planes instead drm_crtc_init -the new atomic_infrastructure needs ->mode_set_nofb callback Signed-off-by: Zhao Junwang --- drivers/gpu/drm/cirrus/cirrus_drv.c |1 - drivers/gpu/drm/cirrus/cirrus_drv.h |3

[PATCH 0/7] drm/cirrus: convert cirrus to atomic mode-setting

2015-07-30 Thread John Hunter
From: Zhao Junwang This patch series aim to convert DRM_CIRRUS to atomic mode-setting. This mostly reference my previous patch series on DRM_BOCHS and Gustavo Padovan;s patch series on drm/exynos. Zhao Junwang (7): drm/cirrus: phase 1 - use the transitional helpers drm/cirrus: phase 2: wire

[PATCH v2 1/8] drm/bochs: phase 1 - use the transitional helpers

2015-07-20 Thread John Hunter
From: Zhao Junwang -register driver's own primary plane -use drm_crtc_init_with_planes instead of drm_crtc_init -split ->mode_set into: 1. set the new hw mode 2. update the primary plane (This is done by ->set_base) -move what ->set_base do into ->atomic_update -the new atomic i

[PATCH 6/8] drm/bochs: phase 3: provide a custom ->atomic_commit implementation

2015-07-20 Thread John Hunter
Hi Pekka, Thanks for the information, I will talk to my mentor Daniel and try to find out what I can do about this. Cheers, Zhao Junwang On Fri, Jul 17, 2015 at 2:08 PM, Pekka Paalanen wrote: > On Thu, 16 Jul 2015 20:20:39 +0800 > John Hunter wrote: > > > From: Zhao Junwan

[PATCH 8/8] drm/bochs: atomic dpms support

2015-07-16 Thread John Hunter
From: Zhao Junwang - use ->disable() and ->enable() for crct and plane - use drm_atomic_helper_connector_dpms for connector Cc: Maarten Lankhorst Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/bochs/bochs_kms.c | 34 ++ 1 file changed, 22

[PATCH 7/8] drm/bochs: phase 3: switch to drm_atomic_helper_page_flip

2015-07-16 Thread John Hunter
From: Zhao Junwang PageFlips now use the atomic helper to work through the atomic modesetting API. v2: -Since driver is now fully atomic, we should add DRIVER_ATOMIC to .driver_features. Cc: Maarten Lankhorst Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/bochs/bochs

[PATCH 6/8] drm/bochs: phase 3: provide a custom ->atomic_commit implementation

2015-07-16 Thread John Hunter
From: Zhao Junwang This supports the asynchronous commits, required for page-flipping Since it's virtual hw it's ok to commit async stuff right away, we never have to wait for vblank. Cc: Maarten Lankhorst Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/bochs/bochs_mm.c |

[PATCH 5/8] drm/bochs: phase 3: use atomic .set_config helper

2015-07-16 Thread John Hunter
From: Zhao Junwang Now that phase 1 and phase 2 are complete, switch .set_config helper to use the atomic one. -since .prepare() callbacks are no more needed, remove them -.mode_set() and .mode_set_base() are no longer needed, remove -as we are not using the transitional helper now, we can use

[PATCH 4/8] drm/bochs: phase 3: for plane updates: switch to atomic helper internally

2015-07-16 Thread John Hunter
From: Zhao Junwang - planes: drm_atomic_helper_update_plane() drm_atomic_helper_diable_plane() - Driver: drm_atomic_helper_check() drm_atomic_helper_commit() Cc: Maarten Lankhorst Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/bochs/bochs_km

[PATCH 3/8] drm/bochs: stop looking at legacy state

2015-07-16 Thread John Hunter
From: Zhao Junwang with transitional helpers we've used the new callbacks, but control flow was still old, but with atomic first step is to call atomic_check, then prepare_fb, and so on, by this time, we can not look at any legacy state like plane->fb or crtc->enabled, because they are updated at

[PATCH 2/8] drm/bochs: phase 2: wire up state reset(), duplicate() and destroy

2015-07-16 Thread John Hunter
From: Zhao Junwang Set CRTC, planes and connectors to use the default implementations from the atomic helper library. Cc: Maarten Lankhorst Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/bochs/bochs_kms.c | 12 1 file changed, 12 insertions(+) diff --git a/

[PATCH 1/8] drm/bochs: phase 1 - use the transitional helpers

2015-07-16 Thread John Hunter
From: Zhao Junwang -register driver's own primary plane -use drm_crtc_init_with_planes instead of drm_crtc_init -split ->mode_set into: 1. set the new hw mode 2. update the primary plane (This is done by ->set_base) -move what ->set_base do into ->atomic_update -the new atomic i

[PATCH 0/8] drm/bochs: convert bochs to atomic mode-setting

2015-07-16 Thread John Hunter
From: Zhao Junwang This patch series aim to convert DRM_BOCHS to atomic mode-setting. I did this mostly reference Gustavo Padovan's patch series on drm/exynos conversion. Zhao Junwang (8): drm/bochs: phase 1 - use the transitional helpers drm/bochs: phase 2: wire up state reset(), duplicate(

[PATCH v3] drm: add a drm_atomic_helper_plane_check_update

2015-07-14 Thread John Hunter
From: Zhao Junwang This is the equivalent helper to drm_plane_helper_check_update for legacy drivers, but using atomic state to check things. Motivated by the atomic conversion of the bochs driver. v2: according to Daniel's comment -polish the kerneldoc comment to match the signatures -

[PATCH v2] drm: add a drm_atomic_helper_plane_check_update

2015-07-13 Thread John Hunter
From: Zhao Junwang This is the equivalent helper to drm_plane_helper_check_update for legacy drivers, but using atomic state to check things. Motivated by the atomic conversion of the bochs driver. v2: according to Daniel's comment -polish the kerneldoc comment to match the signatures -

[PATCH] drm: add a drm_atomic_helper_plane_check_update

2015-07-13 Thread John Hunter
From: Zhao Junwang This is the equivalent helper to drm_plane_helper_check_update for legacy drivers, but using atomic state to check things. Motivated by the atomic conversion of the bochs driver. v2: according to Daniel's comment -polish the kerneldoc comment to match the signatures -

[PATCH] drm: add a drm_atomic_helper_plane_check_update

2015-07-13 Thread John Hunter
ntel.com> wrote: > Op 13-07-15 om 09:12 schreef Maarten Lankhorst: > > Op 13-07-15 om 02:21 schreef John Hunter: > >> From: Zhao Junwang > >> > >> Cc: Daniel Vetter > >> Signed-off-by: Zhao Junwang > >> --- > >> drivers/gpu/drm/drm_

[PATCH] drm: add a drm_atomic_helper_plane_check_update

2015-07-13 Thread John Hunter
From: Zhao Junwang Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/drm_atomic_helper.c | 55 +++ include/drm/drm_atomic_helper.h |7 + 2 files changed, 62 insertions(+) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/

DRM-vmwgfx: Deletion of an unnecessary check before the function call "vfree"

2015-07-09 Thread John Hunter
On Thu, Jul 9, 2015 at 12:55 AM, SF Markus Elfring < elfring at users.sourceforge.net> wrote: > > Reviewed-by: Zhao Junwang > > > > kfree will check that. > > How does this feedback fit to a check before a call > of the vfree() function? > > I might have made a mistake, the ctl^] lead me to the v

[PATCH] drm: sti: add moduleparam to disable fbdev

2015-07-08 Thread John Hunter
On Wed, Jul 8, 2015 at 8:50 PM, Benjamin Gaignard < benjamin.gaignard at linaro.org> wrote: > Useful to avoid recompiling to enable/disable fbdev. > > Signed-off-by: Benjamin Gaignard > Reviewed-by: Zhao Junwang Bochs has the moduleparam for fbdev, lgtm. > --- > drivers/gpu/drm/sti/Kconfig

[PATCH 1/1] DRM-vmwgfx: Deletion of an unnecessary check before the function call "vfree"

2015-07-08 Thread John Hunter
On Mon, Jul 6, 2015 at 4:01 PM, SF Markus Elfring < elfring at users.sourceforge.net> wrote: > From: Markus Elfring > Date: Mon, 6 Jul 2015 09:49:11 +0200 > > The vfree() function performs also input parameter validation. > Thus the test around the call is not needed. > > This issue was detected

[PATCH v2] drm: add a check for x/y in drm_mode_setcrtc

2015-07-07 Thread John Hunter
From: Zhao Junwang legacy setcrtc ioctl does take a 32 bit value which might indeed overflow the checks of crtc_req->x > INT_MAX and crtc_req->y > INT_MAX aren't needed any more with this v2: -polish the annotation according to Daniel's comment Cc: Daniel Vetter Signed-off-by: Zhao Junwang -

[PATCH] drm: add a check for x/y in drm_mode_setcrtc

2015-07-07 Thread John Hunter
From: Zhao Junwang legacy setcrtc ioctl does take a 32 bit value which might indeed overflow the checks of crtc_req->x > INT_MAX and crtc_req->y > INT_MAX aren't needed any more with this Cc: Daniel Vetter Signed-off-by: Zhao Junwang --- drivers/gpu/drm/drm_crtc.c |7 +-- 1 file chan

[PATCH] GPU-DRM: Delete an unnecessary check before drm_property_unreference_blob()

2015-07-06 Thread John Hunter
On Mon, Jul 6, 2015 at 4:00 AM, SF Markus Elfring < elfring at users.sourceforge.net> wrote: > From: Markus Elfring > Date: Sun, 5 Jul 2015 21:55:10 +0200 > > The drm_property_unreference_blob() function tests whether its argument > is NULL and then returns immediately. > Thus the test around the

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread John Hunter
e above patch > from Daniel also accidentally removed the if (crtc_state) check. This > is legal when transitioning to atomic, when the initial state reset > isn't all wired up yet properly. Reinstate that check to fix the bug > John has hit. > > Cc: Daniel Stone > CC: Se

[PATCH] drm: reset empty state in transitional helpers

2015-07-02 Thread John Hunter
On Thu, Jul 2, 2015 at 10:33 PM, Daniel Vetter wrote: > Transitional drivers might not have all the state frobbing lined up > yet. But since the initial code has been merged a lot more state was > added, so we really need this. > > Cc: Daniel Stone > Signed-off-by: Daniel Vetter > Tested-by: Zh

[PATCH] drm/bochs: convert bochs driver to atomic mode-setting

2015-06-11 Thread John Hunter
On Thu, Jun 11, 2015 at 2:29 PM, Gerd Hoffmann wrote: > On Do, 2015-06-11 at 11:18 +0800, John Hunter wrote: > > Hi Gerd, > > Here is what I got: > > > > > > [ 813.137939] [drm:drm_pci_init] > > [ 813.142780] [drm:drm_get_pci_dev] > > [ 813.15698

[PATCH] drm/bochs: convert bochs driver to atomic mode-setting

2015-06-11 Thread John Hunter
o that. I really can't get any clue. If my question is stupid, forgive me, I will try to catch up :) Best Regards, Zhao On Wed, Jun 10, 2015 at 8:20 PM, Gerd Hoffmann wrote: > On Mi, 2015-06-10 at 19:39 +0800, John Hunter wrote: > > Hi Gerd, > > I have tried what you told me.

[PATCH] drm/bochs: convert bochs driver to atomic mode-setting

2015-06-10 Thread John Hunter
ed [ ok ] Starting network connection manager: NetworkManager. [ ok ] Starting Common Unix Printing System: cupsd. The serial console messages also stop here, no other output. I just don't know if I did the right thing, if I did something wrong, I would be appreaciate if you show me. Best Reg

[PATCH] drm/bochs: convert bochs driver to atomic mode-setting

2015-06-10 Thread John Hunter
Thanks for the information, I will try that : ) On Wed, Jun 10, 2015 at 4:04 PM, Gerd Hoffmann wrote: > Hi, > > > As far as I know, the convertion got some problem, and after I > > add the bochs driver when I boot the qemu VM, it just can not > > start up, works fine if I delete bochs-drm.ko f

[PATCH] drm/bochs: convert bochs driver to atomic mode-setting

2015-06-10 Thread John Hunter
From: root convert the bochs driver to atomic mode-setting, referencing the patch serias of drm/exynos sent by Gustavo Padovan Signed-off-by: Zhao Junwang --- Hi all, I am a GSoCer of this year, and my project is to convert bochs driver and cirrus driver to atomic mode-setting. I converted th

[Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-24 Thread John Hunter
ok, let's keep it the the original way. On Fri, Apr 24, 2015 at 12:09 AM, Daniel Vetter wrote: > On Wed, Apr 22, 2015 at 10:50:55AM +0800, John Hunter wrote: > > Sure, but I need Daniel to admit that, because maybe include the two > header > > file make it easier to un

[Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-22 Thread John Hunter
Wed, Apr 22, 2015 at 10:35 AM, yongjun_wei at trendmicro.com.cn < yongjun_wei at trendmicro.com.cn> wrote: > Hi John, > > > > Feel free to submit a new patch. > > > > Regards, > > Yongjun Wei > > > > *From:* John Hunter [mailto:zhjwpku at gmail.com

[Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-22 Thread John Hunter
Hi, I think maybe we should remove both of the two lines: #include "intel_drv.h" #include "i915_drv.h" Because we have* two* "intel_drv.h" and *one* "i915_drv.h", and "i915_drv.h" has already been included in the "intel_drv.h". I not sure whether i am right. If you need me to do the patch, let me

[PATCH] drm: fix trivial typo mistake

2015-04-14 Thread John Hunter
Signed-off-by: John Hunter --- include/drm/drm_crtc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 2e80ad1..8ad8507 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -974,7 +974,7 @@ struct

[PATCH] drm: Fix some typos

2015-04-07 Thread John Hunter
Signed-off-by: John Hunter --- drivers/gpu/drm/drm_atomic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c index 57efdbe..0537f32 100644 --- a/drivers/gpu/drm/drm_atomic.c +++ b/drivers/gpu/drm/drm_atomic.c

Is CIRRUS a suitable driver to convert to atomic mode-setting

2015-04-07 Thread John Hunter
Hi Gerd, Thanks for the suggestion. I will try my best :) On Tue, Apr 7, 2015 at 4:55 PM, Gerd Hoffmann wrote: > Hi, > > > I think at least I should give it a try. If it is really impossible to > > convert, then > > we should talk about whether we should deprecate it in the future. > > Feel fr

Is CIRRUS a suitable driver to convert to atomic mode-setting

2015-04-07 Thread John Hunter
On Tue, Apr 7, 2015 at 2:56 PM, Gerd Hoffmann wrote: > On Do, 2015-04-02 at 09:57 +0800, John Hunter wrote: > > Hi Daniel, > > Sorry to disturb you, I realized you are a busy man of the community. > > So I did some work before I email you. > > > > > > I sub

Is CIRRUS a suitable driver to convert to atomic mode-setting

2015-04-02 Thread John Hunter
Hi Daniel, Sorry to disturb you, I realized you are a busy man of the community. So I did some work before I email you. I submit my proposal as you suggest, convert the two virtual driver (CIRRUS and BOCHS, if I remember it right) to atomic mode-setting. After talked with Martin Peres(the adminis

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread John Hunter
Hi Gerd, I've read the patches about the virtio-gpu, it's a nice design. As far as I know, there are two other drivers used by qemu, CIRRUS and BOCHS. I have a question about the relationship of these three drivers, is that the virtio-gpu designed to replace the other two drivers? I mean are the CI

My GSoC proposal needs help :-)

2015-03-23 Thread John Hunter
Hi Daniel, all I spent the last days on my GSoC 2015 proposal, I've submited it to the google-melange site. Now I post it here for you to review, hope you can spare some time to give it a look and give me some advices. Any comments and criticism is welcomed. Convert the BOCHS and CIRRUS drivers t

[PATCH 2/3] drm: replace the 'for' condition with outside defined variable

2015-03-17 Thread John Hunter
Got it! Cheers, John On Tue, Mar 17, 2015 at 5:24 PM, Daniel Vetter wrote: > On Tue, Mar 17, 2015 at 04:48:23PM +0800, John Hunter wrote: > > Hi Daniel, > > > > On Tue, Mar 17, 2015 at 4:40 PM, Daniel Vetter wrote: > > > > > On Tue, Mar 17, 2015 at

Why no geode subsystem contained in drm

2015-03-17 Thread John Hunter
Thank you, Christian Cheers, John On Tue, Mar 17, 2015 at 4:43 PM, Christian Gmeiner < christian.gmeiner at gmail.com> wrote: > Hi John > > > I noticed that there is no geode subsystem in the drm directory. > > I have a system using the geode LX framebuffer and > > xserver-xorg-video-geode. I am

[PATCH 2/3] drm: replace the 'for' condition with outside defined variable

2015-03-17 Thread John Hunter
Hi Daniel, On Tue, Mar 17, 2015 at 4:40 PM, Daniel Vetter wrote: > On Tue, Mar 17, 2015 at 03:30:27PM +0800, John Hunter wrote: > > use outdise defined variable can reduce the recaculate of the > > count of planes, crtcs and connectors. > > > > Signed-off-by: John

Why no geode subsystem contained in drm

2015-03-17 Thread John Hunter
Hi, I noticed that there is no geode subsystem in the drm directory. I have a system using the geode LX framebuffer and xserver-xorg-video-geode. I am wondering if we can add the drm part of geode lx to let it support the dri? How about VIVANTE then? Best Regards, John -- Best regards Junwang

[PATCH 3/3] drm: change connector to tmp_connector

2015-03-17 Thread John Hunter
This wasn't too harmful since we already look at connector, which has the same effect as the loop for any non-cloned configs. Only when we have a cloned configuration is it important to look at other connectors. Furthermore existing userspace always changes dpms on all of them anyway. Signed-off-b

[PATCH 2/3] drm: replace the 'for' condition with outside defined variable

2015-03-17 Thread John Hunter
use outdise defined variable can reduce the recaculate of the count of planes, crtcs and connectors. Signed-off-by: John Hunter --- drivers/gpu/drm/drm_atomic_helper.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu

[PATCH 1/3] drm: Fix some typo mistake of the annotations

2015-03-17 Thread John Hunter
There are some mistakes that the function name in the annotaions is not matching the real function name. And some duplication word in annotations Signed-off-by: John Hunter --- drivers/gpu/drm/drm_atomic_helper.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a

[PATCH 2/2] drm_atomic_helper.c: connector => tmp_connector

2015-03-17 Thread John Hunter
;t know. :-) Please don't hesitate to criticize if I think I didn't do the right thing. Thanks, John On Mon, Mar 16, 2015 at 4:21 PM, Daniel Vetter wrote: > On Sun, Mar 15, 2015 at 08:59:11PM +0800, John Hunter wrote: > > I'm not sure whether this is right, as far as I can

[PATCH 1/2] drm_atomic_helper.c: cosmetic: fix some typos and add some empty line

2015-03-16 Thread John Hunter
I will read the Documentation/SubmittingPatches, sorry about my mistake. On Mon, Mar 16, 2015 at 4:17 PM, Daniel Vetter wrote: > On Sun, Mar 15, 2015 at 08:59:10PM +0800, John Hunter wrote: > > there are some duplication in the annotations > > add some empty line to make it

[PATCH 2/2] drm_atomic_helper.c: connector => tmp_connector

2015-03-15 Thread John Hunter
I'm not sure whether this is right, as far as I can see, for the macro 'list_for_each_entry', if not use the tmp_connector, it does make sense. Hope I am right with that. --- drivers/gpu/drm/drm_atomic_helper.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/dr

[PATCH 1/2] drm_atomic_helper.c: cosmetic: fix some typos and add some empty line

2015-03-15 Thread John Hunter
there are some duplication in the annotations add some empty line to make it easier to read --- drivers/gpu/drm/drm_atomic_helper.c | 48 + 1 file changed, 38 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm

Connection to the remote recipient's server was denied for unknown reason

2015-03-15 Thread John Hunter
Hi, I was trying to send some patches using git send email, but I got the error msg: "Connection to the remote recipient's server was denied for unknown reason" Can anybody help? -- Best regards Junwang Zhao Microprocessor Research and Develop Center Department of Computer Science &Technology Pek

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-14 Thread John Hunter
Hi Jani, I did as you told me, I subscribed using my other email address, and disable the mail delivery option in the management page of dri-devel. Hope this may work. Thanks. Best regards, John On Fri, Mar 13, 2015 at 8:34 PM, Jani Nikula wrote: > On Fri, 13 Mar 2015, Rob Clark wrote: > > h

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-13 Thread John Hunter
Fine, I will try to catch up. Thanks anyway. Have a nice day, John On Fri, Mar 13, 2015 at 11:07 AM, Rob Clark wrote: > On Thu, Mar 12, 2015 at 10:53 PM, John Hunter wrote: > > Got it, maybe I should work on the drm-next or latest linux master. > > > > I am working on t

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-13 Thread John Hunter
clone. On Fri, Mar 13, 2015 at 10:33 AM, Rob Clark wrote: > On Thu, Mar 12, 2015 at 4:02 AM, John Hunter > wrote: > > From: John Hunter > > > > IMHO, some annotations were copypaste from somewhere else, it is > > obviously not rightly modified. Hope I am right with

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-13 Thread John Hunter
Hi Rob, On Thu, Mar 12, 2015 at 10:34 PM, Rob Clark wrote: > On Thu, Mar 12, 2015 at 10:15 AM, John Hunter wrote: > > Hi, > > I really don't whether I am doing things on right workflow of dri-devel. > > I am new to this community and I want to help. > > Could

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-12 Thread John Hunter
Hi, I really don't whether I am doing things on right workflow of dri-devel. I am new to this community and I want to help. Could anybody spare some time to tell me the right way . Thanks. John On Thu, Mar 12, 2015 at 4:02 PM, John Hunter wrote: > From: John Hunter > > IMHO, so

[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

2015-03-12 Thread John Hunter
From: John Hunter IMHO, some annotations were copypaste from somewhere else, it is obviously not rightly modified. Hope I am right with that. --- drivers/gpu/drm/drm_crtc.c | 115 ++--- 1 file changed, 56 insertions(+), 59 deletions(-) diff --git a

GSOC 2015 project mentor

2015-03-09 Thread John Hunter
Hi Stefan, I came across to contact to Martin Peres the last two days, I asked him about the project "GL/GLSL tests for GL 4.0 and newer", he told me that this was copy-pasted from last years idea. Hope this can help you. John On Sun, Mar 8, 2015 at 4:46 AM, Stefan Lance wrote: > Hello, > >