Messages in this thread | | | Subject | Re: [PATCH] rtlwifi: rtl8821ae: make array static const and remove redundant assignment | From | Kalle Valo <> | Date | Fri, 13 Sep 2019 13:44:40 +0000 (UTC) |
| |
Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com> > > The array channel_all can be make static const rather than populating > it on the stack, this makes the code smaller. Also, variable place > is being initialized with a value that is never read, so this assignment > is redundant and can be removed. > > Before: > text data bss dec hex filename > 118537 9591 0 128128 1f480 realtek/rtlwifi/rtl8821ae/phy.o > > After: > text data bss dec hex filename > 118331 9687 0 128018 1f412 realtek/rtlwifi/rtl8821ae/phy.o > > Saves 110 bytes, (gcc version 9.2.1, amd64) > > Signed-off-by: Colin Ian King <colin.king@canonical.com>
Patch applied to wireless-drivers-next.git, thanks.
569ce0a486fd rtlwifi: rtl8821ae: make array static const and remove redundant assignment
-- https://patchwork.kernel.org/patch/11133295/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
| |