Re: [PATCH] staging: wilc1000: revert "fix TODO to compile spi and sdio components in single module"

Hi Arnd, On Mon, 13 Aug 2018 23:20:33 +0200 Arnd Bergmann wrote: > The TODO item named "make spi and sdio components coexist in one > build" was apparently addressed a long time ago, but never removed > from the TODO file. However, the new patch that tries to address it > actually makes it

[PATCH] android: binder: no outgoing transaction when thread todo has transaction

When a process dies, failed reply is sent to the sender of any transaction queued on a dead thread's todo list. The sender asserts that the received failed reply corresponds to the head of the transaction stack. This assert can fail if the dead thread is allowed to send outgoing transactions when

[PATCH] media: imx: work around false-positive warning, again

A warning that I thought to be solved by a previous patch of mine has resurfaced with gcc-8: media/imx/imx-media-csi.c: In function 'csi_link_validate': media/imx/imx-media-csi.c:1025:20: error: 'upstream_ep' may be used uninitialized in this function [-Werror=maybe-uninitialized]

[PATCH] staging: wilc1000: revert "fix TODO to compile spi and sdio components in single module"

The TODO item named "make spi and sdio components coexist in one build" was apparently addressed a long time ago, but never removed from the TODO file. However, the new patch that tries to address it actually makes it worse again by duplicating the common parts of the driver into two separate

[PATCH] Staging: rtlwifi: efuse: fixed a line length coding style issue

Fixed a code style issue. Line length over 80 characters. Signed-off-by: Tom Todd --- drivers/staging/rtlwifi/efuse.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtlwifi/efuse.c b/drivers/staging/rtlwifi/efuse.c index 1dc71455f270..49ec9728fb04 100644

Re: [PATCH 2/5] vmbus: add driver_override support

On Mon, 13 Aug 2018 19:30:50 + "Michael Kelley (EOSG)" wrote: > From: k...@linuxonhyperv.com Sent: Friday, August > 10, 2018 4:06 PM > > > From: Stephen Hemminger > > > > Add support for overriding the default driver for a VMBus device > > in the same way that it can be done for PCI

Re: [PATCH 2/5] vmbus: add driver_override support

On Mon, Aug 13, 2018 at 07:30:50PM +, Michael Kelley (EOSG) wrote: > From: k...@linuxonhyperv.com Sent: Friday, August > 10, 2018 4:06 PM > > > From: Stephen Hemminger > > > > Add support for overriding the default driver for a VMBus device > > in the same way that it can be done for PCI

RE: [PATCH 2/5] vmbus: add driver_override support

From: k...@linuxonhyperv.com Sent: Friday, August 10, 2018 4:06 PM > From: Stephen Hemminger > > Add support for overriding the default driver for a VMBus device > in the same way that it can be done for PCI devices. This patch > adds the /sys/bus/vmbus/devices/.../driver_override file > and

Re: [PATCH] vmbus: don't return values for uninitalized channels

Stephen Hemminger writes: > For unsupported device types, the vmbus channel ringbuffer is never > initialized, and therefore reading the sysfs files will return garbage > or cause a kernel OOPS. > > Fixes: c2e5df616e1a ("vmbus: add per-channel sysfs info") > Signed-off-by: Stephen Hemminger >

[PATCH] vmbus: don't return values for uninitalized channels

For unsupported device types, the vmbus channel ringbuffer is never initialized, and therefore reading the sysfs files will return garbage or cause a kernel OOPS. Fixes: c2e5df616e1a ("vmbus: add per-channel sysfs info") Signed-off-by: Stephen Hemminger --- drivers/hv/vmbus_drv.c | 3 +++ 1

RE: [PATCH 1/5] Tools: hv: Fix a bug in the key delete code

From: k...@linuxonhyperv.com Sent: Friday, August 10, 2018 4:06 PM > > Fix a bug in the key delete code - the num_records range > from 0 to num_records-1. > > Signed-off-by: K. Y. Srinivasan > Reported-by: David Binderman > Cc: > --- Reviewed-by: Michael Kelley

[PATCH] Staging: octeon-usb: Replaces CVMX_WAIT_FOR_FIELD32 macro with a function

Replaces CVMX_WAIT_FOR_FIELD32 macro with equivalent function. Signed-off-by: Georgios Tsotsos --- drivers/staging/octeon-usb/octeon-hcd.c | 56 + 1 file changed, 29 insertions(+), 27 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c

Re: [PATCH] staging: erofs: change 'unsigned' to 'unsigned int'

Hi Leon, On 2018/8/13 23:20, Leon Imhof wrote: > Fix coding style issue "Prefer 'unsigned int' to bare use of 'unsigned'" > detected by checkpatch.pl > > Signed-off-by: Leon Imhof > --- Thanks for the patch, could be better if use `[PATCH v2]' to indicate that you send a new version of

[PATCH] staging: erofs: change 'unsigned' to 'unsigned int'

Fix coding style issue "Prefer 'unsigned int' to bare use of 'unsigned'" detected by checkpatch.pl Signed-off-by: Leon Imhof --- drivers/staging/erofs/data.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/erofs/data.c b/drivers/staging/erofs/data.c index

Re: [PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset

On Monday, 13 August 2018 17:50:14 MSK Thierry Reding wrote: > From: Thierry Reding > > The BSEV clock has a separate gate bit and can not be assumed to be > always enabled. Add explicit handling for the BSEV clock and reset. > > This fixes an issue on Tegra124 where the BSEV clock is not

[PATCH 09/14] staging: media: tegra-vde: Add IOMMU support

From: Thierry Reding Implement support for using an IOMMU to map physically discontiguous buffers into contiguous I/O virtual mappings that the VDE can use. This allows importing arbitrary DMA-BUFs for use by the VDE. While at it, make sure that the device is detached from any DMA/IOMMU mapping

[PATCH 02/14] staging: media: tegra-vde: Support reference picture marking

From: Thierry Reding Tegra114 and Tegra124 support reference picture marking, which will cause BSEV to write picture marking data to SDRAM. Make sure there is a valid destination address for that data to avoid error messages from the memory controller. Signed-off-by: Thierry Reding ---

[PATCH 03/14] staging: media: tegra-vde: Prepare for interlacing support

From: Thierry Reding The number of frames doubles when decoding interlaced content and the structures describing the frames double in size. Take that into account to prepare for interlacing support. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 73

[PATCH 00/14] staging: media: tegra-vdea: Add Tegra124 support

From: Thierry Reding Hi, this set of patches perform a bit of cleanup and extend support to the VDE implementation found on Tegra114 and Tegra124. This requires adding handling for a clock and a reset for the BSEV block that is separate from the main VDE block. The new VDE revision also

[PATCH 11/14] ARM: tegra: Enable VDE on Tegra124

From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra124.dtsi | 40 + 1 file changed, 40 insertions(+) diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi index b113e47b2b2a..8fdca4723205 100644 ---

[PATCH 14/14] ARM: tegra: Enable SMMU for VDE on Tegra124

From: Thierry Reding The video decode engine can use the SMMU to use buffers that are not physically contiguous in memory. This allows better memory usage for video decoding, since fragmentation may cause contiguous allocations to fail. Signed-off-by: Thierry Reding ---

[PATCH 06/14] staging: media: tegra-vde: Print out invalid FD

From: Thierry Reding Include the invalid file descriptor when reporting an error message to help diagnosing why importing the buffer failed. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 12/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra20

From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra20.dtsi | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi index 15b73bd377f0..abb5738a0705 100644 ---

[PATCH 07/14] staging: media: tegra-vde: Add some clarifying comments

From: Thierry Reding Add some comments specifying what tables are being set up in VRAM. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/staging/media/tegra-vde/tegra-vde.c

[PATCH 10/14] staging: media: tegra-vde: Keep VDE in reset when unused

From: Thierry Reding There is no point in keeping the VDE module out of reset when it is not in use. Reset it on runtime suspend. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 04/14] staging: media: tegra-vde: Use DRM/KMS framebuffer modifiers

From: Thierry Reding VDE on Tegra20 through Tegra114 supports reading and writing frames in 16x16 tiled layout. Similarily, the various block-linear layouts that are supported by the GPU on Tegra124 can also be read from and written to by the Tegra124 VDE. Enable userspace to specify the

[PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset

From: Thierry Reding The BSEV clock has a separate gate bit and can not be assumed to be always enabled. Add explicit handling for the BSEV clock and reset. This fixes an issue on Tegra124 where the BSEV clock is not enabled by default and therefore accessing the BSEV registers will hang the

[PATCH 13/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra30

From: Thierry Reding Signed-off-by: Thierry Reding --- arch/arm/boot/dts/tegra30.dtsi | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi index a6781f653310..492917d61bab 100644 ---

[PATCH 05/14] staging: media: tegra-vde: Properly mark invalid entries

From: Thierry Reding Entries in the reference picture list are marked as invalid by setting the frame ID to 0x3f. Signed-off-by: Thierry Reding --- drivers/staging/media/tegra-vde/tegra-vde.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 08/14] staging: media: tegra-vde: Track struct device *

From: Thierry Reding The pointer to the struct device is frequently used, so store it in struct tegra_vde. Also, pass around a pointer to a struct tegra_vde instead of struct device in some cases to prepare for subsequent patches referencing additional data from that structure. Signed-off-by:

дюгд

взр тднй ттос ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

еюда

фцф йиаэ егея ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

> But it is better to fix them in an independent patch. :) Yah. Of course. This was completely unrelated. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

Hi Dan, On 2018/8/13 20:25, Dan Carpenter wrote: > On Mon, Aug 13, 2018 at 08:17:27PM +0800, Gao Xiang wrote: >>> /* we assume that ofs is aligned with 4 bytes */ >>> it->ofs = EROFS_XATTR_ALIGN(it->ofs); >>> return err; >>> > This might be cleaner if we wrote: > > return (err

Re: [PATCH 4/8] staging: erofs: cleanup z_erofs_vle_work_{lookup, register}

Hi Dan, On 2018/8/13 21:05, Dan Carpenter wrote: > Yeah. You'd have to remove the const. > > Anyway, on looking at it more, I guess this patch is fine for now. We > will probably end up changing z_erofs_vle_work_lookup() and > z_erofs_vle_work_register() some more in the future. > Thanks for

[PATCH 1/2] staging: most: register channel device after init of struct members

This patch moves the call to device_register to the end of the channel initialization section that the devcie belongs to. It is needed to avoid NULL pointer dereferences once the device is registered with sysfs. Signed-off-by: Christian Gromm --- drivers/staging/most/core.c | 8 1 file

Re: [PATCH] staging: erofs: add int to usigned

Hi Leon, On 2018/8/13 20:09, Leon Imhof wrote: > Fix coding style issue "Prefer 'unsigned int' to bare use of 'unsigned'" > detected by checkpatch.pl > > Signed-off-by: Leon Imhof Looks fine, Reviewed-by: Gao Xiang (BTW, could you fix the title of this commit? Thanks.) > --- >

Re: [PATCH 4/8] staging: erofs: cleanup z_erofs_vle_work_{lookup, register}

Yeah. You'd have to remove the const. Anyway, on looking at it more, I guess this patch is fine for now. We will probably end up changing z_erofs_vle_work_lookup() and z_erofs_vle_work_register() some more in the future. regards, dan carpenter ___

[PATCH 2/2] staging: most: do not make interface dependent attrs default for all channels

The channel attribute dbr_size is only relevant for the DIM2 interface. so is the packets_per_xact for USB. This patch cleans up the driver's ABI by not showing all attributes by default for each channel, but only on those they belong to. Signed-off-by: Christian Gromm ---

[PATCH 0/2] staging: most: clean up sysfs interface

This patch set cleans up the driver's ABI. It takes care of proper initialization sequences and prevents interface dependent channel attributes from being displayed by default on all interfaces. Christian Gromm (2): staging: most: register channel device after init of struct members staging:

Re: [PATCH 6/8] staging: erofs: fix vle_decompressed_index_clusterofs

Hi Dan, On 2018/8/13 20:03, Dan Carpenter wrote: >> -static inline unsigned >> -vle_compressed_index_clusterofs(unsigned clustersize, >> -struct z_erofs_vle_decompressed_index *di) >> +static inline int >> +vle_decompressed_index_clusterofs(unsigned int *clusterofs, > > Not related to your

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

Hi Dan, On 2018/8/13 20:40, Dan Carpenter wrote: > On Mon, Aug 13, 2018 at 08:17:27PM +0800, Gao Xiang wrote: @@ -294,8 +322,11 @@ static int inline_getxattr(struct inode *inode, struct getxattr_iter *it) ret = xattr_foreach(>it, _xattr_handlers, ); if

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

On 2018/8/13 20:17, Gao Xiang wrote: >> Generally the rule on likely/unlikely is that they hurt readability so >> we should only add them if it makes a difference in benchmarking. >> > > In my opinion, return values other than 0 and ENOATTR(ENODATA) rarely happens, > it should be in the slow

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

On Mon, Aug 13, 2018 at 08:17:27PM +0800, Gao Xiang wrote: > >> @@ -294,8 +322,11 @@ static int inline_getxattr(struct inode *inode, > >> struct getxattr_iter *it) > >>ret = xattr_foreach(>it, _xattr_handlers, ); > >>if (ret >= 0) > >>break; > >> + > >>

Re: [PATCH 4/8] staging: erofs: cleanup z_erofs_vle_work_{lookup, register}

Hi Dan, On 2018/8/13 20:00, Dan Carpenter wrote: > On Sun, Aug 12, 2018 at 10:01:46PM +0800, Chao Yu wrote: >> From: Gao Xiang >> >> This patch introduces 'struct z_erofs_vle_work_finder' to clean up >> arguments of z_erofs_vle_work_lookup and z_erofs_vle_work_register. >> >> Signed-off-by: Gao

Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

On 2018/8/13 19:04, Dan Carpenter wrote: > On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: >> --- a/drivers/staging/erofs/data.c >> +++ b/drivers/staging/erofs/data.c >> @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) >> } >> >> /* prio -- true is used for dir */ >>

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

On Mon, Aug 13, 2018 at 08:17:27PM +0800, Gao Xiang wrote: > > /* we assume that ofs is aligned with 4 bytes */ > > it->ofs = EROFS_XATTR_ALIGN(it->ofs); > > return err; > > This might be cleaner if we wrote: return (err < 0) ? error : 0; The callers all treate zero and one

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

Hi Dan, On 2018/8/13 19:47, Dan Carpenter wrote: >> -static void xattr_iter_fixup(struct xattr_iter *it) >> +static inline int xattr_iter_fixup(struct xattr_iter *it) >> { >> -if (unlikely(it->ofs >= EROFS_BLKSIZ)) { >> -xattr_iter_end(it, true); >> +if (likely(it->ofs <

[PATCH] staging: erofs: add int to usigned

Fix coding style issue "Prefer 'unsigned int' to bare use of 'unsigned'" detected by checkpatch.pl Signed-off-by: Leon Imhof --- drivers/staging/erofs/data.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/erofs/data.c b/drivers/staging/erofs/data.c index

Re: [PATCH 6/8] staging: erofs: fix vle_decompressed_index_clusterofs

> -static inline unsigned > -vle_compressed_index_clusterofs(unsigned clustersize, > - struct z_erofs_vle_decompressed_index *di) > +static inline int > +vle_decompressed_index_clusterofs(unsigned int *clusterofs, Not related to your patch, but don't make functions inline. Leave it to the

Re: [PATCH 4/8] staging: erofs: cleanup z_erofs_vle_work_{lookup, register}

On Sun, Aug 12, 2018 at 10:01:46PM +0800, Chao Yu wrote: > From: Gao Xiang > > This patch introduces 'struct z_erofs_vle_work_finder' to clean up > arguments of z_erofs_vle_work_lookup and z_erofs_vle_work_register. > > Signed-off-by: Gao Xiang > Reviewed-by: Chao Yu > Signed-off-by: Chao Yu

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

> -static void xattr_iter_fixup(struct xattr_iter *it) > +static inline int xattr_iter_fixup(struct xattr_iter *it) > { > - if (unlikely(it->ofs >= EROFS_BLKSIZ)) { > - xattr_iter_end(it, true); > + if (likely(it->ofs < EROFS_BLKSIZ)) > + return 0; > > -

Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

Hi Dan, On 2018/8/13 19:04, Dan Carpenter wrote: > On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: >> --- a/drivers/staging/erofs/data.c >> +++ b/drivers/staging/erofs/data.c >> @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) >> } >> >> /* prio -- true is used for

Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: > --- a/drivers/staging/erofs/data.c > +++ b/drivers/staging/erofs/data.c > @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) > } > > /* prio -- true is used for dir */ > -struct page *erofs_get_meta_page(struct

mobile apps for you

Do you need mobile apps development? We can do it for you. We are an India base company. Here are the details about us: Years in business: 8 Staffs: 125 App developed: 350 We work on Android, iOS, Ionic, and PhoneGap platforms, we have clients across different kind of industries. Such like

Re: [PATCH 3/8] staging: erofs: add error handling for xattr submodule

Hi Xiang, On 2018/8/13 10:36, Gao Xiang wrote: > Hi Chao, > > On 2018/8/13 10:00, Chao Yu wrote: >> On 2018/8/12 22:01, Chao Yu wrote: >>> From: Gao Xiang >>> >>> This patch enhances the missing error handling code for >>> xattr submodule, which improves the stability for the rare cases. >>>

the mobile apps

Do you need mobile apps development? We can do it for you. We are an India base company. Here are the details about us: Years in business: 8 Staffs: 125 App developed: 350 We work on Android, iOS, Ionic, and PhoneGap platforms, we have clients across different kind of industries. Such like

Re: [PATCH 00/16] staging: gasket: return of the son of cleanups

Btw, can someone explain what 'gasket' even is? A quick look through the directory didn't provide any answers. And please don't just send that anser to me, put it in the source tree (Kconfig, and if needed another documentation file). ___ devel mailing