mbox series

[v2,0/4] J721E: Use external clock in EVM for SERDES

Message ID 20210603143427.28735-1-kishon@ti.com (mailing list archive)
Headers show
Series J721E: Use external clock in EVM for SERDES | expand

Message

Kishon Vijay Abraham I June 3, 2021, 2:34 p.m. UTC
J721E EVM has clock generator that feeds both to the SERDES and to the
PCIe slot present in the EVM. In order to use common reference clock on
either side of the link, configure SERDES to use external reference
clock.

Previously SERDES used internal reference clock and the attached device
used clock from clock generator in the EVM.

Changes from v1:
1) Fixed clock names as suggested by Nishanth
2) Limit to < 100 lines

v1: http://lore.kernell.org/r/20210512151209.27560-1-kishon@ti.com

Kishon Vijay Abraham I (4):
  arm64: dts: ti: k3-j721e-main: Fix external refclk input to SERDES
  arm64: dts: ti: k3-j721e-main: Add #clock-cells property to serdes DT
    node
  arm64: dts: ti: k3-j721e-common-proc-board: Use external clock for
    SERDES
  arm64: dts: ti: k3-j721e-common-proc-board: Re-name "link" name as
    "phy"

 .../dts/ti/k3-j721e-common-proc-board.dts     | 52 ++++++++++-
 arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     | 86 +++++++++++--------
 2 files changed, 98 insertions(+), 40 deletions(-)

Comments

Aswath Govindraju June 7, 2021, 2:38 p.m. UTC | #1
On 03/06/21 8:04 pm, Kishon Vijay Abraham I wrote:
> J721E EVM has clock generator that feeds both to the SERDES and to the
> PCIe slot present in the EVM. In order to use common reference clock on
> either side of the link, configure SERDES to use external reference
> clock.
> 
> Previously SERDES used internal reference clock and the attached device
> used clock from clock generator in the EVM.
> 
> Changes from v1:
> 1) Fixed clock names as suggested by Nishanth
> 2) Limit to < 100 lines
> 
> v1: http://lore.kernell.org/r/20210512151209.27560-1-kishon@ti.com
> 

For the whole series,

Reviewed-by: Aswath Govindraju <a-govindraju@ti.com>

Thanks,
Aswath

> Kishon Vijay Abraham I (4):
>   arm64: dts: ti: k3-j721e-main: Fix external refclk input to SERDES
>   arm64: dts: ti: k3-j721e-main: Add #clock-cells property to serdes DT
>     node
>   arm64: dts: ti: k3-j721e-common-proc-board: Use external clock for
>     SERDES
>   arm64: dts: ti: k3-j721e-common-proc-board: Re-name "link" name as
>     "phy"
> 
>  .../dts/ti/k3-j721e-common-proc-board.dts     | 52 ++++++++++-
>  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     | 86 +++++++++++--------
>  2 files changed, 98 insertions(+), 40 deletions(-)
>
Nishanth Menon June 8, 2021, 2:34 p.m. UTC | #2
On Thu, 3 Jun 2021 20:04:23 +0530, Kishon Vijay Abraham I wrote:
> J721E EVM has clock generator that feeds both to the SERDES and to the
> PCIe slot present in the EVM. In order to use common reference clock on
> either side of the link, configure SERDES to use external reference
> clock.
> 
> Previously SERDES used internal reference clock and the attached device
> used clock from clock generator in the EVM.
> 
> [...]

Hi Kishon Vijay Abraham I,

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/4] arm64: dts: ti: k3-j721e-main: Fix external refclk input to SERDES
      commit: 5c6d0b55b46aeb91355e6a9616decf50a3778c91
[2/4] arm64: dts: ti: k3-j721e-main: Add #clock-cells property to serdes DT node
      commit: 2427bfb335eb5f291a821e91c4c520351ce933df
[3/4] arm64: dts: ti: k3-j721e-common-proc-board: Use external clock for SERDES
      commit: f2a7657ad7a821de9cc77d071a5587b243144cd5
[4/4] arm64: dts: ti: k3-j721e-common-proc-board: Re-name "link" name as "phy"
      commit: 02b4d9186121d842a53e347f53a86ec7f2c6b0c7


All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git