Re: [PATCH v5 5/5] drm/bridge: anx7625: add HDMI audio function

Hi Xin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on linux/master linus/master v5.12-rc3 next-20210318] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

Re: [PATCH v5 5/5] drm/bridge: anx7625: add HDMI audio function

Hi Xin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on linux/master linus/master v5.12-rc3 next-20210318] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

[RFC PATCH] drm/bridge: anx7625: anx7625_audio_hw_params() can be static

Reported-by: kernel test robot Signed-off-by: kernel test robot --- anx7625.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c b/drivers/gpu/drm/bridge/analogix/anx7625.c index f4653f2b58ad4..38bb8a9db216a 100644 ---

Re: [PATCH v5 3/5] drm/bridge: anx7625: add MIPI DPI input feature support

Hi Xin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on linux/master linus/master v5.12-rc3 next-20210318] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

Re: [PATCH v5 5/5] drm/bridge: anx7625: add HDMI audio function

Hi Xin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [also build test WARNING on linux/master linus/master v5.12-rc3 next-20210318] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

Re: [PATCH] staging: comedi: das800: fix request_irq() warn

Thank you all, I fixed the commit log and resent as v2. Thanks, - Tong On Wed, Mar 17, 2021 at 8:48 AM Ian Abbott wrote: > > On 16/03/2021 22:42, Tong Zhang wrote: > > request_irq() wont accept a name which contains slash so we need to > > repalce it with something else -- otherwise it will

[PATCH v2] staging: comedi: das800: fix request_irq() warn

request_irq() won't accept a name which contains slash so we need to replace it with something else -- otherwise it will throw a warning and the entry in /proc/irq/ will not be created. Since the .name might be used by userspace and we don't want to break userspace, so we are changing the

[PATCH v5 5/5] drm/bridge: anx7625: add HDMI audio function

Add audio HDMI codec function support, enable it through device true flag "analogix,audio-enable". Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 227 ++ drivers/gpu/drm/bridge/analogix/anx7625.h | 5 + 2 files changed, 232 insertions(+)

[PATCH v5 4/5] drm/bridge: anx7625: add HDCP support

Add HDCP feature, enable HDCP function through chip internal key and downstream's capability. Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 147 ++ drivers/gpu/drm/bridge/analogix/anx7625.h | 36 2 files changed, 183 insertions(+)

[PATCH v5 3/5] drm/bridge: anx7625: add MIPI DPI input feature support

Add MIPI rx DPI input support. Reported-by: kernel test robot Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 243 ++ drivers/gpu/drm/bridge/analogix/anx7625.h | 18 ++- 2 files changed, 201 insertions(+), 60 deletions(-) diff --git

[PATCH v5 2/5] drm/bridge: anx7625: fix not correct return value

At some time, the original code may return non zero value, force return 0 if operation finished. Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/anx7625.c

[PATCH v5 0/5] Add MIPI rx DPI support

Hi all, this patch series implement MIPI rx DPI feature. Please help to review. This is the v5 version, any mistakes, please let me know, I'll fix it in the next series. Change history: v5: Fix Rob Herring, Hsin-Yi, Robert Foss comments - Rebase code on the branch 'drm-misc-next', refer

[PATCH] staging: rtl8188eu: Fix a typo

s/pasive/passive/ Signed-off-by: Bhaskar Chowdhury --- drivers/staging/rtl8188eu/core/rtw_mlme.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c b/drivers/staging/rtl8188eu/core/rtw_mlme.c index f87dd71934c3..96d15fca48b0 100644

[staging:staging-testing] BUILD SUCCESS 20ad01363a72c8378bd382a026d17c0eef2793cc

defconfig mips allyesconfig mips allmodconfig powerpc allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a001-20210318 i386

[staging:staging-linus] BUILD SUCCESS 2cafd46a714af1e55354bc6dcea9dcc13f9475b5

allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a001-20210318 i386 randconfig-a005-20210318 i386 randconfig-a003-20210318 i386 randconfig-a002-20210318 i386

Re: [PATCH v3 0/3] Binder: Enable App Freezing Capability

On Wed, Mar 17, 2021 at 1:17 PM Jann Horn wrote: > > On Wed, Mar 17, 2021 at 7:00 PM Christian Brauner > wrote: > > On Mon, Mar 15, 2021 at 06:16:27PM -0700, Li Li wrote: > > > To improve the user experience when switching between recently used > > > applications, the background applications

[PATCH] rtl8188eu: Removed Unnecessary ftrace-like logging

prefer using ftrace Signed-off-by: Paul McQuade --- drivers/staging/rtl8188eu/core/rtw_ap.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_ap.c b/drivers/staging/rtl8188eu/core/rtw_ap.c index 182bb944c9b3..008b60e72758 100644 ---

Re: [PATCH 01/43] Staging: rtl8723bs: fix names in rtw_mlme.h

On Thu, Mar 18, 2021 at 03:16:04PM +0300, Dan Carpenter wrote: > On Wed, Mar 17, 2021 at 11:20:48PM +0100, Marco Cesati wrote: > > diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h > > b/drivers/staging/rtl8723bs/include/rtw_mlme.h > > index 1ebc1e183381..ffcceb1fdde6 100644 > > ---

Re: [PATCH 01/43] Staging: rtl8723bs: fix names in rtw_mlme.h

On Thu, Mar 18, 2021 at 03:16:04PM +0300, Dan Carpenter wrote: > On Wed, Mar 17, 2021 at 11:20:48PM +0100, Marco Cesati wrote: > > diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h > > b/drivers/staging/rtl8723bs/include/rtw_mlme.h > > index 1ebc1e183381..ffcceb1fdde6 100644 > > ---

Re: [PATCH v3 0/3] Binder: Enable App Freezing Capability

On Thu, Mar 18, 2021 at 9:20 AM Todd Kjos wrote: > > On Wed, Mar 17, 2021 at 1:17 PM Jann Horn wrote: > > > > On Wed, Mar 17, 2021 at 7:00 PM Christian Brauner > > wrote: > > > On Mon, Mar 15, 2021 at 06:16:27PM -0700, Li Li wrote: > > > > To improve the user experience when switching between

[PATCH 15/15] staging: rtl8723bs: remove unnecessary logging in os_dep/wifi_regd.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 153: FILE: drivers/staging/rtl8723bs/os_dep/wifi_regd.c:153: + DBG_8192C("%s\n", __func__); Signed-off-by: Fabio Aiuto --- drivers/staging/rtl8723bs/os_dep/wifi_regd.c | 2 -- 1 file

[PATCH 14/15] staging: rtl8723bs: remove unnecessary logging in os_dep/os_intfs.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 977: FILE: drivers/staging/rtl8723bs/os_dep/os_intfs.c:977: + DBG_871X("===> %s.\n", __func__); -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1030: FILE:

[PATCH 13/15] staging: rtl8723bs: remove unnecessary logging in os_dep/ioctl_linux.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1207: FILE: drivers/staging/rtl8723bs/os_dep/ioctl_linux.c:1207: + DBG_871X("%s\n", __func__); -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1507: FILE:

[PATCH 12/15] staging: rtl8723bs: remove unnecessary logging in os_dep/ioctl_cfg80211.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 239: FILE: drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c:239: + /* DBG_8192C("%s\n", __func__); */ -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 569: FILE:

[PATCH 11/15] staging: rtl8723bs: remove unnecessary logging in hal/rtl8723b_hal_init.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1685: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:1685: +/* DBG_8192C("%s\n", __func__); */ -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1722: FILE:

[PATCH 10/15] staging: rtl8723bs: remove unnecessary logging in hal/rtl8723b_cmd.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 129: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c:129: + /* DBG_871X("%s\n", __func__); */ -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 222: FILE:

[PATCH 09/15] staging: rtl8723bs: remove unnecessary logging in hal/odm_debug.h

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 134: FILE: drivers/staging/rtl8723bs/hal/odm_debug.h:134: +#define ODM_dbg_enter() { DbgPrint("==> %s\n", __func__); } -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 135:

[PATCH 08/15] staging: rtl8723bs: remove unnecessary logging in hal/odm.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1109: FILE: drivers/staging/rtl8723bs/hal/odm.c:1109: + /* printk("==> %s\n", __func__); */ Signed-off-by: Fabio Aiuto --- drivers/staging/rtl8723bs/hal/odm.c | 2 -- 1 file changed, 2

[PATCH 07/15] staging: rtl8723bs: remove unnecessary logging in hal/hal_com_phycfg.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 201: FILE: drivers/staging/rtl8723bs/hal/hal_com_phycfg.c:201: + /* DBG_871X("===>%s\n", __func__); */ -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 265: FILE:

[PATCH 06/15] staging: rtl8723bs: remove unnecessary logging in hal/HalBtcOutSrc.h

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 126: FILE: drivers/staging/rtl8723bs/hal/HalBtcOutSrc.h:126: + DbgPrint("%s(): ", __func__);\ and remove the whole unused containing macro BTC_PRINT_F Signed-off-by: Fabio Aiuto

[PATCH 05/15] staging: rtl8723bs: remove unnecessary logging in core/rtw_wlan_util.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1140: FILE: drivers/staging/rtl8723bs/core/rtw_wlan_util.c:1140: + DBG_871X("%s\n", __func__); Signed-off-by: Fabio Aiuto --- drivers/staging/rtl8723bs/core/rtw_wlan_util.c | 2 -- 1

[PATCH 04/15] staging: rtl8723bs: remove unnecessary logging in core/rtw_pwrctrl.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 185: FILE: drivers/staging/rtl8723bs/core/rtw_pwrctrl.c:185: + DBG_871X("==>%s\n", __func__); -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 606: FILE:

[PATCH 03/15] staging: rtl8723bs: remove unnecessary logging in core/rtw_mlme_ext.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 987: FILE: drivers/staging/rtl8723bs/core/rtw_mlme_ext.c:987: + DBG_871X("%s\n", __func__); -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1115: FILE:

[PATCH 02/15] staging: rtl8723bs: remove unnecessary logging in core/rtw_mlme.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1134: FILE: drivers/staging/rtl8723bs/core/rtw_mlme.c:1134: + DBG_871X("%s\n", __func__); -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 1223: FILE:

[PATCH 01/15] staging: rtl8723bs: remove unnecessary logging in core/rtw_ap.c

fix the following checkpatch.pl issues: WARNING: Unnecessary ftrace-like logging - prefer using ftrace 66: FILE: drivers/staging/rtl8723bs/core/rtw_ap.c:66: + /* DBG_871X("%s\n", __func__); */ -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace 559: FILE:

[PATCH 00/15] remove unnecessary logging

remove unnsecessary loggings warned by checkpatch Fabio Aiuto (15): staging: rtl8723bs: remove unnecessary logging in core/rtw_ap.c staging: rtl8723bs: remove unnecessary logging in core/rtw_mlme.c staging: rtl8723bs: remove unnecessary logging in core/rtw_mlme_ext.c staging: rtl8723bs:

Re: [PATCH 01/43] Staging: rtl8723bs: fix names in rtw_mlme.h

On Wed, Mar 17, 2021 at 11:20:48PM +0100, Marco Cesati wrote: > diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h > b/drivers/staging/rtl8723bs/include/rtw_mlme.h > index 1ebc1e183381..ffcceb1fdde6 100644 > --- a/drivers/staging/rtl8723bs/include/rtw_mlme.h > +++

Re: [PATCH] rtsx_chip.c:Fix usleep_range is preferred over udelay

On Thu, Mar 18, 2021 at 04:38:37PM +0530, Akshith A V wrote: > changed udelay to usleep_range Trailing whitespace and odd line wrap :( > because usleep_range is preferred over udelay by checkpatch.pl > > Signed-off-by: Akshith A V > --- > drivers/staging/rts5208/rtsx_chip.c | 2 +- > 1 file

Re: staging: rtl8723bs: prefer ftrace

On Thu, Mar 18, 2021 at 10:14:15AM +0100, Fabio Aiuto wrote: > Hi, > > some suggestions before diving in a new task. > The following checkpatch issue: > > -- > WARNING: Unnecessary ftrace-like logging - prefer using ftrace > #559: FILE: drivers/staging/rtl8723bs/core/rtw_ap.c:559: > +

staging: rtl8723bs: prefer ftrace

Hi, some suggestions before diving in a new task. The following checkpatch issue: -- WARNING: Unnecessary ftrace-like logging - prefer using ftrace #559: FILE: drivers/staging/rtl8723bs/core/rtw_ap.c:559: + DBG_871X("%s\n", __func__); simply says to remove the line, due to the existence

[PATCH v6 13/13] arm64: dts: imx8mq: Add node to G2 hardware

Split VPU node in two: one for G1 and one for G2 since they are different hardware blocks. Add syscon for hardware control block. Remove reg-names property that is useless. Each VPU node only need one interrupt. Signed-off-by: Benjamin Gaignard --- version 5: - use syscon instead of VPU reset

[PATCH v6 11/13] media: hantro: Introduce G2/HEVC decoder

Implement all the logic to get G2 hardware decoding HEVC frames. It support up level 5.1 HEVC stream. It doesn't support yet 10 bits formats or scaling feature. Add HANTRO HEVC dedicated control to skip some bits at the beginning of the slice header. That is very specific to this hardware so

[PATCH v6 12/13] media: hantro: IMX8M: add variant for G2/HEVC codec

Add variant to IMX8M to enable G2/HEVC codec. Define the capabilities for the hardware up to 3840x2160. G2 doesn't have postprocessor, use the same clocks and got it own interruption. Signed-off-by: Benjamin Gaignard --- version 5: - remove useless postproc fields for G2 version 2: - remove

[PATCH v6 10/13] media: hantro: handle V4L2_PIX_FMT_HEVC_SLICE control

Make sure that V4L2_PIX_FMT_HEVC_SLICE is correctly handle by v4l2 of the driver. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/hantro/hantro_v4l2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/media/hantro/hantro_v4l2.c

[PATCH v6 09/13] media: uapi: Add a control for HANTRO driver

The HEVC HANTRO driver needs to know the number of bits to skip at the beginning of the slice header. That is a hardware specific requirement so create a dedicated control that this purpose. Signed-off-by: Benjamin Gaignard --- version 5: - Be even more verbose in control documentation. - Do

[PATCH v6 08/13] media: hantro: Only use postproc when post processed formats are defined

If the variant doesn't offert postprocessed formats make sure it will be ok. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/hantro/hantro.h | 8 ++-- drivers/staging/media/hantro/hantro_postproc.c | 14 ++ drivers/staging/media/hantro/hantro_v4l2.c | 4

[PATCH v6 07/13] media: hantro: Define HEVC codec profiles and supported features

Define which HEVC profiles (up to level 5.1) and features (no scaling, no 10 bits) are supported by the driver. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/hantro/hantro.h | 3 ++ drivers/staging/media/hantro/hantro_drv.c | 58 +++ 2 files changed, 61

[PATCH v6 06/13] media: hantro: change hantro_codec_ops run prototype to return errors

Change hantro_codec_ops run prototype from 'void' to 'int'. This allow to cancel the job if an error occur while configuring the hardware. Signed-off-by: Benjamin Gaignard --- version 5: - forward hantro_h264_dec_prepare_run() return value in case of error

[PATCH v6 05/13] media: hevc: Add decode params control

Add decode params control and it associated structure to regroup all the information that are needed to decode a reference frame as it is describe in ITU-T Rec. H.265 section "8.3.2 Decoding process for reference picture set". Adapt Cedrus driver to these changes. Signed-off-by: Benjamin

[PATCH v6 04/13] media: hevc: Add fields and flags for hevc PPS

Add fields and flags as they are defined in 7.4.3.3.1 "General picture parameter set RBSP semantics of the H.265 ITU specification. Signed-off-by: Benjamin Gaignard --- .../userspace-api/media/v4l/ext-ctrls-codec.rst| 14 ++ include/media/hevc-ctrls.h |

[PATCH v6 03/13] media: hantro: Use syscon instead of 'ctrl' register

In order to be able to share the control hardware block between VPUs use a syscon instead a ioremap it in the driver. To keep the compatibility with older DT if 'nxp,imx8mq-vpu-ctrl' phandle is not found look at 'ctrl' reg-name. With the method it becomes useless to provide a list of register

[PATCH v6 00/13] Add HANTRO G2/HEVC decoder support for IMX8MQ

The IMX8MQ got two VPUs but until now only G1 has been enabled. This series aim to add the second VPU (aka G2) and provide basic HEVC decoding support. To be able to decode HEVC it is needed to add/update some of the structures in the uapi. In addition of them one HANTRO dedicated control is

[PATCH v6 02/13] dt-bindings: media: nxp, imx8mq-vpu: Update the bindings for G2 support

Introducing G2 hevc video decoder lead to modify the bindings to allow to get one node per VPUs. VPUs share one hardware control block which is provided as a phandle on an syscon. Each node got now one reg and one interrupt. Add a compatible for G2 hardware block: nxp,imx8mq-vpu-g2. To be

[PATCH v6 01/13] dt-bindings: mfd: Add 'nxp, imx8mq-vpu-ctrl' to syscon list

Add 'nxp,imx8mq-vpu-ctrl' in the list of possible syscon. It will used to access to the VPU control registers. Signed-off-by: Benjamin Gaignard --- Documentation/devicetree/bindings/mfd/syscon.yaml | 1 + 1 file changed, 1 insertion(+) diff --git