From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 35D94C4321A for ; Tue, 11 Jun 2019 11:51:41 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 135E820673 for ; Tue, 11 Jun 2019 11:51:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404099AbfFKLvk (ORCPT ); Tue, 11 Jun 2019 07:51:40 -0400 Received: from metis.ext.pengutronix.de ([85.220.165.71]:38659 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404112AbfFKLvk (ORCPT ); Tue, 11 Jun 2019 07:51:40 -0400 Received: from lupine.hi.pengutronix.de ([2001:67c:670:100:3ad5:47ff:feaf:1a17] helo=lupine) by metis.ext.pengutronix.de with esmtp (Exim 4.89) (envelope-from ) id 1hafJS-0007Ax-Hf; Tue, 11 Jun 2019 13:51:38 +0200 Message-ID: <1560253895.13886.6.camel@pengutronix.de> Subject: Re: [PATCH v3 01/10] rockchip/vpu: rename from rockchip to hantro From: Philipp Zabel To: Boris Brezillon Cc: linux-media@vger.kernel.org, Mauro Carvalho Chehab , Hans Verkuil , Ezequiel Garcia , Nicolas Dufresne , Jonas Karlman , devicetree@vger.kernel.org, kernel@pengutronix.de Date: Tue, 11 Jun 2019 13:51:35 +0200 In-Reply-To: <20190606170614.526d3380@collabora.com> References: <20190531085523.10892-1-p.zabel@pengutronix.de> <20190531085523.10892-2-p.zabel@pengutronix.de> <20190605132242.102fcdd2@collabora.com> <20190606170614.526d3380@collabora.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6-1+deb9u2 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:3ad5:47ff:feaf:1a17 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-media@vger.kernel.org Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org On Thu, 2019-06-06 at 17:06 +0200, Boris Brezillon wrote: > On Wed, 5 Jun 2019 13:22:42 +0200 > Boris Brezillon wrote: > > > On Fri, 31 May 2019 10:55:14 +0200 > > Philipp Zabel wrote: > > > > > Rename the driver and all relevant identifiers from Rockchip to Hantro, > > > as other Hantro IP based VPU implementations can be supported by the > > > same driver. > > > The RK3288 decoder is Hantro G1 based, the encoder is Hantro H1. > > > > > > This patch just renames, no functional changes. > > > > > > Signed-off-by: Philipp Zabel > > > Reviewed-by: Boris Brezillon > > > --- > > > MAINTAINERS | 4 +- > > > drivers/staging/media/Kconfig | 2 +- > > > drivers/staging/media/Makefile | 2 +- > > > .../media/{rockchip/vpu => hantro}/Kconfig | 11 +- > > > drivers/staging/media/hantro/Makefile | 13 + > > > .../media/{rockchip/vpu => hantro}/TODO | 0 > > > .../vpu/rockchip_vpu.h => hantro/hantro.h} | 136 +++++----- > > > .../hantro_drv.c} | 251 +++++++++--------- > > > .../hantro_g1_mpeg2_dec.c} | 55 ++-- > > > .../hantro_h1_jpeg_enc.c} | 44 +-- > > > drivers/staging/media/hantro/hantro_hw.h | 102 +++++++ > > > .../hantro_jpeg.c} | 18 +- > > > drivers/staging/media/hantro/hantro_jpeg.h | 13 + > > > .../hantro_mpeg2.c} | 14 +- > > > .../hantro_v4l2.c} | 234 ++++++++-------- > > > .../hantro_v4l2.h} | 16 +- > > > .../{rockchip/vpu => hantro}/rk3288_vpu_hw.c | 64 ++--- > > > .../vpu => hantro}/rk3288_vpu_regs.h | 2 +- > > > .../{rockchip/vpu => hantro}/rk3399_vpu_hw.c | 64 ++--- > > > .../vpu => hantro}/rk3399_vpu_hw_jpeg_enc.c | 32 +-- > > > .../vpu => hantro}/rk3399_vpu_hw_mpeg2_dec.c | 37 ++- > > > .../vpu => hantro}/rk3399_vpu_regs.h | 2 +- > > > drivers/staging/media/rockchip/vpu/Makefile | 14 - > > > .../media/rockchip/vpu/rockchip_vpu_hw.h | 103 ------- > > > .../media/rockchip/vpu/rockchip_vpu_jpeg.h | 14 - > > > 25 files changed, 616 insertions(+), 631 deletions(-) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/Kconfig (59%) > > > create mode 100644 drivers/staging/media/hantro/Makefile > > > rename drivers/staging/media/{rockchip/vpu => hantro}/TODO (100%) > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu.h => hantro/hantro.h} (69%) > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu_drv.c => hantro/hantro_drv.c} (74%) > > > rename drivers/staging/media/{rockchip/vpu/rk3288_vpu_hw_mpeg2_dec.c => hantro/hantro_g1_mpeg2_dec.c} (87%) > > > rename drivers/staging/media/{rockchip/vpu/rk3288_vpu_hw_jpeg_enc.c => hantro/hantro_h1_jpeg_enc.c} (76%) > > > create mode 100644 drivers/staging/media/hantro/hantro_hw.h > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu_jpeg.c => hantro/hantro_jpeg.c} (95%) > > > create mode 100644 drivers/staging/media/hantro/hantro_jpeg.h > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu_mpeg2.c => hantro/hantro_mpeg2.c} (79%) > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu_v4l2.c => hantro/hantro_v4l2.c} (69%) > > > rename drivers/staging/media/{rockchip/vpu/rockchip_vpu_v4l2.h => hantro/hantro_v4l2.h} (53%) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3288_vpu_hw.c (68%) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3288_vpu_regs.h (99%) > > > > Didn't notice that in my initial review, but rk3288_vpu_regs.h should > > be split in 2: hantro_{g1,h1}_regs.h (g1 being the decoder part and h1 > > the encoder). Will do. I'll also rename the registers from V[DE]PU_REG to [GH]1_REG. > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3399_vpu_hw.c (69%) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3399_vpu_hw_jpeg_enc.c (86%) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3399_vpu_hw_mpeg2_dec.c (92%) > > > rename drivers/staging/media/{rockchip/vpu => hantro}/rk3399_vpu_regs.h (99%) > > > > Same here, rk3399_vpu_regs.h should be split into hantro_{g2,h2}_regs.h > > (I still need to make sure this is actually a g2/h2 combo we have on > > this SoC). That's also true for the > > rk3399_vpu_hw_{mpeg2_dec,jpeg_enc}.c files: they should be renamed > > hantro_{g2_mpeg2_dec,h2_jpeg_enc}.c > > Sorry, I was wrong, it's not a G2/H2 combo we have, it looks like G1/H1 > with shuffled regs/fields. The ID are 0x45142158 (for the decoder) and > 0x3a122158 (for the encoder). Couldn't find any reference to those IDs > in the hantro reference drivers. > > Anyway, I'm working on abstracting the reg layout so we can have the > same base of code for both variants. Ok, I'll keep this part as is for now. regards Philipp