On Tue, Jun 04, 2024 at 08:20:44AM +0200, Krzysztof Kozlowski wrote: > 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 > > I'm sorry I didn't get your point. I've been working on Exynos SoC. But there's no upstream user of this property yet in this patchset. Best Regards, Jung Daehwan
