From: Clayton Craft <clayton@craftyguy.net>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: Denis Kenzior <denkenz@gmail.com>, iwd@lists.linux.dev
Subject: Re: [PATCH] wiphy: include libgen.h to fix implicit def. with musl libc
Date: Thu, 14 Mar 2024 15:30:55 -0700 [thread overview]
Message-ID: <20240314153055.GB8925@craftyguy.net> (raw)
In-Reply-To: <440488C8-0481-4A8F-BF68-A7F97FAADC8C@holtmann.org>
[-- Attachment #1: Type: text/plain, Size: 990 bytes --]
On Wed, 14 Feb 2024 17:52:43 +0100 Marcel Holtmann <marcel@holtmann.org> wrote:
> Hi Denis,
>
> >>>> Add a _get_driver is a good idea as well, but what is the reason for l_sysctl since sysctl is some specific system call.
> >>>
> >>> l_sysctl is used to write stuff into procfs or sysfs. Naming comes from the sysctl(8) utility.
> >> actually no. sysctl just handles /proc/sys/. It has nothing to do with sysfs.
> >
> > You're right about sysctl. ell used the same naming since it was meant to control net/ipv* kernel parameters. However, the same API can be used inside sysfs since the semantics are identical.
> >
> > If you don't like l_sysctl_get_driver, then lets call it l_sysfs_get_driver().
>
> yes, I would prefer l_sysfs_get_driver.
Thanks Marcel & Denis for the replies. It's not clear to me what needs to be
done to resolve the original issue I set out to fix with this patch. Could you
please summarize this so I (or someone else) could try to implement it?
-Clayton
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2024-03-14 22:31 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-12 18:47 [PATCH] wiphy: include libgen.h to fix implicit def. with musl libc Clayton Craft
2024-02-13 23:09 ` Denis Kenzior
2024-02-14 0:13 ` Clayton Craft
2024-02-14 7:03 ` Marcel Holtmann
2024-02-14 16:26 ` Denis Kenzior
2024-02-14 16:30 ` Marcel Holtmann
2024-02-14 16:35 ` Denis Kenzior
2024-02-14 16:39 ` Marcel Holtmann
2024-02-14 16:46 ` Denis Kenzior
2024-02-14 16:52 ` Marcel Holtmann
2024-03-14 22:30 ` Clayton Craft [this message]
2024-03-15 13:49 ` Denis Kenzior
2024-03-15 22:09 ` Clayton Craft
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240314153055.GB8925@craftyguy.net \
--to=clayton@craftyguy.net \
--cc=denkenz@gmail.com \
--cc=iwd@lists.linux.dev \
--cc=marcel@holtmann.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox