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

asus-zephyrus-ga402: remove unused kernel parameter #625

Merged

Conversation

aacebedo
Copy link
Contributor

Description of changes

Removes a flag not required anymore

Things done
  • Tested the changes in your own NixOS Configuration
  • Tested the changes end-to-end by using your fork of nixos-hardware and
    importing it via <nixos-hardware> or Flake input

@aacebedo
Copy link
Contributor Author

@K900 can you check this too?

@K900
Copy link
Contributor

K900 commented May 10, 2023

We should probably force the latest kernel then.

@Mic92
Copy link
Member

Mic92 commented May 11, 2023

Maybe check for a lower kernel bound: #627 (comment)

@aacebedo
Copy link
Contributor Author

We should probably force the latest kernel then.

done using comments from @Mic92 (Thanks !)

@Mic92
Copy link
Member

Mic92 commented May 13, 2023

I will have a look on how to satisfy the unit tests with this, maybe I'll make the latest kernel the new default.

@aacebedo aacebedo force-pushed the aacebedo/ga502_clean_kernel_parameter branch from 1e254aa to 1f92439 Compare May 13, 2023 09:36
@aacebedo
Copy link
Contributor Author

I will have a look on how to satisfy the unit tests with this, maybe I'll make the latest kernel the new default.

fixed the missing config parameter inn the derivation

@aacebedo
Copy link
Contributor Author

I will have a look on how to satisfy the unit tests with this, maybe I'll make the latest kernel the new default.

Ok the problem is now the same as for MR #627. Wonder if we can change the kernel version used in the test or at least mock it.

@colemickens
Copy link
Member

I certainly don't use this kernel param on my ga402 and I don't have any issues AFAICT.

@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

queue

❌ Pull request must be rebased manually

The pull request can't be rebased without conflict and must be rebased manually

@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

queue

❌ Pull request must be rebased manually

The pull request can't be rebased without conflict and must be rebased manually

@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

rebase

✅ Branch has been successfully rebased

@Mic92 Mic92 force-pushed the aacebedo/ga502_clean_kernel_parameter branch from 1f92439 to 938f54b Compare September 19, 2023 05:30
@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

queue

❌ Pull request must be rebased manually

The pull request can't be rebased without conflict and must be rebased manually

@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

rebase

✅ Branch has been successfully rebased

@Mic92 Mic92 force-pushed the aacebedo/ga502_clean_kernel_parameter branch from 938f54b to dc8b25a Compare September 19, 2023 07:34
@Mic92
Copy link
Member

Mic92 commented Sep 19, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at cb4dc98

@mergify mergify bot merged commit cb4dc98 into NixOS:master Sep 19, 2023
3 checks passed
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.

4 participants