Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1714081 > unrolled thread

Re: [PATCH 1/3] sound: ASoC: tegra: Remove SoC-specific Kconfig depends and selects

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 1/3] sound: ASoC: tegra: Remove SoC-specific Kconfig  depends and selects Thierry Reding <thierry.reding@gmail.com> - 2017-08-17 16:30 +0200
    Re: [PATCH 1/3] sound: ASoC: tegra: Remove SoC-specific Kconfig  depends and selects Mark Brown <broonie@kernel.org> - 2017-08-17 16:30 +0200
      Re: [PATCH 1/3] sound: ASoC: tegra: Remove SoC-specific Kconfig  depends and selects Paul Kocialkowski <contact@paulk.fr> - 2017-08-17 21:10 +0200

#1714081 — Re: [PATCH 1/3] sound: ASoC: tegra: Remove SoC-specific Kconfig depends and selects

[Multipart message — attachments visible in raw view] — view raw

On Mon, Jul 31, 2017 at 12:23:09PM +0200, Thierry Reding wrote:
> On Sun, Jul 30, 2017 at 01:22:18PM +0200, Paul Kocialkowski wrote:
> > This removes the SoC-specific dependencies on the platform drivers,
> > as well as SoC-specific selections of platform drivers for the
> > machine drivers. The rationale behind this change is that the
> > dependencies are not actual build dependencies but run-time ones.
> > 
> > The previously listed SoCs were also incomplete: for instance, tegra124
> > uses the tegra30 platform drivers, which could not be built without
> > ARCH_TEGRA_3x_SOC set.
> > 
> > Descriptions of the tristates are also added to allow these options
> > to be set via defconfig/menuconfig.
> > 
> > Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> > ---
> >  sound/soc/tegra/Kconfig | 42 ++++++++++++++----------------------------
> >  1 file changed, 14 insertions(+), 28 deletions(-)
> 
> Acked-by: Thierry Reding <treding@nvidia.com>

Hi Mark,

do you want to pick this up, or would you rather provide an Acked-by so
that I can take it through the Tegra tree along with the defconfig
changes? I can provide a branch with this patch in case we ever need it
for conflict resolution.

Thierry

[toc] | [next] | [standalone]

#1714082

[Multipart message — attachments visible in raw view] — view raw

On Thu, Aug 17, 2017 at 04:22:11PM +0200, Thierry Reding wrote:

> do you want to pick this up, or would you rather provide an Acked-by so
> that I can take it through the Tegra tree along with the defconfig
> changes? I can provide a branch with this patch in case we ever need it
> for conflict resolution.

I don't know what this patch is, sorry.  I think I probably deleted the
mail since the subject line indicates it's not for me.

Please submit patches using subject lines reflecting the style for the
subsystem.  This makes it easier for people to identify relevant
patches.  Look at what existing commits in the area you're changing are
doing and make sure your subject lines visually resemble what they're
doing.

[toc] | [prev] | [next] | [standalone]

#1714312

[Multipart message — attachments visible in raw view] — view raw

Le jeudi 17 août 2017 à 15:26 +0100, Mark Brown a écrit :
> On Thu, Aug 17, 2017 at 04:22:11PM +0200, Thierry Reding wrote:
> 
> > do you want to pick this up, or would you rather provide an Acked-by 
> > so
> > that I can take it through the Tegra tree along with the defconfig
> > changes? I can provide a branch with this patch in case we ever need
> > it
> > for conflict resolution.
> 
> I don't know what this patch is, sorry.  I think I probably deleted
> the
> mail since the subject line indicates it's not for me.
> 
> Please submit patches using subject lines reflecting the style for the
> subsystem.  This makes it easier for people to identify relevant
> patches.  Look at what existing commits in the area you're changing
> are
> doing and make sure your subject lines visually resemble what they're
> doing.

I have just sent out another version with a subject line that matches
what others have been doing (mainly, dropping the "sound:" prefix).

Cheers,

-- 
Paul Kocialkowski,

developer of free digital technology and hardware support.

Website: https://www.paulk.fr/
Coding blog: https://code.paulk.fr/
Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web