2024-01-17 13:44:57

by Daniel Abrecht

[permalink] [raw]
Subject: [PATCH v2] ASoC: soc-core.c: Add "Unknown" and "Unknown Product" to dmi_blacklist

In U-Boot, the default for DMI vendor / product if not set is "Unknown"
and "Unknown Product".
See
https://source.denx.de/u-boot/u-boot/-/blob/v2023.10/lib/smbios.c?ref_type=tags#L272

This patch adds them to the dmi_blacklist.

Signed-off-by: Daniel Abrecht <[email protected]>
Acked-by: Jaroslav Kysela <[email protected]>
---
sound/soc/soc-core.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
index f8524b5bfb33..aa1a31abbe3a 100644
--- a/sound/soc/soc-core.c
+++ b/sound/soc/soc-core.c
@@ -1804,6 +1804,8 @@ static const char * const dmi_blacklist[] = {
"Board Manufacturer",
"Board Vendor Name",
"Board Product Name",
+ "Unknown",
+ "Unknown Product",
NULL, /* terminator */
};

--
2.39.2


2024-01-22 00:07:08

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH v2] ASoC: soc-core.c: Add "Unknown" and "Unknown Product" to dmi_blacklist

On Wed, Jan 17, 2024 at 02:44:43PM +0100, Daniel Abrecht wrote:
> In U-Boot, the default for DMI vendor / product if not set is "Unknown" and
> "Unknown Product".

This doesn't apply against current code, please check and resend.


Attachments:
(No filename) (235.00 B)
signature.asc (499.00 B)
Download all attachments

2024-01-22 10:10:03

by Daniel Abrecht

[permalink] [raw]
Subject: Re: [PATCH v3] ASoC: soc-core.c: Add "Unknown" and "Unknown Product" to dmi_blacklist

I think this time it should work. It's surprisingly difficult to get roundcube to not alter plain text messages. Sorry about that.

2024-01-22 16:58:39

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH v2] ASoC: soc-core.c: Add "Unknown" and "Unknown Product" to dmi_blacklist

On Wed, Jan 17, 2024 at 02:44:43PM +0100, Daniel Abrecht wrote:
> In U-Boot, the default for DMI vendor / product if not set is "Unknown" and
> "Unknown Product".
> See https://source.denx.de/u-boot/u-boot/-/blob/v2023.10/lib/smbios.c?ref_type=tags#L272

This is still corrupted according to git am unfortunately. I can't
immediately spot the issue.


Attachments:
(No filename) (358.00 B)
signature.asc (499.00 B)
Download all attachments

2024-01-22 18:25:05

by Daniel Abrecht

[permalink] [raw]
Subject: Re: [PATCH v2] ASoC: soc-core.c: Add "Unknown" and "Unknown Product" to dmi_blacklist

Am 2024-01-22 16:30, schrieb Mark Brown:
> On Wed, Jan 17, 2024 at 02:44:43PM +0100, Daniel Abrecht wrote:
>> In U-Boot, the default for DMI vendor / product if not set is "Unknown" and
>> "Unknown Product".
>> See https://source.denx.de/u-boot/u-boot/-/blob/v2023.10/lib/smbios.c?ref_type=tags#L272
>
> This is still corrupted according to git am unfortunately. I can't
> immediately spot the issue.

The v3 I sent applies just fine with "git am" as far as I can tell. I just tested it again with the mail form the list.
I think the v3 patch is fine, but this time, the v3 message ended up with the same message ID as the v2 message.
Because of this, some mail clients may get confused, and the threading in mail clients is going to be broken.