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

Used UAPI kernel headers instead of using a copy #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

quic-cgettibo
Copy link
Contributor

No description provided.

README.md Outdated
git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
cd linux/
make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- defconfig
make headers_install INSTALL_HDR_PATH=<Path to install headers>
Copy link
Contributor

@quic-bkumar quic-bkumar May 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use , also can you move this just below "Build and installation" and call it as "pre-requisites" for build and install.

configure.ac Outdated
if test -d "$HEADER_PATH"; then
AC_MSG_RESULT([Header found])
else
AC_MSG_ERROR([Header directory $HEADER_PATH not found])
Copy link
Contributor

@quic-bkumar quic-bkumar May 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use "Linux kernel headers not found in #HEADER_PATH"

configure.ac Outdated
AC_ARG_WITH([kernel_header], [AS_HELP_STRING([--with-kernel_header=PATH], [Specify path to the header])], [HEADER_PATH="$withval"],[AC_MSG_ERROR([Header path is required])])
# Checking if the header path is valid
if test -d "$HEADER_PATH"; then
AC_MSG_RESULT([Header found])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use "Linux kernel header found"

Copy link
Contributor

@quic-bkumar quic-bkumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left few comments.

Signed-off-by: Gettiboina Chanikya Prakash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants