On 2-3-2017 17:38, Arnd Bergmann wrote:
> With KASAN and a couple of other patches applied, this driver is one
> of the few remaining ones that actually use more than 2048 bytes of
> kernel stack:
> 
> broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 
> 'wlc_phy_workarounds_nphy_gainctrl':
> broadcom/brcm80211/brcmsmac/phy/phy_n.c:16065:1: warning: the frame size of 
> 3264 bytes is larger than 2048 bytes [-Wframe-larger-than=]
> broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 
> 'wlc_phy_workarounds_nphy':
> broadcom/brcm80211/brcmsmac/phy/phy_n.c:17138:1: warning: the frame size of 
> 2864 bytes is larger than 2048 bytes [-Wframe-larger-than=]
> 
> Here, I'm reducing the stack size by marking as many local variables as
> 'static const' as I can without changing the actual code.

Acked-by: Arend van Spriel <arend.vanspr...@broadcom.com>
> Signed-off-by: Arnd Bergmann <a...@arndb.de>
> ---
>  .../broadcom/brcm80211/brcmsmac/phy/phy_n.c        | 197 
> ++++++++++-----------
>  1 file changed, 97 insertions(+), 100 deletions(-)
> 
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c 
> b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
> index 42dc8e1f483d..48a4df488d75 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
> @@ -14764,8 +14764,8 @@ static void 
> wlc_phy_ipa_restore_tx_digi_filts_nphy(struct brcms_phy *pi)
>  }
>  
>  static void
> -wlc_phy_set_rfseq_nphy(struct brcms_phy *pi, u8 cmd, u8 *events, u8 *dlys,
> -                    u8 len)
> +wlc_phy_set_rfseq_nphy(struct brcms_phy *pi, u8 cmd, const u8 *events,
> +                    const u8 *dlys, u8 len)
>  {
>       u32 t1_offset, t2_offset;
>       u8 ctr;
> @@ -15240,16 +15240,16 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl_2057_rev5(struct brcms_phy *pi)
>  static void wlc_phy_workarounds_nphy_gainctrl_2057_rev6(struct brcms_phy *pi)
>  {
>       u16 currband;
> -     s8 lna1G_gain_db_rev7[] = { 9, 14, 19, 24 };
> -     s8 *lna1_gain_db = NULL;
> -     s8 *lna1_gain_db_2 = NULL;
> -     s8 *lna2_gain_db = NULL;
> -     s8 tiaA_gain_db_rev7[] = { -9, -6, -3, 0, 3, 3, 3, 3, 3, 3 };
> -     s8 *tia_gain_db;
> -     s8 tiaA_gainbits_rev7[] = { 0, 1, 2, 3, 4, 4, 4, 4, 4, 4 };
> -     s8 *tia_gainbits;
> -     u16 rfseqA_init_gain_rev7[] = { 0x624f, 0x624f };
> -     u16 *rfseq_init_gain;
> +     static const s8 lna1G_gain_db_rev7[] = { 9, 14, 19, 24 };
> +     const s8 *lna1_gain_db = NULL;
> +     const s8 *lna1_gain_db_2 = NULL;
> +     const s8 *lna2_gain_db = NULL;
> +     static const s8 tiaA_gain_db_rev7[] = { -9, -6, -3, 0, 3, 3, 3, 3, 3, 3 
> };
> +     const s8 *tia_gain_db;
> +     static const s8 tiaA_gainbits_rev7[] = { 0, 1, 2, 3, 4, 4, 4, 4, 4, 4 };
> +     const s8 *tia_gainbits;
> +     static const u16 rfseqA_init_gain_rev7[] = { 0x624f, 0x624f };
> +     const u16 *rfseq_init_gain;
>       u16 init_gaincode;
>       u16 clip1hi_gaincode;
>       u16 clip1md_gaincode = 0;
> @@ -15310,10 +15310,9 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl_2057_rev6(struct brcms_phy *pi)
>  
>                       if ((freq <= 5080) || (freq == 5825)) {
>  
> -                             s8 lna1A_gain_db_rev7[] = { 11, 16, 20, 24 };
> -                             s8 lna1A_gain_db_2_rev7[] = {
> -                                     11, 17, 22, 25};
> -                             s8 lna2A_gain_db_rev7[] = { -1, 6, 10, 14 };
> +                             static const s8 lna1A_gain_db_rev7[] = { 11, 
> 16, 20, 24 };
> +                             static const s8 lna1A_gain_db_2_rev7[] = { 11, 
> 17, 22, 25};
> +                             static const s8 lna2A_gain_db_rev7[] = { -1, 6, 
> 10, 14 };
>  
>                               crsminu_th = 0x3e;
>                               lna1_gain_db = lna1A_gain_db_rev7;
> @@ -15321,10 +15320,9 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl_2057_rev6(struct brcms_phy *pi)
>                               lna2_gain_db = lna2A_gain_db_rev7;
>                       } else if ((freq >= 5500) && (freq <= 5700)) {
>  
> -                             s8 lna1A_gain_db_rev7[] = { 11, 17, 21, 25 };
> -                             s8 lna1A_gain_db_2_rev7[] = {
> -                                     12, 18, 22, 26};
> -                             s8 lna2A_gain_db_rev7[] = { 1, 8, 12, 16 };
> +                             static const s8 lna1A_gain_db_rev7[] = { 11, 
> 17, 21, 25 };
> +                             static const s8 lna1A_gain_db_2_rev7[] = { 12, 
> 18, 22, 26};
> +                             static const s8 lna2A_gain_db_rev7[] = { 1, 8, 
> 12, 16 };
>  
>                               crsminu_th = 0x45;
>                               clip1md_gaincode_B = 0x14;
> @@ -15335,10 +15333,9 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl_2057_rev6(struct brcms_phy *pi)
>                               lna2_gain_db = lna2A_gain_db_rev7;
>                       } else {
>  
> -                             s8 lna1A_gain_db_rev7[] = { 12, 18, 22, 26 };
> -                             s8 lna1A_gain_db_2_rev7[] = {
> -                                     12, 18, 22, 26};
> -                             s8 lna2A_gain_db_rev7[] = { -1, 6, 10, 14 };
> +                             static const s8 lna1A_gain_db_rev7[] = { 12, 
> 18, 22, 26 };
> +                             static const s8 lna1A_gain_db_2_rev7[] = { 12, 
> 18, 22, 26};
> +                             static const s8 lna2A_gain_db_rev7[] = { -1, 6, 
> 10, 14 };
>  
>                               crsminu_th = 0x41;
>                               lna1_gain_db = lna1A_gain_db_rev7;
> @@ -15450,65 +15447,65 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl(struct brcms_phy *pi)
>               NPHY_RFSEQ_CMD_CLR_HIQ_DIS,
>               NPHY_RFSEQ_CMD_SET_HPF_BW
>       };
> -     u8 rfseq_updategainu_dlys[] = { 10, 30, 1 };
> -     s8 lna1G_gain_db[] = { 7, 11, 16, 23 };
> -     s8 lna1G_gain_db_rev4[] = { 8, 12, 17, 25 };
> -     s8 lna1G_gain_db_rev5[] = { 9, 13, 18, 26 };
> -     s8 lna1G_gain_db_rev6[] = { 8, 13, 18, 25 };
> -     s8 lna1G_gain_db_rev6_224B0[] = { 10, 14, 19, 27 };
> -     s8 lna1A_gain_db[] = { 7, 11, 17, 23 };
> -     s8 lna1A_gain_db_rev4[] = { 8, 12, 18, 23 };
> -     s8 lna1A_gain_db_rev5[] = { 6, 10, 16, 21 };
> -     s8 lna1A_gain_db_rev6[] = { 6, 10, 16, 21 };
> -     s8 *lna1_gain_db = NULL;
> -     s8 lna2G_gain_db[] = { -5, 6, 10, 14 };
> -     s8 lna2G_gain_db_rev5[] = { -3, 7, 11, 16 };
> -     s8 lna2G_gain_db_rev6[] = { -5, 6, 10, 14 };
> -     s8 lna2G_gain_db_rev6_224B0[] = { -5, 6, 10, 15 };
> -     s8 lna2A_gain_db[] = { -6, 2, 6, 10 };
> -     s8 lna2A_gain_db_rev4[] = { -5, 2, 6, 10 };
> -     s8 lna2A_gain_db_rev5[] = { -7, 0, 4, 8 };
> -     s8 lna2A_gain_db_rev6[] = { -7, 0, 4, 8 };
> -     s8 *lna2_gain_db = NULL;
> -     s8 tiaG_gain_db[] = {
> +     static const u8 rfseq_updategainu_dlys[] = { 10, 30, 1 };
> +     static const s8 lna1G_gain_db[] = { 7, 11, 16, 23 };
> +     static const s8 lna1G_gain_db_rev4[] = { 8, 12, 17, 25 };
> +     static const s8 lna1G_gain_db_rev5[] = { 9, 13, 18, 26 };
> +     static const s8 lna1G_gain_db_rev6[] = { 8, 13, 18, 25 };
> +     static const s8 lna1G_gain_db_rev6_224B0[] = { 10, 14, 19, 27 };
> +     static const s8 lna1A_gain_db[] = { 7, 11, 17, 23 };
> +     static const s8 lna1A_gain_db_rev4[] = { 8, 12, 18, 23 };
> +     static const s8 lna1A_gain_db_rev5[] = { 6, 10, 16, 21 };
> +     static const s8 lna1A_gain_db_rev6[] = { 6, 10, 16, 21 };
> +     const s8 *lna1_gain_db = NULL;
> +     static const s8 lna2G_gain_db[] = { -5, 6, 10, 14 };
> +     static const s8 lna2G_gain_db_rev5[] = { -3, 7, 11, 16 };
> +     static const s8 lna2G_gain_db_rev6[] = { -5, 6, 10, 14 };
> +     static const s8 lna2G_gain_db_rev6_224B0[] = { -5, 6, 10, 15 };
> +     static const s8 lna2A_gain_db[] = { -6, 2, 6, 10 };
> +     static const s8 lna2A_gain_db_rev4[] = { -5, 2, 6, 10 };
> +     static const s8 lna2A_gain_db_rev5[] = { -7, 0, 4, 8 };
> +     static const s8 lna2A_gain_db_rev6[] = { -7, 0, 4, 8 };
> +     const s8 *lna2_gain_db = NULL;
> +     static const s8 tiaG_gain_db[] = {
>               0x0A, 0x0A, 0x0A, 0x0A, 0x0A, 0x0A, 0x0A, 0x0A, 0x0A, 0x0A };
> -     s8 tiaA_gain_db[] = {
> +     static const s8 tiaA_gain_db[] = {
>               0x13, 0x13, 0x13, 0x13, 0x13, 0x13, 0x13, 0x13, 0x13, 0x13 };
> -     s8 tiaA_gain_db_rev4[] = {
> +     static const s8 tiaA_gain_db_rev4[] = {
>               0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d };
> -     s8 tiaA_gain_db_rev5[] = {
> +     static const s8 tiaA_gain_db_rev5[] = {
>               0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d };
> -     s8 tiaA_gain_db_rev6[] = {
> +     static const s8 tiaA_gain_db_rev6[] = {
>               0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d, 0x0d };
> -     s8 *tia_gain_db;
> -     s8 tiaG_gainbits[] = {
> +     const s8 *tia_gain_db;
> +     static const s8 tiaG_gainbits[] = {
>               0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x03, 0x03 };
> -     s8 tiaA_gainbits[] = {
> +     static const s8 tiaA_gainbits[] = {
>               0x06, 0x06, 0x06, 0x06, 0x06, 0x06, 0x06, 0x06, 0x06, 0x06 };
> -     s8 tiaA_gainbits_rev4[] = {
> +     static const s8 tiaA_gainbits_rev4[] = {
>               0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04 };
> -     s8 tiaA_gainbits_rev5[] = {
> +     static const s8 tiaA_gainbits_rev5[] = {
>               0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04 };
> -     s8 tiaA_gainbits_rev6[] = {
> +     static const s8 tiaA_gainbits_rev6[] = {
>               0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04, 0x04 };
> -     s8 *tia_gainbits;
> -     s8 lpf_gain_db[] = { 0x00, 0x06, 0x0c, 0x12, 0x12, 0x12 };
> -     s8 lpf_gainbits[] = { 0x00, 0x01, 0x02, 0x03, 0x03, 0x03 };
> -     u16 rfseqG_init_gain[] = { 0x613f, 0x613f, 0x613f, 0x613f };
> -     u16 rfseqG_init_gain_rev4[] = { 0x513f, 0x513f, 0x513f, 0x513f };
> -     u16 rfseqG_init_gain_rev5[] = { 0x413f, 0x413f, 0x413f, 0x413f };
> -     u16 rfseqG_init_gain_rev5_elna[] = {
> +     const s8 *tia_gainbits;
> +     static const s8 lpf_gain_db[] = { 0x00, 0x06, 0x0c, 0x12, 0x12, 0x12 };
> +     static const s8 lpf_gainbits[] = { 0x00, 0x01, 0x02, 0x03, 0x03, 0x03 };
> +     static const u16 rfseqG_init_gain[] = { 0x613f, 0x613f, 0x613f, 0x613f 
> };
> +     static const u16 rfseqG_init_gain_rev4[] = { 0x513f, 0x513f, 0x513f, 
> 0x513f };
> +     static const u16 rfseqG_init_gain_rev5[] = { 0x413f, 0x413f, 0x413f, 
> 0x413f };
> +     static const u16 rfseqG_init_gain_rev5_elna[] = {
>               0x013f, 0x013f, 0x013f, 0x013f };
> -     u16 rfseqG_init_gain_rev6[] = { 0x513f, 0x513f };
> -     u16 rfseqG_init_gain_rev6_224B0[] = { 0x413f, 0x413f };
> -     u16 rfseqG_init_gain_rev6_elna[] = { 0x113f, 0x113f };
> -     u16 rfseqA_init_gain[] = { 0x516f, 0x516f, 0x516f, 0x516f };
> -     u16 rfseqA_init_gain_rev4[] = { 0x614f, 0x614f, 0x614f, 0x614f };
> -     u16 rfseqA_init_gain_rev4_elna[] = {
> +     static const u16 rfseqG_init_gain_rev6[] = { 0x513f, 0x513f };
> +     static const u16 rfseqG_init_gain_rev6_224B0[] = { 0x413f, 0x413f };
> +     static const u16 rfseqG_init_gain_rev6_elna[] = { 0x113f, 0x113f };
> +     static const u16 rfseqA_init_gain[] = { 0x516f, 0x516f, 0x516f, 0x516f 
> };
> +     static const u16 rfseqA_init_gain_rev4[] = { 0x614f, 0x614f, 0x614f, 
> 0x614f };
> +     static const u16 rfseqA_init_gain_rev4_elna[] = {
>               0x314f, 0x314f, 0x314f, 0x314f };
> -     u16 rfseqA_init_gain_rev5[] = { 0x714f, 0x714f, 0x714f, 0x714f };
> -     u16 rfseqA_init_gain_rev6[] = { 0x714f, 0x714f };
> -     u16 *rfseq_init_gain;
> +     static const u16 rfseqA_init_gain_rev5[] = { 0x714f, 0x714f, 0x714f, 
> 0x714f };
> +     static const u16 rfseqA_init_gain_rev6[] = { 0x714f, 0x714f };
> +     const u16 *rfseq_init_gain;
>       u16 initG_gaincode = 0x627e;
>       u16 initG_gaincode_rev4 = 0x527e;
>       u16 initG_gaincode_rev5 = 0x427e;
> @@ -15538,10 +15535,10 @@ static void 
> wlc_phy_workarounds_nphy_gainctrl(struct brcms_phy *pi)
>       u16 clip1mdA_gaincode_rev6 = 0x2084;
>       u16 clip1md_gaincode = 0;
>       u16 clip1loG_gaincode = 0x0074;
> -     u16 clip1loG_gaincode_rev5[] = {
> +     static const u16 clip1loG_gaincode_rev5[] = {
>               0x0062, 0x0064, 0x006a, 0x106a, 0x106c, 0x1074, 0x107c, 0x207c
>       };
> -     u16 clip1loG_gaincode_rev6[] = {
> +     static const u16 clip1loG_gaincode_rev6[] = {
>               0x106a, 0x106c, 0x1074, 0x107c, 0x007e, 0x107e, 0x207e, 0x307e
>       };
>       u16 clip1loG_gaincode_rev6_224B0 = 0x1074;
> @@ -16066,7 +16063,7 @@ static void wlc_phy_workarounds_nphy_gainctrl(struct 
> brcms_phy *pi)
>  
>  static void wlc_phy_workarounds_nphy(struct brcms_phy *pi)
>  {
> -     u8 rfseq_rx2tx_events[] = {
> +     static const u8 rfseq_rx2tx_events[] = {
>               NPHY_RFSEQ_CMD_NOP,
>               NPHY_RFSEQ_CMD_RXG_FBW,
>               NPHY_RFSEQ_CMD_TR_SWITCH,
> @@ -16076,7 +16073,7 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>               NPHY_RFSEQ_CMD_EXT_PA
>       };
>       u8 rfseq_rx2tx_dlys[] = { 8, 6, 6, 2, 4, 60, 1 };
> -     u8 rfseq_tx2rx_events[] = {
> +     static const u8 rfseq_tx2rx_events[] = {
>               NPHY_RFSEQ_CMD_NOP,
>               NPHY_RFSEQ_CMD_EXT_PA,
>               NPHY_RFSEQ_CMD_TX_GAIN,
> @@ -16085,8 +16082,8 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>               NPHY_RFSEQ_CMD_RXG_FBW,
>               NPHY_RFSEQ_CMD_CLR_HIQ_DIS
>       };
> -     u8 rfseq_tx2rx_dlys[] = { 8, 6, 2, 4, 4, 6, 1 };
> -     u8 rfseq_tx2rx_events_rev3[] = {
> +     static const u8 rfseq_tx2rx_dlys[] = { 8, 6, 2, 4, 4, 6, 1 };
> +     static const u8 rfseq_tx2rx_events_rev3[] = {
>               NPHY_REV3_RFSEQ_CMD_EXT_PA,
>               NPHY_REV3_RFSEQ_CMD_INT_PA_PU,
>               NPHY_REV3_RFSEQ_CMD_TX_GAIN,
> @@ -16096,7 +16093,7 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>               NPHY_REV3_RFSEQ_CMD_CLR_HIQ_DIS,
>               NPHY_REV3_RFSEQ_CMD_END
>       };
> -     u8 rfseq_tx2rx_dlys_rev3[] = { 8, 4, 2, 2, 4, 4, 6, 1 };
> +     static const u8 rfseq_tx2rx_dlys_rev3[] = { 8, 4, 2, 2, 4, 4, 6, 1 };
>       u8 rfseq_rx2tx_events_rev3[] = {
>               NPHY_REV3_RFSEQ_CMD_NOP,
>               NPHY_REV3_RFSEQ_CMD_RXG_FBW,
> @@ -16110,7 +16107,7 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>       };
>       u8 rfseq_rx2tx_dlys_rev3[] = { 8, 6, 6, 4, 4, 18, 42, 1, 1 };
>  
> -     u8 rfseq_rx2tx_events_rev3_ipa[] = {
> +     static const u8 rfseq_rx2tx_events_rev3_ipa[] = {
>               NPHY_REV3_RFSEQ_CMD_NOP,
>               NPHY_REV3_RFSEQ_CMD_RXG_FBW,
>               NPHY_REV3_RFSEQ_CMD_TR_SWITCH,
> @@ -16121,15 +16118,15 @@ static void wlc_phy_workarounds_nphy(struct 
> brcms_phy *pi)
>               NPHY_REV3_RFSEQ_CMD_INT_PA_PU,
>               NPHY_REV3_RFSEQ_CMD_END
>       };
> -     u8 rfseq_rx2tx_dlys_rev3_ipa[] = { 8, 6, 6, 4, 4, 16, 43, 1, 1 };
> -     u16 rfseq_rx2tx_dacbufpu_rev7[] = { 0x10f, 0x10f };
> +     static const u8 rfseq_rx2tx_dlys_rev3_ipa[] = { 8, 6, 6, 4, 4, 16, 43, 
> 1, 1 };
> +     static const u16 rfseq_rx2tx_dacbufpu_rev7[] = { 0x10f, 0x10f };
>  
>       s16 alpha0, alpha1, alpha2;
>       s16 beta0, beta1, beta2;
>       u32 leg_data_weights, ht_data_weights, nss1_data_weights,
>           stbc_data_weights;
>       u8 chan_freq_range = 0;
> -     u16 dac_control = 0x0002;
> +     static const u16 dac_control = 0x0002;
>       u16 aux_adc_vmid_rev7_core0[] = { 0x8e, 0x96, 0x96, 0x96 };
>       u16 aux_adc_vmid_rev7_core1[] = { 0x8f, 0x9f, 0x9f, 0x96 };
>       u16 aux_adc_vmid_rev4[] = { 0xa2, 0xb4, 0xb4, 0x89 };
> @@ -16139,8 +16136,8 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>       u16 aux_adc_gain_rev4[] = { 0x02, 0x02, 0x02, 0x00 };
>       u16 aux_adc_gain_rev3[] = { 0x02, 0x02, 0x02, 0x00 };
>       u16 *aux_adc_gain;
> -     u16 sk_adc_vmid[] = { 0xb4, 0xb4, 0xb4, 0x24 };
> -     u16 sk_adc_gain[] = { 0x02, 0x02, 0x02, 0x02 };
> +     static const u16 sk_adc_vmid[] = { 0xb4, 0xb4, 0xb4, 0x24 };
> +     static const u16 sk_adc_gain[] = { 0x02, 0x02, 0x02, 0x02 };
>       s32 min_nvar_val = 0x18d;
>       s32 min_nvar_offset_6mbps = 20;
>       u8 pdetrange;
> @@ -16151,9 +16148,9 @@ static void wlc_phy_workarounds_nphy(struct brcms_phy 
> *pi)
>       u16 rfseq_rx2tx_lpf_h_hpc_rev7 = 0x77;
>       u16 rfseq_tx2rx_lpf_h_hpc_rev7 = 0x77;
>       u16 rfseq_pktgn_lpf_h_hpc_rev7 = 0x77;
> -     u16 rfseq_htpktgn_lpf_hpc_rev7[] = { 0x77, 0x11, 0x11 };
> -     u16 rfseq_pktgn_lpf_hpc_rev7[] = { 0x11, 0x11 };
> -     u16 rfseq_cckpktgn_lpf_hpc_rev7[] = { 0x11, 0x11 };
> +     static const u16 rfseq_htpktgn_lpf_hpc_rev7[] = { 0x77, 0x11, 0x11 };
> +     static const u16 rfseq_pktgn_lpf_hpc_rev7[] = { 0x11, 0x11 };
> +     static const u16 rfseq_cckpktgn_lpf_hpc_rev7[] = { 0x11, 0x11 };
>       u16 ipalvlshift_3p3_war_en = 0;
>       u16 rccal_bcap_val, rccal_scap_val;
>       u16 rccal_tx20_11b_bcap = 0;
> @@ -24291,13 +24288,13 @@ static void wlc_phy_update_txcal_ladder_nphy(struct 
> brcms_phy *pi, u16 core)
>       u16 bbmult;
>       u16 tblentry;
>  
> -     struct nphy_txiqcal_ladder ladder_lo[] = {
> +     static const struct nphy_txiqcal_ladder ladder_lo[] = {
>               {3, 0}, {4, 0}, {6, 0}, {9, 0}, {13, 0}, {18, 0},
>               {25, 0}, {25, 1}, {25, 2}, {25, 3}, {25, 4}, {25, 5},
>               {25, 6}, {25, 7}, {35, 7}, {50, 7}, {71, 7}, {100, 7}
>       };
>  
> -     struct nphy_txiqcal_ladder ladder_iq[] = {
> +     static const struct nphy_txiqcal_ladder ladder_iq[] = {
>               {3, 0}, {4, 0}, {6, 0}, {9, 0}, {13, 0}, {18, 0},
>               {25, 0}, {35, 0}, {50, 0}, {71, 0}, {100, 0}, {100, 1},
>               {100, 2}, {100, 3}, {100, 4}, {100, 5}, {100, 6}, {100, 7}
> @@ -25773,67 +25770,67 @@ wlc_phy_cal_txiqlo_nphy(struct brcms_phy *pi, 
> struct nphy_txgains target_gain,
>       u16 cal_gain[2];
>       struct nphy_iqcal_params cal_params[2];
>       u32 tbl_len;
> -     void *tbl_ptr;
> +     const void *tbl_ptr;
>       bool ladder_updated[2];
>       u8 mphase_cal_lastphase = 0;
>       int bcmerror = 0;
>       bool phyhang_avoid_state = false;
>  
> -     u16 tbl_tx_iqlo_cal_loft_ladder_20[] = {
> +     static const u16 tbl_tx_iqlo_cal_loft_ladder_20[] = {
>               0x0300, 0x0500, 0x0700, 0x0900, 0x0d00, 0x1100, 0x1900, 0x1901,
>               0x1902,
>               0x1903, 0x1904, 0x1905, 0x1906, 0x1907, 0x2407, 0x3207, 0x4607,
>               0x6407
>       };
>  
> -     u16 tbl_tx_iqlo_cal_iqimb_ladder_20[] = {
> +     static const u16 tbl_tx_iqlo_cal_iqimb_ladder_20[] = {
>               0x0200, 0x0300, 0x0600, 0x0900, 0x0d00, 0x1100, 0x1900, 0x2400,
>               0x3200,
>               0x4600, 0x6400, 0x6401, 0x6402, 0x6403, 0x6404, 0x6405, 0x6406,
>               0x6407
>       };
>  
> -     u16 tbl_tx_iqlo_cal_loft_ladder_40[] = {
> +     static const u16 tbl_tx_iqlo_cal_loft_ladder_40[] = {
>               0x0200, 0x0300, 0x0400, 0x0700, 0x0900, 0x0c00, 0x1200, 0x1201,
>               0x1202,
>               0x1203, 0x1204, 0x1205, 0x1206, 0x1207, 0x1907, 0x2307, 0x3207,
>               0x4707
>       };
>  
> -     u16 tbl_tx_iqlo_cal_iqimb_ladder_40[] = {
> +     static const u16 tbl_tx_iqlo_cal_iqimb_ladder_40[] = {
>               0x0100, 0x0200, 0x0400, 0x0700, 0x0900, 0x0c00, 0x1200, 0x1900,
>               0x2300,
>               0x3200, 0x4700, 0x4701, 0x4702, 0x4703, 0x4704, 0x4705, 0x4706,
>               0x4707
>       };
>  
> -     u16 tbl_tx_iqlo_cal_startcoefs[] = {
> +     static const u16 tbl_tx_iqlo_cal_startcoefs[] = {
>               0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>               0x0000
>       };
>  
> -     u16 tbl_tx_iqlo_cal_cmds_fullcal[] = {
> +     static const u16 tbl_tx_iqlo_cal_cmds_fullcal[] = {
>               0x8123, 0x8264, 0x8086, 0x8245, 0x8056,
>               0x9123, 0x9264, 0x9086, 0x9245, 0x9056
>       };
>  
> -     u16 tbl_tx_iqlo_cal_cmds_recal[] = {
> +     static const u16 tbl_tx_iqlo_cal_cmds_recal[] = {
>               0x8101, 0x8253, 0x8053, 0x8234, 0x8034,
>               0x9101, 0x9253, 0x9053, 0x9234, 0x9034
>       };
>  
> -     u16 tbl_tx_iqlo_cal_startcoefs_nphyrev3[] = {
> +     static const u16 tbl_tx_iqlo_cal_startcoefs_nphyrev3[] = {
>               0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>               0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
>               0x0000
>       };
>  
> -     u16 tbl_tx_iqlo_cal_cmds_fullcal_nphyrev3[] = {
> +     static const u16 tbl_tx_iqlo_cal_cmds_fullcal_nphyrev3[] = {
>               0x8434, 0x8334, 0x8084, 0x8267, 0x8056, 0x8234,
>               0x9434, 0x9334, 0x9084, 0x9267, 0x9056, 0x9234
>       };
>  
> -     u16 tbl_tx_iqlo_cal_cmds_recal_nphyrev3[] = {
> +     static const u16 tbl_tx_iqlo_cal_cmds_recal_nphyrev3[] = {
>               0x8423, 0x8323, 0x8073, 0x8256, 0x8045, 0x8223,
>               0x9423, 0x9323, 0x9073, 0x9256, 0x9045, 0x9223
>       };
> 

Reply via email to