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

fix qcom_cpu_kpssv2 compile on ASUS_AC1300 #776

Closed
wants to merge 1 commit into from

Conversation

igalic
Copy link
Contributor

@igalic igalic commented Jun 11, 2023

follow afdb429 and change the return type
from boolean_t to bool for the definition, too.

PR: 271932
Reported by: Mina Galić
Fixes: afdb429
Pull Request: #776

igalic added a commit to igalic/freebsd-src that referenced this pull request Jun 11, 2023
follow afdb429 and change the return
type from boolean_t to bool for the definition, too.

PR: 271932
Reported by: Mina Galić
Fixes: afdb429
Pull Request: freebsd#776
follow afdb429 and change the return
type from boolean_t to bool for the definition, too.

PR: 271932
Reported by: Mina Galić
Fixes: afdb429
Pull Request: freebsd#776
@bsdimp
Copy link
Member

bsdimp commented Jun 12, 2023

This looks good. Need to tweak the commit message a little (project custom is to not use reported by when you also are fixing the bug) and I reworded the commit message to use only 12 of the characters of the hash and to make it flow a little better. 3ed646d

@bsdimp bsdimp closed this Jun 12, 2023
freebsd-git pushed a commit that referenced this pull request Jun 12, 2023
Complete afdb429 by changing the return type from boolean_t to bool
for the definition for qcom_cpu_kpssv2_regulator_start.

PR: 271932
Fixes: afdb429
Pull Request: #776
@bsdimp bsdimp added the merged label Jun 12, 2023
@igalic
Copy link
Contributor Author

igalic commented Jun 12, 2023

tested with:

poudriere jail -b -c -x -a arm.armv7 -j 14-ctest-armv7 -K ASUS_AC1300 -m git+https -U https://github.com/igalic/freebsd-src.git -v fix/ASUS_AC1300-compile

works.

(merged in 3ed646d )

@igalic igalic deleted the fix/ASUS_AC1300-compile branch June 12, 2023 14:49
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Aug 14, 2023
Complete afdb429 by changing the return type from boolean_t to bool
for the definition for qcom_cpu_kpssv2_regulator_start.

PR: 271932
Fixes: afdb429
Pull Request: freebsd/freebsd-src#776
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants