Re: [RFC] mm: Replace all open encodings for NUMA_NO_NODE

2018-11-12 Thread Hans Verkuil
On 11/12/2018 03:41 AM, Anshuman Khandual wrote: > At present there are multiple places where invalid node number is encoded > as -1. Even though implicitly understood it is always better to have macros > in there. Replace these open encodings for an invalid node number with the > global macro NUMA

Re: [PATCH v3 01/25] dma-mapping: add generic helpers for mapping sgtable objects

2020-05-07 Thread Hans Verkuil
Hi Marek, On 05/05/2020 10:45, Marek Szyprowski wrote: > struct sg_table is a common structure used for describing a memory > buffer. It consists of a scatterlist with memory pages and DMA addresses > (sgl entry), as well as the number of scatterlist entries: CPU pages > (orig_nents entry) and DMA

Re: [PATCH] media: staging: tegra-vde: Disable building with COMPILE_TEST

2019-08-29 Thread Hans Verkuil
On 8/26/19 3:31 PM, YueHaibing wrote: > If COMPILE_TEST is y and IOMMU_SUPPORT is n, selecting TEGRA_VDE > to m will set IOMMU_IOVA to m, this fails the building of > TEGRA_HOST1X and DRM_TEGRA which is y like this: > > drivers/gpu/host1x/cdma.o: In function `host1x_cdma_init': > cdma.c:(.text+0x6

Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-11-27 Thread Hans Verkuil
rted) >> test Composing: OK (Not Supported) >> test Scaling: OK (Not Supported) >> >> Codec ioctls: >> test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported) >> test VIDIOC_G_ENC_INDEX: OK (Not Supported) >

Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-12-04 Thread Hans Verkuil
On 12/05/2017 04:22 AM, Tomasz Figa wrote: > Hi Raj, > > On Tue, Dec 5, 2017 at 9:13 AM, Mani, Rajmohan > wrote: >> Hi Hans, >> >> Thanks for your patience and sharing your thoughts on this. >> >>> Subject: Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset >>> >>> Hi Rajmohan, >>> >>> On 11/17/2017

Re: [PATCH RESEND v9 18/18] media: platform: Add jpeg enc feature

2020-07-16 Thread Hans Verkuil
On 08/07/2020 13:16, Tomasz Figa wrote: > On Wed, Jul 8, 2020 at 9:14 AM Xia Jiang wrote: >> >> On Tue, 2020-07-07 at 15:35 +0200, Tomasz Figa wrote: >>> On Tue, Jul 7, 2020 at 8:47 AM Xia Jiang wrote: On Tue, 2020-06-30 at 16:53 +, Tomasz Figa wrote: > Hi Xia, > > On Tu

Re: [PATCH v10 30/30] videobuf2: use sgtable-based scatterlist wrappers

2020-09-10 Thread Hans Verkuil
ecause the code already properly did all the > scatterlist related calls. > > Signed-off-by: Marek Szyprowski > Reviewed-by: Robin Murphy Acked-by: Hans Verkuil Note that I agree with Marek to keep returning -EIO. If we want to propagate low-level errors, then that should be done

Re: [PATCH v10 29/30] media: pci: fix common ALSA DMA-mapping related codes

2020-09-10 Thread Hans Verkuil
DMA_FROM_DEVICE > definitions. > > Signed-off-by: Marek Szyprowski Acked-by: Hans Verkuil Thanks! Hans > --- > drivers/media/pci/cx23885/cx23885-alsa.c | 4 ++-- > drivers/media/pci/cx25821/cx25821-alsa.c | 4 ++-- > drivers/media/pci/cx88/cx88-alsa.c | 6 +++

Re: [PATCH v2 2/2] qcom_scm: hide Kconfig symbol

2021-10-11 Thread Hans Verkuil
_SUNXI, drop the 'select RESET_CONTROLLER' statement. >According to my testing this still builds fine, and the QCOM >platform selects this symbol already. Acked-by: Hans Verkuil Thanks, Hans > > Acked-by: Kalle Valo > Acked-by: Alex Elder > Signe

Re: [PATCH v9 00/15] Clean up "mediatek,larb"

2021-12-06 Thread Hans Verkuil
Hi Yong Wu, On 12/11/2021 11:54, Yong Wu wrote: > MediaTek IOMMU block diagram always like below: > > M4U > | > smi-common > | > - > | | ... > | | > larb1 larb2 > | | > vdec venc > > All the consumer connect wit

Re: [PATCH v9 00/15] Clean up "mediatek,larb"

2021-12-06 Thread Hans Verkuil
On 06/12/2021 12:52, Joerg Roedel wrote: > On Fri, Nov 12, 2021 at 06:54:54PM +0800, Yong Wu wrote: >> Yong Wu (14): >> dt-binding: mediatek: Get rid of mediatek, larb for multimedia HW >> iommu/mediatek-v1: Free the existed fwspec if the master dev already >> has >> iommu/mediatek: Retur

Re: [PATCH v9 12/15] media: mtk-vcodec: enc: Remove mtk_vcodec_release_enc_pm

2022-01-13 Thread Hans Verkuil
On 13/01/2022 11:11, AngeloGioacchino Del Regno wrote: > Il 11/01/22 11:57, AngeloGioacchino Del Regno ha scritto: >> Il 12/11/21 11:55, Yong Wu ha scritto: >>> After this patchset, mtk_vcodec_release_enc_pm has only one line. >>> then remove that function, use pm_runtime_disable instead. >>> >>> m

Re: [PATCH v10 00/13] Clean up "mediatek,larb"

2022-01-17 Thread Hans Verkuil
Hi Matthias, On 1/17/22 12:49, Matthias Brugger wrote: > > > On 17/01/2022 11:27, AngeloGioacchino Del Regno wrote: >> Il 17/01/22 08:04, Yong Wu ha scritto: >>> MediaTek IOMMU block diagram always like below: >>> >>> M4U >>>   | >>> smi-common >>>   | >>>    --

Re: [PATCH v9 12/15] media: mtk-vcodec: enc: Remove mtk_vcodec_release_enc_pm

2022-01-21 Thread Hans Verkuil
Hi Matthias, On 1/13/22 17:10, Matthias Brugger wrote: > Hi Hans, > > On 13/01/2022 11:15, Hans Verkuil wrote: >> On 13/01/2022 11:11, AngeloGioacchino Del Regno wrote: >>> Il 11/01/22 11:57, AngeloGioacchino Del Regno ha scritto: >>>> Il 12/11/21 11:55,

[GIT PULL FOR v5.18] Clean up "mediatek,larb"

2022-01-22 Thread Hans Verkuil
Hi Mauro, This is the PR of v10 of this series, minus patches 12 and 13 (Matthias will take those once this is merged for 5.18): https://patchwork.linuxtv.org/project/linux-media/cover/20220117070510.17642-1-yong...@mediatek.com/ The first two patches are from: https://patchwork.linuxtv.org/pr

Re: [PATCH v10 00/13] Clean up "mediatek,larb"

2022-01-31 Thread Hans Verkuil
On 1/17/22 12:49, Matthias Brugger wrote: > > > On 17/01/2022 11:27, AngeloGioacchino Del Regno wrote: >> Il 17/01/22 08:04, Yong Wu ha scritto: >>> MediaTek IOMMU block diagram always like below: >>> >>> M4U >>>   | >>> smi-common >>>   | >>>    - >>>

Re: [PATCH v2 1/2] media: v4l2-ctrl: Add control for VP9 profile

2018-06-08 Thread Hans Verkuil
On 06/08/2018 11:31 AM, Tomasz Figa wrote: > Hi Hans, > > On Fri, Jun 8, 2018 at 6:29 PM Hans Verkuil wrote: >> >> On 05/30/2018 09:16 AM, Keiichi Watanabe wrote: >>> Add a new control V4L2_CID_MPEG_VIDEO_VP9_PROFILE for selecting desired >>> profile for

Re: [PATCH v8 4/8] media: platform: Add Cedrus VPU decoder driver

2018-09-06 Thread Hans Verkuil
On 09/06/2018 09:25 AM, Tomasz Figa wrote: > On Thu, Sep 6, 2018 at 4:01 PM Hans Verkuil wrote: >> >> On 09/05/2018 06:29 PM, Paul Kocialkowski wrote: >>> Hi and thanks for the review! >>> >>> Le lundi 03 septembre 2018 à 11:11 +0200, Hans Verkuil a