From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 628983093B8 for ; Thu, 4 Jun 2026 20:58:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780606720; cv=none; b=NaShiNzhBXsPaqKcVZNu9ZV8/bLSOM0Vxz6S9vU7TO14DuBORDj3X2toHGCEPHyaM9f13AW3ybgkfR3ob1XcTuYrm3HtMzKD/sV2/M1Nu+xi5qGJN2uxsZnyJE+QTZ4iRPpI6JOohirCd+jWRBAsI4m3qA/DuUMSSFICVZCYZZM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780606720; c=relaxed/simple; bh=DQTkf1lHMIqbwK47/czYzHj+glOCSShwPoAIlDnSnRw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Oh1KDElVQs4G9MMeRsR84G7A773wD8HDOKCp8OdhDXkKe/Z4gCia7lwHbrwXAbW3tYT8FYy4wp7DBG32dcs8Fu0r7tqTkhVsycB/Jq1wNqvAdG1+03jHX8mxlCA8Qe+fDNOnHUOqvsHIxFGeCoR5EBZlP5Y6L6a6aOb9D2I2Nuo= 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=iknnO2pz; arc=none smtp.client-ip=209.85.128.46 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="iknnO2pz" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-490af320e2aso14117065e9.2 for ; Thu, 04 Jun 2026 13:58:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780606718; x=1781211518; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=DQTkf1lHMIqbwK47/czYzHj+glOCSShwPoAIlDnSnRw=; b=iknnO2pzYAxVBr5i32I5mNFlyUF4hG16k08qD4F71/MXpgO41aPReHIyqAUUCJV6RS ViihSrIEddeu0NhEUPgs9lB8hDS+Ep36XfsjybRB+5GMugVp4FyZhL9U0kHhkOduxGbT maWotG0ZImKwZF7omJlhECYYrhgdvXilFsiefRidN5gYZPFckPzFPAR/C47NAPfVGjps 7RHYlODTmTQhgbviG7ZVGmzhgRzv7pF5ksNClaagSXvoSSJmfCcX2whiiZxpwoscAXQB snl2pwk1CGqeCMF2Foq66rJF/MxGmx74R8cuKe5FYm1qsjt5qmQ8p4H7Ao3iwKimMlol 8Jkw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780606718; x=1781211518; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=DQTkf1lHMIqbwK47/czYzHj+glOCSShwPoAIlDnSnRw=; b=BmxxJHkgbfutVf5yx4hLFY3AybflQs9HHWZrAr6bNrI/SRZrycEBAB9qv+Zc6MNf9Z HD/OK2tHqgddLuEKV1EfP5GJ2flfS9pPz3G3Y5qv/O8nUVqxEOgiqao/XSKXzYPl864R yTeW3p8uGV36SFl7YDiX/EcTiB+NMFp7yfWQixRtplE7TEO2Awo7jf9TOLgN8SHJXIKN WTU/zt+TsrU8MF611w0p+SZMcWbUo6XXBTU5Ox8zFOSzwsuzwBrIbwgyI+wuM0OG9xn/ rPZDQ/1FAVzHkWU7ZfTcgluwBE6IkzhGHHkfN/Oo8aoWNhLNRPeXG2vSAM3/p72ymmOV UwVA== X-Forwarded-Encrypted: i=1; AFNElJ/P+iiKfo6PduMKziYitSYEp7bpQPs28FqtV4HoKZgab6gsT9Gy68933rJEuq04GZ/tW0NeFkiX2+Cn@vger.kernel.org X-Gm-Message-State: AOJu0YygXXOJDIkk0oNDOLS8rODGh3GQ3ftJo0hQh7IGml6QEHpZ/pjD KclMx125dz+t5PcK5Xn/3gPlEiYs/wl/9LEJkAzk0zIAAAZlSTYuRQXj X-Gm-Gg: Acq92OHu14p6GoBEXfFs7qU7Dk7A7PG2LP7AyBrCDd42YOPOGrzWWWqvMvSJoEtEomK 9zTya0PIlNInSivukW0UvcVXrEzDzbBu18JUgHx/Vs82abWtitbaw9WBoMxVctwmYq5v96tL3vk xDn90WRsNJbvULdOeVRTd/pj42AGeOEeqx74ISHPraJ76wCKyGdPyolaOJzrvv7ZK82OE0AvLsp LhYfhrdSDYydWiyopsuA/f3ZA+zzLasjnrNHmkQbMC2kCqQmkmxontsDbQ2bHOFhnMnoE0Mdnr2 yCl1LPovpG+d7ps4+sByjn404KKLjt66VqmPXz2oCy0kZ1bFF3AX9b8d6hN5RnqWoJRfUd9zHNT iyiN1gOEM86vEXAJGFnxE8ye4piPxo0ghFFdeEznIQTr0fNlFdzkQk/445BQ7o8cOPZVPI0akCp gyVHoVjd95DUVcDFMkSr2Th1J8f+2onv2F X-Received: by 2002:a05:600c:83c3:b0:48f:d612:3c59 with SMTP id 5b1f17b1804b1-490c25a21a9mr4351475e9.9.1780606717742; Thu, 04 Jun 2026 13:58:37 -0700 (PDT) Received: from localhost ([94.53.77.213]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-490bc413adbsm100278155e9.15.2026.06.04.13.58.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Jun 2026 13:58:36 -0700 (PDT) Date: Thu, 4 Jun 2026 23:59:21 +0300 From: Catalin Iacob To: Bart Van Assche Cc: Thomas Bogendoerfer , Madhavan Srinivasan , Michael Ellerman , Nicholas Piggin , "Christophe Leroy (CS GROUP)" , Rich Felker , John Paul Adrian Glaubitz , "David S. Miller" , Andreas Larsson , "James E.J. Bottomley" , "Martin K. Petersen" , Jens Axboe , Yoshinori Sato , linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, linux-scsi@vger.kernel.org Subject: Re: [PATCH v3 0/6] Remove remaining references to the pktcdvd driver Message-ID: References: <20260530-remove-pktcdvd-references-v1-1-aa56941d4315@gmail.com> <20260604-remove-pktcdvd-references-v3-0-e2f06fb4eef4@gmail.com> <96a48e65-6ee2-4b81-91f8-821a2de707db@acm.org> Precedence: bulk X-Mailing-List: sparclinux@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <96a48e65-6ee2-4b81-91f8-821a2de707db@acm.org> On Thu, Jun 04, 2026 at 08:50:22AM -0700, Bart Van Assche wrote: > If this series is reposted, please combine patches 1/6 and 2/6. Anyway, > this series looks good to me. Thanks for taking a look. I will repost in a few days when back to a computer to fix the subject of patch 2 since you are right, it does wrongly suggest that the implementation is moved. For learning purposes, do you mind quickly sharing your rationale for combining 1 and 2? To me they are in the same area but still separate things: one removes dead code, the other increases encapsulation. Each is a small but worthwhile cleanup, they can be applied in any order and each can be applied without the other. It's true that, while patch 2 without patch 1 works, it would increase confusion but I find that an argument to order 2 after 1, not to say they are one thing. I'd also find it hard to describe what that combined patch does except by calling out two things: "remove dead code and move the declaration". Your suggested "unexport" seems more confusing than clarifying, I'd argue that today the symbol is not exported since that ifdef always compiles the export out.