Re: [PATCH net-next 2/3] ptp: clockmatrix: Remove IDT references or replace with Renesas.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 18, 2019 at 01:29:56AM EST, David Miller wrote:
>From: vincent.cheng.xh@xxxxxxxxxxx
>Date: Tue, 17 Dec 2019 00:03:07 -0500
>
>> From: Vincent Cheng <vincent.cheng.xh@xxxxxxxxxxx>
>> 
>> Renesas Electronics Corporation completed acquisition of IDT in 2019.
>> 
>> This patch removes IDT references or replaces IDT with Renesas.
>> Renamed idt8a340_reg.h to clockmatrix_reg.h.
>> There were no functional changes.
>> 
>> Signed-off-by: Vincent Cheng <vincent.cheng.xh@xxxxxxxxxxx>
>
>Sorry, we don't do stuff like this.
>
>The driver shall keep the reference to it's old vendor name, and
>this is how we've handled similar situations in the past.

Sorry, was not aware.

>And do you know the worst part about this?  You DID in fact
>functionally change this driver:
>
>> -#define FW_FILENAME	"idtcm.bin"
>> +#define FW_FILENAME	"cm_tcs.bin"
>
>Now everyone would have missing firmware.
>
>So not only is this unacceptable on precedence grounds, and how we
>always handle situations like this, it's functionally wrong and would
>break things for users.
>
>Please remove this patch and resubmit #1 and #3 as a series for
>re-review.

What is the proper way to remove a patch submission?

Regards,
Vincent




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux