[{"id":3684829,"web_url":"http://patchwork.ozlabs.org/comment/3684829/","msgid":"<336f827e-71da-49af-be9b-6f555cd3cbb1@163.com>","list_archive_url":null,"date":"2026-04-30T16:22:07","subject":"Re: [PATCH 00/16] PCI: Use FIELD_MODIFY() to simplify bitfield\n operations","submitter":{"id":89937,"url":"http://patchwork.ozlabs.org/api/people/89937/","name":"Hans Zhang","email":"18255117159@163.com"},"content":"Sorry for the noise.Due to my current system issues, the remaining \npatches were not sent out. I will resend them now.\n\nHans\n\nOn 5/1/26 00:12, Hans Zhang wrote:\n> Replace open-coded bitmask clear + FIELD_PREP() sequences with the\n> dedicated FIELD_MODIFY() macro. FIELD_MODIFY() performs the same\n> operation but is shorter, less error-prone, and includes compile-time\n> checks to prevent field overflow.\n> \n> This series touches the PCI core, PCIe extended capabilities (PTM, IDE,\n> TPH, MSI, REBAR, IOV, CardBus), and several DesignWare-based and\n> other PCI host controllers. No functional change is intended.\n> \n> The patches are split per subsystem or per driver to ease review:\n> ---\n> Hi, If the Maintainers think it's not necessary, please ignore it.\n> ---\n> \n> Hans Zhang (16):\n> PCI: Use FIELD_MODIFY()\n> PCI/PTM: Use FIELD_MODIFY()\n> PCI/IDE: Use FIELD_MODIFY()\n> PCI/IOV: Use FIELD_MODIFY()\n> PCI/TPH: Use FIELD_MODIFY()\n> PCI/MSI: Use FIELD_MODIFY()\n> PCI/REBAR: Use FIELD_MODIFY()\n> PCI/CARDBUS: Use FIELD_MODIFY()\n> PCI: al: Use FIELD_MODIFY()\n> PCI: eswin: Use FIELD_MODIFY()\n> PCI: s32g: Use FIELD_MODIFY()\n> PCI: tegra194: Use FIELD_MODIFY()\n> PCI: qcom: Use FIELD_MODIFY()\n> PCI: dwc: Use FIELD_MODIFY()\n> PCI: mvebu: Use FIELD_MODIFY()\n> PCI: mediatek-gen3: Use FIELD_MODIFY()\n> \n> drivers/pci/controller/dwc/pcie-al.c | 12 ++----\n> .../controller/dwc/pcie-designware-debugfs.c | 23 ++++-------\n> .../pci/controller/dwc/pcie-designware-ep.c | 3 +-\n> drivers/pci/controller/dwc/pcie-designware.c | 3 +-\n> drivers/pci/controller/dwc/pcie-eswin.c | 3 +-\n> drivers/pci/controller/dwc/pcie-nxp-s32g.c | 3 +-\n> drivers/pci/controller/dwc/pcie-qcom-common.c | 40 +++++++------------\n> drivers/pci/controller/dwc/pcie-qcom-ep.c | 6 +--\n> drivers/pci/controller/dwc/pcie-tegra194.c | 8 ++--\n> drivers/pci/controller/pci-mvebu.c | 3 +-\n> drivers/pci/controller/pcie-mediatek-gen3.c | 3 +-\n> drivers/pci/ide.c | 6 +--\n> drivers/pci/iov.c | 3 +-\n> drivers/pci/msi/msi.c | 11 ++---\n> drivers/pci/pci.c | 3 +-\n> drivers/pci/pcie/ptm.c | 3 +-\n> drivers/pci/rebar.c | 6 +--\n> drivers/pci/setup-cardbus.c | 3 +-\n> drivers/pci/tph.c | 10 ++---\n> 19 files changed, 51 insertions(+), 101 deletions(-)\n> \n> \n> base-commit: 3b3bea6d4b9c162f9e555905d96b8c1da67ecd5b","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-pci@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=163.com header.i=@163.com header.a=rsa-sha256\n header.s=s110527 header.b=TQnBJm6d;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c04:e001:36c::12fc:5321; helo=tor.lore.kernel.org;\n envelope-from=linux-pci+bounces-53509-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (1024-bit key) header.d=163.com header.i=@163.com\n header.b=\"TQnBJm6d\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=117.135.210.5","smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=163.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=163.com"],"Received":["from tor.lore.kernel.org (tor.lore.kernel.org\n [IPv6:2600:3c04:e001:36c::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g5zyS6j7Rz1xqf\n\tfor ; Fri, 01 May 2026 02:24:16 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby tor.lore.kernel.org (Postfix) with ESMTP id 1C4F030511CB\n\tfor ; Thu, 30 Apr 2026 16:22:40 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 831BB46AEF5;\n\tThu, 30 Apr 2026 16:22:37 +0000 (UTC)","from m16.mail.163.com (m16.mail.163.com [117.135.210.5])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id A50EC330D54;\n\tThu, 30 Apr 2026 16:22:33 +0000 (UTC)","from [IPV6:240e:b8f:927e:5900:c497:d540:b00d:97f4] (unknown [])\n\tby gzsmtp1 (Coremail) with SMTP id PCgvCgDn1_ivgfNpPCjLBw--.9605S2;\n\tFri, 01 May 2026 00:22:09 +0800 (CST)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1777566157; cv=none;\n b=exQbuK+d/nEdjWhsBPv4LSJ39BYg3KqOweBKL/v1Qv2BU/xJakb1oDtQLIS+6AIjCU7Pma4bJF4cJe8kgwYHMceUWfWxrk6sCzrSbJxDCxiZxAkCsao7uSpm5DqplAu9hvjMAgm21ioSfPenIZWg8sM40f/iIrGuBmL5FEZM0EY=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1777566157; c=relaxed/simple;\n\tbh=Mez5As1RneO/MRM6s7tojfS69kWEmTjgPYktdsfn50o=;\n\th=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From:\n\t In-Reply-To:Content-Type;\n b=GtWYcvWjHDW7AW4u5pqOhL5KpV3gyiqjVuN+Y6WYNYLpzP6/YDfsUy69qHpJ8aRMxmkoDOY4cIgylGGviCSTmahIo+vVAZb7xmxGrf+mjoYMFQ7m8pGSmb5l0Rt3jJcve8KrHBcBV12q5rhGknE5abtbf8A7OM4OOQAlZErDQLQ=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=163.com;\n spf=pass smtp.mailfrom=163.com;\n dkim=pass (1024-bit key) header.d=163.com header.i=@163.com\n header.b=TQnBJm6d; arc=none smtp.client-ip=117.135.210.5","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com;\n\ts=s110527; h=Message-ID:Date:MIME-Version:Subject:To:From:\n\tContent-Type; bh=lgOkgt64wlf+glnOOYyNKpnMDGONDz3qxMKki9FrYto=;\n\tb=TQnBJm6dcoi9EGh/g27w0+KwriZg2FqXJtbHYeSwoID6p78L26SmG6jnZ1q5BF\n\tVV9xWfYDJGKQDQDa5PxwEteQvNbNYr+POy1KGLG37XRA1AQY6rSr5g4xE5mQBjbm\n\tRmyAIM+OaJBv5X2Tdz8vBjl0cWKsPn0spt3yuPuGTeX64=","Message-ID":"<336f827e-71da-49af-be9b-6f555cd3cbb1@163.com>","Date":"Fri, 1 May 2026 00:22:07 +0800","Precedence":"bulk","X-Mailing-List":"linux-pci@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH 00/16] PCI: Use FIELD_MODIFY() to simplify bitfield\n operations","To":"bhelgaas@google.com, lpieralisi@kernel.org, kwilczynski@kernel.org,\n mani@kernel.org, jingoohan1@gmail.com","Cc":"robh@kernel.org, s32@nxp.com, linux-pci@vger.kernel.org,\n linux-kernel@vger.kernel.org","References":"<20260430161254.40278-1-18255117159@163.com>","Content-Language":"en-US","From":"Hans Zhang <18255117159@163.com>","In-Reply-To":"<20260430161254.40278-1-18255117159@163.com>","Content-Type":"text/plain; charset=UTF-8; format=flowed","Content-Transfer-Encoding":"7bit","X-CM-TRANSID":"PCgvCgDn1_ivgfNpPCjLBw--.9605S2","X-Coremail-Antispam":"1Uf129KBjvJXoW7Ar4DJw1rtrWfCw1xKr1fXrb_yoW8KF1rpa\n\t4UKFyxZFy8tr45ZF43Ca1jkFyYvFnak34UtayqgwnrW3ZI9ryUWrWUtFyrJ39xJF4UWFy5\n\tKw1Uta45uFs3JFUanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2\n\t9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07U5R67UUUUU=","X-CM-SenderInfo":"rpryjkyvrrlimvzbiqqrwthudrp/xtbCwxHjh2nzgbE61QAA3z"}}]