-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix issues that trigger compiler warnings with gcc 7.2.1 #7
base: aosp/LA.UM.5.7.r1
Are you sure you want to change the base?
Conversation
Small issues: Misleading indentation looking like an if statement ends in the wrong place, checking a variable of type bool for values > true (impossible), etc. The only change in here that has any effect other than silencing warnings and reducing the headaches of someone trying to read the code is changing the type of enablePowersaveOffload in struct hdd_config: According to the code in wlan_hdd_main.c, it is clearly meant to take 3 different values (0/nodeepsleep/deepsleep), so it can't be bool. Another option may be to keep it as bool and unify 0 and nodeepsleep into false and changing deepsleep to true Signed-off-by: Bernhard Rosenkränzer <bero@lindev.ch>
please do not add/ remove spaces because it will make things harder on the next rebase. |
Adding/changing the spaces is necessary to make the warnings (which the kernel's compiler wrapper script turns into errors) go away -- the indentation in those files is broken.
The other fix would be allowing those warnings in the wrapper script, but that would never be accepted in the upstream kernel. |
Where is the upstream for this code? Fixing the indentation issue there is preferable because it wouldn't cause problems during next rebase... |
the upstream is in code aurora and there is no way to submit code there |
please split the pacth is 2 separate patches |
please update you PR so that I can merge it |
Small issues: Misleading indentation looking like an if statement
ends in the wrong place, checking a variable of type bool for
values > true (impossible), etc.
The only change in here that has any effect other than silencing
warnings and reducing the headaches of someone trying to read
the code is changing the type of enablePowersaveOffload in
struct hdd_config: According to the code in wlan_hdd_main.c, it is
clearly meant to take 3 different values (0/nodeepsleep/deepsleep), so
it can't be bool.
Another option may be to keep it as bool and unify 0 and nodeepsleep
into false and changing deepsleep to true
Signed-off-by: Bernhard Rosenkränzer bero@lindev.ch