[PATCH v2] wl12xx: fix fm_coex parameters configuration

[PATCH v2] wl12xx: fix fm_coex parameters configuration

[Victor Goldenshtein]

Wrong fm_coex parameters were set during wl12xx
init phase, fix it.

Signed-off-by: Ziv Riesel <zivriesel@ti.com>
Signed-off-by: Victor Goldenshtein <victorg@ti.com>
---

v2: changed the subject wlcore -> wl12xx

 drivers/net/wireless/ti/wl12xx/main.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ti/wl12xx/main.c b/drivers/net/wireless/ti/wl12xx/main.c
index d7dd3de..5131375 100644
--- a/drivers/net/wireless/ti/wl12xx/main.c
+++ b/drivers/net/wireless/ti/wl12xx/main.c
@@ -305,8 +305,8 @@ static struct wlcore_conf wl12xx_conf = {
 		.swallow_period               = 5,
 		.n_divider_fref_set_1         = 0xff,       /* default */
 		.n_divider_fref_set_2         = 12,
-		.m_divider_fref_set_1         = 148,
-		.m_divider_fref_set_2         = 0xffff,     /* default */
+		.m_divider_fref_set_1         = 0xffff,
+		.m_divider_fref_set_2         = 148,	    /* default */
 		.coex_pll_stabilization_time  = 0xffffffff, /* default */
 		.ldo_stabilization_time       = 0xffff,     /* default */
 		.fm_disturbed_band_margin     = 0xff,       /* default */
-- 
1.7.5.4

Re: [PATCH v2] wl12xx: fix fm_coex parameters configuration

[Arik Nemtsov]

On Thu, May 10, 2012 at 5:08 PM, Victor Goldenshtein <victorg@ti.com> wrote:
> Wrong fm_coex parameters were set during wl12xx
> init phase, fix it.
>
> Signed-off-by: Ziv Riesel <zivriesel@ti.com>
> Signed-off-by: Victor Goldenshtein <victorg@ti.com>
> ---
>
> v2: changed the subject wlcore -> wl12xx

Are you sure this is not relevant for 18xx? Are the parameters there correct?

Arik

Re: [PATCH v2] wl12xx: fix fm_coex parameters configuration

[Arik Nemtsov]

On Mon, May 14, 2012 at 12:02 PM, Goldenshtein, Victor <victorg@ti.com> wrote:
> On Sat, May 12, 2012 at 10:17 PM, Arik Nemtsov <arik@wizery.com> wrote:
>>
>> On Thu, May 10, 2012 at 5:08 PM, Victor Goldenshtein <victorg@ti.com>
>> wrote:
>> > Wrong fm_coex parameters were set during wl12xx
>> > init phase, fix it.
>> >
>> > Signed-off-by: Ziv Riesel <zivriesel@ti.com>
>> > Signed-off-by: Victor Goldenshtein <victorg@ti.com>
>> > ---
>> >
>> > v2: changed the subject wlcore -> wl12xx
>>
>> Are you sure this is not relevant for 18xx? Are the parameters there
>> correct?
>>
>
> You're right, I will send a patch also for the 18xx. But I"m not sure
> which branch should I use ? To send also to Linux-Wireless ?
> Is the remotes/origin/18xx_upstream branch is good for upstream ?

Better send it rebased on the internal branch. I'll send it to linux
wireless later (together with other pending patches).

Arik

Re: [PATCH v2] wl12xx: fix fm_coex parameters configuration

[Luciano Coelho]

On Thu, 2012-05-10 at 17:08 +0300, Victor Goldenshtein wrote:
> Wrong fm_coex parameters were set during wl12xx
> init phase, fix it.
> 
> Signed-off-by: Ziv Riesel <zivriesel@ti.com>
> Signed-off-by: Victor Goldenshtein <victorg@ti.com>
> ---
> 
> v2: changed the subject wlcore -> wl12xx

Applied and pushed, thanks!

--
Luca.