[patch] Staging: rtl8188eu: overflow in update_sta_support_rate()

2014-04-07 Thread Dan Carpenter
The -SupportedRates[] array has NDIS_802_11_LENGTH_RATES_EX (16) elements. Since ie_len comes from then network and can go up to 255 then it means we should add a range check to prevent memory corruption. Fixes: d6846af679e0 ('staging: r8188eu: Add files for new driver - part 7') Signed-off-by:

[PATCH] imx-drm: imx-drm-core: Fix imx_drm_encoder_get_mux_id

2014-04-07 Thread Philipp Zabel
The decoder mux id is equal to the port id of the encoder's input port that is connected to the given crtc, not to the endpoint id (which is arbitrary and usually zero). Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/staging/imx-drm/imx-drm-core.c | 2 +- 1 file changed, 1

Re: GET YOUR WON PRIZE!!!

2014-04-07 Thread Malik Nadeem Ahmad Soharwardi
i got mail now? ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Paul Bolle
Larry, 0) I stumbled on two minor Kconfig related issues in the r8723au driver in staging-next. 1) drivers/staging/rtl8723au/include/drv_types.h Why is CONFIG_BR_EXT used? I couldn't find a Kconfig symbol BR_EXT nor a preprocessor define for CONFIG_BR_EXT. 2)

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-04-07 Thread Russell King - ARM Linux
On Mon, Apr 07, 2014 at 12:23:37PM +0800, Shawn Guo wrote: And I see another HDMI regression with my testing on mainline kernel. I can have my HDMI work at 1920x1080 with v3.14 kernel, but it can only probes 1024x768 with the mainline today. Works for me here. [20.606] (II) LoadModule:

Re: staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Jes Sorensen
Paul Bolle pebo...@tiscali.nl writes: Larry, 0) I stumbled on two minor Kconfig related issues in the r8723au driver in staging-next. 1) drivers/staging/rtl8723au/include/drv_types.h Why is CONFIG_BR_EXT used? I couldn't find a Kconfig symbol BR_EXT nor a preprocessor define for

Re: staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Paul Bolle
Jes, On Mon, 2014-04-07 at 11:25 +0200, Jes Sorensen wrote: If it's not used, it should be ripped out. That's a rule I can grasp! We've been going through this code for a while ripping things out that are unnecessary and rewriting bad code, but there is still plenty that needs fixing up.

Re: staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Jes Sorensen
Paul Bolle pebo...@tiscali.nl writes: Jes, On Mon, 2014-04-07 at 11:25 +0200, Jes Sorensen wrote: If it's not used, it should be ripped out. That's a rule I can grasp! We've been going through this code for a while ripping things out that are unnecessary and rewriting bad code, but there

Re: staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Paul Bolle
On Mon, 2014-04-07 at 11:38 +0200, Jes Sorensen wrote: I'll try to put them on my list - I have ~40-50 (I think) patches pending on top of what Larry already submitted. If I forget it, please feel free to hollor at me. If this will ever hit mainline I probably will. Paul Bolle

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-04-07 Thread Philipp Zabel
Hi Shawn, Am Montag, den 07.04.2014, 12:23 +0800 schrieb Shawn Guo: On Tue, Mar 11, 2014 at 11:46:11AM +0800, Shawn Guo wrote: I just came across a couple problems when testing the series on my imx6dl-sabresd board in dual display case - HDMI + LVDS. I tested it using Russell's branch

Re: [PATCH v5 00/11] imx-drm dt bindings

2014-04-07 Thread Philipp Zabel
Am Montag, den 07.04.2014, 12:05 +0200 schrieb Philipp Zabel: Hi Shawn, Am Montag, den 07.04.2014, 12:23 +0800 schrieb Shawn Guo: On Tue, Mar 11, 2014 at 11:46:11AM +0800, Shawn Guo wrote: I just came across a couple problems when testing the series on my imx6dl-sabresd board in dual

[PATCH v12][ 01/12] [media] v4l2: add new V4L2_PIX_FMT_RGB666 pixel format.

2014-04-07 Thread Denis Carikli
That new macro is needed by the imx_drm staging driver for supporting the QVGA display of the eukrea-cpuimx51 board. Signed-off-by: Denis Carikli de...@eukrea.com Acked-by: Mauro Carvalho Chehab m.che...@samsung.com Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com Acked-by: Philipp

[PATCH v12][ 04/12] imx-drm: Match ipu_di_signal_cfg's clk_pol with its description.

2014-04-07 Thread Denis Carikli
According to the datasheet, setting the di0_polarity_disp_clk field in the GENERAL di register sets the output clock polarity to active high. Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v9-v10: - New patch that is now needed by the staging: imx-drm: Use de-active and

[PATCH v12][ 02/12] imx-drm: Add RGB666 support for parallel display.

2014-04-07 Thread Denis Carikli
Signed-off-by: Denis Carikli de...@eukrea.com Acked-by: Philipp Zabel p.za...@pengutronix.de --- ChangeLog v8-v9: - Rebased. - Added Philipp Zabel's ack. - Shortened the patch title. ChangeLog v8-v9: - Removed the Cc. They are now set in git-send-email directly. - Rebased. ChangeLog v7-v8: -

Re: [PATCH] staging: cxt1e1: hwprobe: fix incorrect type in assignment

2014-04-07 Thread Matei Oprea
Any comments on this ? ᐧ On Sun, Mar 23, 2014 at 2:13 PM, Matei Oprea e...@opreamatei.ro wrote: This patch fixes incorrect type in argument 1 warning from sparse. Signed-off-by: Matei Oprea e...@opreamatei.ro Cc: ROSEdu Kernel Community fire...@lists.rosedu.org ---

[PATCH v12][ 06/12] ARM: dts: imx5*, imx6*: correct display-timings nodes.

2014-04-07 Thread Denis Carikli
The imx-drm driver can't use the de-active and pixelclk-active display-timings properties yet. Instead the data-enable and the pixel data clock polarity are hardcoded in the imx-drm driver. So theses properties are now set to keep the same behaviour when imx-drm will start using them.

[PATCH v12][ 05/12] imx-drm: use defines for clock polarity settings

2014-04-07 Thread Denis Carikli
Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog 11-v12: - Improved the define names to match the hardware: ENABLE_POL is not a clock signal but instead an enable signal. ChangeLog v9-v10: - New patch which was splitted out from: staging: imx-drm: Use de-active and pixelclk-active

[PATCH v12][ 09/12] drm/panel: Add Eukrea mbimxsd51 displays.

2014-04-07 Thread Denis Carikli
Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v11-v12: - Rebased: It now uses the new DRM_MODE_FLAG_POL_DE flags defines names ChangeLog v10-v11: - New patch. --- .../bindings/panel/eukrea,mbimxsd51-cmo-qvga.txt |7 ++ .../bindings/panel/eukrea,mbimxsd51-dvi-svga.txt |7

[PATCH v12][ 07/12] drm: drm_display_mode: add signal polarity flags

2014-04-07 Thread Denis Carikli
We need a way to pass signal polarity informations between DRM panels, and the display drivers. To do that, a pol_flags field was added to drm_display_mode. Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v11-v12: - Rebased: This patch now applies against drm_modes.h - Rebased: It

[PATCH v12][ 10/12] ARM: dts: mbimx51sd: Add display support.

2014-04-07 Thread Denis Carikli
The CMO-QVGA, DVI-SVGA and DVI-VGA are added. Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v10-v11: - Now uses the drm-panel instead of the display-timings. This is to get regulator support, which is lacking in the imx-drm driver when using the display-timings. ChangeLog

[PATCH v12][ 08/12] imx-drm: Use drm_display_mode timings flags.

2014-04-07 Thread Denis Carikli
The previous hardware behaviour was kept if the flags are not set. Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v11-v12: - Rebased: It now uses the following new flags defines names: CLK_POL, ENABLE_POL - The inversions in ipuv3-crtc.c are now fixed. - ipuv3-crtc.c was still

[PATCH v12][ 12/12] ARM: imx_v6_v7_defconfig: Add more drm drivers.

2014-04-07 Thread Denis Carikli
The DRM_PANEL_SIMPLE is needed by the eukrea mbimxsd51's displays. Signed-off-by: Denis Carikli de...@eukrea.com --- ChangeLog v10-v11: - New patch, splitting it would be overkill. --- arch/arm/configs/imx_v6_v7_defconfig |2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH] staging: cxt1e1: hwprobe: fix incorrect type in assignment

2014-04-07 Thread Greg KH
On Mon, Apr 07, 2014 at 03:38:27PM +0300, Matei Oprea wrote: Any comments on this ? You got an email from me about this, on the 23rd, saying that I can't do anything with it until 3.15-rc1 is out. thanks, greg k-h ___ devel mailing list

Re: [PATCH] staging: cxt1e1: hwprobe: fix incorrect type in assignment

2014-04-07 Thread Matei Oprea
Sorry, I thought that it's out. ᐧ On Mon, Apr 7, 2014 at 5:14 PM, Greg KH gre...@linuxfoundation.org wrote: On Mon, Apr 07, 2014 at 03:38:27PM +0300, Matei Oprea wrote: Any comments on this ? You got an email from me about this, on the 23rd, saying that I can't do anything with it until

Re: [PATCH] staging: rtl8187se: Fix warning symbol should be static

2014-04-07 Thread Jonas Hahnfeld
Am Montag, den 07.04.2014, 01:34 + schrieb Stefan Lippers-Hollmann: Hi On Sunday 06 April 2014, Josh Triplett wrote: On Sun, Apr 06, 2014 at 05:46:04PM +0200, Jonas Hahnfeld wrote: This patch solves some sparse warnings about symbol [...] was not declared. Should it be static? by

Re: staging-next: r8723au: Kconfig related nitpicks

2014-04-07 Thread Larry Finger
On 04/07/2014 04:08 AM, Paul Bolle wrote: Larry, 0) I stumbled on two minor Kconfig related issues in the r8723au driver in staging-next. 1) drivers/staging/rtl8723au/include/drv_types.h Why is CONFIG_BR_EXT used? I couldn't find a Kconfig symbol BR_EXT nor a preprocessor define for

Re: [PATCH] staging/line6: Fix kzalloc coding style issue

2014-04-07 Thread L . Alberto Giménez
On Mon, Apr Apr 2014 at 08:05:04AM +0200, Takashi Iwai said: At Mon, 7 Apr 2014 00:12:30 +0200, L. Alberto Giménez wrote: Pass the actual variable to sizeof instead of a type definition. Signed-off-by: L. Alberto Giménez agime...@sysvalve.es Acked-by: Takashi Iwai ti...@suse.de

Re: [PATCH net-next,v4] hyperv: Add support for virtual Receive Side Scaling (vRSS)

2014-04-07 Thread David Miller
The net-next tree is not open yet, I will announce when it is and you can submit net-next targetted patches. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH 0/1] staging-next: rtl8723au: Fix too many paths

2014-04-07 Thread Jes . Sorensen
From: Jes Sorensen jes.soren...@redhat.com Greg, This one is a major bug in the version of the driver we submitted for staging. It correct a problem where we ended up trying to configure more RF paths than are on the chip. The result was for transmission to hang randomly. I really would like

Re: [visorchipset] invalid opcode: 0000 [#1] PREEMPT SMP

2014-04-07 Thread Greg Kroah-Hartman
On Mon, Apr 07, 2014 at 09:24:37AM -0500, Ken Cox wrote: On 04/07/2014 09:09 AM, Greg Kroah-Hartman wrote: On Mon, Apr 07, 2014 at 07:17:25PM +0800, Fengguang Wu wrote: Hi Ken, I got the below dmesg and the first bad commit is

Re: [visorchipset] invalid opcode: 0000 [#1] PREEMPT SMP

2014-04-07 Thread Greg Kroah-Hartman
On Mon, Apr 07, 2014 at 12:23:47PM -0700, Greg Kroah-Hartman wrote: On Mon, Apr 07, 2014 at 09:24:37AM -0500, Ken Cox wrote: On 04/07/2014 09:09 AM, Greg Kroah-Hartman wrote: On Mon, Apr 07, 2014 at 07:17:25PM +0800, Fengguang Wu wrote: Hi Ken, I got the below dmesg and the first

[PATCH v2 1/5] Staging: rts5139: Camel case labels replaced

2014-04-07 Thread Fabio Falzoi
Replace camel case labels with linux coding style compliant names. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5139/rts51x.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/rts5139/rts51x.c

[PATCH v2 3/5] Staging: rts5139: Splitted one-line multiple assignment

2014-04-07 Thread Fabio Falzoi
Splitted a multiple assignment on two separate lines. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5139/rts51x.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rts5139/rts51x.c b/drivers/staging/rts5139/rts51x.c index

[PATCH v2 2/5] Staging: rts5139: Fixed multi-line code alignment

2014-04-07 Thread Fabio Falzoi
Multi-line code aligned with open parenthesis. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5139/rts51x.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rts5139/rts51x.c b/drivers/staging/rts5139/rts51x.c

[PATCH v2 0/5] Staging: rts5139: Fix coding style

2014-04-07 Thread Fabio Falzoi
Fix some coding style issues in rts51x.c (from rts5139 module). This is for task 10 of the Eudyptula challenge. Fabio Falzoi (5): Camel case labels replaced Fixed multi-line code alignment Splitted one-line multiple assignment pr_warn(...) in place of printk(KERN_WARNING ...) More

[PATCH v2 4/5] Staging: rts5139: pr_warn(...) in place of printk(KERN_WARNING ...)

2014-04-07 Thread Fabio Falzoi
All printk(KERN_WARNING ...) replaced with pr_warn(...). Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5139/rts51x.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/rts5139/rts51x.c

[PATCH v2 5/5] Staging: rts5139: More appropriate sizeof operand

2014-04-07 Thread Fabio Falzoi
Use the pointer rts51x to get the size of the struct. Signed-off-by: Fabio Falzoi fabio.falzo...@gmail.com --- drivers/staging/rts5139/rts51x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rts5139/rts51x.c b/drivers/staging/rts5139/rts51x.c index

Re: [PATCH v2 4/5] Staging: rts5139: pr_warn(...) in place of printk(KERN_WARNING ...)

2014-04-07 Thread Greg KH
On Mon, Apr 07, 2014 at 01:34:47PM -0700, Joe Perches wrote: On Mon, 2014-04-07 at 22:27 +0200, Fabio Falzoi wrote: All printk(KERN_WARNING ...) replaced with pr_warn(...). Please remove the RTS51X_TIP prefix and and and use #define pr_fmt(fmt) KBUILD_MODNAME : fmt before any #include

Re: [PATCH v2 4/5] Staging: rts5139: pr_warn(...) in place of printk(KERN_WARNING ...)

2014-04-07 Thread Joe Perches
On Mon, 2014-04-07 at 13:56 -0700, Greg KH wrote: On Mon, Apr 07, 2014 at 01:34:47PM -0700, Joe Perches wrote: On Mon, 2014-04-07 at 22:27 +0200, Fabio Falzoi wrote: All printk(KERN_WARNING ...) replaced with pr_warn(...). Please remove the RTS51X_TIP prefix and and and use #define

Re: [PATCH net-next, v4] hyperv: Add support for virtual Receive Side Scaling (vRSS)

2014-04-07 Thread Sharat Masetty
Hi Zhang, How is this beneficial when compared to RPS(receive packet steering)? Can you please provide more details on what this patch does? Thanks Sharat On Mon, Apr 7, 2014 at 12:42 PM, David Miller da...@davemloft.net wrote: The net-next tree is not open yet, I will announce when it is

Re: [PATCH v2 4/5] Staging: rts5139: pr_warn(...) in place of printk(KERN_WARNING ...)

2014-04-07 Thread Joe Perches
On Mon, 2014-04-07 at 22:27 +0200, Fabio Falzoi wrote: All printk(KERN_WARNING ...) replaced with pr_warn(...). Please remove the RTS51X_TIP prefix and and and use #define pr_fmt(fmt) KBUILD_MODNAME : fmt before any #include so that pr_level uses are prefixed in a standard way. diff --git

Re: [visorchipset] invalid opcode: 0000 [#1] PREEMPT SMP

2014-04-07 Thread Fengguang Wu
Hi Benjamin, Fengguang, I ran your script against freshly-checked-out source from staging-next, and was not able to reproduce the error with it. My boot log is attached. I noticed that your log did not have Hypervisor detected: KVM in the trace. The KVM options in your script also