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

Linux 3.10 compatibility #35

Merged
merged 1 commit into from Jun 2, 2013
Merged

Linux 3.10 compatibility #35

merged 1 commit into from Jun 2, 2013

Conversation

nowrep
Copy link
Contributor

@nowrep nowrep commented Jun 2, 2013

create_proc_entry() is no longer available in 3.10
also fixed warning issue on x86_64

create_proc_entry() is no longer available in 3.10
also fixed warning issue on x86_64
avilella added a commit that referenced this pull request Jun 2, 2013
Linux 3.10 compatibility
@avilella avilella merged commit 8e86e8e into mkottman:master Jun 2, 2013
@nowrep
Copy link
Contributor Author

nowrep commented Jun 2, 2013

Oops, I overlooked this ..

diff --git a/acpi_call.c b/acpi_call.c
index 4afd9db..3025d97 100644
--- a/acpi_call.c
+++ b/acpi_call.c
@@ -309,9 +309,6 @@ static ssize_t acpi_proc_read( struct file *filp, char __user *buff,
     int len = strlen(result_buffer);

     // output the current result buffer
-    if (copy_to_user(buff, result_buffer, len + 1))
-        return -EFAULT;
-
     ret = simple_read_from_buffer(buff, count, off, result_buffer, len + 1);

     // initialize the result buffer for later

@avilella
Copy link
Collaborator

avilella commented Jun 7, 2013

Is this a pull request?

@nowrep
Copy link
Contributor Author

nowrep commented Jun 7, 2013

It's just a minor change, I'll prefer if you just commit it yourself.
But of course, I can open another pull request if you want.

@avilella
Copy link
Collaborator

avilella commented Jun 7, 2013

please do a pull request, I'll just accept it.

On Fri, Jun 7, 2013 at 4:12 PM, David Rosca notifications@github.comwrote:

It's just a minor change, I'll prefer if you just commit it yourself.
But of course, I can open another pull request if you want.


Reply to this email directly or view it on GitHubhttps://github.com//pull/35#issuecomment-19113021
.

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.

None yet

2 participants