Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751706AbaJTLqL (ORCPT ); Mon, 20 Oct 2014 07:46:11 -0400 Received: from gloria.sntech.de ([95.129.55.99]:43565 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751664AbaJTLqG convert rfc822-to-8bit (ORCPT ); Mon, 20 Oct 2014 07:46:06 -0400 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Mark yao Cc: Dave Airlie , Boris BREZILLON , Rob Clark , Daniel Vetter , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Randy Dunlap , Grant Likely , Greg Kroah-Hartman , John Stultz , Rom Lemarchand , "devicetree@vger.kernel.org" , linux-doc@vger.kernel.org, LKML , dri-devel , linux-api@vger.kernel.org, linux-rockchip@lists.infradead.org, dianders@chromium.org, =?ISO-8859-1?Q?St=E9phane?= Marchesin , dbehr@chromium.org, Olof Johansson , djkurtz@chromium.org, xjq@rock-chips.com, kfx@rock-chips.com, cym@rock-chips.c, om@null.null, cf@rock-chips.com, zyw@rock-chips.com, xxm@rock-chips.com, huangtao@rock-chips.com, kever.yang@rock-chips.com, yxj@rock-chips.com, wxt@rock-chips.com, xw@rock-chips.com Subject: Re: [PATCH v11 0/3] Add drm driver for Rockchip Socs Date: Mon, 20 Oct 2014 13:45:35 +0200 Message-ID: <2468336.fLbrdhjYl7@phil> User-Agent: KMail/4.11.5 (Linux/3.13-1-amd64; KDE/4.11.3; x86_64; ; ) In-Reply-To: <5440D68E.4070103@rock-chips.com> References: <1412763781-31533-1-git-send-email-mark.yao@rock-chips.com> <4044646.m0RgIJvLdJ@phil> <5440D68E.4070103@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, Am Freitag, 17. Oktober 2014, 16:42:54 schrieb Mark yao: > On 2014年10月17日 16:25, Heiko Stübner wrote: > > Am Freitag, 17. Oktober 2014, 16:16:01 schrieb Mark yao: > >> On 2014年10月17日 14:24, Heiko Stübner wrote: > >>> Hi Mark, > >>> > >>> Am Freitag, 17. Oktober 2014, 12:22:53 schrieb Mark yao: > >>>> On 2014年10月17日 08:46, Dave Airlie wrote: > >>>>> On 17 October 2014 10:40, Mark yao wrote: > >>>>>> Hi > >>>>>> I think Rockchip drm driver is ready now, can it land? > >>>>> > >>>>> I probably want to wait until -rc1 though I suppose since its a new > >>>>> driver and self contained we might be able to see if Linus is > >>>>> interested in squeezing it in. > >>>>> > >>>>> Can you send me a git pull request for it against drm-next or even > >>>>> 3.17. > >>>>> > >>>>> Dave. > >>>> > >>>> Hi, Dave > >>>> > >>>> the git pull request: > >>>> The following changes since commit > > > > 4db36870b92cdf5a79615aeabc68efc97df13918: > >>> I think this needs a fix. Your commit 4db36870b is the pending iommu > >>> driver. Which isn't part of neither the mainline kernel nor the drm tree > >>> Dave meant. > >>> > >>> What Dave meant was to base your patches on top of his "drm-next" branch > >>> or > >>> the raw 3.17 release. > >>> > >>> So either base the branch on drm-next from > >>> > >>> http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-next > >>> > >>> or the 3.17 release tag from Linus Torvalds. Apply your patches on top > >>> and > >>> create pull from there. > >>> > >>> I've just checked ... your drm patches apply cleanly against 3.17, so > >>> your > >>> branch to be pulled should probably look something like > >>> > >>> https://github.com/mmind/linux-rockchip/commits/tmp/drmtest > >>> > >>> Heiko > >> > >> Hi Heiko > >> > >> thanks for you check. > >> > >> Hi Dave > >> > >> I have reupload a branch cleanly against 3.17: > > sorry, but I think this might need another round. > > > > Your current base commit 0429fbc0bdc2 is already part of the current 3.18 > > merge window - which Dave won't have in his tree I guess. The commit you > > should base your patches on is bfe01a5ba249 - the 3.17 release. > > > > Simply do a > > > > git checkout v3.17 > > git checkout -b drmupstream > > > > apply your patches and send the pull request with v3.17 as base. > > > > > > Heiko > > I think that would be ok: yes the pull request itself looks ok, but you may have to be more explicit. Normally a pull request mail should have its own subject and body and not be embedded somewhere into the response thread. This enables the maintainers to see what they're asked to do easily. So I guess it would be better to compose a separate mail, with a correct subject, somewhat similar to [0] for example. Heiko [0] http://www.spinics.net/lists/arm-kernel/msg366129.html ----- 8< ---------------- Subject: [GIT PULL] core drm support for Rockchip SoCs The following changes since commit bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9: Linux 3.17 (2014-10-05 12:23:04 -0700) are available in the git repository at: https://github.com/markyzq/kernel-drm-rockchip.git drmrockchip for you to fetch changes up to 45bb5f4e7e82b30e9e7069c73441413680c9a59f: dt-bindings: video: Add documentation for rockchip vop (2014-10-17 16:39:31 +0800) ---------------------------------------------------------------- Mark yao (3): drm: rockchip: Add basic drm driver dt-bindings: video: Add for rockchip display subsytem dt-bindings: video: Add documentation for rockchip vop .../devicetree/bindings/video/rockchip-drm.txt | 19 + .../devicetree/bindings/video/rockchip-vop.txt | 58 + drivers/gpu/drm/Kconfig | 2 + drivers/gpu/drm/Makefile | 1 + drivers/gpu/drm/rockchip/Kconfig | 17 + drivers/gpu/drm/rockchip/Makefile | 8 + drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 449 ++++++ drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 54 + drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 200 +++ drivers/gpu/drm/rockchip/rockchip_drm_fb.h | 28 + drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c | 210 +++ drivers/gpu/drm/rockchip/rockchip_drm_fbdev.h | 20 + drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 293 ++++ drivers/gpu/drm/rockchip/rockchip_drm_gem.h | 54 + drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 1427 ++++++++++++++++++++ drivers/gpu/drm/rockchip/rockchip_drm_vop.h | 196 +++ 16 files changed, 3036 insertions(+) create mode 100644 Documentation/devicetree/bindings/video/rockchip-drm.txt create mode 100644 Documentation/devicetree/bindings/video/rockchip-vop.txt create mode 100644 drivers/gpu/drm/rockchip/Kconfig create mode 100644 drivers/gpu/drm/rockchip/Makefile create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_drv.c create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_drv.h create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_fb.c create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_fb.h create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_fbdev.h create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_gem.c create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_gem.h create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_vop.c create mode 100644 drivers/gpu/drm/rockchip/rockchip_drm_vop.h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/