From nobody Tue Sep 29 13:19:52 2026 Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A150642D770 for ; Fri, 7 Aug 2026 20:06:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786133218; cv=none; b=JIN0zk61QCCbUzIx2USUCYw6eCUtzf1mLk5eHFDSKCLtN7XOLMHN8hmxiWUkGpg9cGVqiI6ToaeeUICe6WiS2f3EwcjaIU0mzXWakrsNsXdy4txRgAG0f/MOkFZeg2cNFg56owcur7i18ehmx1+F4I9yRTQQM/ZOmRZZAUzQliM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786133218; c=relaxed/simple; bh=B5vjso3WytCS2+uPX1Twai7hqC7D2oi2pc6SW+aJLrc=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Content-Type; b=ElUI5rw8gJoAL6Yg+dpXskwcblcsr3LeffEfefVJT/NAdYySOHBed2E7p1B6I4V8PO0tdCrTryS0KiQk+RP4epamkBv+8q1GiytyHjINY1/MMd085dsQnQmehC7rgHrtY0BgHn02afIS8ZNNeJwqRc6yKmYVPl+osKF8hsDQnsY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=VZQKkV94; arc=none smtp.client-ip=209.85.128.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="VZQKkV94" Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-4921eed3fa2so31551525e9.0 for ; Fri, 07 Aug 2026 13:06:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786133211; x=1786738011; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id:date :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to :content-type; bh=DcvzOP3DsrMH8W1cQwtr2Rj2FiH385dungW6zjf9cLs=; b=VZQKkV949n273BG4gXlIn4lK8SKiJQdTRJ66S00hwyWyaXEM6oo8E+9BbjylltzNX6 BxHn1U5qkrB3iT7+Bz9sRKENtoFkecTPAKngE6GnqSoSNIt5jK7AiFkfUGgJQZvrPj9I z7aQBZj8wafowKCCEev0/8Wm0diPQuRt7M0+2u0SFTLsLuemq8T0qil978fDmN1V17wk Lzu1uSir6npK3VTTwnXZog+hbJxMvTLhSaNw0u1TqdLDiJtf0GPI0mos69feucKNgSL9 HR63Y1dCH5GnMhKCLvJxCIMGDrZ49UnzvrNCsH0oJ7IxRMFmI55w4QkxUMLIhJ9KPsQD NxVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786133211; x=1786738011; h=content-transfer-encoding:content-type:mime-version:message-id:date :subject:cc:to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to:content-type; bh=DcvzOP3DsrMH8W1cQwtr2Rj2FiH385dungW6zjf9cLs=; b=K4yOwMfuzbKEf3C0QSmdc6SUE/HpsAUGxXjCFkWcgSwGLJdewIxMCuCbZqeUlEhsmb /u0sDTgMkNpmj49pCOE3XirvJ3XbGxKLKSR4i5J6iZ4VJ7s42Utsi83Ul240yuJ9RGJx jrOmeZSvrjqMwD6+aJXdrXlfy7bMBnY8X2bHeLfkz1vXknvSSOw59bnQWDcoSVo2zf+Q 3kFPbeQseyzp8vbaLL3PS5OsEX1xDbSRr2iR1vHtnhPRC4ZxfIcAZmPz3G4BPPGgAXBI BJr4K1SGV8MFW8s2QEN1lNbNAjQ5cZF5pnsiNc64PGr5r2wUp7A84IOXLKVVnJAFrZKk +y5A== X-Forwarded-Encrypted: i=1; AHgh+Rqg1IqnLTHwSwvXeZiOgtbRQ+CjFfT7dhRZEeXIG/qAKJfRadZwuVc4hf2yW/rKg1x4v2yVaXcLWedae0k=@vger.kernel.org X-Gm-Message-State: AOJu0YzQPBBs3cMd9Cshkp0MDw2LLiJzSEPFCBrrPMix8YlLJPOmSmNI kGDakPhjyUuiO6sOBDHxFxDbJ+16HmyW/mSw0ShKy71iMJZcesX9aXYS X-Gm-Gg: AR+sD11kPvCJjWpmS/28QGGAkgiTFl0Hja/Cy7zHQntQ6Xz44Eg3Cpm65gXlmrsv/PR GxdSTZN2MKG9k5lgC3IfqQoet6lA6q7ZGBIuju/wkBUMdmNuSHBqpXP7aB2picnLv9Nl3jpqYI1 F+S8EIhz6vHlySQ3AmKdIMe+KlF6rhjQVgVfJksAUFuK2hvnY1QQ6PpBA7fLrQhvWHU5fb5aB3A 6CkgIofDVdbY+2p8dXc3HiY8xK+2ICh+fzICtw8by/SZ8ogO5n4XcvU8Fggk/7+nxmHp2v6Pf2Z XSdpN85LBTedMav2ek+nkifOArVLFoOIUZIdDgC7e7NeIz2xnCxvVQswZ/u+R2wBZkjAStaQB32 vKm9bF0zTuxbJcrsUTSwSW37kqwI1LqEmW5q5LEFWiBWKtzNyzYLMW/ONrGZVYcy4zB8IXuvARB KoLuKockLkvboZN/hGE6SvXNT7NfMpzxIRdE1rcIWPdO85m/IFwbCXRlvepT577PA9h7VtWPwQU aN7p2DugAvz2Kb1DLD4 X-Received: by 2002:a05:600c:1991:b0:493:e451:a9e1 with SMTP id 5b1f17b1804b1-4994e7448d9mr312485465e9.2.1786133211126; Fri, 07 Aug 2026 13:06:51 -0700 (PDT) Received: from christian-ThinkPad-T470.. ([2a03:4460:170d:5700:848a:c270:b637:504d]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4995e9eb1a0sm67283895e9.5.2026.08.07.13.06.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Aug 2026 13:06:50 -0700 (PDT) From: =?UTF-8?q?Christian=20Melendez=20Nu=C3=B1ez?= To: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-doc@vger.kernel.org, airlied@gmail.com, simona@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, corbet@lwn.net, skhan@linuxfoundation.org, =?UTF-8?q?Christian=20Melendez=20Nu=C3=B1ez?= Subject: [PATCH] PCI: vgaarb: Remove stale "not implemented" notes from docs Date: Fri, 7 Aug 2026 22:06:39 +0200 Message-Id: <20260807200639.172487-1-chrismelnu@gmail.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable The vgaarb kerneldoc and Documentation/gpu/vgaarbiter.rst both describe "unlock all" and "target default" as "(TODO: not implemented yet)". Both are actually implemented already: - "unlock all" is handled in vga_arb_write() via the "all" strncmp branch, which releases both VGA_RSRC_LEGACY_IO and VGA_RSRC_LEGACY_MEM. - "target default" is handled in the same function via the "default" strncmp branch, which resolves the target to vga_default_device(). Drop the stale disclaimers so the documentation matches actual behavior. This is my first submission to the kernel; happy to take any feedback on the patch itself or how I should be doing this. Signed-off-by: Christian Melendez Nu=C3=B1ez --- Documentation/gpu/vgaarbiter.rst | 9 ++++----- drivers/pci/vgaarb.c | 7 +++---- 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/Documentation/gpu/vgaarbiter.rst b/Documentation/gpu/vgaarbite= r.rst index d1e953712cc2..c3a728130b54 100644 --- a/Documentation/gpu/vgaarbiter.rst +++ b/Documentation/gpu/vgaarbiter.rst @@ -65,8 +65,7 @@ write unlock release locks on target unlock all - release all locks on target held by this user (not impleme= nted - yet) + release all locks on target held by this user decodes set the legacy decoding attributes for the card =20 @@ -74,9 +73,9 @@ write event if something changes on any card (not just the targe= t) =20 card_ID is of the form "PCI:domain:bus:dev.fn". It can be set to "= default" - to go back to the system default card (TODO: not implemented yet).= Currently, - only PCI is supported as a prefix, but the userland API may suppor= t other bus - types in the future, even if the current kernel implementation doe= sn't. + to go back to the system default card. Currently, only PCI is supp= orted + as a prefix, but the userland API may support other bus types in t= he + future, even if the current kernel implementation doesn't. =20 Note about locks: =20 diff --git a/drivers/pci/vgaarb.c b/drivers/pci/vgaarb.c index c360eee11dd9..4ae96ae60d8b 100644 --- a/drivers/pci/vgaarb.c +++ b/drivers/pci/vgaarb.c @@ -1008,10 +1008,9 @@ EXPORT_SYMBOL(vga_client_register); * poll : event if something change on any card (not just the targ= et) * * card_ID is of the form "PCI:domain:bus:dev.fn". It can be set to "defau= lt" - * to go back to the system default card (TODO: not implemented yet). - * Currently, only PCI is supported as a prefix, but the userland API may - * support other bus types in the future, even if the current kernel - * implementation doesn't. + * to go back to the system default card. Currently, only PCI is supported + * as a prefix, but the userland API may support other bus types in the + * future, even if the current kernel implementation doesn't. * * Note about locks: * --=20 2.34.1