Re: [PATCH v2] Staging: bcm2048: Fix function argument alignment in radio-bcm2048.c.

2018-02-26 Thread Hans Verkuil
On 02/27/2018 02:53 AM, Quytelda Kahja wrote: > Hans, > > Thank you very much for your input on the patch; however this patch > has already been applied to the staging tree. Additionally: I have no record of this being applied through linux-media. Did someone else pick this up? Greg perhaps?

Re: [PATCH v2] Staging: bcm2048: Fix function argument alignment in radio-bcm2048.c.

2018-02-26 Thread Quytelda Kahja
Hans, Thank you very much for your input on the patch; however this patch has already been applied to the staging tree. Additionally: > What coding style problem? You should give a short description of > what you are fixing. The subject of the patch (which becomes the subject of the email when

Re: [PATCH 2/2] selftests: ion: Add simple test with the vgem driver

2018-02-26 Thread Laura Abbott
On 02/26/2018 09:07 AM, Shuah Khan wrote: On 02/19/2018 11:33 AM, Daniel Vetter wrote: On Mon, Feb 19, 2018 at 10:18:21AM -0800, Laura Abbott wrote: On 02/19/2018 07:31 AM, Daniel Vetter wrote: On Thu, Feb 15, 2018 at 05:24:45PM -0800, Laura Abbott wrote: Ion is designed to be a framework

Re: [PATCH v2 char-misc 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

2018-02-26 Thread Stephen Hemminger
On Fri, 16 Feb 2018 23:05:33 + Michael Kelley wrote: > Fix bugs in signaling the Hyper-V host when freeing space in the > host->guest ring buffer: > > 1. The interrupt_mask must not be used to determine whether to signal >on the host->guest ring buffer > 2. The

[PATCH 3.16] staging: usbip stub_rx fix static checker warning on unnecessary checks

2018-02-26 Thread Shuah Khan
Upstream commit 10c901209306 ("usbip: stub_rx: fix static checker warning on unnecessary checks") Back-port fix for static checker warning on unnecessary checks smatch warnings: drivers/staging/usbip/stub_rx.c:360 get_pipe() warn: impossible condition '(pdu->u.cmd_submit.transfer_buffer_length >

Re: [PATCH 2/2] selftests: ion: Add simple test with the vgem driver

2018-02-26 Thread Shuah Khan
On 02/19/2018 11:33 AM, Daniel Vetter wrote: > On Mon, Feb 19, 2018 at 10:18:21AM -0800, Laura Abbott wrote: >> On 02/19/2018 07:31 AM, Daniel Vetter wrote: >>> On Thu, Feb 15, 2018 at 05:24:45PM -0800, Laura Abbott wrote: Ion is designed to be a framework used by other clients who perform

Re: [PATCH 1/2] selftests: ion: Remove some prints

2018-02-26 Thread Shuah Khan
On 02/15/2018 06:24 PM, Laura Abbott wrote: > There's no need to print messages each time we alloc and free. Remove them. > > Signed-off-by: Laura Abbott > --- > tools/testing/selftests/android/ion/ionutils.c | 6 -- > 1 file changed, 6 deletions(-) > > diff --git

[PATCH 4/8] staging: wilc1000: move multiple definition of same macro to common header

2018-02-26 Thread Ajay Singh
Move the same #define from multiple '.c' files to common header file. Instead of having same macro in different '.c' files, now kept in common '.h' file. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/linux_mon.c | 3 ---

[PATCH 3/8] staging: wilc1000: fix line over 80 char in wilc_wlan_handle_txq()

2018-02-26 Thread Ajay Singh
Refactor wilc_wlan_handle_txq() to fix 'line over 80 char' issue found by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_wlan.c | 76 ++-- 1 file changed, 39 insertions(+), 37 deletions(-) diff --git

[PATCH 5/8] staging: wilc1000: rename WILC_WFI_mgmt_rx() to avoid camelCase

2018-02-26 Thread Ajay Singh
Fix "Avoid camelCase" issue found by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- drivers/staging/wilc1000/wilc_wlan.c | 2 +- 3 files changed,

[PATCH 6/8] staging: wilc1000: fix line over 80 char in wilc_wlan_handle_rxq()

2018-02-26 Thread Ajay Singh
Fix 'line over 80 character' issue found by checkpatch.pl script. Refactor wilc_wlan_handle_rxq() code to remove the checkpatch.pl warnings. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_wlan.c | 46 1 file changed,

[PATCH 7/8] staging: wilc1000: fix line over 80 char in wilc_wlan_cfg_set()

2018-02-26 Thread Ajay Singh
Fix 'line over 80 character' issue found by checkpatch.pl script. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_wlan.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c

[PATCH 2/8] staging: wilc1000: rename label _end_ in wilc_wlan_handle_txq()

2018-02-26 Thread Ajay Singh
Rename label name starting with '_' to follow as per linux coding style. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_wlan.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c

[PATCH 1/8] staging: wilc1000: remove unnecessary while(0) in wilc_wlan_handle_txq()

2018-02-26 Thread Ajay Singh
Refactor wilc_wlan_handle_txq() by removing unnecessary while(0) loop. "Line over 80 char" issues in wilc_wlan_handle_txq() are fix by reducing extra leading tab. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_wlan.c | 295

[PATCH 0/8] staging: wilc1000: fix coding style & checkpatch reported issues

2018-02-26 Thread Ajay Singh
Cleanup patch series to fix checkpatch.pl reported issue & code modification to follow linux coding style. Ajay Singh (8): staging: wilc1000: remove unnecessary while(0) in wilc_wlan_handle_txq() staging: wilc1000: rename label _end_ in wilc_wlan_handle_txq() staging: wilc1000: fix line

[PATCH 3/4] staging: fsl-dpaa2/eth: Fix incorrect casts

2018-02-26 Thread Ioana Radulescu
The DPAA2 Ethernet driver incorrectly assumes virtual addresses are always 64b long, which causes compiler errors when building for a 32b platform. Fix this by using explicit casts to uintptr_t where necessary. Signed-off-by: Ioana Radulescu ---

[PATCH 0/4] staging: fsl-mc/dpio, fsl-dpaa2/eth: Enable multi-arch compile

2018-02-26 Thread Ioana Radulescu
The DPAA2 DPIO and Ethernet drivers need only a couple of small fixes in order to compile correctly for 32b platforms. Update the drivers and remove ARCH_LAYERSCAPE from their Kconfig dependencies. Ioana Radulescu (4): staging: fsl-mc/dpio: Fix incorrect casts staging: fsl-mc/dpio: allow the

[PATCH 2/4] staging: fsl-mc/dpio: allow the driver to compile multi-arch

2018-02-26 Thread Ioana Radulescu
Drop dependency on ARCH_LAYERSCAPE (which in turn depends on ARM64), thus allowing this driver to compile on all architectures supported by the fsl-mc bus driver. This was compile tested on: - powerpc (corenet_basic_defconfig, ppc64_defconfig) - x86 (i386_defconfig, x86_64_defconfig, needs

[PATCH 1/4] staging: fsl-mc/dpio: Fix incorrect casts

2018-02-26 Thread Ioana Radulescu
The DPIO driver incorrectly assumes virtual addresses are always 64b long, which causes compiler errors when building for a 32b platform. Fix this by using explicit casts to uintptr_t where necessary. Signed-off-by: Ioana Radulescu ---

[PATCH 4/4] staging: fsl-dpaa2/eth: allow the driver to compile multi-arch

2018-02-26 Thread Ioana Radulescu
Drop dependency on ARCH_LAYERSCAPE (which in turn depends on ARM64), thus allowing this driver to compile on all architectures supported by the fsl-mc bus driver. This was compile tested on: - powerpc (corenet_basic_defconfig, ppc64_defconfig) - x86 (i386_defconfig, x86_64_defconfig, needs

Re: [PATCH 2/3] staging: lustre: lmv: correctly iput lmo_root

2018-02-26 Thread James Simmons
> Commit 8f18c8a48b73 ("staging: lustre: lmv: separate master object > with master stripe") changed how lmo_root inodes were managed, > particularly when LMV_HASH_FLAG_MIGRATION is not set. > Previously lsm_md_oinfo[0].lmo_root was always a borrowed > inode reference and didn't need to by iput().

Re: [PATCH 3/3] staging: lustre: lnet/selftest: don't ignore status from lstcon_test_add

2018-02-26 Thread James Simmons
> If lstcon_test_add sets 'ret' (passed by reference) to 1, > then lst_test_add_ioctl() ignores the return value. > This isn't justified - the return value must be zero for 'ret' > to be meaningful. > > Signed-off-by: NeilBrown Reviewed-by: James Simmons

Re: [PATCH 1/3] staging: lustre: lov: use correct env in lov_io_data_version_end()

2018-02-26 Thread James Simmons
> lov - the logical object volume manager - is responsible for > striping data across multiple volumes. > > So when it is given a request, it creates one or more > sub-requests, one for each target volume. Each sub_io > request has a sub_env environment which it operates in. > > When

Re: [PATCH v2] Staging: bcm2048: Fix function argument alignment in radio-bcm2048.c.

2018-02-26 Thread Hans Verkuil
On 02/20/2018 07:53 AM, Quytelda Kahja wrote: > Fix a coding style problem. What coding style problem? You should give a short description of what you are fixing. > > Signed-off-by: Quytelda Kahja > --- > This is the patch without the unnecessary fixes for line length. >

[PATCH 3/3] media: imx: Don't initialize vars that won't be used

2018-02-26 Thread Mauro Carvalho Chehab
As reported by gcc: + drivers/staging/media/imx/imx-media-csi.c: warning: variable 'input_fi' set but not used [-Wunused-but-set-variable]: => 671:33 + drivers/staging/media/imx/imx-media-csi.c: warning: variable 'pinctrl' set but not used [-Wunused-but-set-variable]: => 1742:18 input_fi

CONTACT

2018-02-26 Thread Mrs.Grace Laboso
My Dearest, I am writing this message from my hospital bed, I appeal to you to exercise a little patience and go through this message. I am Mrs.Grace Laboso; I''m 49 yrs old, from Burkina Faso (west Africa) I was married to Late Dr. Walid Ahmed Juffali, Citizen of Saudi Arabia and Billionaire