From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965987AbcAZOBB (ORCPT ); Tue, 26 Jan 2016 09:01:01 -0500 Received: from mail-ig0-f195.google.com ([209.85.213.195]:33503 "EHLO mail-ig0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965768AbcAZOA5 (ORCPT ); Tue, 26 Jan 2016 09:00:57 -0500 MIME-Version: 1.0 In-Reply-To: <56A73CB9.2040802@rock-chips.com> References: <1453799046-307-1-git-send-email-jung.zhao@rock-chips.com> <56A73CB9.2040802@rock-chips.com> Date: Tue, 26 Jan 2016 15:00:56 +0100 Message-ID: Subject: Re: [PATCH v1 0/3] Add VP8 deocder for rk3229 & rk3288 From: Enric Balletbo Serra To: Shawn Lin Cc: Jung Zhao , pawel@osciak.com, m.szyprowski@samsung.com, kyungmin.park@samsung.com, Mauro Carvalho Chehab , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Sakari Ailus , linux-api@vger.kernel.org, linux-kernel@vger.kernel.org, Benoit Parrot , linux-rockchip@lists.infradead.org, Antti Palosaari , Hans Verkuil , alpha.lin@rock-chips.com, Philipp Zabel , Ricardo Ribalda Delgado , herman.chen@rock-chips.com, "linux-arm-kernel@lists.infradead.org" , linux-media@vger.kernel.org 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 Jung, 2016-01-26 10:30 GMT+01:00 Shawn Lin : > Hi jun, > > Where is the dt-bingding documentation about your VP8 controller? > > And would you please share some info about rk3229? I can just find > rk3228 in mainline, otherwise may someone think it's a misspell. > > Thanks. > > > On 2016/1/26 17:04, Jung Zhao wrote: >> >> From: zhaojun >> >> >> ==================== >> Introduction >> ==================== >> >> The purpose of this series is to add the driver for vp8 >> decoder on rk3229 & rk3288 platform, and will support >> more formats in the future. >> >> The driver uses v4l2 framework and RK IOMMU. >> RK IOMMU has not yet been merged. >> Can you share or specify what patches are needed, are they already send to upstream ? So people that want to test your series knows what they need to apply I think that, at least, this patch is required: iommu/rockchip: reconstruct to support multi slaves [1] If this is not already accepted, maybe is a good idea include this patch in the patch series [1] http://www.gossamer-threads.com/lists/linux/kernel/2347458 >> >> >> zhaojun (3): >> media: v4l: Add VP8 format support in V4L2 framework >> media: VPU: support Rockchip VPU >> media: vcodec: rockchip: Add Rockchip VP8 decoder driver >> >> drivers/media/platform/rockchip-vpu/Makefile | 7 + >> .../media/platform/rockchip-vpu/rkvpu_hw_vp8d.c | 798 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vp8d_regs.h | 1594 >> ++++++++++++++++++++ >> drivers/media/platform/rockchip-vpu/rockchip_vpu.c | 799 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vpu_common.h | 439 ++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.c | 1007 +++++++++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.h | 33 + >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.c | 295 ++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.h | 100 ++ >> drivers/media/v4l2-core/v4l2-ctrls.c | 17 +- >> drivers/media/v4l2-core/v4l2-ioctl.c | 3 + >> drivers/media/v4l2-core/videobuf2-dma-contig.c | 51 +- >> include/media/v4l2-ctrls.h | 2 + >> include/media/videobuf2-dma-contig.h | 11 +- >> include/uapi/linux/v4l2-controls.h | 98 ++ >> include/uapi/linux/videodev2.h | 5 + >> 16 files changed, 5238 insertions(+), 21 deletions(-) >> create mode 100644 drivers/media/platform/rockchip-vpu/Makefile >> create mode 100644 drivers/media/platform/rockchip-vpu/rkvpu_hw_vp8d.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vp8d_regs.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_common.h >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.c >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.h >> > > > -- > Best Regards > Shawn Lin > Best Regards, Enric From mboxrd@z Thu Jan 1 00:00:00 1970 From: Enric Balletbo Serra Subject: Re: [PATCH v1 0/3] Add VP8 deocder for rk3229 & rk3288 Date: Tue, 26 Jan 2016 15:00:56 +0100 Message-ID: References: <1453799046-307-1-git-send-email-jung.zhao@rock-chips.com> <56A73CB9.2040802@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <56A73CB9.2040802-TNX95d0MmH7DzftRWevZcw@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Shawn Lin Cc: Jung Zhao , pawel-FA/gS7QP4orQT0dZR+AlfA@public.gmane.org, m.szyprowski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org, kyungmin.park-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org, Mauro Carvalho Chehab , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Sakari Ailus , linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Benoit Parrot , linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Antti Palosaari , Hans Verkuil , alpha.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org, Philipp Zabel , Ricardo Ribalda Delgado , herman.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org, "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-api@vger.kernel.org Hi Jung, 2016-01-26 10:30 GMT+01:00 Shawn Lin : > Hi jun, > > Where is the dt-bingding documentation about your VP8 controller? > > And would you please share some info about rk3229? I can just find > rk3228 in mainline, otherwise may someone think it's a misspell. > > Thanks. > > > On 2016/1/26 17:04, Jung Zhao wrote: >> >> From: zhaojun >> >> >> ==================== >> Introduction >> ==================== >> >> The purpose of this series is to add the driver for vp8 >> decoder on rk3229 & rk3288 platform, and will support >> more formats in the future. >> >> The driver uses v4l2 framework and RK IOMMU. >> RK IOMMU has not yet been merged. >> Can you share or specify what patches are needed, are they already send to upstream ? So people that want to test your series knows what they need to apply I think that, at least, this patch is required: iommu/rockchip: reconstruct to support multi slaves [1] If this is not already accepted, maybe is a good idea include this patch in the patch series [1] http://www.gossamer-threads.com/lists/linux/kernel/2347458 >> >> >> zhaojun (3): >> media: v4l: Add VP8 format support in V4L2 framework >> media: VPU: support Rockchip VPU >> media: vcodec: rockchip: Add Rockchip VP8 decoder driver >> >> drivers/media/platform/rockchip-vpu/Makefile | 7 + >> .../media/platform/rockchip-vpu/rkvpu_hw_vp8d.c | 798 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vp8d_regs.h | 1594 >> ++++++++++++++++++++ >> drivers/media/platform/rockchip-vpu/rockchip_vpu.c | 799 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vpu_common.h | 439 ++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.c | 1007 +++++++++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.h | 33 + >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.c | 295 ++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.h | 100 ++ >> drivers/media/v4l2-core/v4l2-ctrls.c | 17 +- >> drivers/media/v4l2-core/v4l2-ioctl.c | 3 + >> drivers/media/v4l2-core/videobuf2-dma-contig.c | 51 +- >> include/media/v4l2-ctrls.h | 2 + >> include/media/videobuf2-dma-contig.h | 11 +- >> include/uapi/linux/v4l2-controls.h | 98 ++ >> include/uapi/linux/videodev2.h | 5 + >> 16 files changed, 5238 insertions(+), 21 deletions(-) >> create mode 100644 drivers/media/platform/rockchip-vpu/Makefile >> create mode 100644 drivers/media/platform/rockchip-vpu/rkvpu_hw_vp8d.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vp8d_regs.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_common.h >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.c >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.h >> > > > -- > Best Regards > Shawn Lin > Best Regards, Enric From mboxrd@z Thu Jan 1 00:00:00 1970 From: eballetbo@gmail.com (Enric Balletbo Serra) Date: Tue, 26 Jan 2016 15:00:56 +0100 Subject: [PATCH v1 0/3] Add VP8 deocder for rk3229 & rk3288 In-Reply-To: <56A73CB9.2040802@rock-chips.com> References: <1453799046-307-1-git-send-email-jung.zhao@rock-chips.com> <56A73CB9.2040802@rock-chips.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Jung, 2016-01-26 10:30 GMT+01:00 Shawn Lin : > Hi jun, > > Where is the dt-bingding documentation about your VP8 controller? > > And would you please share some info about rk3229? I can just find > rk3228 in mainline, otherwise may someone think it's a misspell. > > Thanks. > > > On 2016/1/26 17:04, Jung Zhao wrote: >> >> From: zhaojun >> >> >> ==================== >> Introduction >> ==================== >> >> The purpose of this series is to add the driver for vp8 >> decoder on rk3229 & rk3288 platform, and will support >> more formats in the future. >> >> The driver uses v4l2 framework and RK IOMMU. >> RK IOMMU has not yet been merged. >> Can you share or specify what patches are needed, are they already send to upstream ? So people that want to test your series knows what they need to apply I think that, at least, this patch is required: iommu/rockchip: reconstruct to support multi slaves [1] If this is not already accepted, maybe is a good idea include this patch in the patch series [1] http://www.gossamer-threads.com/lists/linux/kernel/2347458 >> >> >> zhaojun (3): >> media: v4l: Add VP8 format support in V4L2 framework >> media: VPU: support Rockchip VPU >> media: vcodec: rockchip: Add Rockchip VP8 decoder driver >> >> drivers/media/platform/rockchip-vpu/Makefile | 7 + >> .../media/platform/rockchip-vpu/rkvpu_hw_vp8d.c | 798 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vp8d_regs.h | 1594 >> ++++++++++++++++++++ >> drivers/media/platform/rockchip-vpu/rockchip_vpu.c | 799 ++++++++++ >> .../platform/rockchip-vpu/rockchip_vpu_common.h | 439 ++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.c | 1007 +++++++++++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_dec.h | 33 + >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.c | 295 ++++ >> .../media/platform/rockchip-vpu/rockchip_vpu_hw.h | 100 ++ >> drivers/media/v4l2-core/v4l2-ctrls.c | 17 +- >> drivers/media/v4l2-core/v4l2-ioctl.c | 3 + >> drivers/media/v4l2-core/videobuf2-dma-contig.c | 51 +- >> include/media/v4l2-ctrls.h | 2 + >> include/media/videobuf2-dma-contig.h | 11 +- >> include/uapi/linux/v4l2-controls.h | 98 ++ >> include/uapi/linux/videodev2.h | 5 + >> 16 files changed, 5238 insertions(+), 21 deletions(-) >> create mode 100644 drivers/media/platform/rockchip-vpu/Makefile >> create mode 100644 drivers/media/platform/rockchip-vpu/rkvpu_hw_vp8d.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vp8d_regs.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_common.h >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.c >> create mode 100644 >> drivers/media/platform/rockchip-vpu/rockchip_vpu_dec.h >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.c >> create mode 100644 drivers/media/platform/rockchip-vpu/rockchip_vpu_hw.h >> > > > -- > Best Regards > Shawn Lin > Best Regards, Enric