Skip to content

Add patches for 7.0#2079

Open
Apiznel wants to merge 5 commits into
linux-surface:masterfrom
Apiznel:master
Open

Add patches for 7.0#2079
Apiznel wants to merge 5 commits into
linux-surface:masterfrom
Apiznel:master

Conversation

@Apiznel

@Apiznel Apiznel commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

Changed 0x10 to 0x11 in 0013-cameras.patch as suggested here: #2057

Derived from latest 6.19 patches
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 1, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 2, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 2, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 2, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 7, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
@Apiznel

Apiznel commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

Rebased for: gregkh/linux@9c58e98

@hedmo

hedmo commented May 9, 2026

Copy link
Copy Markdown

no issues from my side (SB3).

MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 10, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 12, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
RevySR pushed a commit to RevySR/linux that referenced this pull request May 19, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 21, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
@orthogonaleety

orthogonaleety commented May 22, 2026

Copy link
Copy Markdown

27577cd & f4fd358 good for me for F43 (not signed), with kernel 7.0.7, installed against Fedora 43, Surface Pro 8
run https://github.com/orthogonaleety/linux-surface/actions/runs/26240487204
built with https://github.com/orthogonaleety/linux-surface/commits/f43-7-0-x
(Can't comment on #2079 (comment) as don't use IPU6 cameras )

MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 24, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
RevySR pushed a commit to RevySR/linux that referenced this pull request May 25, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 25, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 25, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
@cfouche3005

Copy link
Copy Markdown

Regression on Surface Book 1, no touchscreen (tested on NixOS via this PR) on kernel 7.0.9

@orthogonaleety

orthogonaleety commented May 26, 2026

Copy link
Copy Markdown

Regression on Surface Book 1, no touchscreen (tested on NixOS via this PR) on kernel 7.0.9

Hi @cfouche3005

Are you running NixOS stable or unstable?

Prior to using this PR:

  • What libwacom 'were' you using? libwacom-surface or libwacom (standard) ?
  • Which of those libwacom do you see in place now?
  • What Libwacom (standard) version would your NixOS be packaging if you were on standard kernel?

NixOS packages unstable lists libwacom 2.18. libwacom-surface is 2.17. So its plasuible libwacom-surface has been replaced, by libwacom standard (such as happens with Fedora upgrade 43 > 44) , and you will find downgrading to libwacom-surface 2.17 is not an option either.

i.e this matters because in that case the issue is not this PR per-say

@cfouche3005

Copy link
Copy Markdown

I am on full unstable
I use libwacom-surface from Nixos Unstable (installed explicitly via Nix) and there is only libwacom surface installed
image
The weird thing is that I use two surface device with the same flake config and flake.lock (so same version of packages and dependencies) and only the surface book touchscreen doesn't work (the surface laptop studio touchscreen work perfectly).
Also, when checking the log I see kernel error related to the screen (no libwacom error)

@orthogonaleety

Copy link
Copy Markdown

Thanks cfouche. In that case I doubt its relevant. No low hanging fruit here then probably. And I should step back because I'm just complicating the PR discussion now.
However, an aside, and it might be just that I'm not understanding how Nixos package pages can be put together, but the source link for NixOS packages for libwacom-surface, actually just links back to the source for just standard libwacom. (it does that under both 25.11 and unstable). https://search.nixos.org/packages?channel=unstable&query=libwacom-surface > https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/by-name/li/libwacom/package.nix#L86

@cfouche3005

Copy link
Copy Markdown

The link is kind of wrong (but not totally), the package is declared here but as the libwacom-surface use the libwacom build declaration, it links to the libwacom package

@orthogonaleety

orthogonaleety commented May 26, 2026

Copy link
Copy Markdown

Maybe raise that libwacom-surface discrepancy with the NixOS package maintainersas an aside.
You also said

Also, when checking the log I see kernel error related to the screen_

Maybe raise an issue with full details. Not sure how much traction it will get as all this is kind of 'ahead' of linux-surface right now but, I guess its up to who wants to engage and how much. It might help to get news ahead of time. I think there comes a point where one has to say 'ok, the only change was surface 6.x to surface 7.x, but what exactly is is broken where, so it can be fixed'

@cfouche3005

Copy link
Copy Markdown

I will open an issue this evening as I don't have my surface book to get the full logs but from what I have seen, it seem the issue come from the ipts kernel driver

MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request May 26, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request Jun 3, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request Jun 3, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
@orthogonaleety

orthogonaleety commented Jun 3, 2026

Copy link
Copy Markdown

Hi @Apiznel
Sorry to further inflate this PR discussion but I'm not sure what best to do with the following. cfouche's issue has turned out to be a some time standing bug in ipts implemented in the 005 patch file.
The details are in my comment. #2150 (comment)
with my version of a fix near the bottom of that comment
https://github.com/user-attachments/files/28481922/0017-ipts-fix-hid-report-data-size.patch
cfouche has confirmed it tests good here #2150 (comment)

There is also a more succinct commit/PR type description of the matter in an issue I subsequently logged against ipts linux-surface/intel-precise-touch#34 That might be the best think to read first.

If linux-surface had a test branch I would be happy to commend a PR to that, but I'm not a developer, nor experienced in linux kernel concerns at that level, so on principle I'm not happy to risk someone overly assuming something is ready for master (production).

I note you are a past accepted contributor and your work in these commits.
Perhaps you or someone here could examine it, consider where it may perhaps need to be implemented differently or not, and create a commit over the 005 patch file and PR accordingly.

Many thanks.

@cfouche3005

Copy link
Copy Markdown

To add I only tested it on the surface book 1, I don't know if this patch break other surface devices

@dotlambda dotlambda left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The file configs/surface-7.0.config is missing.

Based on the current v6.19 config
@Apiznel

Apiznel commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

@orthogonaleety I probably have as much experience as you do in the matter 😅. This would be better reviewed by @StollD. I'm just a contributor who occasionally rebases the patches as needed (it's nothing complicated). I do appreciate your effort in resolving the issue though.

@Apiznel

Apiznel commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

The file configs/surface-7.0.config is missing.

@dotlambda Added. Thanks!

MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request Jun 4, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
xry111 pushed a commit to AOSC-Tracking/linux that referenced this pull request Jun 4, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
MingcongBai pushed a commit to AOSC-Tracking/linux that referenced this pull request Jun 7, 2026
Experimental camera support for the Surface Pro 9.

Link: linux-surface/linux-surface#1867
Patchset: cameras

[ Mingcong Bai: Since commit 2a7b765 ("platform/x86: int3472: Handle
  GPIO type 0x10 (DOVDD)") took the 0x10 entry used by the original patch
  for Surface Pro 9's secondary power rail, change the latter to 0x11 to
  adapt to this change.

  Link: linux-surface/linux-surface#2057
  Link: linux-surface/linux-surface#2079 ]

Link: https://github.com/linux-surface/linux-surface/tree/4c136ec928ce0b458562c7533852db67a8e0dc91
Signed-off-by: Mingcong Bai <jeffbai@aosc.io>
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.

5 participants