From patchwork Mon Oct 23 14:02:05 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan Cameron X-Patchwork-Id: 13432890 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 11E49125DD for ; Mon, 23 Oct 2023 14:02:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6DBCD103 for ; Mon, 23 Oct 2023 07:02:13 -0700 (PDT) Received: from lhrpeml500005.china.huawei.com (unknown [172.18.147.226]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4SDcLM3hZQz6K9DT; Mon, 23 Oct 2023 22:01:31 +0800 (CST) Received: from SecurePC-101-06.china.huawei.com (10.122.247.231) by lhrpeml500005.china.huawei.com (7.191.163.240) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.31; Mon, 23 Oct 2023 15:02:10 +0100 From: Jonathan Cameron To: , , Michael Tsirkin , Michael Tokarev CC: , Fan Ni , =?utf-8?q?Philippe_M?= =?utf-8?q?athieu-Daud=C3=A9?= Subject: [PATCH v5 0/5] hw/cxl: Line length reduction and related Date: Mon, 23 Oct 2023 15:02:05 +0100 Message-ID: <20231023140210.3089-1-Jonathan.Cameron@huawei.com> X-Mailer: git-send-email 2.39.2 Precedence: bulk X-Mailing-List: linux-cxl@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Originating-IP: [10.122.247.231] X-ClientProxiedBy: lhrpeml500005.china.huawei.com (7.191.163.240) To lhrpeml500005.china.huawei.com (7.191.163.240) X-CFilter-Loop: Reflected v5: - As _Static_assert() is a declaration it cannot follow a label (until C23). Upshot of that is to compile in clang (possibly older GCC) we need to add {} to case statements that start with QEMU_BUILD_BUG_ON() - Add patch to handle another QEMU_BUILD_BUG_ON() case by dropping the unnecessary check. Suggested-by: Michael S. Tsirkin Michael observed that the CXL code regularly went above the 80 character recommendation and in many cases this was not necessary for readability. This series is focused on tidying this up for the existing code so that we can maintain the preferred formatting going forwards. Jonathan Cameron (5): hw/cxl: Use a switch to explicitly check size in caps_reg_read() hw/cxl: Use switch statements for read and write of cachemem registers hw/cxl: CXLDVSECPortExtensions renamed to CXLDVSECPortExt hw/cxl: Line length reductions hw/cxl: Fix a QEMU_BUILD_BUG_ON() in switch statement scope issue. include/hw/cxl/cxl_component.h | 3 +- include/hw/cxl/cxl_device.h | 5 +- include/hw/cxl/cxl_events.h | 3 +- include/hw/cxl/cxl_pci.h | 6 +- hw/cxl/cxl-cdat.c | 3 +- hw/cxl/cxl-component-utils.c | 130 ++++++++++++++++++++------------- hw/cxl/cxl-device-utils.c | 11 ++- hw/cxl/cxl-events.c | 9 ++- hw/cxl/cxl-mailbox-utils.c | 21 ++++-- hw/mem/cxl_type3.c | 31 +++++--- hw/mem/cxl_type3_stubs.c | 5 +- hw/pci-bridge/cxl_downstream.c | 2 +- hw/pci-bridge/cxl_root_port.c | 2 +- hw/pci-bridge/cxl_upstream.c | 2 +- 14 files changed, 144 insertions(+), 89 deletions(-)