On Tue, 2014-11-25 at 21:14 +, Luis de Bethencourt wrote:
On Tue, Nov 25, 2014 at 01:00:07PM -0800, Joe Perches wrote:
In the future, you might consider being more
comprehensive with your patches.
Wasn't sure about the scope of the style fixing
patches. I've been reading Kernel
Fixed the checkpatch warning:
WARNING: simple_strtoul is obsolete, use kstrtoul instead
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/speakup/varhandlers.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/varhandlers.c
On Wed, Nov 26, 2014 at 12:14 PM, Athira Lekshmi C V
andnlnb...@gmail.com wrote:
Fixed the checkpatch warning:
WARNING: simple_strtoul is obsolete, use kstrtoul instead
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/speakup/varhandlers.c |2 +-
1 file
Fixed the checkpatch warning:
WARNING: please, no space before tabs
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/comedi/drivers/ni_at_a2150.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/ni_at_a2150.c
Fixed the checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/octeon/ethernet-rx.c |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/staging/octeon/ethernet-rx.c
On Wed, Nov 26, 2014 at 05:17:48PM +0530, Athira Lekshmi C V wrote:
@@ -294,8 +297,10 @@ static int cvm_oct_napi_poll(struct napi_struct *napi,
int budget)
* CPU.
*/
union cvmx_pow_wq_int_cntx counts;
+
Don't put a blank
Fixed the checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/octeon/ethernet-rx.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/octeon/ethernet-rx.c
On Tue, Nov 25, 2014 at 09:44:21PM +0100, Zahari Doychev wrote:
This patch fixes pointer declarations from void * to void __user * in order
to remove some sparse warnings.
_Are_ those userland addresses, though? Quick grep shows that in the
only caller of lnet_copy_iov2flat() we have something
We found Freescale imx6 and Rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they also have some
lightly differences, such as phy pll configuration, register width(imx hdmi
register is one byte, but rk3288 is 4 bytes width and can only be
CHECK: Alignment should match open parenthesis
+ if ((hdmi-vic == 10) || (hdmi-vic == 11) ||
+ (hdmi-vic == 12) || (hdmi-vic == 13) ||
CHECK: braces {} should be used on all arms of this statement
+ if (hdmi-hdmi_data.video_mode.mdvi)
[...]
+ else {
[...]
the original imx hdmi driver is under drm/imx/,
which depends on imx-drm, so move the imx hdmi
driver out to drm/bridge and rename it to dw_hdmi
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9:
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- correct some spelling mistake
- modify ddc-i2c-bus and interrupt description
Changes in v7: None
Changes in v6: None
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width accesses (writeb,
readb) generate an imprecise external abort.
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
some platform may not support all the display mode,
add mode_valid interface check it
also add drm_connector_register which add a debugfs
interface for dump display modes and edid information
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Changes in v12: None
Changes
HDMI_IH_I2CMPHY_STAT0 is a clear on write register, which indicates i2cm
operation status(i2c transfer done or error), every hdmi phy register
configuration must check this register to make sure the configuration
has complete. But the indication bit should be cleared after check, otherwise
the
RK3288 HDMI will not work without the spare bit of
HDMI_PHY_CONF0 enable
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8: None
Changes in v7: None
Changes in v6: None
Changes
When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary
cpus are sent offline (with echo 0 /sys/devices/system/cpu/cpu$cpu/online)
the system freeze is observed. This happens due to the fact that on newer
hypervisors (Win8, WS2012R2, ...) vmbus channel handlers are distributed
On Wed, 2014-11-26 at 15:42 +, Luis de Bethencourt wrote:
On 26 November 2014 at 01:49, Joe Perches j...@perches.com wrote:
[]
There is a script I posted a while back that
groups various checkpatch types together and
makes it a bit easier to do cleanup style
patches.
Hi Andy,
I have yet to look at this in more detail, but from a quick test
starting with patch 3, the HDMI display stays black on Nitrogen6X,
and starting with patch 8 I get the following error.
imx-drm display-subsystem: [CONNECTOR:21:HDMI-A-1] drm_connector_register
failed: -12
[
Add __acquires() and __releases() function annotations, to fix sparse warnings
related to lock context imbalance.
This fixes the following warnings:
drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c:153:5: warning:
context imbalance in 'cfs_trace_lock_tcd' - wrong count at exit
Am Mittwoch, den 26.11.2014, 21:33 +0800 schrieb Andy Yan:
some platform may not support all the display mode,
add mode_valid interface check it
also add drm_connector_register which add a debugfs
interface for dump display modes and edid information
Signed-off-by: Andy Yan
Am Mittwoch, den 26.11.2014, 21:32 +0800 schrieb Andy Yan:
On rockchip rk3288, only word(32-bit) accesses are
permitted for hdmi registers. Byte width accesses (writeb,
readb) generate an imprecise external abort.
Signed-off-by: Andy Yan andy@rock-chips.com
---
Changes in v13: None
Am Mittwoch, den 26.11.2014, 09:43 +1000 schrieb Dave Airlie:
On 25 November 2014 at 01:33, Philipp Zabel p.za...@pengutronix.de wrote:
The imx-drm driver was put into staging mostly for the following reasons,
all of which have been addressed or superseded:
- convert the irq driver to use
Uma tentativa foi feita para sua conta a partir de um computador desconhecido.
para
a segurança da sua conta, estamos prontos para abrir uma consulta ou
atualização
sua conta. Por favor, preencha os dados abaixo:
nome:
Endereço de email:
password:
Re-digite a senha:
Tel:
Te
On Tue, Nov 11, 2014 at 03:28:57PM +0530, Sanjeev Sharma wrote:
This is a patch to the rtw_cmd.c file that fixes following
Error.
ERROR: that open brace { should be on the previous line
This patch does much more than that :(
___
devel mailing list
On Sun, Nov 23, 2014 at 02:37:57PM +0100, Andreas Ruprecht wrote:
A space which was forgotten in a CDEBUG output is added by this patch.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 2 +-
1 file changed, 1 insertion(+), 1
On Mon, Nov 24, 2014 at 07:55:40PM +0100, Zahari Doychev wrote:
The two patches fix several sparse warning in the lustre module.
Zahari Doychev (2):
[drivers] staging/lustre: fix sparse warnings
[drivers] staging/lustre: fix sparse warnings
On Wed, Nov 26, 2014 at 05:15:48PM +0100, Loic Pefferkorn wrote:
Add __acquires() and __releases() function annotations, to fix sparse
warnings related to lock context imbalance.
This fixes the following warnings:
drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c:153:5:
On Fri, Nov 07, 2014 at 10:17:40AM -0800, Greg KH wrote:
On Fri, Nov 07, 2014 at 12:58:24PM -0500, Ken Depro wrote:
While fixing the CamelCase checks for the functions in the chanstub
source and header files, I discovered they are no longer being used.
This patch removes the two chanstub
On Wed, Nov 12, 2014 at 11:28:15AM -0500, Jeffrey Brown wrote:
Sorry for all of the messy patches from before but here
are the improved patches for parser.h and parser.c.
The camel cases are grouped on functions and structs
that occurs in both files.
You have to do a 'vN' where 'N' is a
On Thu, Nov 13, 2014 at 09:56:39AM -0500, Jeffrey Brown wrote:
Fixed small bug in parser.c by removing cleanups: in parser_init
_guts struct. Replaced it with proper error handling code
and removed the instances of rc = NULL in the code. rc = NULL is
redudant
Signed-off-by: Jeffrey Brown
I _strongly_ suggest unisys get their act together here. The number of
patches that were sent recently that make no sense at all is horrid.
I've now purged all of them from my to-apply queue, please resend
_everything_ that you have, in the correct order, _AFTER_ you all review
them among
On Sun, Nov 23, 2014 at 07:44:08PM +0100, SF Markus Elfring wrote:
From: Markus Elfring elfr...@users.sourceforge.net
Date: Sun, 23 Nov 2014 19:12:29 +0100
The jump label err1 was used by the ion_buffer_create() function in case of
a memory allocation failure just to pass a null pointer to a
On Thu, Nov 20, 2014 at 11:31:14PM +0530, Lekshmi wrote:
The following checkpatch warning was fixed:
WARNING: Missing a blank line after declarations
Signed-off-by: Lekshmi andnlnb...@gmail.com
I need a full name here, sorry.
Please fix and resend.
greg k-h
On Wed, Nov 26, 2014 at 05:58:37PM +0530, Athira Lekshmi C V wrote:
Fixed the checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
What is the C V at the end of the name here? Is that your full
name? I need a real name,
On Wed, Nov 19, 2014 at 09:38:46PM +0100, Mariusz Gorski wrote:
Avoid magic number and use a comparison with a defined value instead
that checks whether module param has been set by the user to some
value at loading time.
Signed-off-by: Mariusz Gorski marius.gor...@gmail.com
Acked-by: Willy
On Thu, Nov 20, 2014 at 04:03:23PM +0100, Abel Moyo wrote:
Changed return type of function get_my_bssid from __inline to inline
Signed-off-by: Abel Moyo abelmoyo...@gmail.com
---
drivers/staging/rtl8188eu/core/rtw_wlan_util.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On Sat, Nov 22, 2014 at 11:36:40AM +0530, Anjana Sasindran wrote:
This patch fixes the five checkpatch.pl warnings:
WARNING:Missing a blank line after declaration
Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 6
On Tue, Nov 25, 2014 at 05:16:40PM +0530, Athira Lekshmi C V wrote:
The following checkpatch warning was fixed
WARNING: line over 80 characters
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
---
drivers/staging/wlan-ng/prism2fw.c | 10 +++---
1 file changed, 7 insertions(+),
On 11/26/2014 10:45 PM, George McCollister wrote:
Output can be held high or low for a specified period of time.
Support for waveform capture could be added in the future.
That's a PWM device?
___
devel mailing list
de...@linuxdriverproject.org
On Wed, Nov 26, 2014 at 01:45:23PM -0800, Greg KH wrote:
On Tue, Nov 25, 2014 at 01:26:14PM +, Luis de Bethencourt wrote:
This patch fixes the checkpatch.pl warnings:
WARNING: line over 80 characters
+ int cores_in_use = core_state.baseline_cores -
On Wed, Nov 26, 2014 at 08:05:55AM -0800, Joe Perches wrote:
On Wed, 2014-11-26 at 15:42 +, Luis de Bethencourt wrote:
On 26 November 2014 at 01:49, Joe Perches j...@perches.com wrote:
[]
There is a script I posted a while back that
groups various checkpatch types together and
On Fri, Nov 07, 2014 at 12:24:28AM +0100, Krzysztof Konopko wrote:
According to Linux coding convention leading spaces are not allowed. This
patch removes leading spaces in rtw_led.c
Signed-off-by: Krzysztof Konopko k...@konagma.com
---
drivers/staging/rtl8188eu/core/rtw_led.c | 5 +++--
On Wed, Nov 26, 2014 at 04:56:55PM +0530, Athira Lekshmi C V wrote:
Fixed the checkpatch warning:
WARNING: please, no space before tabs
Signed-off-by: Athira Lekshmi C V andnlnb...@gmail.com
Same question about the name here as before.
Also, be more specific in your subject as to what you
On Tue, Nov 11, 2014 at 09:03:26PM -0800, Dexuan Cui wrote:
In the case the user-space daemon crashes, hangs or is killed, we
need to down the semaphore, otherwise, after the daemon starts next
time, the obsolete data in fcopy_transaction.message or
fcopy_transaction.fcopy_msg will be used
On Thu, Nov 27, 2014 at 12:35:23AM +, Luis de Bethencourt wrote:
On Wed, Nov 26, 2014 at 01:45:23PM -0800, Greg KH wrote:
On Tue, Nov 25, 2014 at 01:26:14PM +, Luis de Bethencourt wrote:
This patch fixes the checkpatch.pl warnings:
WARNING: line over 80 characters
+
From: Micky Ching micky_ch...@realsil.com.cn
Add helper function to write u32 to registers, if we want to put u32
value to 4 continuous register, this can help us reduce tedious work.
Signed-off-by: Micky Ching micky_ch...@realsil.com.cn
---
include/linux/mfd/rtsx_pci.h | 15 +++
1
From: Micky Ching micky_ch...@realsil.com.cn
This patch is used to change transfer mode for sdio card support
by SD interface.
Micky Ching (2):
mfd: rtsx: add func to split u32 into register
mmc: rtsx: add support for sdio card
drivers/mmc/host/rtsx_pci_sdmmc.c | 366
On Wed, Nov 26, 2014 at 12:54:43PM -0800, Greg KH wrote:
Ugh, how horrid, please just delete these functions and push down the
spin_lock/unlock calls down into the places these are called.
Same for these.
Same thing here.
Hello Greg,
Thanks for your comments, I will write a v2.
--
-Original Message-
From: Greg KH [mailto:gre...@linuxfoundation.org]
Sent: Thursday, November 27, 2014 7:54 AM
To: Dexuan Cui
Cc: linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org;
o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; KY Srinivasan;
Haiyang
In the case the user-space daemon crashes, hangs or is killed, we
need to down the semaphore, otherwise, after the daemon starts next
time, the obsolete data in fcopy_transaction.message or
fcopy_transaction.fcopy_msg will be used immediately.
Reviewed-by: Vitaly Kuznetsov vkuzn...@redhat.com
Cc:
- Original Message -
In the case the user-space daemon crashes, hangs or is killed, we
need to down the semaphore, otherwise, after the daemon starts next
time, the obsolete data in fcopy_transaction.message or
fcopy_transaction.fcopy_msg will be used immediately.
Reviewed-by:
52 matches
Mail list logo