(cc-ing David Bartley, who wrote the MSI patch)
I had a follow-up question: I noticed that support for MSI was added in
December, but there's this conditional that wasn't changed:
/* Only update pwm values for Mitac boards */
if (data->customer_id == NCT6683_CUSTOMER_ID_MITAC)
return attr->mode | S_IWUSR;
Should this continue to only apply for Mitac boards, or should it be changed to
!= NCT6683_CUSTOMER_ID_INTEL?
Blaž
On 1/18/21 9:15 PM, Blaž Hrastnik wrote:
> (cc-ing David Bartley, who wrote the MSI patch)
>
> I had a follow-up question: I noticed that support for MSI was added in
> December, but there's this conditional that wasn't changed:
>
> /* Only update pwm values for Mitac boards */
> if (data->customer_id == NCT6683_CUSTOMER_ID_MITAC)
> return attr->mode | S_IWUSR;
>
> Should this continue to only apply for Mitac boards, or should it be changed to
> != NCT6683_CUSTOMER_ID_INTEL?
>
No, this has to be tested explicitly for each vendor.
Guenter