On Tue, 21 Jun 2011 16:36:38 -0500 Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > On 06/21/2011 01:53 PM, Hauke Mehrtens wrote: > > This fix was done according to si_clock_rate function in broadcom siutils.c > > > > Signed-off-by: Hauke Mehrtens<hauke@xxxxxxxxxx> > > --- > > drivers/ssb/main.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c > > index e568664..57b7b64 100644 > > --- a/drivers/ssb/main.c > > +++ b/drivers/ssb/main.c > > @@ -1002,8 +1002,8 @@ u32 ssb_calc_clock_rate(u32 plltype, u32 n, u32 m) > > switch (plltype) { > > case SSB_PLLTYPE_6: /* 100/200 or 120/240 only */ > > if (m& SSB_CHIPCO_CLK_T6_MMASK) > > - return SSB_CHIPCO_CLK_T6_M0; > > - return SSB_CHIPCO_CLK_T6_M1; > > + return SSB_CHIPCO_CLK_T6_M1; > > + return SSB_CHIPCO_CLK_T6_M0; > > case SSB_PLLTYPE_1: /* 48Mhz base, 3 dividers */ > > case SSB_PLLTYPE_3: /* 25Mhz, 2 dividers */ > > case SSB_PLLTYPE_4: /* 48Mhz, 4 dividers */ > > This patch sent me looking to find the public source for this routine. After > finding it in the source section of Broadcom-wl, I noticed there is some > additional code that is not implemented in the ssb version. Should this > additional material be implemented, or is it not applicable? could you quote that function here? That would be way easier to get what were talking about. -- 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