On 03/06/2024 10:51, Jung Daehwan wrote: >>>>> >>>>> + if (device_property_read_bool(tmpdev, "write-64-hi-lo-quirk")) >>>>> + xhci->quirks |= XHCI_WRITE_64_HI_LO; >>>> >>>> Where is any user of this property (DTS)? Just to clarify: your >>>> downstream does not matter really. >>>> >>> >>> This is set by dwc3 parent node by software node. >>> >>> [PATCH v2 1/5] dt-bindings: usb: snps,dwc3: Add 'snps,xhci-write-64-hi-lo-quirk' quirk >>> https://lore.kernel.org/r/1717135657-120818-2-git-send-email-dh10.jung@xxxxxxxxxxx/ >> >> This is not a patch to DTS. >> >> > > This is set by software node from dwc3. That's why I think this patch doesn't > need DTS patch. I would add DTS patch in dwc3 not xhci if it's needed. > What? I asked you question which upstream SoC (link to DTS) uses it, and you say that "no need for DTS patch"? That's not an answer. Best regards, Krzysztof