On Wed, Aug 06, 2025 at 11:46:04AM +0530, Rangoju, Raju wrote: > > > On 7/28/2025 12:17 PM, Mika Westerberg wrote: > > Hi, > > > > On Tue, Jul 22, 2025 at 11:20:23PM +0530, Raju Rangoju wrote: > > > This patch series aims to update vendor properties for XDomain > > > dynamically for vendors like AMD, Intel and ASMedia. > > > > The XDomain properties pretty much describe "software" not the underlying > > hardware so I don't understand why this is needed? We could have some USB > > IF registered Linux specific ID there but I don't see why this matters at > > all. > > Currently, it is showing up as "Intel" on AMD host controllers during > inter-domain connection. I suppose an alternative is to just call it "Linux" > or "Linux Connection Manager" to ensure we accurately represent the > connections across different systems. > > I appreciate your guidance on this and suggestions you might have. Yeah, something like that (I prefer "Linux"). The "ID" still is 0x8086 though but I don't think that matters. AFAIK we have other "donated" IDs in use in Linux. Let me check on our side if that's okay. > > > > > > Raju Rangoju (3): > > > thunderbolt: Dynamically populate vendor properties for XDomain > > > PCI: Add PCI vendor ID for ASMedia USB4 devices > > > thunderbolt: Add vendor ASMedia in update_property_block for XDomain > > > > > > drivers/thunderbolt/nvm.c | 2 +- > > > drivers/thunderbolt/xdomain.c | 32 +++++++++++++++++++++----------- > > > include/linux/pci_ids.h | 1 + > > > 3 files changed, 23 insertions(+), 12 deletions(-) > > > > > > -- > > > 2.34.1
