Charles Clément | 1 Aug 2010 17:15
Picon

[PATCH 1/3] Staging: vt6655: replace TRUE with in kernel true


Signed-off-by: Charles Clément <caratorn <at> gmail.com>
---
 drivers/staging/vt6655/80211mgr.c    |    8 +-
 drivers/staging/vt6655/IEEE11h.c     |   10 +-
 drivers/staging/vt6655/aes_ccmp.c    |    4 +-
 drivers/staging/vt6655/baseband.c    |   22 ++--
 drivers/staging/vt6655/bssdb.c       |   98 ++++++++--------
 drivers/staging/vt6655/card.c        |  146 ++++++++++++------------
 drivers/staging/vt6655/channel.c     |  170 +++++++++++++-------------
 drivers/staging/vt6655/datarate.c    |    8 +-
 drivers/staging/vt6655/device.h      |    2 +-
 drivers/staging/vt6655/device_cfg.h  |    4 -
 drivers/staging/vt6655/device_main.c |  124 ++++++++++----------
 drivers/staging/vt6655/dpc.c         |   70 ++++++------
 drivers/staging/vt6655/hostap.c      |   26 ++--
 drivers/staging/vt6655/ioctl.c       |   56 +++++-----
 drivers/staging/vt6655/iwctl.c       |   58 +++++-----
 drivers/staging/vt6655/key.c         |   90 +++++++-------
 drivers/staging/vt6655/mac.c         |   50 ++++----
 drivers/staging/vt6655/power.c       |   14 +-
 drivers/staging/vt6655/rf.c          |   74 ++++++------
 drivers/staging/vt6655/rxtx.c        |   88 +++++++-------
 drivers/staging/vt6655/srom.c        |   12 +-
 drivers/staging/vt6655/tether.c      |    4 +-
 drivers/staging/vt6655/ttype.h       |    3 -
 drivers/staging/vt6655/vntwifi.c     |   32 +++---
 drivers/staging/vt6655/wcmd.c        |   64 +++++-----
 drivers/staging/vt6655/wctl.c        |   16 ++--
 drivers/staging/vt6655/wmgr.c        |  216 +++++++++++++++++-----------------
(Continue reading)

Charles Clément | 1 Aug 2010 17:15
Picon

[PATCH 2/3] Staging: vt6655: replace FALSE with in kernel false


Signed-off-by: Charles Clément <caratorn <at> gmail.com>
---
 drivers/staging/vt6655/IEEE11h.c     |    6 +-
 drivers/staging/vt6655/aes_ccmp.c    |    4 +-
 drivers/staging/vt6655/baseband.c    |   22 +++---
 drivers/staging/vt6655/bssdb.c       |   64 ++++++++--------
 drivers/staging/vt6655/card.c        |  106 ++++++++++++++--------------
 drivers/staging/vt6655/channel.c     |   28 ++++----
 drivers/staging/vt6655/datarate.c    |    4 +-
 drivers/staging/vt6655/device.h      |    2 +-
 drivers/staging/vt6655/device_cfg.h  |    4 -
 drivers/staging/vt6655/device_main.c |  132 +++++++++++++++++-----------------
 drivers/staging/vt6655/dpc.c         |   88 +++++++++++-----------
 drivers/staging/vt6655/hostap.c      |   26 ++++----
 drivers/staging/vt6655/ioctl.c       |   36 +++++-----
 drivers/staging/vt6655/iwctl.c       |   38 +++++-----
 drivers/staging/vt6655/key.c         |  102 +++++++++++++-------------
 drivers/staging/vt6655/mac.c         |   68 +++++++++---------
 drivers/staging/vt6655/power.c       |   36 +++++-----
 drivers/staging/vt6655/rf.c          |   52 +++++++-------
 drivers/staging/vt6655/rxtx.c        |   66 +++++++++---------
 drivers/staging/vt6655/srom.c        |   12 ++--
 drivers/staging/vt6655/tether.c      |    4 +-
 drivers/staging/vt6655/ttype.h       |    4 -
 drivers/staging/vt6655/vntwifi.c     |   18 +++---
 drivers/staging/vt6655/wcmd.c        |   46 ++++++------
 drivers/staging/vt6655/wctl.c        |   24 +++---
 drivers/staging/vt6655/wmgr.c        |  122 ++++++++++++++++----------------
 drivers/staging/vt6655/wpa.c         |   18 +++---
(Continue reading)

Charles Clément | 1 Aug 2010 17:15
Picon

[PATCH 3/3] Staging: vt6655: replace BOOL with in kernel bool


Signed-off-by: Charles Clément <caratorn <at> gmail.com>
---
 drivers/staging/vt6655/IEEE11h.c     |   10 +-
 drivers/staging/vt6655/IEEE11h.h     |    2 +-
 drivers/staging/vt6655/aes_ccmp.c    |    4 +-
 drivers/staging/vt6655/aes_ccmp.h    |    2 +-
 drivers/staging/vt6655/baseband.c    |   16 ++--
 drivers/staging/vt6655/baseband.h    |   10 +-
 drivers/staging/vt6655/bssdb.c       |   18 ++--
 drivers/staging/vt6655/bssdb.h       |   38 +++++-----
 drivers/staging/vt6655/card.c        |   58 +++++++-------
 drivers/staging/vt6655/card.h        |   55 +++++++-------
 drivers/staging/vt6655/channel.c     |   14 ++--
 drivers/staging/vt6655/channel.h     |   10 +-
 drivers/staging/vt6655/datarate.c    |    4 +-
 drivers/staging/vt6655/datarate.h    |    2 +-
 drivers/staging/vt6655/device.h      |  134 +++++++++++++++++-----------------
 drivers/staging/vt6655/device_main.c |   42 +++++-----
 drivers/staging/vt6655/dpc.c         |   48 ++++++------
 drivers/staging/vt6655/dpc.h         |    2 +-
 drivers/staging/vt6655/hostap.c      |    2 +-
 drivers/staging/vt6655/iocmd.h       |   23 +++---
 drivers/staging/vt6655/key.c         |   16 ++--
 drivers/staging/vt6655/key.h         |   24 +++---
 drivers/staging/vt6655/mac.c         |   30 ++++----
 drivers/staging/vt6655/mac.h         |   30 ++++----
 drivers/staging/vt6655/power.c       |   12 ++--
 drivers/staging/vt6655/power.h       |   10 +-
 drivers/staging/vt6655/rf.c          |   44 ++++++------
(Continue reading)

Online Coodinator | 2 Aug 2010 04:56
Picon

Congratulations

Dear Microsoft User,

This is to inform you that you have been selected for a cash prize of 
£500,000.00 (Five Hundred Thousand Great British Pound) from Microsoft 
Corporation 2010 International Award Program held on the 27th of July 2010 in 
London, United Kingdom.

The selection process was carried out through random selection in our 
computerized email selection system (ESS) from a database of over 125,000 
email addresses composed of various email domains worldwide.

The Microsoft Corporation Award Programme is approved by the British Gaming 
Board and also licensed by The International Association of Gaming Regulators 
(IAGR). 

To begin the processing of your prize you are to contact our MS Claims Office 
for more information as regards procedures to claim your prize.

REF: NLU/63120973/LY AND BATCH: 08/H3401/3UK

To file for your claim, contact the processing consultant:

MS CLAIMS OFFICE
U.K OFFICE.ENGLAND, LHART STREET 34, WALSALL.WS1 3PE
WEST MIDLAND, LONDON. 70 1NL UNITED KINGDOM.

Claims Officer: Mr. Terry Owen
Foreign Services Directorate/Operation Personnel
Emails: mcaclaims10 <at> live.co.uk
Tel: +447045795051
(Continue reading)

Javier Martinez Canillas | 2 Aug 2010 06:43
Picon
Gravatar

[PATCH] staging/wlags49_h2: use common PCI_VENDOR/DEVICE_ID name format

There is a common name format for PCI_VENDOR/DEVICE_ID constants.
wlags49_h2 names doesn't fit in the standard. 
This patch change the names and also fix some style issues.

Thanks a lot,

Signed-off-by: Javier Martinez Canillas <martinez.javier <at> gmail.com>
---
 drivers/staging/wlags49_h2/wl_pci.c |   10 +++++++---
 drivers/staging/wlags49_h2/wl_pci.h |    8 ++++----
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/wlags49_h2/wl_pci.c b/drivers/staging/wlags49_h2/wl_pci.c
index af4f4f8..020b17a 100644
--- a/drivers/staging/wlags49_h2/wl_pci.c
+++ b/drivers/staging/wlags49_h2/wl_pci.c
 <at>  <at>  -117,9 +117,13  <at>  <at>  enum hermes_pci_versions {
 };

 static struct pci_device_id wl_pci_tbl[] __devinitdata = {
-	{ WL_LKM_PCI_VENDOR_ID, WL_LKM_PCI_DEVICE_ID_0, PCI_ANY_ID, PCI_ANY_ID, 0, 0,
CH_Agere_Systems_Mini_PCI_V1 },
-    { WL_LKM_PCI_VENDOR_ID, WL_LKM_PCI_DEVICE_ID_1, PCI_ANY_ID, PCI_ANY_ID, 0, 0,
CH_Agere_Systems_Mini_PCI_V1 },
-    { WL_LKM_PCI_VENDOR_ID, WL_LKM_PCI_DEVICE_ID_2, PCI_ANY_ID, PCI_ANY_ID, 0, 0,
CH_Agere_Systems_Mini_PCI_V1 },
+	{ PCI_VENDOR_ID_WL_LKM, PCI_DEVICE_ID_WL_LKM_0,
+	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, CH_Agere_Systems_Mini_PCI_V1 },
+	{ PCI_VENDOR_ID_WL_LKM, PCI_DEVICE_ID_WL_LKM_1,
+	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, CH_Agere_Systems_Mini_PCI_V1 },
(Continue reading)

Kulikov Vasiliy | 2 Aug 2010 10:31
Picon

[PATCH 1/9 v2] staging: dt3155: check put_user() return value

put_user() return code was not checked for errors. To simplify the code
fill local struct dt3155_read with data and copy it with single
copy_to_user() call.

Signed-off-by: Kulikov Vasiliy <segooon <at> gmail.com>
---
 drivers/staging/dt3155/dt3155_drv.c |   17 ++++++++---------
 1 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c
index 7316996..4013f09 100644
--- a/drivers/staging/dt3155/dt3155_drv.c
+++ b/drivers/staging/dt3155/dt3155_drv.c
 <at>  <at>  -745,10 +745,11  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,
   struct dt3155_status *dts = &dt3155_status[minor];
   struct dt3155_fbuffer *fb = &dts->fbuffer;
   struct frame_info	*frame_info;
+  struct dt3155_read dt_read;

   /* TODO: this should check the error flag and */
   /*   return an error on hardware failures */
-  if (count != sizeof(struct dt3155_read))
+  if (count != sizeof(dt_read))
     {
       printk(KERN_INFO "DT3155 ERROR (NJC): count is not right\n");
       return -EINVAL;
 <at>  <at>  -797,17 +798,15  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,

   /* make this an offset */
   offset = frame_info->addr - dts->mem_addr;
(Continue reading)

Simon Horman | 2 Aug 2010 14:41
Picon
Gravatar

Re: [PATCH 1/9 v2] staging: dt3155: check put_user() return value

On Mon, Aug 02, 2010 at 12:31:12PM +0400, Kulikov Vasiliy wrote:
> put_user() return code was not checked for errors. To simplify the code
> fill local struct dt3155_read with data and copy it with single
> copy_to_user() call.
> 
> Signed-off-by: Kulikov Vasiliy <segooon <at> gmail.com>
> ---
>  drivers/staging/dt3155/dt3155_drv.c |   17 ++++++++---------
>  1 files changed, 8 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c
> index 7316996..4013f09 100644
> --- a/drivers/staging/dt3155/dt3155_drv.c
> +++ b/drivers/staging/dt3155/dt3155_drv.c
>  <at>  <at>  -745,10 +745,11  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,
>    struct dt3155_status *dts = &dt3155_status[minor];
>    struct dt3155_fbuffer *fb = &dts->fbuffer;
>    struct frame_info	*frame_info;
> +  struct dt3155_read dt_read;
>  
>    /* TODO: this should check the error flag and */
>    /*   return an error on hardware failures */
> -  if (count != sizeof(struct dt3155_read))
> +  if (count != sizeof(dt_read))

I believe that sizeof(struct dt3155_read) is preferred,
so this change isn't needed and sizeof(struct dt3155_read)
should be used below (three times).

>      {
(Continue reading)

Vasiliy Kulikov | 2 Aug 2010 14:56
Picon

[PATCH 1/9 v3] staging: dt3155: check put_user() return value

put_user() returns code was not checked for errors. To simplify the code
fill local struct dt3155_read with data and copy it with single
copy_to_user() call.

Signed-off-by: Kulikov Vasiliy <segooon <at> gmail.com>
---
 drivers/staging/dt3155/dt3155_drv.c |   13 ++++++-------
 1 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c
index 7316996..6031837 100644
--- a/drivers/staging/dt3155/dt3155_drv.c
+++ b/drivers/staging/dt3155/dt3155_drv.c
 <at>  <at>  -745,6 +745,7  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,
   struct dt3155_status *dts = &dt3155_status[minor];
   struct dt3155_fbuffer *fb = &dts->fbuffer;
   struct frame_info	*frame_info;
+  struct dt3155_read dt_read;

   /* TODO: this should check the error flag and */
   /*   return an error on hardware failures */
 <at>  <at>  -797,14 +798,12  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,

   /* make this an offset */
   offset = frame_info->addr - dts->mem_addr;
+  dt_read.offset = offset;
+  dt_read.frame_seq = fb->frame_count;
+  dt_read.state = dts->state;
+  memcpy(&dt_read.frame_info, frame_info, sizeof(struct frame_info));

(Continue reading)

Simon Horman | 2 Aug 2010 15:51
Picon
Gravatar

Re: [PATCH 1/9 v3] staging: dt3155: check put_user() return value

On Mon, Aug 02, 2010 at 04:56:38PM +0400, Vasiliy Kulikov wrote:
> put_user() returns code was not checked for errors. To simplify the code
> fill local struct dt3155_read with data and copy it with single
> copy_to_user() call.
> 
> Signed-off-by: Kulikov Vasiliy <segooon <at> gmail.com>

Looks good to me.

Reviewed-by: Simon Horman <horms <at> verge.net.au>

> ---
>  drivers/staging/dt3155/dt3155_drv.c |   13 ++++++-------
>  1 files changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/staging/dt3155/dt3155_drv.c b/drivers/staging/dt3155/dt3155_drv.c
> index 7316996..6031837 100644
> --- a/drivers/staging/dt3155/dt3155_drv.c
> +++ b/drivers/staging/dt3155/dt3155_drv.c
>  <at>  <at>  -745,6 +745,7  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,
>    struct dt3155_status *dts = &dt3155_status[minor];
>    struct dt3155_fbuffer *fb = &dts->fbuffer;
>    struct frame_info	*frame_info;
> +  struct dt3155_read dt_read;
>  
>    /* TODO: this should check the error flag and */
>    /*   return an error on hardware failures */
>  <at>  <at>  -797,14 +798,12  <at>  <at>  static ssize_t dt3155_read(struct file *filep, char __user *buf,
>  
>    /* make this an offset */
(Continue reading)

Haiyang Zhang | 2 Aug 2010 19:00
Picon
Favicon

[PATCH 1/1] staging: hv: Fix missing functions for net_device_ops

From: Haiyang Zhang <haiyangz <at> microsoft.com>

Subject: staging: hv: Fix missing functions for net_device_ops
  Unlike in older kernels, ether_setup() no longer sets default
functions. We need to put them into net_device_ops. For example,
eth_change_mtu prevents setting MTU beyond 1500. This is necessary
because we don't support jumbo frame at this time.

Signed-off-by: Haiyang Zhang <haiyangz <at> microsoft.com>
Signed-off-by: Hank Janssen <hjanssen <at> microsoft.com>

---
 drivers/staging/hv/netvsc_drv.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_drv.c
index 56e1157..64a0114 100644
--- a/drivers/staging/hv/netvsc_drv.c
+++ b/drivers/staging/hv/netvsc_drv.c
 <at>  <at>  -327,6 +327,9  <at>  <at>  static const struct net_device_ops device_ops = {
 	.ndo_stop =			netvsc_close,
 	.ndo_start_xmit =		netvsc_start_xmit,
 	.ndo_set_multicast_list =	netvsc_set_multicast_list,
+	.ndo_change_mtu =		eth_change_mtu,
+	.ndo_validate_addr =		eth_validate_addr,
+	.ndo_set_mac_address =		eth_mac_addr,
 };

 static int netvsc_probe(struct device *device)
--

-- 
(Continue reading)


Gmane