On 2/17/2018 7:29 AM, Andy Shevchenko wrote:
> On Fri, Feb 16, 2018 at 11:55 PM, Thinh Nguyen
> <thinh.ngu...@synopsys.com> wrote:
>> Add the ability to add device properties dynamically. Currently, device
>> properties are added to platform device using
>> platform_device_add_properties(). However, this function does not allow
>> adding properties incrementally. It is useful to have this ability when
>> the driver needs to set common device properties across different HW
> 
> I'm not sure it's useful anyhow.
> 
>> or
>> if IP and FPGA validation test different configurations for different
>> HW.
> 
> Shouldn't be a separate stuff for FPGA exclusively?

Can you clarify/expand your question?

> 
>> To address this issue, update dwc3_pci to store device properties to
>> an array and dynamically manage them here.
>>
>> Introduce two new functions to do so:
>>   * dwc3_pci_add_one_property() - this function adds one property to
>>     dwc->properties array and increase its size as needed
>>   * dwc3_pci_add_properties() - this function takes a null terminated
>>     array of device properties and add them to dwc->properties
> 
> So, why you can't use ACPI / DT here?
> 

dwc3_pci_add_properties() is a convenient function that takes statically 
allocated array of (quirks) properties for different HW and store them 
to dwc->properties. The idea is to add more properties on top of these 
required quirks.

Thanks,
Thinh
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to