On Sat, Sep 17, 2022 at 12:06:56AM +0100, Daniel Golle wrote:
> +static void rt2800_loft_search(struct rt2x00_dev *rt2x00dev, u8 ch_idx,
> + u8 alc_idx, u8 dc_result[][RF_ALC_NUM][2])
> +{
> + u32 p0 = 0, p1 = 0, pf = 0;
> + char idx0 = 0, idx1 = 0;
> + u8 idxf[] = {0x00, 0x00};
> + u8 ibit = 0x20;
> + u8 iorq;
> + char bidx;
> +
> + rt2800_bbp_write(rt2x00dev, 158, 0xb0);
> + rt2800_bbp_write(rt2x00dev, 159, 0x80);
> +
> + for (bidx = 5; bidx >= 0; bidx--) {
> + for (iorq = 0; iorq <= 1; iorq++) {
> + rt2x00_dbg(rt2x00dev, "\n========================================================\n");
Is this useful for debugging ?
> +void rt2800_loft_iq_calibration(struct rt2x00_dev *rt2x00dev)
> +{
> + struct rf_reg_pair rf_store[CHAIN_NUM][13];
> + u32 macorg1 = 0;
> + u32 macorg2 = 0;
> + u32 macorg3 = 0;
> + u32 macorg4 = 0;
> + u32 macorg5 = 0;
> + u32 orig528 = 0;
> + u32 orig52c = 0;
> +
> + u32 savemacsysctrl = 0, mtxcycle = 0;
> + u32 macvalue = 0;
> + u32 mac13b8 = 0;
> + u32 p0 = 0, p1 = 0;
> + u32 p0_idx10 = 0, p1_idx10 = 0;
> +
> + u8 rfvalue;
> + u8 loft_dc_search_result[CHAIN_NUM][RF_ALC_NUM][2];
> + u8 ger[CHAIN_NUM], per[CHAIN_NUM];
> + u8 rf_gain[] = {0x00, 0x01, 0x02, 0x04, 0x08, 0x0c};
> + u8 rfvga_gain_table[] = {0x24, 0x25, 0x26, 0x27, 0x28, 0x2c, 0x2d, 0x2e, 0x2f, 0x30,
> + 0x31, 0x38, 0x39, 0x3a, 0x3b, 0x3c, 0x3d, 0x3e, 0x3F};
> + u8 vga_gain[] = {14, 14};
> + u8 bbp_2324gain[] = {0x16, 0x14, 0x12, 0x10, 0x0c, 0x08};
We should use static const for those.
> + u8 bbp = 0, ch_idx = 0, rf_alc_idx = 0, idx = 0;
> + u8 bbpr30, rfb0r39, rfb0r42;
> + u8 bbpr1;
> + u8 bbpr4;
> + u8 bbpr241, bbpr242;
> + u8 count_step;
This function one is also huge too and I think require a bit refacing
int smaller sub functions. Also utilize lot of stack, there are no
warnings about too much stack consuming ?
