All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Alexander Tsoy <alexander@tsoy.me>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH 10/10] ALSA: usb-audio: Replace blacklist word
Date: Wed, 15 Jul 2020 19:20:25 +0200	[thread overview]
Message-ID: <s5himeopuee.wl-tiwai@suse.de> (raw)
In-Reply-To: <e1282824eaa7ab0b220b44e5573e3ed0b8c34b9e.camel@tsoy.me>

On Wed, 15 Jul 2020 19:04:37 +0200,
Alexander Tsoy wrote:
> 
> В Вт, 14/07/2020 в 19:26 +0200, Takashi Iwai пишет:
> > Follow the recent inclusive terminology guidelines and replace the
> > word "blacklist" appropriately.
> > 
> > Only a comment fix, no functional changes.
> > 
> > Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > ---
> >  sound/usb/quirks-table.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
> > index 4ec491011b19..336d81bb5fd1 100644
> > --- a/sound/usb/quirks-table.h
> > +++ b/sound/usb/quirks-table.h
> > @@ -127,7 +127,7 @@
> >  /*
> >   * HP Wireless Audio
> >   * When not ignored, causes instability issues for some users,
> > forcing them to
> > - * blacklist the entire module.
> > + * skip the entire module.
> 
> Maybe something like "disable loading" is more appropriate here?
> "blacklist" here means literally "blacklist" option in modprobe.d
> or modprobe.blacklist= cmdline option.

Well, in the end the behavior is same: the driver skips the probe and
quits.  I don't mind much about the choice of the words, but IMO the
difference here is too subtle to discuss.


thanks,

Takashi

      reply	other threads:[~2020-07-15 17:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-14 17:26 [PATCH 00/10] ALSA: Replace words for inclusive terminalogy guideline Takashi Iwai
2020-07-14 17:26 ` [PATCH 01/10] ALSA: Replace the word "slave" in vmaster API Takashi Iwai
2020-07-14 18:52   ` Pierre-Louis Bossart
2020-07-15  6:59     ` Takashi Iwai
2020-07-15 14:34       ` Pierre-Louis Bossart
2020-07-15 16:41         ` Takashi Iwai
2020-07-14 17:26 ` [PATCH 02/10] ALSA: ac97: Replace the words white/blacklist Takashi Iwai
2020-07-14 17:26 ` [PATCH 03/10] ALSA: bt87x: " Takashi Iwai
2020-07-14 17:26 ` [PATCH 04/10] ALSA: ctxfi: Replace the word blacklist Takashi Iwai
2020-07-14 17:26 ` [PATCH 05/10] ALSA: es1968: Replace the words white/blacklist Takashi Iwai
2020-07-14 17:26 ` [PATCH 06/10] ALSA: hda: " Takashi Iwai
2020-07-14 17:26 ` [PATCH 07/10] ALSA: intel8x0: " Takashi Iwai
2020-07-14 17:26 ` [PATCH 08/10] ALSA: nm256: Replace the words blacklist Takashi Iwai
2020-07-14 17:26 ` [PATCH 09/10] ALSA: via82xx: Replace the words whitelist Takashi Iwai
2020-07-14 17:26 ` [PATCH 10/10] ALSA: usb-audio: Replace blacklist word Takashi Iwai
2020-07-15 17:04   ` Alexander Tsoy
2020-07-15 17:20     ` Takashi Iwai [this message]

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=s5himeopuee.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alexander@tsoy.me \
    --cc=alsa-devel@alsa-project.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.