[PATCH] erofs: fix compile warnings when moving out include/trace/events/erofs.h

2019-08-26 Thread Gao Xiang
As Stephon reported [1], many compile warnings are raised when moving out include/trace/events/erofs.h: In file included from include/trace/events/erofs.h:8, from : include/trace/events/erofs.h:28:37: warning: 'struct dentry' declared inside parameter list will not be visible out

[PATCH RESEND] erofs: fix compile warnings when moving out include/trace/events/erofs.h

2019-08-26 Thread Gao Xiang
As Stephon reported [1], many compile warnings are raised when moving out include/trace/events/erofs.h: In file included from include/trace/events/erofs.h:8, from : include/trace/events/erofs.h:28:37: warning: 'struct dentry' declared inside parameter list will not be visible out

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

2019-08-26 Thread YueHaibing
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+0x66c): undefined reference to `alloc_iova' cdma.c:(.

Re: [PATCH RESEND] erofs: fix compile warnings when moving out include/trace/events/erofs.h

2019-08-26 Thread Chao Yu
On 2019-8-26 21:26, Gao Xiang wrote: > As Stephon reported [1], many compile warnings are raised when > moving out include/trace/events/erofs.h: > > In file included from include/trace/events/erofs.h:8, > from : > include/trace/events/erofs.h:28:37: warning: 'struct dentry' declar

Re: [PATCH RESEND] erofs: fix compile warnings when moving out include/trace/events/erofs.h

2019-08-26 Thread Gao Xiang
Hi Chao, On Mon, Aug 26, 2019 at 09:51:35PM +0800, Chao Yu wrote: > On 2019-8-26 21:26, Gao Xiang wrote: [] > > TRACE_EVENT(erofs_lookup, > > ^~~ > > include/trace/events/erofs.h:28:2: note: in expansion of macro 'TP_PROTO' > > TP_PROTO(struct inode *dir, struct dentry *dentry, unsig

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

2019-08-26 Thread Dmitry Osipenko
Hello Yue, 26.08.2019 16:31, YueHaibing пишет: > 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:

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

2019-08-26 Thread Dmitry Osipenko
26.08.2019 18:18, Dmitry Osipenko пишет: > Hello Yue, > > 26.08.2019 16:31, YueHaibing пишет: >> 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/host1

Re: [PATCH] staging: android: ion: cma heap: Limit size of allocated buffer

2019-08-26 Thread Laura Abbott
On 8/23/19 10:28 PM, Alexey Skidanov wrote: In ion_cma_heap, the allocated buffer is represented by a single struct scatterlist instance. The length field of this struct is 32 bit, hence the maximal size of requested buffer should be less than 4GB. The len paramer of the allocation function is 6

Re: [PATCH] staging: android: ion: cma heap: Limit size of allocated buffer

2019-08-26 Thread Alexey Skidanov
On 8/26/19 11:36 AM, Laura Abbott wrote: > On 8/23/19 10:28 PM, Alexey Skidanov wrote: >> In ion_cma_heap, the allocated buffer is represented by a single >> struct scatterlist instance. The length field of this struct is >> 32 bit, hence the maximal size of requested buffer should be >> less than

Re: [PATCH] staging: android: ion: cma heap: Limit size of allocated buffer

2019-08-26 Thread Laura Abbott
On 8/26/19 3:55 AM, Alexey Skidanov wrote: On 8/26/19 11:36 AM, Laura Abbott wrote: On 8/23/19 10:28 PM, Alexey Skidanov wrote: In ion_cma_heap, the allocated buffer is represented by a single struct scatterlist instance. The length field of this struct is 32 bit, hence the maximal size of req

[PATCH v2 1/2] staging: rtl8192u: Add or remove spaces to fix style issues

2019-08-26 Thread Sumera Priyadarsini
This patch fixes the file r8190_rtl8256.c to avoid the following checkpatch.pl warnings: CHECK: spaces preferred around that '<<' (ctx:VxV) CHECK: spaces preferred around that '-' (ctx:VxV) CHECK: No space is necessary after a cast Signed-off-by: Sumera Priyadarsini --- Ch

[PATCH v2 2/2] staging: rtl8192u: Add or remove blank lines as required

2019-08-26 Thread Sumera Priyadarsini
This patch fixes the file r8190_rtl8256.c to avoid the following checkpatch.pl warnings: CHECK: Blank lines aren't necessary before a close brace '}' CHECK: Please use a blank line after function/struct/union/enum declarations CHECK: Please don't use multiple blank line Si

Re: [PATCH 5/8] media: cedrus: Detect first slice of a frame

2019-08-26 Thread Boris Brezillon
Hi Jernej, On Thu, 22 Aug 2019 21:44:57 +0200 Jernej Skrabec wrote: > When codec supports multiple slices in one frame, VPU has to know when > first slice of each frame is being processed, presumably to correctly > clear/set data in auxiliary buffers. > > Add first_slice field to cedrus_run str

Re: [PATCH 2/8] videodev2.h: add V4L2_DEC_CMD_FLUSH

2019-08-26 Thread Boris Brezillon
On Thu, 22 Aug 2019 21:44:54 +0200 Jernej Skrabec wrote: > From: Hans Verkuil > > Add this new V4L2_DEC_CMD_FLUSH decoder command and document it. > > Signed-off-by: Hans Verkuil > Signed-off-by: Jernej Skrabec Reviewed-by: Boris Brezillon > --- > Documentation/media/uapi/v4l/vidioc-deco

Re: [PATCH 7/8] media: cedrus: Add support for holding capture buffer

2019-08-26 Thread Boris Brezillon
On Thu, 22 Aug 2019 21:44:59 +0200 Jernej Skrabec wrote: > When frame contains multiple slices and driver works in slice mode, it's > more efficient to hold capture buffer in queue until all slices of a > same frame are decoded. > > Add support for that to Cedrus driver by exposing and implement

Re: [PATCH 5/8] media: cedrus: Detect first slice of a frame

2019-08-26 Thread Jernej Škrabec
Dne ponedeljek, 26. avgust 2019 ob 20:28:31 CEST je Boris Brezillon napisal(a): > Hi Jernej, > > On Thu, 22 Aug 2019 21:44:57 +0200 > > Jernej Skrabec wrote: > > When codec supports multiple slices in one frame, VPU has to know when > > first slice of each frame is being processed, presumably t

Re: [PATCH 8/8] media: cedrus: Add support for V4L2_DEC_CMD_FLUSH

2019-08-26 Thread Boris Brezillon
On Thu, 22 Aug 2019 21:45:00 +0200 Jernej Skrabec wrote: > This command is useful for explicitly flushing last decoded frame. > > Signed-off-by: Jernej Skrabec Reviewed-by: Boris Brezillon > --- > .../staging/media/sunxi/cedrus/cedrus_video.c | 34 +++ > 1 file changed, 34 i