nixos-hardware icon indicating copy to clipboard operation
nixos-hardware copied to clipboard

XPS 13 9310 question

Open seqizz opened this issue 4 years ago • 6 comments

Hi, I saw the related config but I didn't need any of those when I use boot.kernelPackages = pkgs.linuxPackages_latest;. So my question is, is this the same hardware? If not I'd also like to add config for this model, maybe suffixing with "2-in-1". My result is:

~> sudo dmidecode| grep -i xps
        Product Name: XPS 13 9310 2-in-1

Maybe @mitchmindtree ?

seqizz avatar Mar 27 '21 08:03 seqizz

Is it possible that these patches became upstream in one way or the other? Maybe we can drop the whole custom kernel thing.

Mic92 avatar Mar 28 '21 16:03 Mic92

Maybe you and @mitchmindtree could both post the wifi chipset that was used. Most patches seems to be related to this.

Mic92 avatar Mar 28 '21 16:03 Mic92

Aaah makes sense. Looks like there are 2 different variants of wifi on this model. Mine is indeed AX201:

00:14.3 Network controller: Intel Corporation Wi-Fi 6 AX201 (rev 20)

Since there is no "if" we can use, should I send my version with -ax201 suffix?

seqizz avatar Mar 28 '21 17:03 seqizz

@seqizz we might be better off creating a module option (see here for info about module structure) for specifying either one in the configuration.

mitchmindtree avatar Apr 04 '21 13:04 mitchmindtree

Honestly I don't have much experience with modules, will check it out. Hopefully there is an example in the repo already.

seqizz avatar Apr 07 '21 07:04 seqizz

I would add an option to enable AX500 support and deprecate it once mainline supports this wifi chipset.

This is an example for a module option:

https://github.com/NixOS/nixpkgs/blob/d9f9a4a9760ef3cf3df63771cebefacd1b0178b4/nixos/modules/hardware/keyboard/zsa.nix#L23

Mic92 avatar Apr 07 '21 08:04 Mic92