Received-SPF: pass (google.com: domain of linux-kernel+bounces-93163-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Sebastian Reichel <sebastian.reichel@collabora.com>
To: Sebastian Reichel <sre@kernel.org>, 
 "Ricardo B. Marliere" <ricardo@marliere.net>
Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, 
 kernel test robot <lkp@intel.com>
In-Reply-To: <20240303-class_cleanup-power-v2-1-e248b7128519@marliere.net>
References: <20240303-class_cleanup-power-v2-1-e248b7128519@marliere.net>
Subject: Re: [PATCH v2] power: supply: move power_supply_attr_groups
 definition back to sysfs
Message-Id: <170968466622.645753.4432514364980212313.b4-ty@collabora.com>
Date: Wed, 06 Mar 2024 01:24:26 +0100
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


On Sun, 03 Mar 2024 13:30:50 -0300, Ricardo B. Marliere wrote:
> As reported by the kernel test robot, 'power_supply_attr_group' is defined
> but not used when CONFIG_SYSFS is not set. Sebastian suggested that the
> correct fix implemented by this patch, instead of my attempt in commit
> ea4367c40c79 ("power: supply: core: move power_supply_attr_group into #ifdef
> block"), is to define power_supply_attr_groups in power_supply_sysfs.c and
> expose it in the power_supply.h header. For the case where CONFIG_SYSFS=n,
> define it as NULL.
> 
> [...]

Applied, thanks!

[1/1] power: supply: move power_supply_attr_groups definition back to sysfs
      commit: f107e6b82392a4d64507e219c57235d3fe09e9d7

Best regards,
-- 
Sebastian Reichel <sebastian.reichel@collabora.com>