headers
Johannes Berg | 7 Oct 11:04
Favicon

mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

Hi,

I just realised that for whatever reason mac80211 isn't setting the
IEEE80211_CONF_SHORT_SLOT_TIME flag. I'm not sure when or why that was
removed, but clearly it isn't used.

Can you please fix the drivers using it (b43, b43legacy, iwlwifi, p54,
rt2x00, rtl8180, rtl8187) to use the bss_conf's use_short_slot instead?
I'll fix mac80211 to set that appropriately.

johannes
Permalink | Reply |
headers
Ivo van Doorn | 7 Oct 19:57

Re: mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

Hi,

> I just realised that for whatever reason mac80211 isn't setting the
> IEEE80211_CONF_SHORT_SLOT_TIME flag. I'm not sure when or why that was
> removed, but clearly it isn't used.
> 
> Can you please fix the drivers using it (b43, b43legacy, iwlwifi, p54,
> rt2x00, rtl8180, rtl8187) to use the bss_conf's use_short_slot instead?
> I'll fix mac80211 to set that appropriately.

Didn't you have such a patch for rt2x00 already?
If not I'll see what I can do in a few days.

Ivo

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@...
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Permalink | Reply |
headers
Johannes Berg | 7 Oct 20:03
Favicon

Re: mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

On Tue, 2008-10-07 at 19:57 +0200, Ivo van Doorn wrote:
> Hi,
> 
> > I just realised that for whatever reason mac80211 isn't setting the
> > IEEE80211_CONF_SHORT_SLOT_TIME flag. I'm not sure when or why that was
> > removed, but clearly it isn't used.
> > 
> > Can you please fix the drivers using it (b43, b43legacy, iwlwifi, p54,
> > rt2x00, rtl8180, rtl8187) to use the bss_conf's use_short_slot instead?
> > I'll fix mac80211 to set that appropriately.
> 
> Didn't you have such a patch for rt2x00 already?

I had something a long time ago, but it never worked out right. I  still
have it in my patches dir as *-BROKEN-mac80211-bss-slot-time.patch.

> If not I'll see what I can do in a few days.

Thanks,
Johannes
Permalink | Reply |
headers
Herton Ronaldo Krzesinski | 8 Oct 03:14
Favicon

Re: mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

On Tuesday 07 October 2008 06:05:24 Johannes Berg wrote:
> Hi,
>
> I just realised that for whatever reason mac80211 isn't setting the
> IEEE80211_CONF_SHORT_SLOT_TIME flag. I'm not sure when or why that was
> removed, but clearly it isn't used.
>
> Can you please fix the drivers using it (b43, b43legacy, iwlwifi, p54,
> rt2x00, rtl8180, rtl8187) to use the bss_conf's use_short_slot instead?
> I'll fix mac80211 to set that appropriately.

About 8187 and 8187b, I think the changes below may handle it, I have yet to 
test them, just posting to have some feedback in case there is something 
missing or wrong. I'll try to review the other things too (like 
use_short_preamble warned in the other thread):

diff --git a/drivers/net/wireless/rtl8187_dev.c 
b/drivers/net/wireless/rtl8187_dev.c
index e990261..7f1cac8 100644
--- a/drivers/net/wireless/rtl8187_dev.c
+++ b/drivers/net/wireless/rtl8187_dev.c
@@ -870,6 +870,32 @@ static void rtl8187_remove_interface(struct ieee80211_hw 
*dev,
 	mutex_unlock(&priv->conf_mutex);
 }

+static void rtl8187_conf_slot(struct rtl8187_priv *priv, bool use_short_slot)
+{
+	if (priv->is_rtl8187b) {
+		if (use_short_slot) {
(Continue reading)

Permalink | Reply |
headers
Herton Ronaldo Krzesinski | 8 Oct 05:29
Favicon

Re: mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

On Tuesday 07 October 2008 22:14:12 Herton Ronaldo Krzesinski wrote:
> On Tuesday 07 October 2008 06:05:24 Johannes Berg wrote:
> > Hi,
> >
> > I just realised that for whatever reason mac80211 isn't setting the
> > IEEE80211_CONF_SHORT_SLOT_TIME flag. I'm not sure when or why that was
> > removed, but clearly it isn't used.
> >
> > Can you please fix the drivers using it (b43, b43legacy, iwlwifi, p54,
> > rt2x00, rtl8180, rtl8187) to use the bss_conf's use_short_slot instead?
> > I'll fix mac80211 to set that appropriately.
>
> About 8187 and 8187b, I think the changes below may handle it, I have yet
> to test them, just posting to have some feedback in case there is something
> missing or wrong. I'll try to review the other things too (like
> use_short_preamble warned in the other thread):
>
> diff --git a/drivers/net/wireless/rtl8187_dev.c
> b/drivers/net/wireless/rtl8187_dev.c
> index e990261..7f1cac8 100644
> --- a/drivers/net/wireless/rtl8187_dev.c
> +++ b/drivers/net/wireless/rtl8187_dev.c
> @@ -870,6 +870,32 @@ static void rtl8187_remove_interface(struct
> ieee80211_hw *dev,
>  	mutex_unlock(&priv->conf_mutex);
>  }
>
> +static void rtl8187_conf_slot(struct rtl8187_priv *priv, bool
> use_short_slot) +{
> +	if (priv->is_rtl8187b) {
(Continue reading)

Permalink | Reply |
headers
Johannes Berg | 8 Oct 10:44
Favicon

Re: mac80211 and IEEE80211_CONF_SHORT_SLOT_TIME

On Tue, 2008-10-07 at 22:14 -0300, Herton Ronaldo Krzesinski wrote:

> About 8187 and 8187b, I think the changes below may handle it, I have yet to 
> test them, just posting to have some feedback in case there is something 
> missing or wrong. I'll try to review the other things too (like 
> use_short_preamble warned in the other thread):

Looks ok to me, except for the stuff you yourself pointed out.

> The values used for SLOT etc. are kept for 8187, for 8187b I got the values 
> from realtek GPL driver (nothing is specified in the datasheet).

They seem to be very close to just the spec values in usecs, so maybe
double-check 802.11-2007, I don't know the values off the top of my
head.

johannes
Permalink | Reply |

Gmane