<div dir="ltr"><div>Hi Brian</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Nov 2018 at 00:44, Brian Dodge <<a href="mailto:bdodge09@gmail.com">bdodge09@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Thank you Pavel, that is a good point.<br>
<br>
The chip vendor has indicated that there is no reason to maintain the <br>
old/improper prefix and wishes to go forward (only) with the "arctic" <br>
prefix and any existing dts files are or will be updated.<br></blockquote><div><br></div><div>Looks like this patch series has fallen into the cracks a little.</div><div><br></div><div>I think I assumed this info would end in the description of patch v2 1/3 (in order to answer Rob's feedback) and I sat and waited for a respin. On the other hand... I didn't actually say that explicitly anywhere! So... I'd recommend a respin perhaps with a small bit of text explaining how the vendor can state that any existing dts files will be updated. This is a peripheral device so these strings are probably embedded into OEM devicetrees rather than exclusively under the control of the vendor.</div><div><br></div><div><br></div><div>Daniel.</div><div><br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On 11/11/18 6:30 AM, Pavel Machek wrote:<br>
> Hi!<br>
><br>
>> The vendor-prefixes.txt file properly refers to ArcticSand<br>
>> as arctic but the driver improperly abbreviated the prefix<br>
>> to arc. This was a mistake in the original patch<br>
>><br>
>> Signed-off-by: Brian Dodge <<a href="mailto:bdodge09@gmail.com" target="_blank">bdodge09@gmail.com</a>><br>
>> ---<br>
>>   drivers/video/backlight/arcxcnn_bl.c | 22 +++++++++++-----------<br>
>>   1 file changed, 11 insertions(+), 11 deletions(-)<br>
>><br>
>>    *<br>
>> - * Copyright 2016 ArcticSand, Inc.<br>
>> - * Author : Brian Dodge <<a href="mailto:bdodge@arcticsand.com" target="_blank">bdodge@arcticsand.com</a>><br>
>> + * Copyright 2018 pSemi, Inc.<br>
>> + * Author : Brian Dodge <<a href="mailto:bdodge@psemi.com" target="_blank">bdodge@psemi.com</a>><br>
> Ummm. Copyright 2016-2018?<br>
><br>
>> @@ -202,27 +202,27 @@ static void arcxcnn_parse_dt(struct arcxcnn *lp)<br>
>>      if (ret == 0)<br>
>>              lp->pdata->initial_brightness = prog_val;<br>
>><br>
>> -    ret = of_property_read_u32(node, "arc,led-config-0", &prog_val);<br>
>> +    ret = of_property_read_u32(node, "arctic,led-config-0", &prog_val);<br>
>>      if (ret == 0)<br>
>>              lp->pdata->led_config_0 = (u8)prog_val;<br>
>><br>
> If there's a dts using this, you want to update it at the same<br>
> time. You may want to support both names going forward.<br>
>                                                                       Pavel<br>
</blockquote></div></div>