All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>
To: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [GIT PULL 1/4] soc/tegra: Changes for v4.13-rc1
Date: Sun, 18 Jun 2017 21:00:45 -0700	[thread overview]
Message-ID: <20170619040045.GD30022@localhost> (raw)
In-Reply-To: <20170616081306.11505-1-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Hi,

On Fri, Jun 16, 2017 at 10:13:03AM +0200, Thierry Reding wrote:
> Hi ARM SoC maintainers,
> 
> The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
> 
>   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git tags/tegra-for-4.13-soc
> 
> for you to fetch changes up to da1dbec1be2b54c16649e33e479708a55156e311:
> 
>   soc/tegra: flowctrl: Fix error handling (2017-06-13 16:47:44 +0200)
> 
> Thanks,
> Thierry
> 
> ----------------------------------------------------------------
> soc/tegra: Changes for v4.13-rc1
> 
> This contains an implementation of generic PM domains for Tegra186,
> based on the BPMP powergate request.
> 
> ----------------------------------------------------------------
> Christophe Jaillet (1):
>       soc/tegra: flowctrl: Fix error handling
> 
> Thierry Reding (3):
>       PM / Domains: Allow overriding the ->xlate() callback
>       soc/tegra: bpmp: Update ABI header
>       soc/tegra: bpmp: Implement generic PM domains
> 
>  drivers/base/power/domain.c        |   8 +-
>  drivers/firmware/tegra/bpmp.c      |   4 +
>  drivers/soc/tegra/Kconfig          |   5 +
>  drivers/soc/tegra/Makefile         |   1 +
>  drivers/soc/tegra/flowctrl.c       |   2 +-
>  drivers/soc/tegra/powergate-bpmp.c | 359 +++++++++++++++++++++++++++++++
>  include/linux/pm_domain.h          |   4 +
>  include/soc/tegra/bpmp-abi.h       | 418 ++++++++++++++++++++++++++++++++++++-
>  include/soc/tegra/bpmp.h           |  12 ++

Merged, but does the bpmp-abi.h really belong in include/soc? Seems like most
of it is internal ABI for whatever abstraction communicates with bpmp, in this
case powergate. Locating it as a headerfile under drivers/soc/tegra/ seems to
make more sense. Care to move it?


-Olof

WARNING: multiple messages have this Message-ID (diff)
From: olof@lixom.net (Olof Johansson)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL 1/4] soc/tegra: Changes for v4.13-rc1
Date: Sun, 18 Jun 2017 21:00:45 -0700	[thread overview]
Message-ID: <20170619040045.GD30022@localhost> (raw)
In-Reply-To: <20170616081306.11505-1-thierry.reding@gmail.com>

Hi,

On Fri, Jun 16, 2017 at 10:13:03AM +0200, Thierry Reding wrote:
> Hi ARM SoC maintainers,
> 
> The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:
> 
>   Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git tags/tegra-for-4.13-soc
> 
> for you to fetch changes up to da1dbec1be2b54c16649e33e479708a55156e311:
> 
>   soc/tegra: flowctrl: Fix error handling (2017-06-13 16:47:44 +0200)
> 
> Thanks,
> Thierry
> 
> ----------------------------------------------------------------
> soc/tegra: Changes for v4.13-rc1
> 
> This contains an implementation of generic PM domains for Tegra186,
> based on the BPMP powergate request.
> 
> ----------------------------------------------------------------
> Christophe Jaillet (1):
>       soc/tegra: flowctrl: Fix error handling
> 
> Thierry Reding (3):
>       PM / Domains: Allow overriding the ->xlate() callback
>       soc/tegra: bpmp: Update ABI header
>       soc/tegra: bpmp: Implement generic PM domains
> 
>  drivers/base/power/domain.c        |   8 +-
>  drivers/firmware/tegra/bpmp.c      |   4 +
>  drivers/soc/tegra/Kconfig          |   5 +
>  drivers/soc/tegra/Makefile         |   1 +
>  drivers/soc/tegra/flowctrl.c       |   2 +-
>  drivers/soc/tegra/powergate-bpmp.c | 359 +++++++++++++++++++++++++++++++
>  include/linux/pm_domain.h          |   4 +
>  include/soc/tegra/bpmp-abi.h       | 418 ++++++++++++++++++++++++++++++++++++-
>  include/soc/tegra/bpmp.h           |  12 ++

Merged, but does the bpmp-abi.h really belong in include/soc? Seems like most
of it is internal ABI for whatever abstraction communicates with bpmp, in this
case powergate. Locating it as a headerfile under drivers/soc/tegra/ seems to
make more sense. Care to move it?


-Olof

  parent reply	other threads:[~2017-06-19  4:00 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-16  8:13 [GIT PULL 1/4] soc/tegra: Changes for v4.13-rc1 Thierry Reding
2017-06-16  8:13 ` Thierry Reding
     [not found] ` <20170616081306.11505-1-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-06-16  8:13   ` [GIT PULL 2/4] firmware: tegra: " Thierry Reding
2017-06-16  8:13     ` Thierry Reding
     [not found]     ` <20170616081306.11505-2-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-06-19  4:01       ` Olof Johansson
2017-06-19  4:01         ` Olof Johansson
2017-06-16  8:13   ` [GIT PULL 3/4] ARM: tegra: Device tree changes " Thierry Reding
2017-06-16  8:13     ` Thierry Reding
     [not found]     ` <20170616081306.11505-3-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-06-19  4:01       ` Olof Johansson
2017-06-19  4:01         ` Olof Johansson
2017-06-16  8:13   ` [GIT PULL 4/4] arm64: " Thierry Reding
2017-06-16  8:13     ` Thierry Reding
     [not found]     ` <20170616081306.11505-4-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-06-19  4:02       ` Olof Johansson
2017-06-19  4:02         ` Olof Johansson
2017-06-19  4:00   ` Olof Johansson [this message]
2017-06-19  4:00     ` [GIT PULL 1/4] soc/tegra: Changes " Olof Johansson
2017-06-19  5:59     ` Mikko Perttunen
2017-06-19  5:59       ` Mikko Perttunen
     [not found]       ` <7de8de14-b0ad-883c-58ec-847789be118b-/1wQRMveznE@public.gmane.org>
2017-06-19  6:04         ` Olof Johansson
2017-06-19  6:04           ` Olof Johansson

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=20170619040045.GD30022@localhost \
    --to=olof-nzht3qvonbneowh0uzbu5w@public.gmane.org \
    --cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /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.