* [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-14 12:26 ` Paul Bolle
0 siblings, 0 replies; 9+ messages in thread
From: Paul Bolle @ 2013-07-14 12:26 UTC (permalink / raw)
To: Ben Dooks, Kukjin Kim, Sangbeom Kim, Liam Girdwood, Mark Brown,
Jaroslav Kysela, Takashi Iwai
Cc: linux-arm-kernel, linux-samsung-soc, alsa-devel, linux-kernel
Commit 383ffda2fa ("ARM: EXYNOS: no more support non-DT for EXYNOS
SoCs") removed the Kconfig symbols MACH_SMDKV310 and MACH_SMDKC210.
Now remove the references to both of these symbols from the Kconfig
entry for SND_SOC_SAMSUNG_SMDK_WM9713. They're not needed anymore.
Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
---
Untested.
sound/soc/samsung/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/samsung/Kconfig b/sound/soc/samsung/Kconfig
index 9855dfc..3d76659 100644
--- a/sound/soc/samsung/Kconfig
+++ b/sound/soc/samsung/Kconfig
@@ -136,7 +136,7 @@ config SND_SOC_SAMSUNG_RX1950_UDA1380
config SND_SOC_SAMSUNG_SMDK_WM9713
tristate "SoC AC97 Audio support for SMDK with WM9713"
- depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
+ depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
select SND_SOC_WM9713
select SND_SAMSUNG_AC97
help
--
1.8.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-14 12:26 ` Paul Bolle
0 siblings, 0 replies; 9+ messages in thread
From: Paul Bolle @ 2013-07-14 12:26 UTC (permalink / raw)
To: linux-arm-kernel
Commit 383ffda2fa ("ARM: EXYNOS: no more support non-DT for EXYNOS
SoCs") removed the Kconfig symbols MACH_SMDKV310 and MACH_SMDKC210.
Now remove the references to both of these symbols from the Kconfig
entry for SND_SOC_SAMSUNG_SMDK_WM9713. They're not needed anymore.
Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
---
Untested.
sound/soc/samsung/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/samsung/Kconfig b/sound/soc/samsung/Kconfig
index 9855dfc..3d76659 100644
--- a/sound/soc/samsung/Kconfig
+++ b/sound/soc/samsung/Kconfig
@@ -136,7 +136,7 @@ config SND_SOC_SAMSUNG_RX1950_UDA1380
config SND_SOC_SAMSUNG_SMDK_WM9713
tristate "SoC AC97 Audio support for SMDK with WM9713"
- depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
+ depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
select SND_SOC_WM9713
select SND_SAMSUNG_AC97
help
--
1.8.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
2013-07-14 12:26 ` Paul Bolle
(?)
@ 2013-07-14 12:45 ` Mark Brown
-1 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2013-07-14 12:45 UTC (permalink / raw)
To: Paul Bolle
Cc: Ben Dooks, Kukjin Kim, Sangbeom Kim, Liam Girdwood,
Jaroslav Kysela, Takashi Iwai, linux-arm-kernel,
linux-samsung-soc, alsa-devel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1137 bytes --]
On Sun, Jul 14, 2013 at 02:26:52PM +0200, Paul Bolle wrote:
> Commit 383ffda2fa ("ARM: EXYNOS: no more support non-DT for EXYNOS
> SoCs") removed the Kconfig symbols MACH_SMDKV310 and MACH_SMDKC210.
> Now remove the references to both of these symbols from the Kconfig
> entry for SND_SOC_SAMSUNG_SMDK_WM9713. They're not needed anymore.
Look at the code you are editing here:
> - depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
> + depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
Simply removing the dependencies is not going to give the clearly
intended effect here. The dependency on the DT only machine needs to be
replaced with something, not just deleted, otherwise the driver can't be
enabled in Kconfig.
Also let me again remind you to submit patches with subject lines that
are appropriate for the subsystem you're updating. I very nearly
deleted this unread since it looks like a patch for arch/arm with no
relationship with audio but in fact it doesn't touch arch/arm at all.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-14 12:45 ` Mark Brown
0 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2013-07-14 12:45 UTC (permalink / raw)
To: Paul Bolle
Cc: alsa-devel, Kukjin Kim, Sangbeom Kim, linux-kernel,
Liam Girdwood, Takashi Iwai, linux-samsung-soc, Ben Dooks,
linux-arm-kernel
[-- Attachment #1.1: Type: text/plain, Size: 1137 bytes --]
On Sun, Jul 14, 2013 at 02:26:52PM +0200, Paul Bolle wrote:
> Commit 383ffda2fa ("ARM: EXYNOS: no more support non-DT for EXYNOS
> SoCs") removed the Kconfig symbols MACH_SMDKV310 and MACH_SMDKC210.
> Now remove the references to both of these symbols from the Kconfig
> entry for SND_SOC_SAMSUNG_SMDK_WM9713. They're not needed anymore.
Look at the code you are editing here:
> - depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
> + depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
Simply removing the dependencies is not going to give the clearly
intended effect here. The dependency on the DT only machine needs to be
replaced with something, not just deleted, otherwise the driver can't be
enabled in Kconfig.
Also let me again remind you to submit patches with subject lines that
are appropriate for the subsystem you're updating. I very nearly
deleted this unread since it looks like a patch for arch/arm with no
relationship with audio but in fact it doesn't touch arch/arm at all.
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-14 12:45 ` Mark Brown
0 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2013-07-14 12:45 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Jul 14, 2013 at 02:26:52PM +0200, Paul Bolle wrote:
> Commit 383ffda2fa ("ARM: EXYNOS: no more support non-DT for EXYNOS
> SoCs") removed the Kconfig symbols MACH_SMDKV310 and MACH_SMDKC210.
> Now remove the references to both of these symbols from the Kconfig
> entry for SND_SOC_SAMSUNG_SMDK_WM9713. They're not needed anymore.
Look at the code you are editing here:
> - depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
> + depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
Simply removing the dependencies is not going to give the clearly
intended effect here. The dependency on the DT only machine needs to be
replaced with something, not just deleted, otherwise the driver can't be
enabled in Kconfig.
Also let me again remind you to submit patches with subject lines that
are appropriate for the subsystem you're updating. I very nearly
deleted this unread since it looks like a patch for arch/arm with no
relationship with audio but in fact it doesn't touch arch/arm at all.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130714/5cfbc8a3/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
2013-07-14 12:45 ` Mark Brown
@ 2013-07-14 13:18 ` Paul Bolle
-1 siblings, 0 replies; 9+ messages in thread
From: Paul Bolle @ 2013-07-14 13:18 UTC (permalink / raw)
To: Mark Brown
Cc: Ben Dooks, Kukjin Kim, Sangbeom Kim, Liam Girdwood,
Jaroslav Kysela, Takashi Iwai, linux-arm-kernel,
linux-samsung-soc, alsa-devel, linux-kernel
On Sun, 2013-07-14 at 13:45 +0100, Mark Brown wrote:
> Look at the code you are editing here:
>
> > - depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
> > + depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
>
> Simply removing the dependencies is not going to give the clearly
> intended effect here. The dependency on the DT only machine needs to be
> replaced with something, not just deleted, otherwise the driver can't be
> enabled in Kconfig.
I interpreted commit 383ffda2fa as a commit that just removed a lot of
stuff: see its commit explanation and its diffstat (it adds one line and
deletes over 900 lines). Moreover, after that commit the references to
MACH_SMDKV310 and MACH_SMDKC210 both are nops. Deleting them can't
possibly change anything.
But, anyhow, what would be the symbols to use here?
> Also let me again remind you to submit patches with subject lines that
> are appropriate for the subsystem you're updating. I very nearly
> deleted this unread since it looks like a patch for arch/arm with no
> relationship with audio but in fact it doesn't touch arch/arm at all.
I do try to use subject lines that are used in the sections I touch. I
seem to remember that you asked me _once_, I think one or two years ago,
to adjust to the system used for SoC. Since then you've received a
number of patches from me that, as far as I can remember, always used
that system.
Besides, this patch was intended as a follow up to commit 383ffda2fa, to
clean up two references it apparently forgot to remove, and therefor
basically copied the subject line used in that commit.
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-14 13:18 ` Paul Bolle
0 siblings, 0 replies; 9+ messages in thread
From: Paul Bolle @ 2013-07-14 13:18 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, 2013-07-14 at 13:45 +0100, Mark Brown wrote:
> Look at the code you are editing here:
>
> > - depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110 || MACH_SMDKV310 || MACH_SMDKC210)
> > + depends on SND_SOC_SAMSUNG && (MACH_SMDK6410 || MACH_SMDKC100 || MACH_SMDKV210 || MACH_SMDKC110)
>
> Simply removing the dependencies is not going to give the clearly
> intended effect here. The dependency on the DT only machine needs to be
> replaced with something, not just deleted, otherwise the driver can't be
> enabled in Kconfig.
I interpreted commit 383ffda2fa as a commit that just removed a lot of
stuff: see its commit explanation and its diffstat (it adds one line and
deletes over 900 lines). Moreover, after that commit the references to
MACH_SMDKV310 and MACH_SMDKC210 both are nops. Deleting them can't
possibly change anything.
But, anyhow, what would be the symbols to use here?
> Also let me again remind you to submit patches with subject lines that
> are appropriate for the subsystem you're updating. I very nearly
> deleted this unread since it looks like a patch for arch/arm with no
> relationship with audio but in fact it doesn't touch arch/arm at all.
I do try to use subject lines that are used in the sections I touch. I
seem to remember that you asked me _once_, I think one or two years ago,
to adjust to the system used for SoC. Since then you've received a
number of patches from me that, as far as I can remember, always used
that system.
Besides, this patch was intended as a follow up to commit 383ffda2fa, to
clean up two references it apparently forgot to remove, and therefor
basically copied the subject line used in that commit.
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
2013-07-14 13:18 ` Paul Bolle
@ 2013-07-15 11:25 ` Mark Brown
-1 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2013-07-15 11:25 UTC (permalink / raw)
To: Paul Bolle
Cc: Ben Dooks, Kukjin Kim, Sangbeom Kim, Liam Girdwood,
Jaroslav Kysela, Takashi Iwai, linux-arm-kernel,
linux-samsung-soc, alsa-devel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1505 bytes --]
On Sun, Jul 14, 2013 at 03:18:16PM +0200, Paul Bolle wrote:
> On Sun, 2013-07-14 at 13:45 +0100, Mark Brown wrote:
> > Simply removing the dependencies is not going to give the clearly
> > intended effect here. The dependency on the DT only machine needs to be
> > replaced with something, not just deleted, otherwise the driver can't be
> > enabled in Kconfig.
> I interpreted commit 383ffda2fa as a commit that just removed a lot of
Always provide human readable descriptions of commits in email.
> stuff: see its commit explanation and its diffstat (it adds one line and
> deletes over 900 lines). Moreover, after that commit the references to
> MACH_SMDKV310 and MACH_SMDKC210 both are nops. Deleting them can't
> possibly change anything.
This still doesn't have much to do with your commit message or indicate
much understanding of what the change is actually doing or why it might
be a problem.
> But, anyhow, what would be the symbols to use here?
Well, let's think about this for a minute - what is going on when
someone does a device tree conversion on a platform? What is replacing
the board files? What would an equivalent change be for the audio
driver?
> Besides, this patch was intended as a follow up to commit 383ffda2fa, to
> clean up two references it apparently forgot to remove, and therefor
> basically copied the subject line used in that commit.
That doesn't really help - putting in a completely irrelevant subject
line is a really good way to get your patch ignored.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: EXYNOS: remove references to non-DT SoCs
@ 2013-07-15 11:25 ` Mark Brown
0 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2013-07-15 11:25 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Jul 14, 2013 at 03:18:16PM +0200, Paul Bolle wrote:
> On Sun, 2013-07-14 at 13:45 +0100, Mark Brown wrote:
> > Simply removing the dependencies is not going to give the clearly
> > intended effect here. The dependency on the DT only machine needs to be
> > replaced with something, not just deleted, otherwise the driver can't be
> > enabled in Kconfig.
> I interpreted commit 383ffda2fa as a commit that just removed a lot of
Always provide human readable descriptions of commits in email.
> stuff: see its commit explanation and its diffstat (it adds one line and
> deletes over 900 lines). Moreover, after that commit the references to
> MACH_SMDKV310 and MACH_SMDKC210 both are nops. Deleting them can't
> possibly change anything.
This still doesn't have much to do with your commit message or indicate
much understanding of what the change is actually doing or why it might
be a problem.
> But, anyhow, what would be the symbols to use here?
Well, let's think about this for a minute - what is going on when
someone does a device tree conversion on a platform? What is replacing
the board files? What would an equivalent change be for the audio
driver?
> Besides, this patch was intended as a follow up to commit 383ffda2fa, to
> clean up two references it apparently forgot to remove, and therefor
> basically copied the subject line used in that commit.
That doesn't really help - putting in a completely irrelevant subject
line is a really good way to get your patch ignored.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130715/1ec4cc47/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-07-15 11:25 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-14 12:26 [PATCH] ARM: EXYNOS: remove references to non-DT SoCs Paul Bolle
2013-07-14 12:26 ` Paul Bolle
2013-07-14 12:45 ` Mark Brown
2013-07-14 12:45 ` Mark Brown
2013-07-14 12:45 ` Mark Brown
2013-07-14 13:18 ` Paul Bolle
2013-07-14 13:18 ` Paul Bolle
2013-07-15 11:25 ` Mark Brown
2013-07-15 11:25 ` Mark Brown
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.