Skip to content

Remove <linux/synclink.h> ioctls because the synclink_gt driver is being removed from the kernel#185

Open
EthanNelson-Moore wants to merge 1 commit into
sunfishcode:mainfrom
EthanNelson-Moore:remove_synclink_ioctls
Open

Remove <linux/synclink.h> ioctls because the synclink_gt driver is being removed from the kernel#185
EthanNelson-Moore wants to merge 1 commit into
sunfishcode:mainfrom
EthanNelson-Moore:remove_synclink_ioctls

Conversation

@EthanNelson-Moore
Copy link
Copy Markdown

The synclink_gt driver is being removed from the kernel due to long-unfixed code style and correctness issues and a lack of known users. <linux/synclink.h> will be removed along with it. To allow this crate to continue compiling, remove the code to generate ioctl bindings for this header. Also remove the generated ioctl values, since they will no longer function.

The synclink_gt driver is being removed from the kernel due to
long-unfixed code style and correctness issues and a lack of known
users. <linux/synclink.h> will be removed along with it. To allow this
crate to continue compiling, remove the code to generate ioctl bindings
for this header. Also remove the generated ioctl values, since they
will no longer function.
klarasm pushed a commit to klarasm/linux that referenced this pull request May 12, 2026
The synclink_gt driver was marked as broken in commit 426263d
("tty: synclink_gt: mark as BROKEN") in July 2023 because it had severe
structural problems and there had been no evidence of users since 2016.
Since then, no meaningful improvements have been made to the driver,
and it is unlikely that will ever happen due to the lack of interest.
Drop the driver and references to it in comments and documentation.
include/uapi/linux/synclink.h is also removed. The only use of this
header I have found is the linux-raw-sys Rust crate. It generates
bindings for all UAPI headers, but has a hardcoded list of headers and
ioctls, including this one, so that does not indicate that anyone is
using it. I have sent a pull request to remove the include and ioctl
definitions for this header (see the link below).

Link: sunfishcode/linux-raw-sys#185
Signed-off-by: Ethan Nelson-Moore <enelsonmoore@gmail.com>
Acked-by: Jakub Kicinski <kuba@kernel.org>
Link: https://patch.msgid.link/20260504031519.18877-1-enelsonmoore@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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.

1 participant