Re: [PATCH 0/7] EDAC: remove unused structure members

From: Jiri Slaby <jirislaby@kernel.org>
To: bp@alien8.de
Cc: linux-kernel@vger.kernel.org,
	Andre Przywara <andre.przywara@arm.com>,
	Bjorn Andersson <andersson@kernel.org>,
	Dinh Nguyen <dinguyen@kernel.org>,
	Douglas Thompson <dougthompson@xmission.com>,
	James Morse <james.morse@arm.com>,
	Jan Luebbe <jlu@pengutronix.de>,
	Johannes Thumshirn <morbidrsa@gmail.com>,
	Khuong Dinh <khuong@os.amperecomputing.com>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	linux-edac@vger.kernel.org,
	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Michal Simek <michal.simek@amd.com>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Ralf Baechle <ralf@linux-mips.org>,
	Robert Richter <rric@kernel.org>,
	Sai Krishna Potthuri <sai.krishna.potthuri@amd.com>,
	Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>,
	Tony Luck <tony.luck@intel.com>,
	Yazen Ghannam <yazen.ghannam@amd.com>
Subject: Re: [PATCH 0/7] EDAC: remove unused structure members
Date: Fri, 8 Mar 2024 10:01:14 +0100	[thread overview]
Message-ID: <c46cc960-1b7c-4e61-977a-f22ea5fdd944@kernel.org> (raw)
In-Reply-To: <20240213112051.27715-1-jirislaby@kernel.org>

Any plans on looking into/commenting/merging this :)?

On 13. 02. 24, 12:20, Jiri Slaby (SUSE) wrote:
> Hi,
> 
> this series removes unused EDAC structure members as found by
> clang-struct (and manually checked by me).
> 
> I admit I could not even compile-test octeon files, hopefully
> kernel-test robot would catch mistakes in there.
> 
> Cc: Andre Przywara <andre.przywara@arm.com>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Borislav Petkov <bp@alien8.de>
> Cc: Dinh Nguyen <dinguyen@kernel.org>
> Cc: Douglas Thompson <dougthompson@xmission.com>
> Cc: James Morse <james.morse@arm.com>
> Cc: Jan Luebbe <jlu@pengutronix.de>
> Cc: Johannes Thumshirn <morbidrsa@gmail.com>
> Cc: Khuong Dinh <khuong@os.amperecomputing.com>
> Cc: Konrad Dybcio <konrad.dybcio@linaro.org>
> Cc: linux-edac@vger.kernel.org
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Michal Simek <michal.simek@amd.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Robert Richter <rric@kernel.org>
> Cc: Sai Krishna Potthuri <sai.krishna.potthuri@amd.com>
> Cc: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
> Cc: Tony Luck <tony.luck@intel.com>
> Cc: Yazen Ghannam <yazen.ghannam@amd.com>
> 
> Jiri Slaby (SUSE) (7):
>    EDAC/amd64: Remove amd64_pvt::ext_nbcfg
>    EDAC/device: Remove edac_dev_sysfs_block_attribute::{block,value}
>    EDAC/device: Remove edac_dev_sysfs_block_attribute::store()
>    EDAC: Remove dynamic attributes from edac_device_alloc_ctl_info()
>    EDAC: Remove edac_pci_ctl_info::edac_subsys
>    EDAC: Remove edac_pci_ctl_info::complete
>    EDAC: Remove edac_device_ctl_info::removal_complete
> 
>   drivers/edac/altera_edac.c       |  8 ++---
>   drivers/edac/amd64_edac.h        |  1 -
>   drivers/edac/amd8111_edac.c      |  3 +-
>   drivers/edac/armada_xp_edac.c    |  2 +-
>   drivers/edac/cpc925_edac.c       |  2 +-
>   drivers/edac/edac_device.c       | 53 ++------------------------------
>   drivers/edac/edac_device.h       | 22 ++-----------
>   drivers/edac/edac_device_sysfs.c | 22 ++-----------
>   drivers/edac/edac_pci.h          |  5 ---
>   drivers/edac/highbank_l2_edac.c  |  2 +-
>   drivers/edac/mpc85xx_edac.c      |  2 +-
>   drivers/edac/octeon_edac-l2c.c   |  2 +-
>   drivers/edac/octeon_edac-pc.c    |  2 +-
>   drivers/edac/qcom_edac.c         |  1 -
>   drivers/edac/sifive_edac.c       |  3 +-
>   drivers/edac/thunderx_edac.c     |  6 ++--
>   drivers/edac/xgene_edac.c        | 10 +++---
>   drivers/edac/zynqmp_edac.c       |  2 +-
>   18 files changed, 26 insertions(+), 122 deletions(-)
> 

-- 
js
suse labs