Re: [PATCH v3 1/4] dt-bindings: vendor-prefixes: Add fdhisi, titanmec, princeton, winrise, wxicore
From: Jean-François Lessard <hidden>
Date: 2025-08-22 02:35:21
Also in:
linux-leds, lkml
Le 21 août 2025 16 h 13 min 44 s HAE, Conor Dooley [off-list ref] a écrit :
On Thu, Aug 21, 2025 at 03:35:51PM -0400, Jean-François Lessard wrote:quoted
Le 20 août 2025 16 h 08 min 01 s HAE, Conor Dooley [off-list ref] a écrit :quoted
On Wed, Aug 20, 2025 at 12:31:14PM -0400, Jean-François Lessard wrote:quoted
Add vendor prefixes: - fdhisi: Fuzhou Fuda Hisi Microelectronics Co., Ltd. - titanmec: Shenzhen Titan Micro Electronics Co., Ltd. - princeton: Princeton Technology Corp. - winrise: Shenzhen Winrise Technology Co., Ltd. - wxicore: Wuxi i-Core Electronics Co., Ltd. The titanmec prefix is based on the company's domain name titanmec.com. The remaining prefixes are based on company names, as these manufacturers either lack active .com domains or their .com domains are occupied by unrelated businesses. The fdhisi and titanmec prefixes were originally identified by Andreas Färber. CC: Andreas Färber <afaerber@suse.de> Signed-off-by: Jean-François Lessard <redacted>Acked-by: Conor Dooley <conor.dooley@microchip.com>Thanks for your acknowledgement, I will include it in the v4 submission.quoted
but some reason why all these are being added together would be nice.Do you mean repeating this cover letter v3 changelog note in this patch: v3: - Update vendor prefixes with documented rationale, in a single patch, per maintainer feedback See https://lore.kernel.org/linux-devicetree/491ec8dd-8ca5-45bb-b5f4-dfd08a10e8de@kernel.org/#t (local) Or do you mean the relationship between these vendors being added here, such as replacing the fist line of the commit message to:Ye, this. The relation between the vendors.
Well received. I will update the commit message accordingly in v4.
quoted
Add vendor prefixes of chip manufacturers supported by the TM16xx 7-segment LED matrix display controllers driver: - fdhisi...quoted
quoted
---... Best regards, Jean-Francois Lessard