Message ID | 20240906211445.3924724-1-joshuapius@google.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | ALSA: usb-audio: Add logitech Audio profile quirk | expand |
On Fri, 06 Sep 2024 23:14:38 +0200, Joshua Pius wrote: > > Specify shortnames for the following Logitech Devices: Rally bar, Rally > bar mini, Tap, MeetUp and Huddle. > > Signed-off-by: Joshua Pius <joshuapius@google.com> Is this change needed only for UCM profiles? UCM v2 should be able to handle better to identify models, and such short name updates aren't needed for them. OTOH, I don't mind much to take this kind of small harmless changes (unless it happens too frequently). So I'll likely take this, but just for verifying the situation. thanks, Takashi > --- > sound/usb/card.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/sound/usb/card.c b/sound/usb/card.c > index 778de9244f1e..9c411b82a218 100644 > --- a/sound/usb/card.c > +++ b/sound/usb/card.c > @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { > /* Creative/Toshiba Multimedia Center SB-0500 */ > DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), > > + /* Logitech Audio Devices */ > + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), > + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), > + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), > + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), > + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), > DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), > > DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"), > -- > 2.46.0.598.g6f2099f65c-goog >
Yes, this change is for UCM profiles. Yes this should be a one time occurrence as afterwards effort is being made to migrate over to UCM v2. Thanks, Joshua P On Sun, Sep 8, 2024 at 3:47 AM Takashi Iwai <tiwai@suse.de> wrote: > > On Fri, 06 Sep 2024 23:14:38 +0200, > Joshua Pius wrote: > > > > Specify shortnames for the following Logitech Devices: Rally bar, Rally > > bar mini, Tap, MeetUp and Huddle. > > > > Signed-off-by: Joshua Pius <joshuapius@google.com> > > Is this change needed only for UCM profiles? UCM v2 should be able to > handle better to identify models, and such short name updates aren't > needed for them. > > OTOH, I don't mind much to take this kind of small harmless changes > (unless it happens too frequently). So I'll likely take this, but > just for verifying the situation. > > > thanks, > > Takashi > > > > --- > > sound/usb/card.c | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/sound/usb/card.c b/sound/usb/card.c > > index 778de9244f1e..9c411b82a218 100644 > > --- a/sound/usb/card.c > > +++ b/sound/usb/card.c > > @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { > > /* Creative/Toshiba Multimedia Center SB-0500 */ > > DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), > > > > + /* Logitech Audio Devices */ > > + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), > > + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), > > + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), > > + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), > > + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), > > DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), > > > > DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"), > > -- > > 2.46.0.598.g6f2099f65c-goog > >
On Mon, 09 Sep 2024 04:03:36 +0200, Joshua Pius wrote: > > Yes, this change is for UCM profiles. > > Yes this should be a one time occurrence as afterwards effort is being > made to migrate over to UCM v2. OK, then I'll take it. But, I noticed that your Signed-off-by tag was with google.com address while you submitted from chromium.org. Could you align those? thanks, Takashi > > Thanks, > > Joshua P > > > > > > On Sun, Sep 8, 2024 at 3:47 AM Takashi Iwai <tiwai@suse.de> wrote: > > > > On Fri, 06 Sep 2024 23:14:38 +0200, > > Joshua Pius wrote: > > > > > > Specify shortnames for the following Logitech Devices: Rally bar, Rally > > > bar mini, Tap, MeetUp and Huddle. > > > > > > Signed-off-by: Joshua Pius <joshuapius@google.com> > > > > Is this change needed only for UCM profiles? UCM v2 should be able to > > handle better to identify models, and such short name updates aren't > > needed for them. > > > > OTOH, I don't mind much to take this kind of small harmless changes > > (unless it happens too frequently). So I'll likely take this, but > > just for verifying the situation. > > > > > > thanks, > > > > Takashi > > > > > > > --- > > > sound/usb/card.c | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/sound/usb/card.c b/sound/usb/card.c > > > index 778de9244f1e..9c411b82a218 100644 > > > --- a/sound/usb/card.c > > > +++ b/sound/usb/card.c > > > @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { > > > /* Creative/Toshiba Multimedia Center SB-0500 */ > > > DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), > > > > > > + /* Logitech Audio Devices */ > > > + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), > > > + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), > > > + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), > > > + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), > > > + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), > > > DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), > > > > > > DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"), > > > -- > > > 2.46.0.598.g6f2099f65c-goog > > >
Done. On Tue, Sep 10, 2024 at 4:25 AM Takashi Iwai <tiwai@suse.de> wrote: > > On Mon, 09 Sep 2024 04:03:36 +0200, > Joshua Pius wrote: > > > > Yes, this change is for UCM profiles. > > > > Yes this should be a one time occurrence as afterwards effort is being > > made to migrate over to UCM v2. > > OK, then I'll take it. But, I noticed that your Signed-off-by tag was > with google.com address while you submitted from chromium.org. > Could you align those? > > > thanks, > > Takashi > > > > > Thanks, > > > > Joshua P > > > > > > > > > > > > On Sun, Sep 8, 2024 at 3:47 AM Takashi Iwai <tiwai@suse.de> wrote: > > > > > > On Fri, 06 Sep 2024 23:14:38 +0200, > > > Joshua Pius wrote: > > > > > > > > Specify shortnames for the following Logitech Devices: Rally bar, Rally > > > > bar mini, Tap, MeetUp and Huddle. > > > > > > > > Signed-off-by: Joshua Pius <joshuapius@chromium.org> > > > > > > Is this change needed only for UCM profiles? UCM v2 should be able to > > > handle better to identify models, and such short name updates aren't > > > needed for them. > > > > > > OTOH, I don't mind much to take this kind of small harmless changes > > > (unless it happens too frequently). So I'll likely take this, but > > > just for verifying the situation. > > > > > > > > > thanks, > > > > > > Takashi > > > > > > > > > > --- > > > > sound/usb/card.c | 6 ++++++ > > > > 1 file changed, 6 insertions(+) > > > > > > > > diff --git a/sound/usb/card.c b/sound/usb/card.c > > > > index 778de9244f1e..9c411b82a218 100644 > > > > --- a/sound/usb/card.c > > > > +++ b/sound/usb/card.c > > > > @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { > > > > /* Creative/Toshiba Multimedia Center SB-0500 */ > > > > DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), > > > > > > > > + /* Logitech Audio Devices */ > > > > + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), > > > > + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), > > > > + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), > > > > + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), > > > > + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), > > > > DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), > > > > > > > > DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"), > > > > -- > > > > 2.46.0.598.g6f2099f65c-goog > > > >
On Wed, 11 Sep 2024 21:27:34 +0200, Joshua Pius wrote: > > Done. Err, I meant to resubmit a new patch with the consistent From and Signed-off-by tag. I haven't seen any new patch yet. Takashi > > > On Tue, Sep 10, 2024 at 4:25 AM Takashi Iwai <tiwai@suse.de> wrote: > > > > On Mon, 09 Sep 2024 04:03:36 +0200, > > Joshua Pius wrote: > > > > > > Yes, this change is for UCM profiles. > > > > > > Yes this should be a one time occurrence as afterwards effort is being > > > made to migrate over to UCM v2. > > > > OK, then I'll take it. But, I noticed that your Signed-off-by tag was > > with google.com address while you submitted from chromium.org. > > Could you align those? > > > > > > thanks, > > > > Takashi > > > > > > > > Thanks, > > > > > > Joshua P > > > > > > > > > > > > > > > > > > On Sun, Sep 8, 2024 at 3:47 AM Takashi Iwai <tiwai@suse.de> wrote: > > > > > > > > On Fri, 06 Sep 2024 23:14:38 +0200, > > > > Joshua Pius wrote: > > > > > > > > > > Specify shortnames for the following Logitech Devices: Rally bar, Rally > > > > > bar mini, Tap, MeetUp and Huddle. > > > > > > > > > > Signed-off-by: Joshua Pius <joshuapius@chromium.org> > > > > > > > > Is this change needed only for UCM profiles? UCM v2 should be able to > > > > handle better to identify models, and such short name updates aren't > > > > needed for them. > > > > > > > > OTOH, I don't mind much to take this kind of small harmless changes > > > > (unless it happens too frequently). So I'll likely take this, but > > > > just for verifying the situation. > > > > > > > > > > > > thanks, > > > > > > > > Takashi > > > > > > > > > > > > > --- > > > > > sound/usb/card.c | 6 ++++++ > > > > > 1 file changed, 6 insertions(+) > > > > > > > > > > diff --git a/sound/usb/card.c b/sound/usb/card.c > > > > > index 778de9244f1e..9c411b82a218 100644 > > > > > --- a/sound/usb/card.c > > > > > +++ b/sound/usb/card.c > > > > > @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { > > > > > /* Creative/Toshiba Multimedia Center SB-0500 */ > > > > > DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), > > > > > > > > > > + /* Logitech Audio Devices */ > > > > > + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), > > > > > + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), > > > > > + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), > > > > > + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), > > > > > + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), > > > > > DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), > > > > > > > > > > DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"), > > > > > -- > > > > > 2.46.0.598.g6f2099f65c-goog > > > > > >
diff --git a/sound/usb/card.c b/sound/usb/card.c index 778de9244f1e..9c411b82a218 100644 --- a/sound/usb/card.c +++ b/sound/usb/card.c @@ -384,6 +384,12 @@ static const struct usb_audio_device_name usb_audio_names[] = { /* Creative/Toshiba Multimedia Center SB-0500 */ DEVICE_NAME(0x041e, 0x3048, "Toshiba", "SB-0500"), + /* Logitech Audio Devices */ + DEVICE_NAME(0x046d, 0x0867, "Logitech, Inc.", "Logi-MeetUp"), + DEVICE_NAME(0x046d, 0x0874, "Logitech, Inc.", "Logi-Tap-Audio"), + DEVICE_NAME(0x046d, 0x087c, "Logitech, Inc.", "Logi-Huddle"), + DEVICE_NAME(0x046d, 0x0898, "Logitech, Inc.", "Logi-RB-Audio"), + DEVICE_NAME(0x046d, 0x08d2, "Logitech, Inc.", "Logi-RBM-Audio"), DEVICE_NAME(0x046d, 0x0990, "Logitech, Inc.", "QuickCam Pro 9000"), DEVICE_NAME(0x05e1, 0x0408, "Syntek", "STK1160"),
Specify shortnames for the following Logitech Devices: Rally bar, Rally bar mini, Tap, MeetUp and Huddle. Signed-off-by: Joshua Pius <joshuapius@google.com> --- sound/usb/card.c | 6 ++++++ 1 file changed, 6 insertions(+)