Wireless Daemon for Linux
 help / color / mirror / Atom feed
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 --]

  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