Skip to content
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

Logical instead of bitwise and #1

Open
jrisom opened this issue Mar 12, 2013 · 1 comment
Open

Logical instead of bitwise and #1

jrisom opened this issue Mar 12, 2013 · 1 comment
Labels

Comments

@jrisom
Copy link

jrisom commented Mar 12, 2013

ar9300_misc.c contains an error discovered by clang:

/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_misc.c:853:38:
error: use of logical '&&' with constant operand
[-Werror,-Wconstant-logical-operand]
if ((ahp->ah_enterprise_mode && AR_ENT_OTP_MIN_PKT_SIZE_DISABLE) &&
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

erikarn referenced this issue in erikarn/qcamain_open_hal_public Mar 12, 2013
clang complains on it.

Issue:	#1
@erikarn
Copy link
Contributor

erikarn commented Mar 12, 2013

Hi!

I've fixed this in my branch; I'll fix it upstream soon.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants