On Wed, Apr 6, 2011 at 9:13 PM, Luciano Coelho <coelho@xxxxxx> wrote:
> On Mon, 2011-04-04 at 10:07 +0300, Shahar Levi wrote:
>> Add support to FM WLAN coexistence (STA only).
>> Few WiFi harmonics may interfere FM operation, to
>> avoid this problem special coexistence techniques are
>> activated around some FM frequencies.
>>
>> Signed-off-by: Shahar Levi <shahar_levi@xxxxxx>
>> ---
>
> Some comments.
>
>
>> diff --git a/drivers/net/wireless/wl12xx/acx.c b/drivers/net/wireless/wl12xx/acx.c
>> index e005aa4..5f5afbe 100644
>> --- a/drivers/net/wireless/wl12xx/acx.c
>> +++ b/drivers/net/wireless/wl12xx/acx.c
>> @@ -1604,3 +1604,45 @@ out:
>> kfree(acx);
>> return ret;
>> }
>> +
>> +int wl1271_acx_fm_coex(struct wl1271 *wl)
>> +{
>> + struct wl1271_acx_fm_coex *acx;
>> + int ret;
>> +
>> + wl1271_debug(DEBUG_ACX, "acx fm coex setting");
>> +
>> + acx = kzalloc(sizeof(*acx), GFP_KERNEL);
>> + if (!acx) {
>> + ret = -ENOMEM;
>> + goto out;
>> + }
>> +
>> + acx->enable = wl->conf.fm_coex.enable;
>> + acx->swallow_period = wl->conf.fm_coex.swallow_period;
>> + acx->n_divider_fref_set_1 = wl->conf.fm_coex.n_divider_fref_set_1;
>> + acx->n_divider_fref_set_2 = wl->conf.fm_coex.n_divider_fref_set_2;
>> + acx->m_divider_fref_set_1 =
>> + cpu_to_le16(wl->conf.fm_coex.m_divider_fref_set_1);
>> + acx->m_divider_fref_set_2 =
>> + cpu_to_le16(wl->conf.fm_coex.m_divider_fref_set_2);
>> + acx->coex_pll_stabilization_time =
>> + cpu_to_le32(wl->conf.fm_coex.coex_pll_stabilization_time);
>
>> + acx->ldo_stabilization_timwl12xx-fm-coex-v2/e =
>> + cpu_to_le16(wl->conf.fm_coex.ldo_stabilization_time);
>
> What's this? There is some garbage pasted into this statement here!
How that get in? This is a typo of copy&paste of "wl12xx-fm-coex-v2/"
Will be fix.
>
>
>> diff --git a/drivers/net/wireless/wl12xx/acx.h b/drivers/net/wireless/wl12xx/acx.h
>> index 0a40cae..45242db 100644
>> --- a/drivers/net/wireless/wl12xx/acx.h
>> +++ b/drivers/net/wireless/wl12xx/acx.h
>> @@ -1172,6 +1172,65 @@ struct wl1271_acx_inconnection_sta {
>> u8 padding1[2];
>> } __packed;
>>
>> +/*
>> + * ACX_FM_COEX_CFG
>> + * set the FM co-existence parameters.
>> + */
>> +struct wl1271_acx_fm_coex {
>> + struct acx_header header;
>> + /* enable(1) / disable(0) the FM Coex feature */
>> + u8 enable;
>> + /*
>> + * Swallow period used in COEX PLL swallowing mechanism.
>> + * 0xFF = use FW default
>> + */
>> + u8 swallow_period;
>> + /*
>> + * The N divider used in COEX PLL swallowing mechanism for Fref of
>> + * 38.4/19.2 Mhz. 0xFF = use FW default
>> + */
>> + u8 n_divider_fref_set_1;
>> + /*
>> + * The N divider used in COEX PLL swallowing mechanism for Fref of
>> + * 26/52 Mhz. 0xFF = use FW default
>> + */
>> + u8 n_divider_fref_set_2;
>> + /*
>> + * The M divider used in COEX PLL swallowing mechanism for Fref of
>> + * 38.4/19.2 Mhz. 0xFFFF = use FW default
>> + */
>> + u16 m_divider_fref_set_1;
>
> This must be __le16.
Will be fix
>
>
>> + /*
>> + * The M divider used in COEX PLL swallowing mechanism for Fref of
>> + * 26/52 Mhz. 0xFFFF = use FW default
>> + */
>> + u16 m_divider_fref_set_2;
>
> Same here.
Will be fix
>
>
>> + /*
>> + * The time duration in uSec required for COEX PLL to stabilize.
>> + * 0xFFFFFFFF = use FW default
>> + */
>> + u32 coex_pll_stabilization_time;
>
> __le32
Will be fix
>
>
>> + /*
>> + * The time duration in uSec required for LDO to stabilize.
>> + * 0xFFFFFFFF = use FW default
>> + */
>> + u16 ldo_stabilization_time;
>
> __le16
Will be fix
>
>
> --
> Cheers,
> Luca.
--
All the best,
Shahar
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
