diff mbox

staging: vt6656: vnt_rf_setpower: fix missing rate RATE_12M

Message ID 1425746197-6614-1-git-send-email-tvboxspy@gmail.com (mailing list archive)
State Not Applicable
Delegated to: Kalle Valo
Headers show

Commit Message

Malcolm Priestley March 7, 2015, 4:36 p.m. UTC
When the driver sets this rate a power of zero value is set causing
data flow stoppage until another rate is tried.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Cc: <stable@vger.kernel.org> # v3.17+
---
 drivers/staging/vt6656/rf.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Luis Henriques March 26, 2015, 3:22 p.m. UTC | #1
Hi Malcolm,

On Sat, Mar 07, 2015 at 04:36:37PM +0000, Malcolm Priestley wrote:
> When the driver sets this rate a power of zero value is set causing
> data flow stoppage until another rate is tried.
> 
> Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
> Cc: <stable@vger.kernel.org> # v3.17+

Is there a reason for this patch being tagged for stable v3.17+ ?

I'm asking because there's a similar commit for vt6655 (40c8790bcb7a
"vt6655: RFbSetPower fix missing rate RATE_12M") which was tagged for
stable without the "v3.17+", and thus I assume applicable to other
trees as well (e.g. 3.16).

Cheers,
--
Luís

> ---
>  drivers/staging/vt6656/rf.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
> index c42cde5..c4286cc 100644
> --- a/drivers/staging/vt6656/rf.c
> +++ b/drivers/staging/vt6656/rf.c
> @@ -640,6 +640,7 @@ int vnt_rf_setpower(struct vnt_private *priv, u32 rate, u32 channel)
>  		break;
>  	case RATE_6M:
>  	case RATE_9M:
> +	case RATE_12M:
>  	case RATE_18M:
>  	case RATE_24M:
>  	case RATE_36M:
> -- 
> 2.1.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Malcolm Priestley March 26, 2015, 6:33 p.m. UTC | #2
On 26/03/15 15:22, Luis Henriques wrote:
> Hi Malcolm,
>
> On Sat, Mar 07, 2015 at 04:36:37PM +0000, Malcolm Priestley wrote:
>> When the driver sets this rate a power of zero value is set causing
>> data flow stoppage until another rate is tried.
>>
>> Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
>> Cc: <stable@vger.kernel.org> # v3.17+
>
> Is there a reason for this patch being tagged for stable v3.17+ ?
Hi Luis

Sorry I thought it wouldn't apply to 3.16.

I need to do a backport patch for earlier kernels.

Regards

Malcolm

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Luis Henriques March 26, 2015, 6:48 p.m. UTC | #3
On Thu, Mar 26, 2015 at 06:33:18PM +0000, Malcolm Priestley wrote:
> 
> 
> On 26/03/15 15:22, Luis Henriques wrote:
> >Hi Malcolm,
> >
> >On Sat, Mar 07, 2015 at 04:36:37PM +0000, Malcolm Priestley wrote:
> >>When the driver sets this rate a power of zero value is set causing
> >>data flow stoppage until another rate is tried.
> >>
> >>Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
> >>Cc: <stable@vger.kernel.org> # v3.17+
> >
> >Is there a reason for this patch being tagged for stable v3.17+ ?
> Hi Luis
> 
> Sorry I thought it wouldn't apply to 3.16.
> 
> I need to do a backport patch for earlier kernels.
> 

Awesome, thanks for the quick reply.

Commit 163fe301b9f78b6de57d0014eafe504fd20c0cd4 is a clean cherry-pick
to 3.16 kernel, but I haven't check which other kernels it should also
be applied to.

Cheers,
--
Luís

> Regards
> 
> Malcolm
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index c42cde5..c4286cc 100644
--- a/drivers/staging/vt6656/rf.c
+++ b/drivers/staging/vt6656/rf.c
@@ -640,6 +640,7 @@  int vnt_rf_setpower(struct vnt_private *priv, u32 rate, u32 channel)
 		break;
 	case RATE_6M:
 	case RATE_9M:
+	case RATE_12M:
 	case RATE_18M:
 	case RATE_24M:
 	case RATE_36M: