[GIT,PULL,4/9] soc/tegra: Changes for v5.1-rc1

Message ID 20190208150935.32108-5-thierry.reding@gmail.com
State New
Headers show
Series
  • Tegra changes for v5.1-rc1
Related show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git tags/tegra-for-5.1-soc

Message

Thierry Reding Feb. 8, 2019, 3:09 p.m.
Hi ARM SoC maintainers,

The following changes since commit bfeffd155283772bbe78c6a05dec7c0128ee500c:

  Linux 5.0-rc1 (2019-01-06 17:08:20 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git tags/tegra-for-5.1-soc

for you to fetch changes up to e247deae1a55089cb04cc25c91faeba083d0c39c:

  soc/tegra: pmc: Support systems where PMC is marked secure (2019-01-25 16:18:24 +0100)

Thanks,
Thierry

----------------------------------------------------------------
soc/tegra: Changes for v5.1-rc1

This contains a couple of miscellaneous fixes for minor issues and a
largish rework of the PMC driver to make it work on systems where the
PMC has been locked down and can only be accessed from secure firmware.

----------------------------------------------------------------
Jon Hunter (1):
      soc/tegra: pmc: Make tegra_powergate_is_powered() a local function

Joseph Lo (1):
      soc/tegra: fuse: Fix typo in tegra210_init_speedo_data

Mikko Perttunen (1):
      soc/tegra: pmc: Support systems where PMC is marked secure

Thierry Reding (6):
      soc/tegra: pmc: Use TEGRA186_ prefix for GPIO names
      soc/tegra: pmc: Sort includes alphabetically
      soc/tegra: pmc: Add missing kerneldoc
      soc/tegra: pmc: Pass struct tegra_pmc * where possible
      soc/tegra: pmc: Make alignment consistent
      soc/tegra: pmc: Explicitly initialize all fields

Timo Alho (1):
      soc/tegra: fuse: Fix illegal free of IO base address

 drivers/soc/tegra/fuse/fuse-tegra.c      |  12 +-
 drivers/soc/tegra/fuse/speedo-tegra210.c |   2 +-
 drivers/soc/tegra/pmc.c                  | 424 +++++++++++++++++++++----------
 include/soc/tegra/pmc.h                  |   6 -
 4 files changed, 298 insertions(+), 146 deletions(-)

Comments

Arnd Bergmann Feb. 15, 2019, 4:25 p.m. | #1
On Fri, Feb 8, 2019 at 4:09 PM Thierry Reding <thierry.reding@gmail.com> wrote:

> ----------------------------------------------------------------
> soc/tegra: Changes for v5.1-rc1
>
> This contains a couple of miscellaneous fixes for minor issues and a
> largish rework of the PMC driver to make it work on systems where the
> PMC has been locked down and can only be accessed from secure firmware.

Pulled into arm/drivers.

> ----------------------------------------------------------------
> Jon Hunter (1):
>       soc/tegra: pmc: Make tegra_powergate_is_powered() a local function
>
> Joseph Lo (1):
>       soc/tegra: fuse: Fix typo in tegra210_init_speedo_data
>
> Mikko Perttunen (1):
>       soc/tegra: pmc: Support systems where PMC is marked secure
>
> Thierry Reding (6):
>       soc/tegra: pmc: Use TEGRA186_ prefix for GPIO names
>       soc/tegra: pmc: Sort includes alphabetically
>       soc/tegra: pmc: Add missing kerneldoc
>       soc/tegra: pmc: Pass struct tegra_pmc * where possible
>       soc/tegra: pmc: Make alignment consistent
>       soc/tegra: pmc: Explicitly initialize all fields
>
> Timo Alho (1):
>       soc/tegra: fuse: Fix illegal free of IO base address

Should that last one have been marked 'Cc:stable' ?

       Arnd
Thierry Reding Feb. 15, 2019, 9:42 p.m. | #2
On Fri, Feb 15, 2019 at 05:25:39PM +0100, Arnd Bergmann wrote:
> On Fri, Feb 8, 2019 at 4:09 PM Thierry Reding <thierry.reding@gmail.com> wrote:
> 
> > ----------------------------------------------------------------
> > soc/tegra: Changes for v5.1-rc1
> >
> > This contains a couple of miscellaneous fixes for minor issues and a
> > largish rework of the PMC driver to make it work on systems where the
> > PMC has been locked down and can only be accessed from secure firmware.
> 
> Pulled into arm/drivers.
> 
> > ----------------------------------------------------------------
> > Jon Hunter (1):
> >       soc/tegra: pmc: Make tegra_powergate_is_powered() a local function
> >
> > Joseph Lo (1):
> >       soc/tegra: fuse: Fix typo in tegra210_init_speedo_data
> >
> > Mikko Perttunen (1):
> >       soc/tegra: pmc: Support systems where PMC is marked secure
> >
> > Thierry Reding (6):
> >       soc/tegra: pmc: Use TEGRA186_ prefix for GPIO names
> >       soc/tegra: pmc: Sort includes alphabetically
> >       soc/tegra: pmc: Add missing kerneldoc
> >       soc/tegra: pmc: Pass struct tegra_pmc * where possible
> >       soc/tegra: pmc: Make alignment consistent
> >       soc/tegra: pmc: Explicitly initialize all fields
> >
> > Timo Alho (1):
> >       soc/tegra: fuse: Fix illegal free of IO base address
> 
> Should that last one have been marked 'Cc:stable' ?

My understanding is that this currently only happens if we boot the
system using a DTB from a downstream build where these FUSE and BPMP
device tree nodes are sorted differently and therefore cause them to
be probed in the wrong order.

Booting an upstream kernel with a downstream DTB isn't known to work
very well in general. It's something that I hope we'll be able to do
in some distant future when the gap between upstream and downstream
DTBs has become sufficiently small, but I don't think we need to
bother fixing this in stable releases.

Adding Timo to clarify whether he thinks this is important enough to
backport.

Thierry