On Mon, Jun 23, 2014 at 2:33 PM, Kristina Martšenko
kristina.martse...@gmail.com wrote:
Here is an example of another driver being moved out:
http://www.spinics.net/lists/linux-usb/msg100924.html
Thanks Kristina. checkpatch reports 1 errors and 17 warnings on the
current driver.
I guess I
This patch fixes the following checkpatch warning:
CHECK: Blank lines aren't necessary after an open brace '{'
Signed-off-by: Ben Chan benc...@chromium.org
---
drivers/staging/gdm72xx/gdm_usb.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/gdm72xx/gdm_usb.c
This patch fixes the following checkpatch warnings, which are issued
when the gdm72xx driver is moved out of staging into drivers/net/wimax:
WARNING: networking block comments don't use an empty /* line, use /*
Comment...
WARNING: networking block comments start with * on subsequent lines
On 23/06/14 16:05, Maurice Moss wrote:
Hi All,
Thanks for your reply. I have been away the last few weeks.
Dan, I am using 64 bit Debian 3.2.57. So I am not using the latest
kernel. Would this be a problem?
That's probably quite old now. Depending on your hardware there may be
important
On Tue, 2014-06-24 at 09:27 +1000, Julian Calaby wrote:
- x = (T)pci_alloc_consistent(E1,E2,E3);
+ x = pci_zalloc_consistent(E1,E2,E3);
if ((x==NULL) || ...) S
- memset((T2)x,0,E2);
I don't know much about SmPL, but wouldn't having that if statement
there reduce your matches?
Code
On 2014-06-23 18:45, Hartley Sweeten wrote:
On Monday, June 23, 2014 8:10 AM, Ian Abbott wrote:
On 2014-06-20 21:13, H Hartley Sweeten wrote:
The DT2821 series board have jumpers that set the output range for
the two Analog Output channels. The range_table for the Analog Output
subdevice
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
From: Jes Sorensen jes.soren...@redhat.com
We need to make sure to update the network info as well as the IE
array.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/wifi.h | 13 -
1 file changed, 13 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/wifi.h
b/drivers/staging/rtl8723au/include/wifi.h
index
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/wifi.h | 8
1 file changed, 8 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/wifi.h
b/drivers/staging/rtl8723au/include/wifi.h
index
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_xmit.c | 21 -
drivers/staging/rtl8723au/include/rtl8723a_xmit.h | 1 -
2 files changed, 22 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Clean up and correct the beacon frame validation using a full beacon
frame, and pass that to rtw_check_beacon_data23a(). Previous we went
through hoops to construct the frame, minus the ieee80211_3addr header
which just made it more complicated, and
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_cmd.c | 5 -
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 2 --
drivers/staging/rtl8723au/include/rtl8723a_cmd.h | 2 --
3 files changed, 9
From: Jes Sorensen jes.soren...@redhat.com
Greg,
Please find attach my latest changeset for the rtl8723au. The primary
change is how we handle the beacon frame information within the
driver. Previously the driver would keep half of the ieee80211
management frame plus the IE (information
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 35 +--
1 file changed, 14 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 9 +
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 2 +-
drivers/staging/rtl8723au/include/ieee80211.h | 2 +-
3 files changed, 3
From: Jes Sorensen jes.soren...@redhat.com
Use ieee80211_mgmt to construct the frame instead of relying on hard
coded offsets.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_cmd.c | 31 +---
1 file changed, 19
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 18 +++---
1 file changed, 7 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 1 -
drivers/staging/rtl8723au/include/ieee80211.h | 1 -
2 files changed, 2 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
We validate the parameters in cfg80211_rtw_add_key() so no need to do
it all again in rtw_cfg80211_{ap_}set_encryption()
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 76
From: Jes Sorensen jes.soren...@redhat.com
struct ieee_parms is obsolete, so stop passing it on.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 19 ++-
1 file changed, 2 insertions(+), 17 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/wifi.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/wifi.h
b/drivers/staging/rtl8723au/include/wifi.h
index
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 55 ++-
1 file changed, 24 insertions(+), 31 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
No point checking it again in here, twice!
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 237 ++
1 file changed, 110 insertions(+), 127 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Passing strings around for something we have defines for is just
silly.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 47 +++
1 file changed, 31 insertions(+), 16
From: Jes Sorensen jes.soren...@redhat.com
This should help catching all the users once we stop carrying the
fixed beacon elements in the IE array.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 14 --
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 28 +--
1 file changed, 11 insertions(+), 17 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 62 +--
1 file changed, 30 insertions(+), 32 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 37 ---
1 file changed, 13 insertions(+), 24 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Rewrite rtw_restruct_wmm_ie23a() using cfg80211_find_vendor_ie()
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 34 ---
1 file changed, 13 insertions(+), 21 deletions(-)
From: Jes Sorensen jes.soren...@redhat.com
This gets rid of the odd carrying of half the beacon frame in the IE
array stored for the network. Instead we rely on the relevant fields
(timestamp, beacon_interval, and capability) stored in struct
wlan_bssid_ex.
Carrying only half the ieee80211_mgmt
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 62 ++-
1 file changed, 26 insertions(+), 36 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
From: Jes Sorensen jes.soren...@redhat.com
Another case using hard coded offsets counting on the presence of
fixed beacon elements in the IEs array.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 4 ++--
1 file changed, 2 insertions(+), 2
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 37 ---
1 file changed, 37 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
On Tue, Jun 24, 2014 at 12:12:55AM -0700, Ben Chan wrote:
On Mon, Jun 23, 2014 at 2:33 PM, Kristina Martšenko
kristina.martse...@gmail.com wrote:
Here is an example of another driver being moved out:
http://www.spinics.net/lists/linux-usb/msg100924.html
Thanks Kristina. checkpatch
On Tue, Jun 24, 2014 at 11:14:40AM +0200, Luca Ellero wrote:
On 23/06/2014 22:19, Kristina Martšenko wrote: Hi Angelo,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving quickstart out of staging in
over a year. Are there
On Tue, Jun 24, 2014 at 08:29:17AM +0800, Dexuan Cui wrote:
Currently the VSC has no chance to notify the VSP of the dirty rectangle on VM
panic because the notification work is done in a workqueue, and in panic() the
kernel typically ends up in an infinite loop, and a typical kernel config has
On Tue, Jun 24, 2014 at 11:54:10AM +0800, Stefan Hajnoczi wrote:
On Tue, Jun 24, 2014 at 4:23 AM, Kristina Martšenko
kristina.martse...@gmail.com wrote:
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving line6 out of staging in
On Mon, Jun 16, 2014 at 12:11:20PM +0200, Denis Carikli wrote:
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
This patch needs an
Denis,
This patch creates binding documentation. Any patch which does so
should be copied to the DT people so they can review the bindings
and give appropriate acks. It would be better if you separate the
binding documentation updates from the other functional changes too.
I've added them on
On Mon, Jun 16, 2014 at 12:11:18PM +0200, Denis Carikli wrote:
Signed-off-by: Denis Carikli de...@eukrea.com
It would be nice to have a little more explanation in the commit messages
for these patches. If you'd like to send me better commit messages for
these patches, I'll add them to what I
On 06/24/2014 05:13 PM, Russell King - ARM Linux wrote:
[...]
If you'd like to send me better commit messages for
these patches, I'll add them to what I already have:
imx-drm: use defines for clock polarity settings
The comment of the clk_pol field of the ipu_di_signal_cfg struct was
On Tue, Jun 24, 2014 at 06:25:19PM +0200, Denis Carikli wrote:
On 06/24/2014 05:13 PM, Russell King - ARM Linux wrote:
[...]
If you'd like to send me better commit messages for
these patches, I'll add them to what I already have:
imx-drm: use defines for clock polarity settings
The
Permit masters to call component_master_add_child() and match the same
child multiple times. This may happen if there's multiple connections
to a single component device from other devices. In such scenarios,
we should not return a failure, but instead ignore the attempt.
Signed-off-by: Russell
In try_to_bring_up_master(), we tear down the master's component list
for each error case, except for devres group failure. Fix this
oversight by making the code less prone to such mistakes.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
drivers/base/component.c | 62
Clean up the code a little; we don't need to check that the master is
unbound for every invocation of try_to_bring_up_master(), so let's move
it to where it's really needed - try_to_bring_up_masters(), where we may
encounter already bound masters.
Signed-off-by: Russell King
Add support for generating a set of component matches at master probe
time, and submitting them to the component layer. This allows the
component layer to perform the matches internally without needing to
call into the master driver, and allows for further restructuring of
the component helper.
Update the imx-drm driver to use the component match support rather than
add_components.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
drivers/staging/imx-drm/imx-drm-core.c | 57 +++---
1 file changed, 4 insertions(+), 53 deletions(-)
diff --git
Since we now have an array which defines each component, maintain the
components to be bound in the array rather than a separate list. We
also need duplicate tracking so we can eliminate multiple bind calls
for the same component: we preserve the list-based component order in
that the first match
On Tue, Jun 24, 2014 at 10:06 AM, Russell King - ARM Linux
li...@arm.linux.org.uk wrote:
Denis,
This patch creates binding documentation. Any patch which does so
should be copied to the DT people so they can review the bindings
and give appropriate acks. It would be better if you separate
From: Clément Calmels clement.calm...@free.fr
Found running sparse on the rtl8723au driver.
CHECK drivers/staging/rtl8723au/core/rtw_cmd.c
drivers/staging/rtl8723au/core/rtw_cmd.c:1479:54: warning: incorrect type in
argument 2 (different base types)
On Tue, Jun 24, 2014 at 08:29:17AM +0800, Dexuan Cui wrote:
Currently the VSC has no chance to notify the VSP of the dirty rectangle on
VM
panic because the notification work is done in a workqueue, and in panic()
the
kernel typically ends up in an infinite loop, and a typical kernel config
On Mon, Jun 16, 2014 at 12:11:22PM +0200, Denis Carikli wrote:
[...]
diff --git
a/Documentation/devicetree/bindings/panel/eukrea,mbimxsd51-dvi-svga.txt
b/Documentation/devicetree/bindings/panel/eukrea,mbimxsd51-dvi-svga.txt
[...]
@@ -0,0 +1,7 @@
+Eukrea DVI-SVGA (800x600 pixels) DVI output.
Hi Thierry,
Le Tue, 24 Jun 2014 23:49:37 +0200,
Thierry Reding thierry.red...@gmail.com a écrit :
On Mon, Jun 16, 2014 at 12:11:22PM +0200, Denis Carikli wrote:
[...]
diff --git
a/Documentation/devicetree/bindings/panel/eukrea,mbimxsd51-dvi-svga.txt
On Tue, Jun 24, 2014 at 11:56:39PM +0200, Eric Bénard wrote:
Hi Thierry,
Le Tue, 24 Jun 2014 23:49:37 +0200,
Thierry Reding thierry.red...@gmail.com a écrit :
On Mon, Jun 16, 2014 at 12:11:22PM +0200, Denis Carikli wrote:
[...]
diff --git
On Tue, Jun 24, 2014 at 03:57:46PM +0100, Russell King - ARM Linux wrote:
On Mon, Jun 16, 2014 at 12:11:20PM +0200, Denis Carikli wrote:
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
On Tue, Jun 24, 2014 at 04:01:58PM +0100, Russell King - ARM Linux wrote:
On Mon, Jun 16, 2014 at 12:11:19PM +0200, Denis Carikli wrote:
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
Hi,
I recently joined the Linux Driver Development mailing list and would
like to contribute to the Linux Driver Project. Specifically, I would
like to contribute by working on ACPI Support for HP Pavillion laptops
(Referred list at
_init_listhead is just an inline wrapper around INIT_LIST_HEAD. This
patch removes the wrapper and directly uses INIT_LIST_HEAD instead.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |7 +--
drivers/staging/rtl8712/rtl8712_recv.c|
is_list_empty is just an inline wrapper around list_empty. This patch
removes the wrapper and directly uses list_empty instead.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h | 10 +-
drivers/staging/rtl8712/rtl8712_recv.c |4 ++--
list_insert_tail is just an inline wrapper around list_add_tail. This
patch removes the wrapper and directly uses list_add_tail.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |6 --
drivers/staging/rtl8712/rtl8712_recv.c| 10
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/rtl8712/osdep_service.h
b/drivers/staging/rtl8712/osdep_service.h
index 77b1443..3dfc473 100644
---
get_next is just an inline wrapper around return list-next. This
patch removes the wrapper and directly uses list-next where
applicable.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |5 -
drivers/staging/rtl8712/rtl8712_recv.c
_queue_empty is an inline wrapper around list_empty. This patch removes this
wrapper function and instead calls list_empty directly.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |5 -
drivers/staging/rtl8712/rtl8712_cmd.c |
get_list_head is an inline that returns list-head. This patch removes this
inline and directly applies list-head where applicable.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |5 -
drivers/staging/rtl8712/rtl8712_recv.c|
list_delete is just an inline wrapper around list_del_init. This patch
removes the wrapper and directly uses list_del_init.
Signed-off-by: James A Shackleford sh...@linux.com
---
drivers/staging/rtl8712/osdep_service.h |5 -
drivers/staging/rtl8712/rtl8712_recv.c|8
Good day,
I have something of importance which you will be interested in, it will be my
utmost appreciation if you contact me so I can furnish you with details.
Roberts
___
devel mailing list
de...@linuxdriverproject.org
71 matches
Mail list logo