NAK: [PATCH 0/3][SRU][J] Add ODM driver f81604 usb-can

[Tim Gardner]

On 12/27/23 8:29 PM, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao at canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/2045387
> 
> [ Impact ]
> f81604 driver needed for ODM partner
> 
> [ Test Plan ]
> Install can-utils
> 
> apt install can-utils
> 
> insert the driver:
> modprobe f81604
> 
> two socket can interfaces will be created, can0 and can1
> 
> connect the DB9 cable and termination provided.
> Set up the two interfaces:
> ip link set can1 up type can bitrate 500000
> ip link set can1 up type can bitrate 500000
> candump can0
> 
> In another shell run:
> cansend can1 123#DEADBEEF
> 
> The first shell should report the received frame. Switch the interfaces in
> the last commands to test the frame flow in the other way around.
> 
> [ Where problems could occur ]
> USB bus miss-behavior could cause issues to the driver.
> 
> Chia-Lin Kao (AceLan) (1):
>    UBUNTU: [Config] updateconfigs for ODM drivers CONFIG_CAN_F81604
> 
> Filippo Copetti (1):
>    UBUNTU: SAUCE: ODM: can: usb: f81604: backport of Fintek F81604
> 
> Ji-Ze Hong (1):
>    can: usb: f81604: add Fintek F81604 support
> 
>   MAINTAINERS                      |    6 +
>   debian.master/config/annotations |    1 +
>   drivers/net/can/usb/Kconfig      |   12 +
>   drivers/net/can/usb/Makefile     |    1 +
>   drivers/net/can/usb/f81604.c     | 1204 ++++++++++++++++++++++++++++++
>   5 files changed, 1224 insertions(+)
>   create mode 100644 drivers/net/can/usb/f81604.c
> 

Patches 1 and 2 should be squashed and declared a backport. Patch 1 as a 
standalone cherry pick will not compile.
-- 
-----------
Tim Gardner
Canonical, Inc