Feedback-ID: ibe194615:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-221-gec32977366-fm-20230306.001-gec329773
Mime-Version: 1.0
Message-Id: <dd77e7df-022f-4299-8827-4622c1d2d1f3@app.fastmail.com>
In-Reply-To: <1af9e9c5-681b-483e-620f-c1866fafd6db@redhat.com>
References: <20230314-think-lmi-sysfs_ops-v1-1-9d4f1cf9caec@weissschuh.net>
 <1af9e9c5-681b-483e-620f-c1866fafd6db@redhat.com>
Date:   Fri, 17 Mar 2023 09:37:19 -0400
From:   "Mark Pearson" <mpearson-lenovo@squebb.ca>
To:     "Hans de Goede" <hdegoede@redhat.com>,
        =?UTF-8?Q?Thomas_Wei=C3=9Fschuh?= <linux@weissschuh.net>,
        "Mark Pearson" <markpearson@lenovo.com>,
        "markgross@kernel.org" <markgross@kernel.org>
Cc:     "platform-driver-x86@vger.kernel.org" 
        <platform-driver-x86@vger.kernel.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] platform/x86: think-lmi: Remove custom kobject sysfs_ops
Content-Type: text/plain;charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Thanks Thomas,

Tested on a couple of my platforms - all looked good.

Tested-by: mpearson-lenovo@squebb.ca

Mark

On Thu, Mar 16, 2023, at 10:44 AM, Hans de Goede wrote:
> Hi,
>
> On 3/14/23 01:07, Thomas Wei=C3=9Fschuh wrote:
>> think-lmi defines its own sysfs_ops that are identical to the standard
>> kobj_sysfs_ops. Use the standard definitions.
>>=20
>> Signed-off-by: Thomas Wei=C3=9Fschuh <linux@weissschuh.net>
>
> Thank you for your patch, I've applied this patch to my review-hans=20
> branch:
> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers=
-x86.git/log/?h=3Dreview-hans
>
> Note it will show up in my review-hans branch once I've pushed my
> local branch there, which might take a while.
>
> Once I've run some tests on this branch the patches there will be
> added to the platform-drivers-x86/for-next branch and eventually
> will be included in the pdx86 pull-request to Linus for the next
> merge-window.
>
> Regards,
>
> Hans
>
>
>> ---
>>=20
>> This has only been compile-tested.
>> ---
>>  drivers/platform/x86/think-lmi.c | 31 ++-----------------------------
>>  1 file changed, 2 insertions(+), 29 deletions(-)
>>=20
>> diff --git a/drivers/platform/x86/think-lmi.c b/drivers/platform/x86/=
think-lmi.c
>> index 86b33b74519b..2b5ae68ffce8 100644
>> --- a/drivers/platform/x86/think-lmi.c
>> +++ b/drivers/platform/x86/think-lmi.c
>> @@ -1047,33 +1047,6 @@ static const struct attribute_group tlmi_attr_=
group =3D {
>>  	.attrs =3D tlmi_attrs,
>>  };
>> =20
>> -static ssize_t tlmi_attr_show(struct kobject *kobj, struct attribute=
 *attr,
>> -				    char *buf)
>> -{
>> -	struct kobj_attribute *kattr;
>> -
>> -	kattr =3D container_of(attr, struct kobj_attribute, attr);
>> -	if (kattr->show)
>> -		return kattr->show(kobj, kattr, buf);
>> -	return -EIO;
>> -}
>> -
>> -static ssize_t tlmi_attr_store(struct kobject *kobj, struct attribut=
e *attr,
>> -				     const char *buf, size_t count)
>> -{
>> -	struct kobj_attribute *kattr;
>> -
>> -	kattr =3D container_of(attr, struct kobj_attribute, attr);
>> -	if (kattr->store)
>> -		return kattr->store(kobj, kattr, buf, count);
>> -	return -EIO;
>> -}
>> -
>> -static const struct sysfs_ops tlmi_kobj_sysfs_ops =3D {
>> -	.show	=3D tlmi_attr_show,
>> -	.store	=3D tlmi_attr_store,
>> -};
>> -
>>  static void tlmi_attr_setting_release(struct kobject *kobj)
>>  {
>>  	struct tlmi_attr_setting *setting =3D to_tlmi_attr_setting(kobj);
>> @@ -1091,12 +1064,12 @@ static void tlmi_pwd_setting_release(struct k=
object *kobj)
>> =20
>>  static const struct kobj_type tlmi_attr_setting_ktype =3D {
>>  	.release        =3D &tlmi_attr_setting_release,
>> -	.sysfs_ops	=3D &tlmi_kobj_sysfs_ops,
>> +	.sysfs_ops	=3D &kobj_sysfs_ops,
>>  };
>> =20
>>  static const struct kobj_type tlmi_pwd_setting_ktype =3D {
>>  	.release        =3D &tlmi_pwd_setting_release,
>> -	.sysfs_ops	=3D &tlmi_kobj_sysfs_ops,
>> +	.sysfs_ops	=3D &kobj_sysfs_ops,
>>  };
>> =20
>>  static ssize_t pending_reboot_show(struct kobject *kobj, struct kobj=
_attribute *attr,
>>=20
>> ---
>> base-commit: eeac8ede17557680855031c6f305ece2378af326
>> change-id: 20230314-think-lmi-sysfs_ops-e849a18299bb
>>=20
>> Best regards,