Re: Patch "soundwire: fix regmap dependencies and align with other serial links" has been added to the 5.2-stable tree
On Sun, Oct 06, 2019 at 09:44:47AM -0400, Sasha Levin wrote: > This is a note to let you know that I've just added the patch titled > > soundwire: fix regmap dependencies and align with other serial links > > to the 5.2-stable tree which can be found at: > > http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary > > The filename of the patch is: > soundwire-fix-regmap-dependencies-and-align-with-oth.patch > and it can be found in the queue-5.2 subdirectory. > > If you, or anyone else, feels it should not be added to the stable tree, > please let know about it. > > > > commit 3287dff699a585506096287541cc4414600899c7 > Author: Pierre-Louis Bossart > Date: Thu Jul 18 18:02:15 2019 -0500 > > soundwire: fix regmap dependencies and align with other serial links > > [ Upstream commit 8676b3ca4673517650fd509d7fa586aff87b3c28 ] > > The existing code has a mixed select/depend usage which makes no sense. > > config SOUNDWIRE_BUS >tristate >select REGMAP_SOUNDWIRE > > config REGMAP_SOUNDWIRE > tristate > depends on SOUNDWIRE_BUS > > Let's remove one layer of Kconfig definitions and align with the > solutions used by all other serial links. > > Signed-off-by: Pierre-Louis Bossart > Link: > https://lore.kernel.org/r/20190718230215.18675-1-pierre-louis.boss...@linux.intel.com > Signed-off-by: Vinod Koul > Signed-off-by: Sasha Levin > > diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig > index 6ad5ef48b61ee..8cd2ac650b505 100644 > --- a/drivers/base/regmap/Kconfig > +++ b/drivers/base/regmap/Kconfig > @@ -44,7 +44,7 @@ config REGMAP_IRQ > > config REGMAP_SOUNDWIRE > tristate > - depends on SOUNDWIRE_BUS > + depends on SOUNDWIRE > > config REGMAP_SCCB > tristate > diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig > index 3a01cfd70fdcd..f518273cfbe3c 100644 > --- a/drivers/soundwire/Kconfig > +++ b/drivers/soundwire/Kconfig > @@ -4,7 +4,7 @@ > # > > menuconfig SOUNDWIRE > - bool "SoundWire support" > + tristate "SoundWire support" > help > SoundWire is a 2-Pin interface with data and clock line ratified > by the MIPI Alliance. SoundWire is used for transporting data > @@ -17,17 +17,12 @@ if SOUNDWIRE > > comment "SoundWire Devices" > > -config SOUNDWIRE_BUS > - tristate > - select REGMAP_SOUNDWIRE > - > config SOUNDWIRE_CADENCE > tristate > > config SOUNDWIRE_INTEL > tristate "Intel SoundWire Master driver" > select SOUNDWIRE_CADENCE > - select SOUNDWIRE_BUS > depends on X86 && ACPI && SND_SOC > help > SoundWire Intel Master driver. > diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile > index fd99a831b92a0..45b7e50016539 100644 > --- a/drivers/soundwire/Makefile > +++ b/drivers/soundwire/Makefile > @@ -5,7 +5,7 @@ > > #Bus Objs > soundwire-bus-objs := bus_type.o bus.o slave.o mipi_disco.o stream.o > -obj-$(CONFIG_SOUNDWIRE_BUS) += soundwire-bus.o > +obj-$(CONFIG_SOUNDWIRE) += soundwire-bus.o > > #Cadence Objs > soundwire-cadence-objs := cadence_master.o Do any of these Kconfig changes actually fix any real issue? It looks like they are all just "cleanups" to me. What problem are they solving? thanks, greg k-h
Re: [PATCH] soundwire: fix regmap dependencies and align with other serial links
On 18-07-19, 18:02, Pierre-Louis Bossart wrote: > The existing code has a mixed select/depend usage which makes no sense. > > config SOUNDWIRE_BUS >tristate >select REGMAP_SOUNDWIRE > > config REGMAP_SOUNDWIRE > tristate > depends on SOUNDWIRE_BUS > > Let's remove one layer of Kconfig definitions and align with the > solutions used by all other serial links. Applied, thanks -- ~Vinod
Re: [PATCH] soundwire: fix regmap dependencies and align with other serial links
On 19/07/2019 00:02, Pierre-Louis Bossart wrote: The existing code has a mixed select/depend usage which makes no sense. config SOUNDWIRE_BUS tristate select REGMAP_SOUNDWIRE config REGMAP_SOUNDWIRE tristate depends on SOUNDWIRE_BUS Let's remove one layer of Kconfig definitions and align with the solutions used by all other serial links. Signed-off-by: Pierre-Louis Bossart --- drivers/base/regmap/Kconfig | 2 +- drivers/soundwire/Kconfig | 7 +-- drivers/soundwire/Makefile | 2 +- 3 files changed, 3 insertions(+), 8 deletions(-) I have been using some thing similar in my setup. I did test this with Qualcomm WSA881x codec. Tested-by: Srinivas Kandagatla --srini
Re: [alsa-devel] [PATCH] soundwire: fix regmap dependencies and align with other serial links
On Wed, Aug 07, 2019 at 01:17:20PM -0500, Pierre-Louis Bossart wrote: > I don't have the knowledge or means to test what I suggested initially for > the other buses, and the optimization was minimal anyways, so this patch > takes the path of least resistance and aligns with others. > if there are no objections it's probably easier to push this patch through > the SoundWire tree, with the relevant Acks. Makes sense I think Acked-by: Mark Brown signature.asc Description: PGP signature
Re: [alsa-devel] [PATCH] soundwire: fix regmap dependencies and align with other serial links
Hi Mark, Vinod, Mark, any feedback? There will be a set of SoundWire codec drivers provided upstream soonish and we'll get a number of kbuild errors without this patch. I think I'm missing context here, I've basically been zoning out all the soundwire stuff - the patch series are huge and generate a bunch of discusion. Is the patch below the full thing? I don't see any obvious problems. Here's a bit of context: This patch is really independent from the 40-odd fixes I pushed about 10 days ago. I provided an initial version back in April ('[PATCH v2 2/2] regmap: soundwire: fix Kconfig select/depend issue') during the first batch of updates. At the time, the suggested solution for the compilation issues was not agreed on, so the build errors remained - not a big deal they only show-up with codec drivers that were not upstreamed so far. It took me a while to come back to it but that was the first in my TODO list after my Summer break and now that we are almost ready to upstream those codec drivers it's a good time to revisit this issue. Your initial feedback was: "This now makes _SOUNDWIRE different to all the other bus types; if this is a good change then surely the same thing should be done for all the other bus types. " and "Alignment is a requirement. If you want to optimize this then it'd be better to optimize all the bus types rather than just having the one weird bus type that does something different for no documented reason." I don't have the knowledge or means to test what I suggested initially for the other buses, and the optimization was minimal anyways, so this patch takes the path of least resistance and aligns with others. if there are no objections it's probably easier to push this patch through the SoundWire tree, with the relevant Acks.
Re: [alsa-devel] [PATCH] soundwire: fix regmap dependencies and align with other serial links
On Wed, Aug 07, 2019 at 10:09:27AM -0500, Pierre-Louis Bossart wrote: > Vinod, Mark, any feedback? > There will be a set of SoundWire codec drivers provided upstream soonish and > we'll get a number of kbuild errors without this patch. I think I'm missing context here, I've basically been zoning out all the soundwire stuff - the patch series are huge and generate a bunch of discusion. Is the patch below the full thing? I don't see any obvious problems. > > > > > > --- > > > drivers/base/regmap/Kconfig | 2 +- > > > drivers/soundwire/Kconfig | 7 +-- > > > drivers/soundwire/Makefile | 2 +- > > > 3 files changed, 3 insertions(+), 8 deletions(-) > > > > > > diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig > > > index 6ad5ef48b61e..8cd2ac650b50 100644 > > > --- a/drivers/base/regmap/Kconfig > > > +++ b/drivers/base/regmap/Kconfig > > > @@ -44,7 +44,7 @@ config REGMAP_IRQ > > > > > > config REGMAP_SOUNDWIRE > > > tristate > > > - depends on SOUNDWIRE_BUS > > > + depends on SOUNDWIRE > > > > > > config REGMAP_SCCB > > > tristate > > > diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig > > > index 3a01cfd70fdc..f518273cfbe3 100644 > > > --- a/drivers/soundwire/Kconfig > > > +++ b/drivers/soundwire/Kconfig > > > @@ -4,7 +4,7 @@ > > > # > > > > > > menuconfig SOUNDWIRE > > > - bool "SoundWire support" > > > + tristate "SoundWire support" > > > help > > >SoundWire is a 2-Pin interface with data and clock line > > > ratified > > >by the MIPI Alliance. SoundWire is used for transporting data > > > @@ -17,17 +17,12 @@ if SOUNDWIRE > > > > > > comment "SoundWire Devices" > > > > > > -config SOUNDWIRE_BUS > > > - tristate > > > - select REGMAP_SOUNDWIRE > > > - > > > config SOUNDWIRE_CADENCE > > > tristate > > > > > > config SOUNDWIRE_INTEL > > > tristate "Intel SoundWire Master driver" > > > select SOUNDWIRE_CADENCE > > > - select SOUNDWIRE_BUS > > > depends on X86 && ACPI && SND_SOC > > > help > > >SoundWire Intel Master driver. > > > diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile > > > index fd99a831b92a..45b7e5001653 100644 > > > --- a/drivers/soundwire/Makefile > > > +++ b/drivers/soundwire/Makefile > > > @@ -5,7 +5,7 @@ > > > > > > #Bus Objs > > > soundwire-bus-objs := bus_type.o bus.o slave.o mipi_disco.o stream.o > > > -obj-$(CONFIG_SOUNDWIRE_BUS) += soundwire-bus.o > > > +obj-$(CONFIG_SOUNDWIRE) += soundwire-bus.o > > > > > > #Cadence Objs > > > soundwire-cadence-objs := cadence_master.o > > > -- > > > 2.20.1 > > > > > ___ > > Alsa-devel mailing list > > alsa-de...@alsa-project.org > > https://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > signature.asc Description: PGP signature
Re: [alsa-devel] [PATCH] soundwire: fix regmap dependencies and align with other serial links
On 7/19/19 4:04 AM, Rafael J. Wysocki wrote: On Fri, Jul 19, 2019 at 1:02 AM Pierre-Louis Bossart wrote: The existing code has a mixed select/depend usage which makes no sense. config SOUNDWIRE_BUS tristate select REGMAP_SOUNDWIRE config REGMAP_SOUNDWIRE tristate depends on SOUNDWIRE_BUS Let's remove one layer of Kconfig definitions and align with the solutions used by all other serial links. Signed-off-by: Pierre-Louis Bossart No issues found: Reviewed-by: Rafael J. Wysocki Vinod, Mark, any feedback? There will be a set of SoundWire codec drivers provided upstream soonish and we'll get a number of kbuild errors without this patch. --- drivers/base/regmap/Kconfig | 2 +- drivers/soundwire/Kconfig | 7 +-- drivers/soundwire/Makefile | 2 +- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig index 6ad5ef48b61e..8cd2ac650b50 100644 --- a/drivers/base/regmap/Kconfig +++ b/drivers/base/regmap/Kconfig @@ -44,7 +44,7 @@ config REGMAP_IRQ config REGMAP_SOUNDWIRE tristate - depends on SOUNDWIRE_BUS + depends on SOUNDWIRE config REGMAP_SCCB tristate diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig index 3a01cfd70fdc..f518273cfbe3 100644 --- a/drivers/soundwire/Kconfig +++ b/drivers/soundwire/Kconfig @@ -4,7 +4,7 @@ # menuconfig SOUNDWIRE - bool "SoundWire support" + tristate "SoundWire support" help SoundWire is a 2-Pin interface with data and clock line ratified by the MIPI Alliance. SoundWire is used for transporting data @@ -17,17 +17,12 @@ if SOUNDWIRE comment "SoundWire Devices" -config SOUNDWIRE_BUS - tristate - select REGMAP_SOUNDWIRE - config SOUNDWIRE_CADENCE tristate config SOUNDWIRE_INTEL tristate "Intel SoundWire Master driver" select SOUNDWIRE_CADENCE - select SOUNDWIRE_BUS depends on X86 && ACPI && SND_SOC help SoundWire Intel Master driver. diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile index fd99a831b92a..45b7e5001653 100644 --- a/drivers/soundwire/Makefile +++ b/drivers/soundwire/Makefile @@ -5,7 +5,7 @@ #Bus Objs soundwire-bus-objs := bus_type.o bus.o slave.o mipi_disco.o stream.o -obj-$(CONFIG_SOUNDWIRE_BUS) += soundwire-bus.o +obj-$(CONFIG_SOUNDWIRE) += soundwire-bus.o #Cadence Objs soundwire-cadence-objs := cadence_master.o -- 2.20.1 ___ Alsa-devel mailing list alsa-de...@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
Re: [PATCH] soundwire: fix regmap dependencies and align with other serial links
On Fri, Jul 19, 2019 at 1:02 AM Pierre-Louis Bossart wrote: > > The existing code has a mixed select/depend usage which makes no sense. > > config SOUNDWIRE_BUS >tristate >select REGMAP_SOUNDWIRE > > config REGMAP_SOUNDWIRE > tristate > depends on SOUNDWIRE_BUS > > Let's remove one layer of Kconfig definitions and align with the > solutions used by all other serial links. > > Signed-off-by: Pierre-Louis Bossart No issues found: Reviewed-by: Rafael J. Wysocki > --- > drivers/base/regmap/Kconfig | 2 +- > drivers/soundwire/Kconfig | 7 +-- > drivers/soundwire/Makefile | 2 +- > 3 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig > index 6ad5ef48b61e..8cd2ac650b50 100644 > --- a/drivers/base/regmap/Kconfig > +++ b/drivers/base/regmap/Kconfig > @@ -44,7 +44,7 @@ config REGMAP_IRQ > > config REGMAP_SOUNDWIRE > tristate > - depends on SOUNDWIRE_BUS > + depends on SOUNDWIRE > > config REGMAP_SCCB > tristate > diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig > index 3a01cfd70fdc..f518273cfbe3 100644 > --- a/drivers/soundwire/Kconfig > +++ b/drivers/soundwire/Kconfig > @@ -4,7 +4,7 @@ > # > > menuconfig SOUNDWIRE > - bool "SoundWire support" > + tristate "SoundWire support" > help > SoundWire is a 2-Pin interface with data and clock line ratified > by the MIPI Alliance. SoundWire is used for transporting data > @@ -17,17 +17,12 @@ if SOUNDWIRE > > comment "SoundWire Devices" > > -config SOUNDWIRE_BUS > - tristate > - select REGMAP_SOUNDWIRE > - > config SOUNDWIRE_CADENCE > tristate > > config SOUNDWIRE_INTEL > tristate "Intel SoundWire Master driver" > select SOUNDWIRE_CADENCE > - select SOUNDWIRE_BUS > depends on X86 && ACPI && SND_SOC > help > SoundWire Intel Master driver. > diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile > index fd99a831b92a..45b7e5001653 100644 > --- a/drivers/soundwire/Makefile > +++ b/drivers/soundwire/Makefile > @@ -5,7 +5,7 @@ > > #Bus Objs > soundwire-bus-objs := bus_type.o bus.o slave.o mipi_disco.o stream.o > -obj-$(CONFIG_SOUNDWIRE_BUS) += soundwire-bus.o > +obj-$(CONFIG_SOUNDWIRE) += soundwire-bus.o > > #Cadence Objs > soundwire-cadence-objs := cadence_master.o > -- > 2.20.1 >
[PATCH] soundwire: fix regmap dependencies and align with other serial links
The existing code has a mixed select/depend usage which makes no sense. config SOUNDWIRE_BUS tristate select REGMAP_SOUNDWIRE config REGMAP_SOUNDWIRE tristate depends on SOUNDWIRE_BUS Let's remove one layer of Kconfig definitions and align with the solutions used by all other serial links. Signed-off-by: Pierre-Louis Bossart --- drivers/base/regmap/Kconfig | 2 +- drivers/soundwire/Kconfig | 7 +-- drivers/soundwire/Makefile | 2 +- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/base/regmap/Kconfig b/drivers/base/regmap/Kconfig index 6ad5ef48b61e..8cd2ac650b50 100644 --- a/drivers/base/regmap/Kconfig +++ b/drivers/base/regmap/Kconfig @@ -44,7 +44,7 @@ config REGMAP_IRQ config REGMAP_SOUNDWIRE tristate - depends on SOUNDWIRE_BUS + depends on SOUNDWIRE config REGMAP_SCCB tristate diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig index 3a01cfd70fdc..f518273cfbe3 100644 --- a/drivers/soundwire/Kconfig +++ b/drivers/soundwire/Kconfig @@ -4,7 +4,7 @@ # menuconfig SOUNDWIRE - bool "SoundWire support" + tristate "SoundWire support" help SoundWire is a 2-Pin interface with data and clock line ratified by the MIPI Alliance. SoundWire is used for transporting data @@ -17,17 +17,12 @@ if SOUNDWIRE comment "SoundWire Devices" -config SOUNDWIRE_BUS - tristate - select REGMAP_SOUNDWIRE - config SOUNDWIRE_CADENCE tristate config SOUNDWIRE_INTEL tristate "Intel SoundWire Master driver" select SOUNDWIRE_CADENCE - select SOUNDWIRE_BUS depends on X86 && ACPI && SND_SOC help SoundWire Intel Master driver. diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile index fd99a831b92a..45b7e5001653 100644 --- a/drivers/soundwire/Makefile +++ b/drivers/soundwire/Makefile @@ -5,7 +5,7 @@ #Bus Objs soundwire-bus-objs := bus_type.o bus.o slave.o mipi_disco.o stream.o -obj-$(CONFIG_SOUNDWIRE_BUS) += soundwire-bus.o +obj-$(CONFIG_SOUNDWIRE) += soundwire-bus.o #Cadence Objs soundwire-cadence-objs := cadence_master.o -- 2.20.1