Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> wrote: > @@ -1607,7 +1607,7 @@ static u8 *rtw_build_rsvd_page(struct rtw_dev *rtwdev, u32 *size) > struct rtw_rsvd_page *rsvd_pkt; > u32 page = 0; > u8 total_page = 0; > - u8 page_size, page_margin, tx_desc_sz; > + u16 page_size, page_margin, tx_desc_sz; > u8 *buf; > int ret; In reverse X'mas tree order, but still keep struct at beginning. > > @@ -2013,7 +2013,8 @@ static int _rtw_hw_scan_update_probe_req(struct rtw_dev *rtwdev, u8 num_probes, > { > const struct rtw_chip_info *chip = rtwdev->chip; > struct sk_buff *skb, *tmp; > - u8 page_offset = 1, *buf, page_size = chip->page_size; > + u8 page_offset = 1, *buf; > + u16 page_size = chip->page_size; > u16 pg_addr = rtwdev->fifo.rsvd_h2c_info_addr, loc; > u16 buf_offset = page_size * page_offset; > u8 tx_desc_sz = chip->tx_pkt_desc_sz; ditto.