Date: Wed, 24 Jul 2024 02:46:47 +0300 From: Vladimir Kondratyev <vladimir@kondratyev.su> To: "Bjoern A. Zeeb" <bz@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: <75796855-cf79-4538-81c5-ae6f7765d2d3@kondratyev.su> In-Reply-To: <0964qqop-qrsn-2rpq-1n4s-snr76ps12no9@SerrOFQ.bet> References: <202407211314.46LDEoW4023107@gitrepo.freebsd.org> <0964qqop-qrsn-2rpq-1n4s-snr76ps12no9@SerrOFQ.bet>
next in thread | previous in thread | raw e-mail | index | archive | help
On 22.07.2024 21:55, Bjoern A. Zeeb wrote: > On Sun, 21 Jul 2024, Vladimir Kondratyev wrote: >=20 >> The branch main has been updated by wulf: >> >> URL:=20 >> https://cgit.FreeBSD.org/src/commit/?id=3Da1181662d3707d8165d7153a0aa5= d76ea1ddc11a >> >> commit a1181662d3707d8165d7153a0aa5d76ea1ddc11a >> Author:=C2=A0=C2=A0=C2=A0=C2=A0 Vladimir Kondratyev <wulf@FreeBSD.org> >> AuthorDate: 2024-07-21 13:09:12 +0000 >> Commit:=C2=A0=C2=A0=C2=A0=C2=A0 Vladimir Kondratyev <wulf@FreeBSD.org> >> CommitDate: 2024-07-21 13:09:12 +0000 >> >> =C2=A0=C2=A0 LinuxKPI: Add devm_device_add_group to linux/device.h >> >> =C2=A0=C2=A0 devm_device_add_group creates a managed attribute group f= or a device. >> >> =C2=A0=C2=A0 Sponsored by:=C2=A0=C2=A0 Serenity Cyber Security, LLC >> =C2=A0=C2=A0 MFC after:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 1 week >> =C2=A0=C2=A0 Reviewed by:=C2=A0=C2=A0=C2=A0 manu >> =C2=A0=C2=A0 Differential Revision:=C2=A0 https://reviews.freebsd.org/= D45845 >> --- >> sys/compat/linuxkpi/common/include/linux/device.h |=C2=A0 4 +++ >> sys/compat/linuxkpi/common/src/linux_compat.c=C2=A0=C2=A0=C2=A0=C2=A0 = | 30 +++++++++++++++++++++++ >> 2 files changed, 34 insertions(+) >> >> diff --git a/sys/compat/linuxkpi/common/include/linux/device.h=20 >> 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 *d= ev, void=20 >> (*action)(void *), vo >> #define=C2=A0=C2=A0=C2=A0 devm_add_action_or_reset(dev, action, data)=C2= =A0=C2=A0=C2=A0 \ >> =C2=A0=C2=A0=C2=A0=C2=A0lkpi_devm_add_action_or_reset(dev, action, dat= a) >> >> +int lkpi_devm_device_add_group(struct device *dev, const struct=20 >> attribute_group *group); >> +#define=C2=A0=C2=A0=C2=A0 devm_device_add_group(dev, group)=C2=A0=C2=A0= =C2=A0 \ >> +=C2=A0=C2=A0=C2=A0 lkpi_devm_device_add_group(dev, group) >=20 > I thought we would use linuxkpi_ as prefix for publicly exposed LinuxKP= I > related functions and lkpi_ for "internal" FreeBSD specific code which > does not follow Linux KPI? >=20 And I thought that we use lkpi_ as HPS told me that once. And really if we take a look at sys/compat/linuxkpi/common/src % grep ^linux_ * | wc -l 247 % grep ^lkpi_ * | wc -l 228 % grep ^linuxkpi_ * | wc -l 99 lkpi_ is used twice more times. I would prefer to use lkpi_ as prefix for publicly exposed functions and = _lkpi_=20 or __lkpi_ for internal code as lkpi is four symbols shorter. > Can this be fixed before MFC? >=20 Yes if you insist on that. --=20 WBR Vladimir Kondratyev
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?75796855-cf79-4538-81c5-ae6f7765d2d3>