- 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
> -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;
> Haiya
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
Cc: K. Y. Srinivasan
S
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: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On
> Behalf Of Greg Kroah-Hartman
> Sent: Thursday, November 27, 2014 11:03 AM
> To: Vitaly Kuznetsov
> Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux-
> ker...@vger.kernel.org
> Subject: Re:
On Wed, Nov 26, 2014 at 02:52:22PM +0100, Vitaly Kuznetsov wrote:
> 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
>
From: Micky Ching
Add support for sdio card by SD interface. The main change is data
transfer mode, When read data, host wait data transfer while command
start. When write data, host will start data transfer after command get
response. The transfer mode modify can be applied both for SD/MMC card
From: Micky Ching
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
---
include/linux/mfd/rtsx_pci.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/include/
From: Micky Ching
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 ++
include/l
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
>
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 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
Same question about the name here as before.
Also, be more specific in your subject as to what you are changing.
"fix
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
> ---
> drivers/staging/rtl8188eu/core/rtw_led.c | 5 +++--
> 1 file ch
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 wrote:
> []
> > > There is a script I posted a while back that
> > > groups various checkpatch "types" together and
> > > makes
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 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
http:
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
> ---
> drivers/staging/wlan-ng/prism2fw.c | 10 +++---
> 1 file changed, 7 insertions(+), 3 deletions(-
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
> ---
> drivers/staging/rtl8188eu/hal/usb_halinit.c | 6 ++
> 1 file changed,
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
> ---
> drivers/staging/rtl8188eu/core/rtw_wlan_util.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/
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
> Acked-by: Willy Tarreau
> ---
>
Output can be held high or low for a specified period of time.
Support for waveform capture could be added in the future.
Signed-off-by: George McCollister
---
Documentation/ABI/testing/sysfs-bus-iio | 7 +++
drivers/iio/industrialio-core.c | 3 +++
driver
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 -
> atomic_read(&core_state.available_cores);
>
> WARNING: line over 80
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
What is the "C V" at the end of the name here? Is that your "full
name"? I need a real name, not initials
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
I need a "full name" here, sorry.
Please fix and resend.
greg k-h
___
On Sun, Nov 23, 2014 at 07:44:08PM +0100, SF Markus Elfring wrote:
> From: Markus Elfring
> 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 vfree() function
> cal
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 yoursel
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
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 nu
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 ch
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: wa
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
>
> drivers/staging/lustre/include/linux/
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
> ---
> drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> di
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 li
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 agradece
Am Mittwoch, den 26.11.2014, 09:43 +1000 schrieb Dave Airlie:
> On 25 November 2014 at 01:33, Philipp Zabel 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 linear irq domai
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
>
> ---
>
> Changes in v13: None
> Changes in v
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
> ---
>
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
drive
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
[ cu
On Wed, 2014-11-26 at 15:42 +, Luis de Bethencourt wrote:
> On 26 November 2014 at 01:49, Joe Perches 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.
> >
> > https://lkml.or
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
Rockchip RK3288 hdmi is compatible with dw_hdmi
this patch is depend on patch by Mark Yao Add drm
driver for Rockchip Socs
see https://lkml.org/lkml/2014/11/19/1153
Signed-off-by: Andy Yan
---
Changes in v13: None
Changes in v12:
- add comment for the depend on patch
Changes in v11: None
Cha
Signed-off-by: Andy Yan
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8:
- Add documentation for rockchip dw hdmi
Changes in v7: None
Changes in v6: None
Changes in v5: None
Changes in v4: None
Changes in v3: None
.../d
RK3288 HDMI will not work without the spare bit of
HDMI_PHY_CONF0 enable
Signed-off-by: Andy Yan
---
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 in v5: None
Changes in
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 corr
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
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v
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
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes
Signed-off-by: Andy Yan
---
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
Changes in v5: None
Changes i
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
---
Changes in v13: None
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8: None
hdmi phy configuration is platform specific, which can be adusted
according to the board to get the best SI
Signed-off-by: Andy Yan
---
Changes in v13:
- split phy configuration from patch#4
Changes in v12: None
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes in v8: None
IMX6 and Rockchip RK3288 and JZ4780 (Ingenic Xburst/MIPS)
use the interface compatible Designware HDMI IP, but they
also have some lightly differences, such as phy pll configuration,
register width, 4K support, clk useage, and the crtc mux configuration
is also platform specific.
To reuse the imx
drm driver may probe before the i2c bus, so the driver should
defer probing until it is available
Signed-off-by: Andy Yan
Reviewed-by: Daniel Kurtz
---
Changes in v13: None
Changes in v12:
- refactor of_node_put(ddc_node)
Changes in v11: None
Changes in v10: None
Changes in v9: None
Changes i
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 {
[...]
Sign
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 access
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 somethin
Fixed the checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Athira Lekshmi C V
---
drivers/staging/octeon/ethernet-rx.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/octeon/ethernet-rx.c
b/drivers/staging/octeon/ethernet-rx.c
index 44
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 bl
Fixed the checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Athira Lekshmi C V
---
drivers/staging/octeon/ethernet-rx.c |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/staging/octeon/ethernet-rx.c
b/drivers/staging/octeon/ethernet-rx.c
index
Fixed the checkpatch warning:
WARNING: please, no space before tabs
Signed-off-by: Athira Lekshmi C V
---
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
b/drivers/staging/comedi/dri
On Wed, Nov 26, 2014 at 12:14 PM, Athira Lekshmi C V
wrote:
> Fixed the checkpatch warning:
> WARNING: simple_strtoul is obsolete, use kstrtoul instead
>
> Signed-off-by: Athira Lekshmi C V
> ---
> drivers/staging/speakup/varhandlers.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Fixed the checkpatch warning:
WARNING: simple_strtoul is obsolete, use kstrtoul instead
Signed-off-by: Athira Lekshmi C V
---
drivers/staging/speakup/varhandlers.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/varhandlers.c
b/drivers/staging/spea
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
62 matches
Mail list logo