On 2015/11/3 5:33, ja...@microsoft.com wrote:
> From: Jake Oshins
>
> This patch adds an fwnode_handle to struct pci_sysdata, which is
> used by the next patch in the series when trying to locate an
> IRQ domain associated with a root PCI bus.
>
> Signed-off-by: Jake Oshins
> ---
> arch/x86/in
Attention Please!,
This is to official inform you that we have been having meetings for
the past Three (3) Weeks which ended two days ago with Mr. Jim Yong
Kim the World Bank president and other seven continent presidents on
the Congress we treated on solution to Scam victim problems.
Not
On 2015/11/3 5:33, ja...@microsoft.com wrote:
> From: Jake Oshins
>
> This patch introduces a new driver which exposes a root PCI bus whenever a PCI
> Express device is passed through to a guest VM under Hyper-V. The device can
> be single- or multi-function. The interrupts for the devices are ma
There is such a field both in struct mlme_ext_info and in struct
registry_priv. Rename both.
Also fix checkpatch issue in the lines touched:
WARNING: line over 80 characters
Signed-off-by: Luca Ceresoli
Cc: Larry Finger
Cc: Greg Kroah-Hartman
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.
Using the ternary operator allows to more concisely write the same
code, and to stay within 80 characters without even increasing the
number of lines.
Signed-off-by: Luca Ceresoli
Cc: Larry Finger
Cc: Greg Kroah-Hartman
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 8
1 file cha
rtw_AcceptAddbaReq is a static variable, it is set once and never
modified. It is referenced only once, to assign its value to a member
of struct registry_priv with practically the same name.
Get rid of the variable, and move the meaningful part of the comment
near the declaration of the relevant
The first argument to memset() is (void *), the explicit typecasts are
not needed. They just make code less readable.
Signed-off-by: Luca Ceresoli
Cc: Larry Finger
Cc: Greg Kroah-Hartman
---
drivers/staging/rtl8188eu/core/rtw_ap.c | 4 ++--
drivers/staging/rtl8188eu/core/rtw_efuse.c
This line is connented since the initial import in commit 7b464c9fa5cc
("staging: r8188eu: Add files for new driver - part 4") and there's no
comment stating how it could ever be useful.
Signed-off-by: Luca Ceresoli
Cc: Larry Finger
Cc: Greg Kroah-Hartman
---
drivers/staging/rtl8188eu/core/rtw
Fix checkpatch issue:
CHECK: spaces preferred around that '*' (ctx:VxV)
Signed-off-by: Luca Ceresoli
Cc: Larry Finger
Cc: Greg Kroah-Hartman
---
drivers/staging/rtl8188eu/core/rtw_efuse.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/core/r
> -Original Message-
> From: ja...@microsoft.com [mailto:ja...@microsoft.com]
> Sent: Monday, November 2, 2015 1:33 PM
> To: gre...@linuxfoundation.org; KY Srinivasan ; linux-
> ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de;
> a...@canonical.com; vkuzn...@redhat.com;
From: Mitko Haralanov
It is possible for an SDMA transmission error to happen
during the processing of an user SDMA transfer. In that
case it is better to detect it early and abort any further
attempts to send more packets.
Reviewed-by: Ira Weiny
Signed-off-by: Mitko Haralanov
---
drivers/sta
From: Mitko Haralanov
Clean-up unnecessary goto statements based on feedback from the
mailing list on previous patch submissions.
Reviewed-by: Ira Weiny
Signed-off-by: Mitko Haralanov
---
drivers/staging/rdma/hfi1/user_sdma.c | 37 +--
1 file changed, 14 insert
From: Mitko Haralanov
The driver pins pages on behalf of user processes in two
separate instances - when the process has submitted a
SDMA transfer and when the process programs an expected
receive buffer.
When pinning pages, the driver is required to observe the
locked page limit set by the syst
From: Mitko Haralanov
There is no need to cleck if the packet queue is allocated
when cleaning up a user context. The hfi1_user_sdma_free_queues()
function already does all the required checks.
Reviewed-by: Ira Weiny
Signed-off-by: Mitko Haralanov
---
drivers/staging/rdma/hfi1/file_ops.c | 3
From: Ira Weiny
Various improvements to the SDMA engine code.
Mitko Haralanov (5):
staging/rdma/hfi1: Convert to use get_user_pages_fast
staging/rdma/hfi1: Unconditionally clean-up SDMA queues
staging/rdma/hfi1: Clean-up unnecessary goto statements
staging/rdma/hfi1: Detect SDMA transmis
From: Mitko Haralanov
Convert hfi1_get_user_pages() to use get_user_pages_fast(),
which is much fatster. The mm semaphore is still taken to
update the pinned page count but is for a much shorter
amount of time.
Signed-off-by: Mitko Haralanov
Reviewed-by: Ira Weiny
---
drivers/staging/rdma/hfi
From: Sebastian Lawniczak
Fix warnings reported by checkpatch.pl in block comments.
Signed-off-by: Sebastian Lawniczak
---
drivers/staging/sm750fb/ddk750_chip.c | 70 +++
1 file changed, 39 insertions(+), 31 deletions(-)
diff --git a/drivers/staging/sm750fb/ddk
OK. Since the maintainers are CC'd, I guess I should wait for a
clarification about this?
El 02/12/15 a las 10:29, Dan Carpenter escribió:
Put v2 in the subject. Also the subsystem prefix is:
[PATCH v3] staging: wilc1000: ...
On Mon, Nov 30, 2015 at 09:09:04PM -0300, Mario J. Rugiero wrote:
Telekom Malaysia Berhad
G.03B, Ground Floor, Kompleks Antarabangsa,
Jln Sultan Ismail, Off Jalan Ampang
50088 50250
Kuala Lumpur.
NOTIS RASMI HADIAH TELEKOM MALAYSIA
Pihak Telekom Malaysia @Program Kemenangan yang telah diadakan pada 1 Dec 2015
di mana alamat email anda yang disertakan be
On Wed, Dec 02, 2015 at 04:20:59PM +0300, Alexander Zarochentsev wrote:
> > BAD: if (rc != 0)
> > GOOD: if (rc)
>
> The latest suggestion is not correct,
> from http://wiki.lustre.org/Lustre_Coding_Guidelines :
> Conditional boolean (if (expr)), scalar (if (val != 0)) and pointer
> (if (ptr !
Hello,
On Wed, Dec 2, 2015 at 3:59 PM, Dan Carpenter wrote:
> Actually we're going to have to redo so much code that it's not worth it
> for me to review the rest of these patches. Please just look over
> everything again:
>
> BAD: return -1;
> GOOD: return -EINVAL;
>
> BAD: failed0:
> GO
Put v2 in the subject. Also the subsystem prefix is:
[PATCH v3] staging: wilc1000: ...
On Mon, Nov 30, 2015 at 09:09:04PM -0300, Mario J. Rugiero wrote:
> This patch replaces an "if (ptr > 0)" comparison that seems to be a
> confusing way to check for null by a simpler "if (ptr)" check.
>
> Sig
On Mon, Nov 30, 2015 at 02:46:14PM -0500, Benjamin Romer wrote:
> @@ -1651,7 +1651,7 @@ service_resp_queue(struct uiscmdrsp *cmdrsp, struct
> visornic_devdata *devdata,
>* netif_wake_queue()
>*/
> if (vni
Probably it's better to create a different struct:
struct udata_s {
unsigned int size;
void __user *buf;
};
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/list
Actually we're going to have to redo so much code that it's not worth it
for me to review the rest of these patches. Please just look over
everything again:
BAD: return -1;
GOOD: return -EINVAL;
BAD: failed0:
GOOD: free_something:
BAD: if (rc != 0)
GOOD: if (rc)
Do one thing per p
Hi,
Our company is a manufactures/assembles/distributor of LED Lighting, as
will as LED Signage products..
I myself have over 30 years in lighting and signage field and over 15 years
in LED products.
We are running a 40% off Holiday sales on all of our LED products, with
free energy audits as wel
Fold this into the original patch. (Which you're going to have to redo
anyway to fix the other bug and because of the GW-BASIC label names).
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.
On Fri, Nov 20, 2015 at 06:35:55PM -0500, James Simmons wrote:
> From: Liang Zhen
>
> - libcfs_ioctl_popdata should copy out inline buffers.
> - code cleanup for libcfs ioctl handler
> - error number fix for obd_ioctl_getdata
> - add new function libcfs_ioctl_unpack for upcoming patches
>
On Fri, Nov 20, 2015 at 06:35:51PM -0500, James Simmons wrote:
> From: Amir Shehata
>
> This is the sixth patch of a set of patches that enables DLC.
>
> This patch enables the user space to call into the kernel space
> DLC code. Added handlers in the LNetCtl function to call
> the new function
On Fri, Nov 20, 2015 at 06:35:49PM -0500, James Simmons wrote:
> +int lnet_get_peers(int count, __u64 *nid, char *aliveness,
> +int *ncpt, int *refcount,
> +int *ni_peer_tx_credits, int *peer_tx_credits,
> +int *peer_rtr_credits, int *peer_min_rtr_cre
This feels like we are fixing a bug introduce in PATCH 11 when we
removed a NULL check. Don't introduce bugs and then fix them in the
same patchset; the fix has to be folded into the original patch.
regards,
dan carpenter
___
devel mailing list
de...@l
Telekom Malaysia Berhad
G.03B, Ground Floor, Kompleks Antarabangsa,
Jln Sultan Ismail, Off Jalan Ampang
50088 50250
Kuala Lumpur.
NOTIS RASMI HADIAH TELEKOM MALAYSIA
Pihak Telekom Malaysia @Program Kemenangan yang telah diadakan pada 1 Dec 2015
di mana alamat email anda yang disertakan be
On Fri, Nov 20, 2015 at 06:35:49PM -0500, James Simmons wrote:
> From: Amir Shehata
>
> This is the fifth patch of a set of patches that enables DLC.
>
> This patch adds the new structures which will be used
> in the IOCTL communication. It also added a set of
> show operations to show buffers,
On Fri, Nov 20, 2015 at 06:35:48PM -0500, James Simmons wrote:
> +int libcfs_ioctl_getdata_len(const struct libcfs_ioctl_hdr __user *arg,
> + __u32 *len)
> +{
> + struct libcfs_ioctl_hdr hdr;
>
> - orig_len = hdr->ioc_len;
> - if (copy_from_user(buf, arg, hdr-
On Fri, Nov 20, 2015 at 06:35:47PM -0500, James Simmons wrote:
> +
> + return 0;
>
> failed4:
> - lnet_ping_target_fini();
> - failed3:
> the_lnet.ln_refcount = 0;
> + lnet_ping_md_unlink(pinfo, &md_handle);
> + lnet_ping_info_free(pinfo);
> + failed3:
> lnet_accepto
Hi Brian,
On Tue, 1 Dec 2015 14:17:47 -0800
Brian Norris wrote:
> On Tue, Dec 01, 2015 at 12:03:14PM +0100, Boris Brezillon wrote:
> > mtd_to_nand() now uses the container_of() approach to transform an
> > mtd_info pointer into a nand_chip one. Drop useless mtd->priv
> > assignments from NAND co
mtd_to_nand() now uses the container_of() approach to transform an
mtd_info pointer into a nand_chip one. Drop useless mtd->priv
assignments from NAND controller drivers.
Signed-off-by: Boris Brezillon
---
Patch generated with the following coccinelle script:
---8<
virtual patch
@@
struct m
struct nand_chip now embeds an mtd device. Patch all drivers to make use
of this mtd instance instead of using the instance embedded in their
private struct or dynamically allocated.
Signed-off-by: Boris Brezillon
Cc: Julia Lawall
---
Most of those changes were generated with the coccinelle scri
38 matches
Mail list logo
