All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] wireless/atmel: remove time_t usage
@ 2017-02-23 16:14 ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-02-23 16:14 UTC (permalink / raw)
  To: Simon Kelley, Kalle Valo
  Cc: linux-wireless, netdev, linux-arm-kernel, Alexandre Belloni

last_qual never really holds a time. It only holds jiffies. Make it the
same type as jiffies.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/net/wireless/atmel/atmel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/atmel/atmel.c b/drivers/net/wireless/atmel/atmel.c
index e12f62356fd1..27b110dc8cc6 100644
--- a/drivers/net/wireless/atmel/atmel.c
+++ b/drivers/net/wireless/atmel/atmel.c
@@ -513,7 +513,7 @@ struct atmel_private {
 	} station_state;
 
 	int operating_mode, power_mode;
-	time_t last_qual;
+	unsigned long last_qual;
 	int beacons_this_sec;
 	int channel;
 	int reg_domain, config_reg_domain;
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH] wireless/atmel: remove time_t usage
@ 2017-02-23 16:14 ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-02-23 16:14 UTC (permalink / raw)
  To: Simon Kelley, Kalle Valo
  Cc: netdev, Alexandre Belloni, linux-wireless, linux-arm-kernel

last_qual never really holds a time. It only holds jiffies. Make it the
same type as jiffies.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/net/wireless/atmel/atmel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/atmel/atmel.c b/drivers/net/wireless/atmel/atmel.c
index e12f62356fd1..27b110dc8cc6 100644
--- a/drivers/net/wireless/atmel/atmel.c
+++ b/drivers/net/wireless/atmel/atmel.c
@@ -513,7 +513,7 @@ struct atmel_private {
 	} station_state;
 
 	int operating_mode, power_mode;
-	time_t last_qual;
+	unsigned long last_qual;
 	int beacons_this_sec;
 	int channel;
 	int reg_domain, config_reg_domain;
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH] wireless/atmel: remove time_t usage
@ 2017-02-23 16:14 ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-02-23 16:14 UTC (permalink / raw)
  To: linux-arm-kernel

last_qual never really holds a time. It only holds jiffies. Make it the
same type as jiffies.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/net/wireless/atmel/atmel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/atmel/atmel.c b/drivers/net/wireless/atmel/atmel.c
index e12f62356fd1..27b110dc8cc6 100644
--- a/drivers/net/wireless/atmel/atmel.c
+++ b/drivers/net/wireless/atmel/atmel.c
@@ -513,7 +513,7 @@ struct atmel_private {
 	} station_state;
 
 	int operating_mode, power_mode;
-	time_t last_qual;
+	unsigned long last_qual;
 	int beacons_this_sec;
 	int channel;
 	int reg_domain, config_reg_domain;
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
  2017-02-23 16:14 ` Alexandre Belloni
  (?)
  (?)
@ 2017-03-08 15:13 ` Kalle Valo
  -1 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 15:13 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Simon Kelley, linux-wireless, netdev, linux-arm-kernel,
	Alexandre Belloni

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

The title should start with "atmel:", I can fix that during commit.

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
  2017-02-23 16:14 ` Alexandre Belloni
@ 2017-03-08 15:13   ` Kalle Valo
  -1 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 15:13 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Simon Kelley, Alexandre Belloni, linux-wireless,
	linux-arm-kernel, netdev

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

The title should start with "atmel:", I can fix that during commit.

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
* wireless/atmel: remove time_t usage
@ 2017-03-08 15:13   ` Kalle Valo
  0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 15:13 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

The title should start with "atmel:", I can fix that during commit.

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
       [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
  2017-03-08 16:10     ` Alexandre Belloni
@ 2017-03-08 16:10     ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-03-08 16:10 UTC (permalink / raw)
  To: Kalle Valo; +Cc: Simon Kelley, linux-wireless, netdev, linux-arm-kernel

Hi,

On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> > last_qual never really holds a time. It only holds jiffies. Make it the
> > same type as jiffies.
> > 
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> The title should start with "atmel:", I can fix that during commit.
> 

If you prefer, I can send a v2

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
@ 2017-03-08 16:10     ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-03-08 16:10 UTC (permalink / raw)
  To: Kalle Valo; +Cc: Simon Kelley, linux-wireless, linux-arm-kernel, netdev

Hi,

On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> > last_qual never really holds a time. It only holds jiffies. Make it the
> > same type as jiffies.
> > 
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> The title should start with "atmel:", I can fix that during commit.
> 

If you prefer, I can send a v2

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 15+ messages in thread
* wireless/atmel: remove time_t usage
@ 2017-03-08 16:10     ` Alexandre Belloni
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2017-03-08 16:10 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> > last_qual never really holds a time. It only holds jiffies. Make it the
> > same type as jiffies.
> > 
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> The title should start with "atmel:", I can fix that during commit.
> 

If you prefer, I can send a v2

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
  2017-03-08 16:10     ` Alexandre Belloni
  (?)
@ 2017-03-08 18:53       ` Kalle Valo
  -1 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 18:53 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: Simon Kelley, linux-wireless, netdev, linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> writes:

> Hi,
>
> On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
>> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
>> > last_qual never really holds a time. It only holds jiffies. Make it the
>> > same type as jiffies.
>> > 
>> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> 
>> The title should start with "atmel:", I can fix that during commit.
>> 
>
> If you prefer, I can send a v2

Thanks, but no need. I can easily edit the commit log when I commit it.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: wireless/atmel: remove time_t usage
@ 2017-03-08 18:53       ` Kalle Valo
  0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 18:53 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: Simon Kelley, linux-wireless, linux-arm-kernel, netdev

Alexandre Belloni <alexandre.belloni@free-electrons.com> writes:

> Hi,
>
> On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
>> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
>> > last_qual never really holds a time. It only holds jiffies. Make it the
>> > same type as jiffies.
>> > 
>> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> 
>> The title should start with "atmel:", I can fix that during commit.
>> 
>
> If you prefer, I can send a v2

Thanks, but no need. I can easily edit the commit log when I commit it.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 15+ messages in thread
* wireless/atmel: remove time_t usage
@ 2017-03-08 18:53       ` Kalle Valo
  0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-08 18:53 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> writes:

> Hi,
>
> On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
>> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
>> > last_qual never really holds a time. It only holds jiffies. Make it the
>> > same type as jiffies.
>> > 
>> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> 
>> The title should start with "atmel:", I can fix that during commit.
>> 
>
> If you prefer, I can send a v2

Thanks, but no need. I can easily edit the commit log when I commit it.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: atmel: remove time_t usage
  2017-02-23 16:14 ` Alexandre Belloni
@ 2017-03-20 16:43   ` Kalle Valo
  -1 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-20 16:43 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Simon Kelley, Alexandre Belloni, linux-wireless,
	linux-arm-kernel, netdev

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Patch applied to wireless-drivers-next.git, thanks.

5c5105666de5 atmel: remove time_t usage

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
* atmel: remove time_t usage
@ 2017-03-20 16:43   ` Kalle Valo
  0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-20 16:43 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Patch applied to wireless-drivers-next.git, thanks.

5c5105666de5 atmel: remove time_t usage

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: atmel: remove time_t usage
  2017-02-23 16:14 ` Alexandre Belloni
                   ` (5 preceding siblings ...)
  (?)
@ 2017-03-20 16:43 ` Kalle Valo
  -1 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2017-03-20 16:43 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Simon Kelley, linux-wireless, netdev, linux-arm-kernel,
	Alexandre Belloni

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Patch applied to wireless-drivers-next.git, thanks.

5c5105666de5 atmel: remove time_t usage

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 15+ messages in thread
end of thread, other threads:[~2017-03-20 16:52 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-23 16:14 [PATCH] wireless/atmel: remove time_t usage Alexandre Belloni
2017-02-23 16:14 ` Alexandre Belloni
2017-02-23 16:14 ` Alexandre Belloni
2017-03-08 15:13 ` Kalle Valo
2017-03-08 15:13 ` Kalle Valo
2017-03-08 15:13   ` Kalle Valo
     [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
2017-03-08 16:10   ` Alexandre Belloni
2017-03-08 16:10     ` Alexandre Belloni
2017-03-08 16:10     ` Alexandre Belloni
2017-03-08 18:53     ` Kalle Valo
2017-03-08 18:53       ` Kalle Valo
2017-03-08 18:53       ` Kalle Valo
2017-03-20 16:43 ` atmel: " Kalle Valo
2017-03-20 16:43   ` Kalle Valo
2017-03-20 16:43 ` Kalle Valo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.