Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Shivani Bhardwaj
On Sat, Nov 7, 2015 at 1:02 PM, Dilger, Andreas wrote: > On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: > >>Remove the prototype of function cfs_trace_free_string_buffer() as it is >>no longer needed. > > These patches would be a lot more useful if the summary contained the name > of the functio

[PATCH 9/9] Staging: lustre: llite_internal: Remove function prototype

2015-11-06 Thread Shivani Bhardwaj
Remove the prototype of the function ll_finish_md_op_data() as it is no longer needed. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_internal.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_internal.h b/drivers/stagi

[PATCH 7/9] Staging: lustre: llite_close: Substitute function calls

2015-11-06 Thread Shivani Bhardwaj
Substitute standard function kfree() in place of the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_close.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_close.c b/dri

[PATCH 8/9] Staging: lustre: llite_lib: Remove wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function ll_finish_md_op_data() and replace all its calls with the standrd function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_lib.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/sta

[PATCH 6/9] Staging: lustre: llite_nfs: Replace with standard function

2015-11-06 Thread Shivani Bhardwaj
Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/llite_nfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_nfs.c b/

[PATCH 5/9] Staging: lustre: symlink: Substitute standard function

2015-11-06 Thread Shivani Bhardwaj
Substitute the standard function kfree() for the function ll_finish_md_op_data(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/symlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/symlink.c b/drivers/staging

[PATCH 4/9] Staging: lustre: xattr_cache: Change function calls

2015-11-06 Thread Shivani Bhardwaj
Change the calls of the function ll_finish_md_op_data() to the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/xattr_cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/llite/xattr_cache.c b

[PATCH 3/9] Staging: lustre: namei: Replace calls with kfree

2015-11-06 Thread Shivani Bhardwaj
Replace the calls of the function ll_finish_md_op_data() with thr stndard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/namei.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/namei.c

[PATCH 2/9] Staging: lustre: file: Replace function calls with standard function

2015-11-06 Thread Shivani Bhardwaj
Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/file.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/lustre/lustre

[PATCH 1/9] Staging: lustre: dir: Replace function calls

2015-11-06 Thread Shivani Bhardwaj
Replace the calls of the function ll_finish_md_op_data() with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/dir.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c

[PATCH 0/9] Remove wrapper function and tidy up the code

2015-11-06 Thread Shivani Bhardwaj
This patchset removes an unnecessary wrapper function and replaces all its calls in different files with the standard function that it wrapped. Also, one patch removes its prototype. After applying this patch, code becomes cleaner. Shivani Bhardwaj (9): Staging: lustre: dir: Replace function cal

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Dilger, Andreas
On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: >Remove the prototype of function cfs_trace_free_string_buffer() as it is >no longer needed. These patches would be a lot more useful if the summary contained the name of the function being removed, rather than "remove function prototype" and varia

Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch

2015-11-06 Thread Ivan Safonov
On 11/07/2015 10:14 AM, Greg Kroah-Hartman wrote: On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: goto is not needed here. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-)

[PATCH] Staging: fwserial: Declare fwtty_port_get as static

2015-11-06 Thread Shraddha Barke
Declare the function fwtty_port_get as static since it is used only in this particular file. Also remove the corresponding declaration from header file. Signed-off-by: Shraddha Barke --- drivers/staging/fwserial/fwserial.c | 3 +-- drivers/staging/fwserial/fwserial.h | 1 - 2 files changed, 1 in

Re: [PATCH v2 1/7] staging: sm750fb: remove unused modedb.h

2015-11-06 Thread Greg Kroah-Hartman
On Wed, Nov 04, 2015 at 12:38:05PM +0200, Mike Rapoport wrote: > The modedb.h defines unused set of videomodes and can be removed > > Signed-off-by: Mike Rapoport > --- > drivers/staging/sm750fb/modedb.h | 233 > --- > drivers/staging/sm750fb/sm750.c | 2 -

Re: [PATCH] Staging: fwserial: Declare fwtty_port_put as static

2015-11-06 Thread Greg KH
On Thu, Nov 05, 2015 at 07:35:36PM +0530, Shraddha Barke wrote: > Declare the function fwtty_port_put as static since it is used > only in this particular file. Also remove the corresponding > declaration from header file. > > Signed-off-by: Shraddha Barke > --- > drivers/staging/fwserial/fwseri

Re: [PATCH] staging: lustre: remove IOC_LIBCFS_PING_TEST ioctl

2015-11-06 Thread Greg Kroah-Hartman
On Fri, Nov 06, 2015 at 09:08:03PM -0500, y...@ccs.ornl.gov wrote: > From: James Simmons > > The ioctl IOC_LIBCFS_PING_TEST has not been used in > ages. The recent nidstring changes which moved all > the nidstring operations from libcfs to the LNet > layer but this ioctl code was still using an >

Re: [PATCH RESEND 11/11] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 08:07:06PM -0500, Jubin John wrote: > --- a/drivers/staging/rdma/hfi1/hfi.h > +++ b/drivers/staging/rdma/hfi1/hfi.h > @@ -1084,6 +1084,10 @@ struct hfi1_devdata { > /* Save the enabled LCB error bits */ > u64 lcb_err_en; > u8 dc_shutdown; > + > + /* rec

Re: [PATCH RESEND 10/11] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 08:07:05PM -0500, Jubin John wrote: > From: Ira Weiny > > sdma_select_engine_vl only needs to protect itself from an invalid VL. > Something higher up the stack should be warning the user when they try > to use an SL which maps to an invalid VL. > > Reviewed-by: Dean Luic

Re: [PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch

2015-11-06 Thread Greg Kroah-Hartman
On Fri, Nov 06, 2015 at 10:17:29PM +0700, Ivan Safonov wrote: > goto is not needed here. > > Signed-off-by: Ivan Safonov > --- > drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- > 1 file changed, 11 insertions(+), 14 deletions(-) > I've applied most of these patches, bu

Re: [PATCH v4 6/9] staging/rdma/hfi1: Implement Expected Receive TID caching

2015-11-06 Thread Greg KH
On Fri, Oct 30, 2015 at 06:58:45PM -0400, ira.we...@intel.com wrote: > From: Mitko Haralanov > > Expected receives work by user-space libraries (PSM) calling into the driver > with information about the user's receive buffer and have the driver DMA-map > that buffer and program the HFI to receive

Re: [PATCH] lustre: obdclass: fix sparse warning

2015-11-06 Thread Greg KH
On Sun, Nov 01, 2015 at 02:12:29PM +0530, Paul Davies C wrote: > Fix the following warning given by sparse: > > drivers/staging/lustre/lustre/obdclass/cl_lock.c:690:5: warning: symbol > 'cl_lock_mutex_try' was not declared. Should it be static? > > Signed-off-by: Paul Davies C > --- > drivers/

Re: [PATCH v3 5/5] staging: lustre: libcfs.h: remove IS_PO2 and __is_po2

2015-11-06 Thread Greg KH
On Thu, Oct 29, 2015 at 03:00:24AM +0200, Aya Mahfouz wrote: > Removes IS_PO2 and __is_po2 since the uses of IS_PO2 have > been replaced by is_power_of_2 > > Signed-off-by: Aya Mahfouz > --- > v2: > -became patch number 5 in the series > v3: > -no change > > drivers/staging/lustre

Re: [lustre-devel] [PATCH 07/10] staging: lustre: Handle nodemask on UMP machines

2015-11-06 Thread Greg Kroah-Hartman
On Fri, Nov 06, 2015 at 10:18:17PM +, Simmons, James A. wrote: > >All warnings (new ones prefixed by >>): > > > > In file included from include/linux/bitops.h:36:0, > >from > > drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/linux/libcfs.h:44, > >

Re: [PATCH v3 2/5] staging: lustre: hash.c: Replace IS_PO2 by is_power_of_2

2015-11-06 Thread Greg KH
On Thu, Oct 29, 2015 at 02:55:50AM +0200, Aya Mahfouz wrote: > Replaces IS_PO2 by is_power_of_2. It is more accurate to use > is_power_of_2 since it returns 1 for numbers that are powers > of 2 only whereas IS_PO2 returns 1 for 0 and numbers that are > powers of 2. > > Reviewed-by: Andreas Dilger

Re: [PATCH 5/5] comedi: driver: Fix - BIT macro used coding style issue

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 07:57:46PM +0530, Ranjith wrote: > BIT macro is used for defining bit location instead of shifting > operator - coding style issue > > Signed-off-by: Ranjith T > --- > drivers/staging/comedi/drivers/ii_pci20kc.c | 70 > +-- > 1 file changed, 35

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread Greg KH
On Sat, Nov 07, 2015 at 03:29:23AM +0530, Punit Vara wrote: > This patch is to the linux_wlan.c file that fixes declaration of *wilc > to remove following error while building it. > > make drivers/staging/wilc1000/linux_wlan.o > drivers/staging/wilc1000/linux_wlan.c:1824:24: error: ‘wilc’ undeclar

Re: [PATCH v4 5/9] staging/rdma/hfi1: Add function stubs for TID caching

2015-11-06 Thread Greg KH
On Fri, Oct 30, 2015 at 06:58:44PM -0400, ira.we...@intel.com wrote: > From: Mitko Haralanov > > Add mmu notify helper functions and TID caching function stubs in preparation > for the TID caching implementation. > > TID caching makes use of the MMU notifier to allow the driver to respond to >

Re: [PATCH 3/3] staging/rdma/hfi1: Method to toggle "fast ECN" detection

2015-11-06 Thread Greg KH
On Thu, Nov 05, 2015 at 12:02:25PM -0500, ira.weiny wrote: > On Thu, Nov 05, 2015 at 10:56:27AM +0300, Dan Carpenter wrote: > > On Wed, Nov 04, 2015 at 09:10:11PM -0500, ira.we...@intel.com wrote: > > > From: Vennila Megavannan > > > > > > Add a module paramter to toggle prescan/Fast ECN Detectio

Re: [PATCH] staging: lustre: lnet: selftest: Move extern declarations to headers

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 02:04:21AM +0530, Amitoj Kaur Chawla wrote: > This patch moves extern declarations to respective header files. > > This patch also removes extern keyword from function declarations > since functions have the extern specifier by default. > > Signed-off-by: Amitoj Kaur Chawl

[PATCH] staging: lustre: remove IOC_LIBCFS_PING_TEST ioctl

2015-11-06 Thread y
From: James Simmons The ioctl IOC_LIBCFS_PING_TEST has not been used in ages. The recent nidstring changes which moved all the nidstring operations from libcfs to the LNet layer but this ioctl code was still using an nidstring operation that was causing an circular dependency loop between libcfs

[PATCH v3 10/11] staging: fsl-mc: Added DPRC interrupt handler

2015-11-06 Thread J. German Rivera
The interrupt handler for DPRC IRQs is added. DPRC IRQs are generated for hot plug events related to DPAA2 objects in a given DPRC. These events include, creating/destroying DPAA2 objects in the DPRC, changing the "plugged" state of DPAA2 objects and moving objects between DPRCs. Signed-off-by: J.

[PATCH v3 09/11] staging: fsl-mc: Fixed bug in dprc_probe() error path

2015-11-06 Thread J. German Rivera
Destroy mc_io in error path in dprc_probe() only if the mc_io was created in this function. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: none - Addressed comment from Dan Carpenter * Renamed goto error labels to indicate what the goto does drivers/st

[PATCH v3 07/11] staging: fsl-mc: Populate the IRQ pool for an MC bus instance

2015-11-06 Thread J. German Rivera
Scan the corresponding DPRC container to get total count of IRQs needed by all its child DPAA2 objects. Then, preallocate a set of MSI IRQs with the DPRC's ICID (GIT-ITS device Id) to populate the the DPRC's IRQ pool. Each child DPAA2 object in the DPRC and the DPRC object itself will allocate thei

[PATCH v3 05/11] staging: fsl-mc: Extended MC bus allocator to include IRQs

2015-11-06 Thread J. German Rivera
All the IRQs for DPAA2 objects in the same DPRC must use the ICID of that DPRC, as their device Id in the GIC-ITS. Thus, all these IRQs must share the same ITT table in the GIC. As a result, a pool of IRQs with the same device Id must be preallocated per DPRC (fsl-mc bus instance). So, the fsl-mc b

[PATCH v3 01/11] irqdomain: Added domain bus token DOMAIN_BUS_FSL_MC_MSI

2015-11-06 Thread J. German Rivera
Since an FSL-MC bus is a new bus type that is neither PCI nor PLATFORM, we need a new domain bus token to disambiguate the IRQ domain for FSL-MC MSIs. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: none include/linux/irqdomain.h | 1 + 1 file changed, 1

[PATCH v3 11/11] staging: fsl-mc: Added MSI support to the MC bus driver

2015-11-06 Thread J. German Rivera
Initialize/Cleanup ITS-MSI support for the MC bus driver at driver init/exit time. Associate an MSI domain with each DPAA2 child device. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: none drivers/staging/fsl-mc/bus/mc-bus.c | 12 1 file cha

[PATCH v3 02/11] fsl-mc: msi: Added FSL-MC-specific member to the msi_desc's union

2015-11-06 Thread J. German Rivera
FSL-MC is a bus type different from PCI and platform, so it needs its own member in the msi_desc's union. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: - Addressed comment from Jiang Liu * Added a dedicated structure for FSL-MC in struct msi_desc incl

[PATCH v3 06/11] staging: fsl-mc: Changed DPRC built-in portal's mc_io to be atomic

2015-11-06 Thread J. German Rivera
The DPRC built-in portal's mc_io is used to send commands to the MC to program MSIs for MC objects. This is done by the fsl_mc_msi_write_msg() callback, which is invoked by the generic MSI layer with interrupts disabled. As a result, the mc_io used in fsl_mc_msi_write_msg needs to be an atomic mc_i

[PATCH v3 04/11] staging: fsl-mc: Added GICv3-ITS support for FSL-MC MSIs

2015-11-06 Thread J. German Rivera
Added platform-specific MSI support layer for FSL-MC devices. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: none drivers/staging/fsl-mc/bus/Makefile| 1 + .../staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 127 +

[PATCH v3 03/11] staging: fsl-mc: Added generic MSI support for FSL-MC devices

2015-11-06 Thread J. German Rivera
Created an MSI domain for the fsl-mc bus-- including functions to create a domain, find a domain, alloc/free domain irqs, and bus specific overrides for domain and irq_chip ops. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none - Addressed comments from Marc Zyngier: * Add

[PATCH v3 00/11] staging: fsl-mc: MC bus MSI support

2015-11-06 Thread J. German Rivera
This patch series addresses the following item from the TODO list for the MC bus driver to exit staging: * Interrupt support. For meaningful driver support we need interrupts, and thus need message interrupt support by the bus driver. MC Bus MSI Support Architecture ==

[PATCH v3 08/11] staging: fsl-mc: set MSI domain for DPRC objects

2015-11-06 Thread J. German Rivera
THE MSI domain associated with a root DPRC object is obtained form the device tree. Child DPRCs inherit the parent DPRC MSI domain. Signed-off-by: J. German Rivera --- CHANGE HISTORY Changes in v3: none Changes in v2: none drivers/staging/fsl-mc/bus/dprc-driver.c | 39

[PATCH RESEND 11/11] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-06 Thread Jubin John
From: Mark F. Brown Disabling one receive context when RX_DMA is receiving a packet can cause incorrect packet delivery for a subsequent packet on another receive context. This is resolved by doing the following: 1. Programming dummy tail address for every receive context before enabling it 2

[PATCH RESEND 07/11] staging/rdma/hfi1: Fix for opaportconfig ledon by not checking for portNum

2015-11-06 Thread Jubin John
From: Sebastian Sanchez opaportconfig ledon fails with error message due to port number being checked in the attr modifier. This change removes the check for the port number in AttrMod, so the P field is ignored. Reviewed-by: Easwar Hariharan Signed-off-by: Sebastian Sanchez Signed-off-by: Jub

[PATCH RESEND 03/11] staging/rdma/hfi1: Enable WFR PCIe extended tags from the driver

2015-11-06 Thread Jubin John
From: Vennila Megavannan Some BIOS implementations turn off extended tags in DevCtl (a RW field) even though it was originally set and is advertised in DevCap Fix is to set it in the driver Reviewed-by: Dean Luick Reviewed-by: Mike Marciniszyn Reviewed-by: Ashutosh Dixit Signed-off-by: Vennil

[PATCH RESEND 04/11] staging/rdma/hfi1: Always download SBus firmware

2015-11-06 Thread Jubin John
From: Dean Luick B0 dual port parts require the SBus firmware to always be downloaded. Remove reset of the SBus Master spico. It is not necessary since the SBus firmware download already does that. Reviewed-by: Dennis Dalessandro Signed-off-by: Dean Luick Signed-off-by: Jubin John --- driv

[PATCH RESEND 06/11] staging/rdma/hfi1: Select only devices with active links

2015-11-06 Thread Jubin John
From: Dean Luick When looking for or validating a user device, only use devices that are currently active. Reviewed-by: Mitko Haralanov Signed-off-by: Dean Luick Signed-off-by: Jubin John --- drivers/staging/rdma/hfi1/file_ops.c | 21 ++--- 1 files changed, 18 insertions(+)

[PATCH RESEND 10/11] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-06 Thread Jubin John
From: Ira Weiny sdma_select_engine_vl only needs to protect itself from an invalid VL. Something higher up the stack should be warning the user when they try to use an SL which maps to an invalid VL. Reviewed-by: Dean Luick Reviewed-by: Mike Marciniszyn Reviewed-by: Kaike Wan Signed-off-by: I

[PATCH RESEND 01/11] staging/rdma/hfi1: Convert dd_dev_info() to hfi1_cdbg() in process startup

2015-11-06 Thread Jubin John
From: Sebastian Sanchez Replacing dd_dev_info() for hfi1_cdbg() to avoid generating syslog output for every context that is open by PSM. Reviewed-by: Mitko Haralanov Signed-off-by: Sebastian Sanchez Signed-off-by: Jubin John --- drivers/staging/rdma/hfi1/file_ops.c | 10 +- drivers

[PATCH RESEND 08/11] staging/rdma/hfi1: Remove spurious error messages

2015-11-06 Thread Jubin John
From: Ignacio Hernandez Changed the order in which diagnostics messages are printed, taking into account the cases where the errors are handled in rcv_hdrerr() and no further message is needed to report. Reviewed-by: Mark Debbage Reviewed-by: Arthur Kepner Reviewed-by: Mike Marciniszyn Signed

[PATCH RESEND 05/11] staging/rdma/hfi1: Disable thermal polling before sensor initialization

2015-11-06 Thread Jubin John
From: Jareer Abdel-Qader During driver load the thermal sensor needs to be reset prior to initialization of the sensor. This prevents a possible sensor lock up which can cause the wrong temperature value to be reported. This fix leads to remove disabling thermal polling from reset_asic_csrs() fun

[PATCH RESEND 09/11] staging/rdma/hfi1: use one-shot LCB write

2015-11-06 Thread Jubin John
From: Dean Luick Use the one-shot LCB write implemented in the 8051 firmware. This speeds up 8051 LCB writes by 2x. Use old method for older firmwares. Reviewed-by: Dennis Dalessandro Signed-off-by: Dean Luick Signed-off-by: Jubin John --- drivers/staging/rdma/hfi1/chip.c | 42 +++

[PATCH RESEND 02/11] staging/rdma/hfi1: Clear the QSFP reset that is asserted on FLR

2015-11-06 Thread Jubin John
From: Easwar Hariharan The FLR on driver load asserts the QSFP reset pin and the driver does not deassert it after. This patch allows the external QSFP cable to exit reset by writing 1 to all the QSFP pins. Reviewed-by: Dean Luick Signed-off-by: Easwar Hariharan Signed-off-by: Jubin John ---

[PATCH RESEND 00/11] staging/rdma/hfi1: Miscellaneous Fixes

2015-11-06 Thread Jubin John
Resending these previously sent hfi1 driver patches as a series, instead of individual patches, based on feedback about order of patch application from a friendly patch-bot of Greg KH. Dean Luick (3): staging/rdma/hfi1: Always download SBus firmware staging/rdma/hfi1: Select only devices with

Re: [lustre-devel] Build Dependency Issue in Lustre Mainline Linux

2015-11-06 Thread Chris J Arges
On 11/06/2015 06:10 PM, Simmons, James A. wrote: > >> Just did a fresh git pull --rebase and tried to do 'make deb-pkg' with >> an Ubuntu config (which enables lustre). I got the following: >> >> depmod: WARNING: found 2 modules in dependency cycles! >> depmod: WARNING: >> /home/arges/src/kernel/l

RE: [lustre-devel] Build Dependency Issue in Lustre Mainline Linux

2015-11-06 Thread Simmons, James A.
>Just did a fresh git pull --rebase and tried to do 'make deb-pkg' with >an Ubuntu config (which enables lustre). I got the following: > >depmod: WARNING: found 2 modules in dependency cycles! >depmod: WARNING: >/home/arges/src/kernel/linux/./debian/tmp/lib/modules/4.3.0+/kernel/drivers/staging/lu

RE: [PATCH v2 03/11] staging: fsl-mc: Added generic MSI support for FSL-MC devices

2015-11-06 Thread Jose Rivera
> -Original Message- > From: Marc Zyngier [mailto:marc.zyng...@arm.com] > Sent: Friday, November 06, 2015 11:51 AM > To: Rivera Jose-B46482; gre...@linuxfoundation.org; a...@arndb.de; > de...@driverdev.osuosl.org; linux-ker...@vger.kernel.org > Cc: Yoder Stuart-B08248; katz Itai-RM05202; Pa

Re: [PATCH 01/26] staging: wilc1000: separate hif_sdio and hif_spi into different module

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 02:45:22PM -0800, Greg KH wrote: > On Fri, Nov 06, 2015 at 06:39:58PM +0900, Glen Lee wrote: > > hif_sdio and hif_spi objects are compiled all the time even though one of > > SPI > > or SDIO is selected. This patch separates hif_sdio and hif_spi into > > different > > modu

Re: [PATCH 01/26] staging: wilc1000: separate hif_sdio and hif_spi into different module

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 06:39:58PM +0900, Glen Lee wrote: > hif_sdio and hif_spi objects are compiled all the time even though one of SPI > or SDIO is selected. This patch separates hif_sdio and hif_spi into different > modules using ifdef define. > After rework SPI and SDIO modules with only one h

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread kbuild test robot
Hi Punit, [auto build test WARNING on staging/staging-testing] [also build test WARNING on next-20151106] [cannot apply to v4.3] url: https://github.com/0day-ci/linux/commits/Punit-Vara/Staging-wilc1000-Declare-wilc-in-init_wilc_driver-to-fix-build-error/20151107-060255 config: i386

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:47 AM, punit vara wrote: > On Sat, Nov 7, 2015 at 3:39 AM, punit vara wrote: >> On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara wrote: >>> This patch is to the linux_wlan.c file that fixes declaration of *wilc >>> to remove following error while building it. >>> >>> make driv

RE: [lustre-devel] [PATCH 07/10] staging: lustre: Handle nodemask on UMP machines

2015-11-06 Thread Simmons, James A.
>All warnings (new ones prefixed by >>): > > In file included from include/linux/bitops.h:36:0, >from > drivers/staging/lustre/lustre/libcfs/../../include/linux/libcfs/linux/libcfs.h:44, >from > drivers/staging/lustre/lustre/libcfs/../../include/linux/lib

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:39 AM, punit vara wrote: > On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara wrote: >> This patch is to the linux_wlan.c file that fixes declaration of *wilc >> to remove following error while building it. >> >> make drivers/staging/wilc1000/linux_wlan.o >> drivers/staging/wilc1

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread Dan Carpenter
Oh... Sorry. I got your email confused with someone else. > > diff --git a/drivers/staging/wilc1000/linux_wlan.c > > b/drivers/staging/wilc1000/linux_wlan.c > > index 2a5b36f..0805050 100644 > > --- a/drivers/staging/wilc1000/linux_wlan.c > > +++ b/drivers/staging/wilc1000/linux_wlan.c > > @@ -

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread punit vara
On Sat, Nov 7, 2015 at 3:29 AM, Punit Vara wrote: > This patch is to the linux_wlan.c file that fixes declaration of *wilc > to remove following error while building it. > > make drivers/staging/wilc1000/linux_wlan.o > drivers/staging/wilc1000/linux_wlan.c:1824:24: error: ‘wilc’ undeclared > > Sig

RE: [lustre-devel] [PATCH] staging: lustre: lnet: klnds: socklnd: Move extern declarations to header

2015-11-06 Thread Simmons, James A.
>From: lustre-devel [mailto:lustre-devel-boun...@lists.lustre.org] On Behalf Of >Amitoj Kaur Chawla >Sent: Friday, November 06, 2015 9:57 AM >To: oleg.dro...@intel.com; andreas.dil...@intel.com; >gre...@linuxfoundation.org; lustre-de...@lists.lustre.org; >de...@driverdev.osuosl.org; linux-ker...

Re: [PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread Dan Carpenter
On Sat, Nov 07, 2015 at 03:29:23AM +0530, Punit Vara wrote: > This patch is to the linux_wlan.c file that fixes declaration of *wilc > to remove following error while building it. > > make drivers/staging/wilc1000/linux_wlan.o > drivers/staging/wilc1000/linux_wlan.c:1824:24: error: ‘wilc’ undeclar

RE: [PATCH] Staging: lustre: dir: Remove wrapper function

2015-11-06 Thread Simmons, James A.
>From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 11:55 AM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...@vger.kernel.org; >lustre-

RE: [PATCH 1/3] Staging: lustre: module: Replace function calls

2015-11-06 Thread Simmons, James A.
>From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 12:18 PM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...@vger.kernel.org >Subject: [

RE: [PATCH 2/3] Staging: lustre: tracefile: Remove wrapper function

2015-11-06 Thread Simmons, James A.
>-Original Message- >From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 12:19 PM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...

RE: [PATCH 1/3] Staging: lustre: ldlm_pool: Remove unneeded wrapper function

2015-11-06 Thread Simmons, James A.
>-Original Message- >From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 12:43 PM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...

RE: [PATCH 2/3] Staging: lustre: ldlm_pool: Drop wrapper function

2015-11-06 Thread Simmons, James A.
>-Original Message- >From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 12:44 PM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...

RE: [PATCH 3/3] Staging: lustre: ldlm_pool: Drop unneeded wrapper function

2015-11-06 Thread Simmons, James A.
>-Original Message- >From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf >Of Shivani Bhardwaj >Sent: Friday, November 06, 2015 12:45 PM >To: gre...@linuxfoundation.org >Cc: oleg.dro...@intel.com; de...@driverdev.osuosl.org; >andreas.dil...@intel.com; linux-ker...

[PATCH] Staging: wilc1000: Declare *wilc in init_wilc_driver to fix build error

2015-11-06 Thread Punit Vara
This patch is to the linux_wlan.c file that fixes declaration of *wilc to remove following error while building it. make drivers/staging/wilc1000/linux_wlan.o drivers/staging/wilc1000/linux_wlan.c:1824:24: error: ‘wilc’ undeclared Signed-off-by: Punit Vara --- drivers/staging/wilc1000/linux_wla

Re: [PATCH 2/3] staging: rtl8188eu: if/else replaced by min_t

2015-11-06 Thread Dan Carpenter
On Fri, Nov 06, 2015 at 11:42:50PM +0200, Andy Shevchenko wrote: > On Fri, Nov 6, 2015 at 5:18 PM, Ivan Safonov wrote: > > psecuritypriv->authenticator_ie[0] = (unsigned > > char)psecnetwork->IELength; > > > > - if ((psecnetwork->IELength-12) < (256-1)) > > - memcpy(&

Re: [PATCH 2/3] staging: rtl8188eu: if/else replaced by min_t

2015-11-06 Thread Andy Shevchenko
On Fri, Nov 6, 2015 at 5:18 PM, Ivan Safonov wrote: > Duplicated code removed. > > Signed-off-by: Ivan Safonov > --- > drivers/staging/rtl8188eu/core/rtw_cmd.c | 5 + > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c > b/drivers/stagi

Re: linux-next network throughput performance regression

2015-11-06 Thread David Ahern
On 11/6/15 2:18 PM, Simon Xiao wrote: The .config file used to build linux-next kernel is attached to this mail. Thanks. Failed to notice this on the first response; my brain filled in. Why linux-next tree? Can you try net-next which is more relevant for this mailing list, post the top commi

Re: linux-next network throughput performance regression

2015-11-06 Thread David Ahern
On 11/6/15 1:31 PM, Simon Xiao wrote: I compared the network throughput performance on SLES12 bare metal servers, between SLES12 default kernel and latest linux-next (2015-11-05) kernel, based on the test results, I suspect there is a network regression exists on Linux-Next over the 40G Ethern

linux-next network throughput performance regression

2015-11-06 Thread Simon Xiao
I compared the network throughput performance on SLES12 bare metal servers, between SLES12 default kernel and latest linux-next (2015-11-05) kernel, based on the test results, I suspect there is a network regression exists on Linux-Next over the 40G Ethernet network: a) iperf3 reports 50% perfor

Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue

2015-11-06 Thread Dan Carpenter
On Thu, Nov 05, 2015 at 09:27:45PM +0530, Ranjith wrote: > BIT macro is used for defining bit location instead of shifting > operator - coding style issue > > Signed-off-by: Ranjith T You're From line and your signed-off line need to match and they both need to be your legal name that you use to

Re: [PATCH v2 03/11] staging: fsl-mc: Added generic MSI support for FSL-MC devices

2015-11-06 Thread Marc Zyngier
On 30/10/15 19:43, J. German Rivera wrote: > Created an MSI domain for the fsl-mc bus-- including functions > to create a domain, find a domain, alloc/free domain irqs, and > bus specific overrides for domain and irq_chip ops. > > Signed-off-by: J. German Rivera > --- > Changes in v2: none > >

[PATCH 3/3] Staging: lustre: ldlm_pool: Drop unneeded wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function ldlm_pool_set_limit() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ld

[PATCH 2/3] Staging: lustre: ldlm_pool: Drop wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function ldlm_pool_get_limit() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_p

[PATCH 1/3] Staging: lustre: ldlm_pool: Remove unneeded wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function ldlm_pl2ns() and replace its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/staging/lustre/lustre/ld

[PATCH 0/3] Remove wrapper functions from ldlm_pool

2015-11-06 Thread Shivani Bhardwaj
This patchset removes different wrapper functions from the file ldlm_pool and replaces their calls with the function they wrapped. After applying this patchset, code becomes cleaner. Shivani Bhardwaj (3): Staging: lustre: ldlm_pool: Remove unneeded wrapper function Staging: lustre: ldlm_pool:

Awards

2015-11-06 Thread RENEE CAMERON
Prize Notification Contact: unitednationdiplomaticen...@outlook.it ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 14/22] staging: comedi: adv_pci1710: support external analog output reference

2015-11-06 Thread Ian Abbott
On 05/11/15 19:03, Hartley Sweeten wrote: On Thursday, November 05, 2015 5:43 AM, Ian Abbott wrote: On 04/11/15 16:55, H Hartley Sweeten wrote: The analog outputs can use an external reference to create the D/A output range. Add an entry to the comedi_lrange table for it and modify the (*insn_w

Re: [PATCH 15/22] staging: comedi: adv_pci1710: tidy up analog output subdev_flags

2015-11-06 Thread Ian Abbott
On 05/11/15 18:56, Hartley Sweeten wrote: On Thursday, November 05, 2015 6:01 AM, Ian Abbott wrote: On 04/11/15 16:55, H Hartley Sweeten wrote: The SDF_GROUND and SDF_COMMON flags are not needed for the analog output subdevice. Just remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott

Re: [PATCH 19/22] staging: comedi: adv_pci1710: fix counter 0 internal clock source

2015-11-06 Thread Ian Abbott
On 05/11/15 17:42, Hartley Sweeten wrote: On Thursday, November 05, 2015 6:17 AM, Ian Abbott wrote: On 04/11/15 16:55, H Hartley Sweeten wrote: Counters 1 and 2 of the 8254 are cascaeded to create the 32-bit timer used for the analog input pacer trigger. The base clock to these counters is 10 M

[PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Shivani Bhardwaj
Remove the prototype of function cfs_trace_free_string_buffer() as it is no longer needed. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/tracefile.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/libcfs/tracefile.h b/drivers/staging/l

[PATCH 2/3] Staging: lustre: tracefile: Remove wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function cfs_trace_free_string_buffer() as it can be replaced with the standard function kfree(). Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/tracefile.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lu

[PATCH 1/3] Staging: lustre: module: Replace function calls

2015-11-06 Thread Shivani Bhardwaj
Replace the calls of function cfs_trace_free_string_buffer() with kfree() as the former function is not required. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/libcfs/module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustr

[PATCH 0/3] Remove wrapper function and clean up the code

2015-11-06 Thread Shivani Bhardwaj
This patchset removes a wrapper function, its prototype and replace its calls in different files with a standard function. After applying this patch, code becomes cleaner. Shivani Bhardwaj (3): Staging: lustre: module: Replace function calls Staging: lustre: tracefile: Remove wrapper function

[PATCH] Staging: lustre: dir: Remove wrapper function

2015-11-06 Thread Shivani Bhardwaj
Remove the function ll_check_page() and replace all its calls with the function it wrapped. Signed-off-by: Shivani Bhardwaj --- drivers/staging/lustre/lustre/llite/dir.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/dir.c b/driv

Re: [PATCH 3/3] comedi: driver: Fix - BIT macro used coding style issue

2015-11-06 Thread Greg KH
On Fri, Nov 06, 2015 at 07:24:18PM +0530, Ranjith T wrote: > Is this patch is fine?..Because I didn't get any reply for this patch You sent it 1 day ago, while we are in the middle of the merge window and I can't touch anything for 2 weeks at the earliest. Please be patient. greg k-h ___

Awards

2015-11-06 Thread RENEE CAMERON
Prize Notification Contact: unitednationdiplomaticen...@outlook.it ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

RE: [PATCH v2 1/1] staging: rdma: hfi1 : Prefer using the BIT macro

2015-11-06 Thread Marciniszyn, Mike
> Cc: Sunny Kumar > Subject: [PATCH v2 1/1] staging: rdma: hfi1 : Prefer using the BIT macro > > This patch replaces bit shifting on 1 with the BIT(x) macro > > Signed-off-by: Sunny Kumar > --- Thanks for the patch! Acked-by: Mike Marciniszyn ___ de

[PATCH 3/3] staging: rtl8188eu: goto removed

2015-11-06 Thread Ivan Safonov
malloc error handling moved into one place. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 67 +++- 1 file changed, 13 insertions(+), 54 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/r

[PATCH 1/3] staging: rtl8188eu: goto replaced by 'else' branch

2015-11-06 Thread Ivan Safonov
goto is not needed here. Signed-off-by: Ivan Safonov --- drivers/staging/rtl8188eu/core/rtw_cmd.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/rtw_cmd.c index 1d03727..c0

  1   2   >