On 3/7/2017 10:52 AM, Reshetova, Elena wrote:
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-after-free
situations.
Signed-off-by: Elena
On 3/7/2017 10:52 AM, Reshetova, Elena wrote:
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-after-free
situations.
Signed-off-by: Elena
Hello.
On 03/06/2017 05:20 PM, Elena Reshetova wrote:
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-after-free
situations.
Signed-off-by:
Hello.
On 03/06/2017 05:20 PM, Elena Reshetova wrote:
refcount_t type and corresponding API should be
used instead of atomic_t when the variable is used as
a reference counter. This allows to avoid accidental
refcounter overflows that might lead to use-after-free
situations.
Signed-off-by:
On 3/6/2017 10:21 AM, jsun4 wrote:
If asid_cache(cpu) overflows, there may be two tasks with the same
asid. It is a risk that the two different tasks may have the same
address space.
A process will update its asid to newer version only when switch_mm()
is called and matches the following
On 3/6/2017 10:21 AM, jsun4 wrote:
If asid_cache(cpu) overflows, there may be two tasks with the same
asid. It is a risk that the two different tasks may have the same
address space.
A process will update its asid to newer version only when switch_mm()
is called and matches the following
Hello!
On 3/5/2017 6:24 AM, Jiwei Sun wrote:
If asid_cache(cpu) overflows, there may be two tasks with the same
asid. It is a risk that the two different tasks may have the same
address space.
A process will update its asid to newer version only when switch_mm()
is called and matches the
Hello!
On 3/5/2017 6:24 AM, Jiwei Sun wrote:
If asid_cache(cpu) overflows, there may be two tasks with the same
asid. It is a risk that the two different tasks may have the same
address space.
A process will update its asid to newer version only when switch_mm()
is called and matches the
On 3/1/2017 6:55 PM, Romain Perier wrote:
The PCI pool API is deprecated. This commits replaces the PCI pool old
Commit.
API by the appropriated function with the DMA pool API.
Appropriate perhaps?
Signed-off-by: Romain Perier
Acked-by: Peter Senna
On 3/1/2017 6:55 PM, Romain Perier wrote:
The PCI pool API is deprecated. This commits replaces the PCI pool old
Commit.
API by the appropriated function with the DMA pool API.
Appropriate perhaps?
Signed-off-by: Romain Perier
Acked-by: Peter Senna Tschudin
Tested-by: Peter
Hello!
On 3/1/2017 6:55 PM, Romain Perier wrote:
The PCI pool API is deprecated. This commits replaces the PCI pool old
Commit.
API by the appropriated function with the DMA pool API. It also updates
Appropriate perhaps?
the name of some variables and the content of comments,
Hello!
On 3/1/2017 6:55 PM, Romain Perier wrote:
The PCI pool API is deprecated. This commits replaces the PCI pool old
Commit.
API by the appropriated function with the DMA pool API. It also updates
Appropriate perhaps?
the name of some variables and the content of comments,
Hello!
On 3/3/2017 2:36 AM, John Stultz wrote:
I had seen some odd behavior with HiKey's usb-gadget interface
that I finally seemed to have chased down. Basically every other
time I pluged in the OTG port, the gadget interface would
Plugged. Perhaps could be fixed while applying...
Hello!
On 3/3/2017 2:36 AM, John Stultz wrote:
I had seen some odd behavior with HiKey's usb-gadget interface
that I finally seemed to have chased down. Basically every other
time I pluged in the OTG port, the gadget interface would
Plugged. Perhaps could be fixed while applying...
Hello!
Your subject is too long, mught want to move i(part of) it to the patch
description...
And you need to provide your Signed-off-by:, the patch can't be applied
without it.
MBR, Sergei
Hello!
Your subject is too long, mught want to move i(part of) it to the patch
description...
And you need to provide your Signed-off-by:, the patch can't be applied
without it.
MBR, Sergei
On 2/23/2017 4:13 AM, Dmitry V. Levin wrote:
Include in uapi/linux/seg6.h to fix the following
linux/seg6.h userspace compilation error:
/usr/include/linux/seg6.h:31:18: error: array type has incomplete element type
'struct in6_addr'
struct in6_addr segments[0];
Include in
On 2/23/2017 4:13 AM, Dmitry V. Levin wrote:
Include in uapi/linux/seg6.h to fix the following
linux/seg6.h userspace compilation error:
/usr/include/linux/seg6.h:31:18: error: array type has incomplete element type
'struct in6_addr'
struct in6_addr segments[0];
Include in
Hello!
On 2/23/2017 4:13 AM, Dmitry V. Levin wrote:
Consistently use types from linux/types.h to fix the following
linux/rds.h userspace compilation errors:
/usr/include/linux/rds.h:198:2: error: unknown type name 'u8'
u8 rx_traces;
/usr/include/linux/rds.h:199:2: error: unknown type name
Hello!
On 2/23/2017 4:13 AM, Dmitry V. Levin wrote:
Consistently use types from linux/types.h to fix the following
linux/rds.h userspace compilation errors:
/usr/include/linux/rds.h:198:2: error: unknown type name 'u8'
u8 rx_traces;
/usr/include/linux/rds.h:199:2: error: unknown type name
On 2/22/2017 11:55 AM, Chunfeng Yun wrote:
there is a reference clock for each port, HighSpeed port is 48M,
and SuperSpeed port is 26M which usually comes from 26M oscillator
directly, but some SoCs is not. it is flexible to move it into port
... but on some SoCs does not?
node.
On 2/22/2017 11:55 AM, Chunfeng Yun wrote:
there is a reference clock for each port, HighSpeed port is 48M,
and SuperSpeed port is 26M which usually comes from 26M oscillator
directly, but some SoCs is not. it is flexible to move it into port
... but on some SoCs does not?
node.
Hello!
On 2/22/2017 11:55 AM, Chunfeng Yun wrote:
the reference clock of HighSpeed port is 48M which comes from PLL;
the reference clock of SuperSpeed port is 26M which usually comes
from 26M oscillator directly, but some SoCs are not, add it for
... but on some SoCs does not?
Hello!
On 2/22/2017 11:55 AM, Chunfeng Yun wrote:
the reference clock of HighSpeed port is 48M which comes from PLL;
the reference clock of SuperSpeed port is 26M which usually comes
from 26M oscillator directly, but some SoCs are not, add it for
... but on some SoCs does not?
Hello!
On 2/14/2017 5:27 AM, Lu Baolu wrote:
Add a simple udelay calibration in x86 architecture-specific
boot-time initializations. This will get a workable estimate
for loops_per_jiffy. Hence, udelay() could be used after this
initialization.
Cc: Ingo Molnar
Cc:
Hello!
On 2/14/2017 5:27 AM, Lu Baolu wrote:
Add a simple udelay calibration in x86 architecture-specific
boot-time initializations. This will get a workable estimate
for loops_per_jiffy. Hence, udelay() could be used after this
initialization.
Cc: Ingo Molnar
Cc: x...@kernel.org
Hello!
On 2/10/2017 8:38 PM, Shannon Nelson wrote:
When the sunvnet_common code was split out for use by both sunvnet
and the newer ldmvsw, it was made into a static kernel library, which
limits the usefulness of sunvnet and ldmvsw as loadables, since most
of the real work is being done in the
Hello!
On 2/10/2017 8:38 PM, Shannon Nelson wrote:
When the sunvnet_common code was split out for use by both sunvnet
and the newer ldmvsw, it was made into a static kernel library, which
limits the usefulness of sunvnet and ldmvsw as loadables, since most
of the real work is being done in the
Hello!
On 2/12/2017 5:57 AM, Tobin C. Harding wrote:
This patch fixes two trivial whitespace messages (ERROR/WARNING).
Fixes trailing whitespace ERROR and fixes space before tabs WARNING.
Signed-off-by: Tobin C. Harding
---
include/linux/idr.h | 4 ++--
1 file changed, 2
Hello!
On 2/12/2017 5:57 AM, Tobin C. Harding wrote:
This patch fixes two trivial whitespace messages (ERROR/WARNING).
Fixes trailing whitespace ERROR and fixes space before tabs WARNING.
Signed-off-by: Tobin C. Harding
---
include/linux/idr.h | 4 ++--
1 file changed, 2 insertions(+), 2
Hello!
On 2/8/2017 11:31 AM, Corentin Labbe wrote:
Checkpatch complains about some code style problem on stmmac_mdio.c.
This patch fix them.
Fixes.
Signed-off-by: Corentin Labbe
Acked-by: Giuseppe Cavallaro
---
Hello!
On 2/8/2017 11:31 AM, Corentin Labbe wrote:
Checkpatch complains about some code style problem on stmmac_mdio.c.
This patch fix them.
Fixes.
Signed-off-by: Corentin Labbe
Acked-by: Giuseppe Cavallaro
---
drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c | 7 ---
1 file
Hello!
On 02/06/2017 05:03 PM, Richard Leitner wrote:
For USB string descriptors we need to convert ASCII strings to UTF16-LE.
Therefore make a simple helper function (based on ascii2desc from
drivers/usb/core/hcd.c) for that purpose.
Signed-off-by: Richard Leitner
Hello!
On 02/06/2017 05:03 PM, Richard Leitner wrote:
For USB string descriptors we need to convert ASCII strings to UTF16-LE.
Therefore make a simple helper function (based on ascii2desc from
drivers/usb/core/hcd.c) for that purpose.
Signed-off-by: Richard Leitner
[...]
diff --git
Hello!
On 2/5/2017 11:21 PM, Alban wrote:
From: Alban Bedel
Normally compressed images have to be loaded at a different address to
allow the decompressor to run. This add an option to let vmlinuz copy
itself to the correct address from the normal vmlinux address.
Hello!
On 2/5/2017 11:21 PM, Alban wrote:
From: Alban Bedel
Normally compressed images have to be loaded at a different address to
allow the decompressor to run. This add an option to let vmlinuz copy
itself to the correct address from the normal vmlinux address.
Signed-off-by: Alban Bedel
On 2/2/2017 1:39 AM, Jon Mason wrote:
From: Hari Vyas
ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to
bgmac_set_mac_address() but code assumed u8 *. This caused two bytes
chopping and the wrong mac address was configured.
Signed-off-by: Hari Vyas
On 2/2/2017 1:39 AM, Jon Mason wrote:
From: Hari Vyas
ndo_set_mac_address() passes struct sockaddr * as 2nd parameter to
bgmac_set_mac_address() but code assumed u8 *. This caused two bytes
chopping and the wrong mac address was configured.
Signed-off-by: Hari Vyas
Signed-off-by: Jon Mason
Hello!
On 2/2/2017 1:39 AM, Jon Mason wrote:
From: Zac Schroff
Fix a bug in the 'bgmac' driver init sequence that blind writes for init
sequence where it should preserve most bits other than the ones it is
deliberately manipulating.
Signed-off-by: Zac Schroff
Hello!
On 2/2/2017 1:39 AM, Jon Mason wrote:
From: Zac Schroff
Fix a bug in the 'bgmac' driver init sequence that blind writes for init
sequence where it should preserve most bits other than the ones it is
deliberately manipulating.
Signed-off-by: Zac Schroff
Signed-off-by: Jon Mason
On 2/1/2017 9:53 AM, Hannes Reinecke wrote:
[1] which were a pain in the ass to untangle and debug during development,
it's really time for it to die..
Outside of the patch series in question, how to we expedite the
euthanasia of IDE? What explicit features/support are we missing through
On 2/1/2017 9:53 AM, Hannes Reinecke wrote:
[1] which were a pain in the ass to untangle and debug during development,
it's really time for it to die..
Outside of the patch series in question, how to we expedite the
euthanasia of IDE? What explicit features/support are we missing through
On 01/31/2017 09:51 PM, James Bottomley wrote:
[1] which were a pain in the ass to untangle and debug during
development, it's really time for it to die..
Outside of the patch series in question, how to we expedite the
euthanasia of IDE? What explicit features/support are we missing
through
On 01/31/2017 09:51 PM, James Bottomley wrote:
[1] which were a pain in the ass to untangle and debug during
development, it's really time for it to die..
Outside of the patch series in question, how to we expedite the
euthanasia of IDE? What explicit features/support are we missing
through
.pinchart+rene...@ideasonboard.com>
Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>
---
Changes in version 3:
- ported the patch to the modern kernel;
- reformatted rcar_du_live_fb_create_handle()/rcar_du_live_fb_create()'s
parameter lists;
- added my signoff.
drive
From: Laurent Pinchart
Register live sources for VSPD0 and VSPD1 and configure the plane source
at plane setup time to source frames from memory or from the VSP1.
[Sergei: ported to the modern kernel.]
Signed-off-by: Laurent Pinchart
Signed-off-by: Sergei Shtylyov
---
Changes in version 3
Sergei: ported to the modern kernel.]
Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>
---
Changes in version 3:
- moved drm_live_source_{init|cleanup}() from drm_crtc.c to drm_plane.c and
the
-by: Laurent Pinchart
Signed-off-by: Sergei Shtylyov
---
Changes in version 3:
- moved drm_live_source_{init|cleanup}() from drm_crtc.c to drm_plane.c and
their prototypes from to (along with
*struct* drm_live_source[_funcs]);
- moved drm_mode_getsource[_res]() prototypes from
plane to connect it with the live source.
[Sergei: ported to the modern kernel.]
Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>
---
Changes in version 3:
- ported the patch to the modern kernel;
: ported to the modern kernel.]
Signed-off-by: Laurent Pinchart
Signed-off-by: Sergei Shtylyov
---
Changes in version 3:
- ported the patch to the modern kernel;
- added my signoff.
drivers/gpu/drm/drm_framebuffer.c | 134 ++
include/uapi/drm/drm_mode.h
Hello.
Here's the set of 3 patches against the 'drm-next' branch of David Airlie's
'linux.git' repo. This is a port of Laurent's DRM/DU live source patches to the
recent kernel, see [1] for the version 2 of the patchset (including a Laurent's
big blurb :-)). For the patch #3 to work one
Hello.
Here's the set of 3 patches against the 'drm-next' branch of David Airlie's
'linux.git' repo. This is a port of Laurent's DRM/DU live source patches to the
recent kernel, see [1] for the version 2 of the patchset (including a Laurent's
big blurb :-)). For the patch #3 to work one
On 01/19/2017 04:29 PM, Bartosz Golaszewski wrote:
Add the SATA node to the da850 device tree.
Signed-off-by: Bartosz Golaszewski
---
arch/arm/boot/dts/da850.dtsi | 6 ++
1 file changed, 6 insertions(+)
diff --git a/arch/arm/boot/dts/da850.dtsi
On 01/19/2017 04:29 PM, Bartosz Golaszewski wrote:
Add the SATA node to the da850 device tree.
Signed-off-by: Bartosz Golaszewski
---
arch/arm/boot/dts/da850.dtsi | 6 ++
1 file changed, 6 insertions(+)
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index
Hello!
On 1/17/2017 6:18 PM, Arnd Bergmann wrote:
The filter-out macro needs two arguments, passing only one is
clearly the result of a typo that leads to 'make clean' failing
on MIPS:
arch/mips/boot/compressed/Makefile:21: *** insufficient number of arguments (1)
to function 'filter-out'.
Hello!
On 1/17/2017 6:18 PM, Arnd Bergmann wrote:
The filter-out macro needs two arguments, passing only one is
clearly the result of a typo that leads to 'make clean' failing
on MIPS:
arch/mips/boot/compressed/Makefile:21: *** insufficient number of arguments (1)
to function 'filter-out'.
Hello!
On 01/17/2017 03:26 PM, Bartosz Golaszewski wrote:
In preparation for using two clocks in the driver (the sysclk2-based
clock and the external REFCLK), check if we got a functional clock
after calling ahci_platform_get_resources(). If not, retry calling
get_clk() with con_id specified.
Hello!
On 01/17/2017 03:26 PM, Bartosz Golaszewski wrote:
In preparation for using two clocks in the driver (the sysclk2-based
clock and the external REFCLK), check if we got a functional clock
after calling ahci_platform_get_resources(). If not, retry calling
get_clk() with con_id specified.
struct */
-static struct mdiobb_ops bb_ops = {
+static const struct mdiobb_ops bb_ops = {
.owner = THIS_MODULE,
.set_mdc = sh_mdc_ctrl,
.set_mdio_dir = sh_mmd_ctrl,
[...]
For the above 2 drivers:
Acked-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>
MBR, Sergei
mdiobb_ops bb_ops = {
+static const struct mdiobb_ops bb_ops = {
.owner = THIS_MODULE,
.set_mdc = sh_mdc_ctrl,
.set_mdio_dir = sh_mmd_ctrl,
[...]
For the above 2 drivers:
Acked-by: Sergei Shtylyov
MBR, Sergei
Hello!
On 1/15/2017 12:47 AM, Florian Fainelli wrote:
Move the assignment of ports in _dsa_register_switch() closer to where
it is checked, no functional change. Re-order declarations to be
"Be" not needed.
preserve the inverted christmas tree style.
Signed-off-by: Florian Fainelli
Hello!
On 1/15/2017 12:47 AM, Florian Fainelli wrote:
Move the assignment of ports in _dsa_register_switch() closer to where
it is checked, no functional change. Re-order declarations to be
"Be" not needed.
preserve the inverted christmas tree style.
Signed-off-by: Florian Fainelli
Hello!
On 01/10/2017 05:21 AM, Shuah Khan wrote:
Fix dwc3_exynos_probe() to call clk_prepare_enable() only when suspend
clock is specified. Call clk_disable_unprepare() from remove and probe
error path only when susp_clk has been set from remove and probe error
paths.
Signed-off-by: Shuah
Hello!
On 01/10/2017 05:21 AM, Shuah Khan wrote:
Fix dwc3_exynos_probe() to call clk_prepare_enable() only when suspend
clock is specified. Call clk_disable_unprepare() from remove and probe
error path only when susp_clk has been set from remove and probe error
paths.
Signed-off-by: Shuah
Hello!
On 1/6/2017 5:45 PM, Nikita Yushchenko wrote:
It is possible that device is capable of 64-bit DMA addresses, and
device driver tries to set wide DMA mask, but bridge or bus used to
connect device to the system can't handle wide addresses.
With swiotlb, memory above 4G still can be used
Hello!
On 1/6/2017 5:45 PM, Nikita Yushchenko wrote:
It is possible that device is capable of 64-bit DMA addresses, and
device driver tries to set wide DMA mask, but bridge or bus used to
connect device to the system can't handle wide addresses.
With swiotlb, memory above 4G still can be used
On 12/30/2016 12:46 PM, Sergei Shtylyov wrote:
It is possible that PCI device supports 64-bit DMA addressing, and thus
it's driver sets device's dma_mask to DMA_BIT_MASK(64), however PCI host
Its.
bridge has limitations on inbound transactions addressing. Example of
such setup is NVME
On 12/30/2016 12:46 PM, Sergei Shtylyov wrote:
It is possible that PCI device supports 64-bit DMA addressing, and thus
it's driver sets device's dma_mask to DMA_BIT_MASK(64), however PCI host
Its.
bridge has limitations on inbound transactions addressing. Example of
such setup is NVME
Hello!
On 12/29/2016 11:45 PM, Nikita Yushchenko wrote:
It is possible that PCI device supports 64-bit DMA addressing, and thus
it's driver sets device's dma_mask to DMA_BIT_MASK(64), however PCI host
Its.
bridge has limitations on inbound transactions addressing. Example of
such setup
Hello!
On 12/29/2016 11:45 PM, Nikita Yushchenko wrote:
It is possible that PCI device supports 64-bit DMA addressing, and thus
it's driver sets device's dma_mask to DMA_BIT_MASK(64), however PCI host
Its.
bridge has limitations on inbound transactions addressing. Example of
such setup
Hello!
On 12/27/2016 10:52 AM, Wei Zhang wrote:
When we send a packet for our own local address on a non-loopback interface
(e.g. eth0), due to the change had been introduced from commit 0b922b7a829c
("net: original ingress device index in PKTINFO"), the original ingress
device index would be
Hello!
On 12/27/2016 10:52 AM, Wei Zhang wrote:
When we send a packet for our own local address on a non-loopback interface
(e.g. eth0), due to the change had been introduced from commit 0b922b7a829c
("net: original ingress device index in PKTINFO"), the original ingress
device index would be
Hello!
On 12/25/2016 3:30 AM, Thomas Preisner wrote:
In some cases the return value of a failing function is not being used
and the function typhoon_init_one() returns another negative error
code instead.
Signed-off-by: Thomas Preisner
Signed-off-by: Milan
Hello!
On 12/25/2016 3:30 AM, Thomas Preisner wrote:
In some cases the return value of a failing function is not being used
and the function typhoon_init_one() returns another negative error
code instead.
Signed-off-by: Thomas Preisner
Signed-off-by: Milan Stephan
---
Hello!
On 12/24/2016 03:02 PM, Thomas Preisner wrote:
In a few cases the err-variable is not set to a negative error code if a
function call fails and thus 0 is returned instead.
It may be better to set err to the appropriate negative error code
before returning.
Bugzilla:
Hello!
On 12/24/2016 03:02 PM, Thomas Preisner wrote:
In a few cases the err-variable is not set to a negative error code if a
function call fails and thus 0 is returned instead.
It may be better to set err to the appropriate negative error code
before returning.
Bugzilla:
Hello!
On 12/23/2016 9:52 AM, Lu Baolu wrote:
If xhci host fails to response to a command, the command
s/response/respond/.
watchdog timer will be fired. The callback function will
abort and clear current command and restart the command
execution. If driver fails to restart command
Hello!
On 12/23/2016 9:52 AM, Lu Baolu wrote:
If xhci host fails to response to a command, the command
s/response/respond/.
watchdog timer will be fired. The callback function will
abort and clear current command and restart the command
execution. If driver fails to restart command
Hello.
On 12/16/2016 09:21 PM, Ozgur Karatas wrote:
This patch fixed to keyboard typo, brackets not closed.
I think, it should be close to parenthes.
Signed-off-by: Ozgur Karatas
---
tools/net/bpf_dbg.c | 2 +-
1 files changed, 1 insertion(+), 1 deletions(-)
Hello.
On 12/16/2016 09:21 PM, Ozgur Karatas wrote:
This patch fixed to keyboard typo, brackets not closed.
I think, it should be close to parenthes.
Signed-off-by: Ozgur Karatas
---
tools/net/bpf_dbg.c | 2 +-
1 files changed, 1 insertion(+), 1 deletions(-)
diff --git
Hello!
On 12/12/2016 11:54 PM, Andy Lutomirski wrote:
smbencrypt() points a scatterlist to the stack, which is breaks if
s/is//.
CONFIG_VMAP_STACK=y.
Fix it by switching to crypto_cipher_encrypt_one(). The new code
should be considerably faster as an added benefit.
This code is
Hello!
On 12/12/2016 11:54 PM, Andy Lutomirski wrote:
smbencrypt() points a scatterlist to the stack, which is breaks if
s/is//.
CONFIG_VMAP_STACK=y.
Fix it by switching to crypto_cipher_encrypt_one(). The new code
should be considerably faster as an added benefit.
This code is
Hello!
On 12/9/2016 6:08 AM, Zheng Li wrote:
From: zheng li
There is an inconsitent conditional judgement in __ip_append_data and
Inconsistent.
ip_finish_output functions, the variable length in __ip_append_data just
include the length of applicatoin's payload
Hello!
On 12/9/2016 6:08 AM, Zheng Li wrote:
From: zheng li
There is an inconsitent conditional judgement in __ip_append_data and
Inconsistent.
ip_finish_output functions, the variable length in __ip_append_data just
include the length of applicatoin's payload and udp header, don't
Hello!
On 12/5/2016 9:57 AM, Santosh Shilimkar wrote:
Fixes below warnings:
warning: symbol 'rds_send_probe' was not declared. Should it be static?
warning: symbol 'rds_send_ping' was not declared. Should it be static?
warning: symbol 'rds_tcp_accept_one_path' was not declared. Should it be
Hello!
On 12/5/2016 9:57 AM, Santosh Shilimkar wrote:
Fixes below warnings:
warning: symbol 'rds_send_probe' was not declared. Should it be static?
warning: symbol 'rds_send_ping' was not declared. Should it be static?
warning: symbol 'rds_tcp_accept_one_path' was not declared. Should it be
On 12/4/2016 1:33 PM, Pan Bian wrote:
You now need to indicate the patch version in hte subject, like this:
[PATCH 1/1 v2] isdn:...
From: Pan Bian
In function hfc4s8s_probe(), the value of return variable err should be
negative on failures. However, when the call to
On 12/4/2016 1:33 PM, Pan Bian wrote:
You now need to indicate the patch version in hte subject, like this:
[PATCH 1/1 v2] isdn:...
From: Pan Bian
In function hfc4s8s_probe(), the value of return variable err should be
negative on failures. However, when the call to request_region()
On 12/4/2016 11:39 AM, Pan Bian wrote:
From: Pan Bian
Marco BNX2X_ALLOC_AND_SET(arr, lbl, func) calls kmalloc() to allocate
Macro.
memory, and jumps to label "lbl" if the allocation fails. Label "lbl"
first cleans memory and then returns variable rc. Before calling
On 12/4/2016 11:39 AM, Pan Bian wrote:
From: Pan Bian
Marco BNX2X_ALLOC_AND_SET(arr, lbl, func) calls kmalloc() to allocate
Macro.
memory, and jumps to label "lbl" if the allocation fails. Label "lbl"
first cleans memory and then returns variable rc. Before calling the
macro, the value
Hello.
On 12/4/2016 8:15 AM, Pan Bian wrote:
From: Pan Bian
In function hfc4s8s_probe(), the value of return variable err should be
negative on failures. However, when the call to request_region() returns
NULL, the value of err is 0. This patch fixes the bug, assiging
Hello.
On 12/4/2016 8:15 AM, Pan Bian wrote:
From: Pan Bian
In function hfc4s8s_probe(), the value of return variable err should be
negative on failures. However, when the call to request_region() returns
NULL, the value of err is 0. This patch fixes the bug, assiging
"-ENOMEM" to err on the
On 12/03/2016 06:38 PM, Pan Bian wrote:
In function caif_sktinit_module(), the check of the return value of
sock_register() seems ineffective. This patch fixes it.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751
Signed-off-by: Pan Bian
---
On 12/03/2016 06:38 PM, Pan Bian wrote:
In function caif_sktinit_module(), the check of the return value of
sock_register() seems ineffective. This patch fixes it.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751
Signed-off-by: Pan Bian
---
net/caif/caif_socket.c | 2 +-
1 file
Hello.
On 12/3/2016 2:18 PM, Pan Bian wrote:
In function caif_sktinit_module(), the check of the return value of
sock_register() seems ineffective. This patch fixes it.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751
Signed-off-by: Pan Bian
---
Hello.
On 12/3/2016 2:18 PM, Pan Bian wrote:
In function caif_sktinit_module(), the check of the return value of
sock_register() seems ineffective. This patch fixes it.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751
Signed-off-by: Pan Bian
---
net/caif/caif_socket.c | 2 +-
1
Hello.
On 11/25/2016 06:24 AM, Changming Huang wrote:
The EHCI specification states the following in the SUSP bit description:
In the Suspend state, the port is senstive to resume detection.
Sensitive.
Note that the bit status does not change untile the port is suspended and
Until.
Hello.
On 11/25/2016 06:24 AM, Changming Huang wrote:
The EHCI specification states the following in the SUSP bit description:
In the Suspend state, the port is senstive to resume detection.
Sensitive.
Note that the bit status does not change untile the port is suspended and
Until.
Hello.
On 11/24/2016 10:10 PM, Joe Perches wrote:
These just add unnecessary vertical whitespace.
Signed-off-by: Joe Perches
---
drivers/net/irda/w83977af_ir.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/net/irda/w83977af_ir.c
Hello.
On 11/24/2016 10:10 PM, Joe Perches wrote:
These just add unnecessary vertical whitespace.
Signed-off-by: Joe Perches
---
drivers/net/irda/w83977af_ir.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/net/irda/w83977af_ir.c
801 - 900 of 4469 matches
Mail list logo