All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding@gmail.com>
To: Svyatoslav Ryhel <clamor95@gmail.com>
Cc: Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com>,
	Tom Warren <twarren@nvidia.com>, Marek Vasut <marex@denx.de>,
	Maxim Schwalm <maxim.schwalm@gmail.com>,
	Dmitry Osipenko <digetx@gmail.com>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>,
	Michal Simek <michal.simek@amd.com>, Stefan Roese <sr@denx.de>,
	Eugen Hristev <eugen.hristev@microchip.com>,
	Michael Walle <michael@walle.cc>, Simon Glass <sjg@chromium.org>,
	Jim Liu <jim.t90615@gmail.com>,
	William Zhang <william.zhang@broadcom.com>,
	Rick Chen <rick@andestech.com>,
	Stefan Herbrechtsmeier <stefan.herbrechtsmeier@weidmueller.com>,
	Andre Przywara <andre.przywara@arm.com>,
	Jaehoon Chung <jh80.chung@samsung.com>,
	u-boot@lists.denx.de
Subject: Re: [PATCH v6 0/3] Timer support for ARM Tegra
Date: Wed, 25 Jan 2023 17:41:08 +0100	[thread overview]
Message-ID: <Y9FbpLKesEN/40FC@orome> (raw)
In-Reply-To: <20230124065751.5973-1-clamor95@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2364 bytes --]

On Tue, Jan 24, 2023 at 08:57:48AM +0200, Svyatoslav Ryhel wrote:
> - ARM: tegra: remap clock_osc_freq for all Tegra family
> Enum clock_osc_freq was designed to use only with T20.
> This patch remaps it to use additional frequencies, added in
> T30+ SoC while maintaining backwards compatibility with T20.
> 
> - drivers: timer: add timer driver for ARMv7 based Tegra devices
> Add timer support for T20/T30/T114 and T124 based devices.
> Driver is based on DM, has device tree support and can be
> used on SPL and early boot stage.
> 
> - ARM: tegra: include timer as default option
> Enable TIMER as default option for all Tegra devices and
> enable TEGRA_TIMER for TEGRA_ARMV7_COMMON. Additionally
> enable SPL_TIMER if build as SPL part and drop deprecated
> configs from common header.
> 
> P. S. I have no arm64 Tegra and according to comment in 
> tegra-common.h
> Use the Tegra US timer on ARMv7, but the architected timer on ARMv8.
> 
> Svyatoslav Ryhel (3):
>   ARM: tegra: remap clock_osc_freq for all Tegra family
>   drivers: timer: add timer driver for ARMv7 based Tegra devices
>   ARM: tegra: include timer as default option

This causes a regression on Tegra210 (Jetson TX1). I'm trying to
investigate, but it's complicated by the fact that I'm not getting out
any debug prints, so I suspect the issue is happening quite early.

Thierry

> 
>  arch/arm/Kconfig                        |   1 +
>  arch/arm/include/asm/arch-tegra/clock.h |   9 +-
>  arch/arm/mach-tegra/Kconfig             |   2 +
>  arch/arm/mach-tegra/clock.c             |  17 +++-
>  arch/arm/mach-tegra/cpu.c               |  70 ++++++++++---
>  arch/arm/mach-tegra/tegra114/clock.c    |  13 +--
>  arch/arm/mach-tegra/tegra124/clock.c    |  13 +--
>  arch/arm/mach-tegra/tegra20/clock.c     |   4 +-
>  arch/arm/mach-tegra/tegra210/clock.c    |  22 +----
>  arch/arm/mach-tegra/tegra30/clock.c     |  10 +-
>  drivers/timer/Kconfig                   |   8 ++
>  drivers/timer/Makefile                  |   1 +
>  drivers/timer/tegra-timer.c             | 126 ++++++++++++++++++++++++
>  drivers/usb/host/ehci-tegra.c           |  46 +++++++--
>  include/configs/tegra-common.h          |   6 --
>  15 files changed, 269 insertions(+), 79 deletions(-)
>  create mode 100644 drivers/timer/tegra-timer.c
> 
> -- 
> 2.37.2
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  parent reply	other threads:[~2023-01-25 16:41 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-24  6:57 [PATCH v6 0/3] Timer support for ARM Tegra Svyatoslav Ryhel
2023-01-24  6:57 ` [PATCH v6 1/3] ARM: tegra: remap clock_osc_freq for all Tegra family Svyatoslav Ryhel
2023-01-24  6:57 ` [PATCH v6 2/3] drivers: timer: add timer driver for ARMv7 based Tegra devices Svyatoslav Ryhel
2023-01-24  6:57 ` [PATCH v6 3/3] ARM: tegra: include timer as default option Svyatoslav Ryhel
2023-01-25 16:41 ` Thierry Reding [this message]
2023-01-26 10:34   ` [PATCH v6 0/3] Timer support for ARM Tegra Thierry Reding
2023-01-26 11:40     ` Thierry Reding
2023-01-26 16:49       ` Tom Warren
2023-01-26 17:12         ` Svyatoslav Ryhel
2023-01-26 17:58           ` Thierry Reding
2023-01-26 18:10             ` Svyatoslav Ryhel
2023-01-26 22:12               ` Tom Warren
2023-01-26 18:28             ` Svyatoslav Ryhel
2023-01-26 17:08     ` Svyatoslav Ryhel
2023-01-26 17:54       ` Thierry Reding
2023-01-26 22:00         ` Dmitry Osipenko
2023-01-26 22:12           ` Dmitry Osipenko
2023-01-27  4:51             ` Svyatoslav Ryhel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Y9FbpLKesEN/40FC@orome \
    --to=thierry.reding@gmail.com \
    --cc=andre.przywara@arm.com \
    --cc=clamor95@gmail.com \
    --cc=digetx@gmail.com \
    --cc=eugen.hristev@microchip.com \
    --cc=jh80.chung@samsung.com \
    --cc=jim.t90615@gmail.com \
    --cc=marex@denx.de \
    --cc=maxim.schwalm@gmail.com \
    --cc=michael@walle.cc \
    --cc=michal.simek@amd.com \
    --cc=rayagonda.kokatanur@broadcom.com \
    --cc=rick@andestech.com \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=stefan.herbrechtsmeier@weidmueller.com \
    --cc=twarren@nvidia.com \
    --cc=u-boot@lists.denx.de \
    --cc=william.zhang@broadcom.com \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.