ACK/Cmnt: [jammy xilinx-zynqmp v2 0/2] Add support for KD240 platform

Tim Gardner tim.gardner at canonical.com
Thu Jan 4 16:06:55 UTC 2024


On 1/3/24 4:56 PM, Portia Stephens wrote:
> [ Impact ]
> * This change adds support for the KD240 Kria platform.
> * It is the same SoC as other Kria/ZCU platforms
> * This change will enable us to certify the KD240 platform with the
> linux-xilinx-zynqmp kernel
> 
> [ Test Plan ]
> * Normal regression testing and certification testing will be run on currently
> certified platform
> * A release candidate KD240 image will be generated from -proposed and QA will
> run certification testing on it
> 
> [ Where problems could occur ]
> * The device tree changes include changes to dtsi's used by other certified
> Xilinx platforms.
> 
> [ Other Info ]
> * KD240 changes are backported from Xilinx 6.1 tree.
> 
> Portia Stephens (1):
>    Sync KD240 DTS from xlnx_rebase_v6.1_LTS
> 
> Srinivas Neeli (1):
>    can: xilinx_can: Add support for controller reset
> 
>   .../arm64/boot/dts/xilinx/zynqmp-clk-ccf.dtsi |  31 +--
>   .../boot/dts/xilinx/zynqmp-sck-kd-g-revA.dts  |   7 +-
>   .../boot/dts/xilinx/zynqmp-sck-kr-g-revA.dts  |   9 +-
>   .../boot/dts/xilinx/zynqmp-sck-kr-g-revB.dts  |  14 +-
>   .../boot/dts/xilinx/zynqmp-sck-kv-g-revA.dts  |   6 +-
>   .../boot/dts/xilinx/zynqmp-sck-kv-g-revB.dts  |   9 +-
>   .../boot/dts/xilinx/zynqmp-sm-k26-revA.dts    | 186 ++++++++--------
>   arch/arm64/boot/dts/xilinx/zynqmp.dtsi        | 199 ++++++++----------
>   drivers/net/can/xilinx_can.c                  |  22 +-
>   9 files changed, 247 insertions(+), 236 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>

Subsequent versions should have a description in the cover letter of the 
changes from one version to the next. For example,

v2: Include S-O-B from patch author in the chain of provenance

This is an impossible patch to review with all of the device tree 
changes. I trust that certification testing will catch any regressions.
-- 
-----------
Tim Gardner
Canonical, Inc




More information about the kernel-team mailing list