Re: [PATCH v1 09/18] media: hantro: move hantro_needs_postproc function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v1 09/18] media: hantro: move hantro_needs_postproc function
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Thu, 18 Feb 2021 13:59:49 +0300
Cc: ezequiel@xxxxxxxxxxxxx, p.zabel@xxxxxxxxxxxxxx, mchehab@xxxxxxxxxx, robh+dt@xxxxxxxxxx, shawnguo@xxxxxxxxxx, s.hauer@xxxxxxxxxxxxxx, kernel@xxxxxxxxxxxxxx, festevam@xxxxxxxxx, linux-imx@xxxxxxx, gregkh@xxxxxxxxxxxxxxxxxxx, mripard@xxxxxxxxxx, paul.kocialkowski@xxxxxxxxxxx, wens@xxxxxxxx, jernej.skrabec@xxxxxxxx, krzk@xxxxxxxxxx, shengjiu.wang@xxxxxxx, adrian.ratiu@xxxxxxxxxxxxx, aisheng.dong@xxxxxxx, peng.fan@xxxxxxx, Anson.Huang@xxxxxxx, hverkuil-cisco@xxxxxxxxx, devel@xxxxxxxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-rockchip@xxxxxxxxxxxxxxxxxxx, kernel@xxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-media@xxxxxxxxxxxxxxx
In-reply-to: <20210217080306.157876-10-benjamin.gaignard@collabora.com>
User-agent: Mutt/1.9.4 (2018-02-28)

On Wed, Feb 17, 2021 at 09:02:57AM +0100, Benjamin Gaignard wrote:
> hantro_needs_postproc function becoming to much complex to
> stray inline in .h file move it to .c file.
> 

Yeah.  I do think this would make more sense folded in with patch 7.

regards,
dan carpenter

References:
- [PATCH v1 00/18] Add HANTRO G2/HEVC decoder support for IMX8MQ
  - From: Benjamin Gaignard
- [PATCH v1 09/18] media: hantro: move hantro_needs_postproc function
  - From: Benjamin Gaignard

Prev by Date: Re: [PATCH v1 07/18] media: hantro: Add a field to distinguish the hardware versions
Next by Date: Re: [PATCH 1/3] dt-bindings: Add YAML bindings for Host1x and NVDEC
Previous by thread: [PATCH v1 09/18] media: hantro: move hantro_needs_postproc function
Next by thread: [PATCH v1 10/18] media: hantro: Add helper functions for buffer information
Index(es):
- Date
- Thread

[Index of Archives] [Device Tree Compilter] [Device Tree Spec] [Linux Driver Backports] [Video for Linux] [Linux USB Devel] [Linux PCI Devel] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [XFree86] [Yosemite Backpacking]