Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 22 Jul 2024 18:55:03 +0000 (UTC)
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        Vladimir Kondratyev <wulf@FreeBSD.org>
Cc:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org,  dev-commits-src-main@FreeBSD.org
Subject:   Re: git: a1181662d370 - main - LinuxKPI: Add devm_device_add_group to linux/device.h
Message-ID:  <0964qqop-qrsn-2rpq-1n4s-snr76ps12no9@SerrOFQ.bet>
In-Reply-To: <202407211314.46LDEoW4023107@gitrepo.freebsd.org>
References:  <202407211314.46LDEoW4023107@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 21 Jul 2024, Vladimir Kondratyev wrote:

> The branch main has been updated by wulf:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=a1181662d3707d8165d7153a0aa5d76ea1ddc11a
>
> commit a1181662d3707d8165d7153a0aa5d76ea1ddc11a
> Author:     Vladimir Kondratyev <wulf@FreeBSD.org>
> AuthorDate: 2024-07-21 13:09:12 +0000
> Commit:     Vladimir Kondratyev <wulf@FreeBSD.org>
> CommitDate: 2024-07-21 13:09:12 +0000
>
>    LinuxKPI: Add devm_device_add_group to linux/device.h
>
>    devm_device_add_group creates a managed attribute group for a device.
>
>    Sponsored by:   Serenity Cyber Security, LLC
>    MFC after:      1 week
>    Reviewed by:    manu
>    Differential Revision:  https://reviews.freebsd.org/D45845
> ---
> sys/compat/linuxkpi/common/include/linux/device.h |  4 +++
> sys/compat/linuxkpi/common/src/linux_compat.c     | 30 +++++++++++++++++++++++
> 2 files changed, 34 insertions(+)
>
> diff --git a/sys/compat/linuxkpi/common/include/linux/device.h b/sys/compat/linuxkpi/common/include/linux/device.h
> index 00c55b31b248..912be8fbae8b 100644
> --- a/sys/compat/linuxkpi/common/include/linux/device.h
> +++ b/sys/compat/linuxkpi/common/include/linux/device.h
> @@ -701,4 +701,8 @@ int lkpi_devm_add_action_or_reset(struct device *dev, void (*action)(void *), vo
> #define	devm_add_action_or_reset(dev, action, data)	\
> 	lkpi_devm_add_action_or_reset(dev, action, data)
>
> +int lkpi_devm_device_add_group(struct device *dev, const struct attribute_group *group);
> +#define	devm_device_add_group(dev, group)	\
> +	lkpi_devm_device_add_group(dev, group)

I thought we would use linuxkpi_ as prefix for publicly exposed LinuxKPI
related functions and lkpi_ for "internal" FreeBSD specific code which
does not follow Linux KPI?

Can this be fixed before MFC?

-- 
Bjoern A. Zeeb                                                     r15:7



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0964qqop-qrsn-2rpq-1n4s-snr76ps12no9>