From: Mark Brown <broonie@kernel.org> To: Lee Jones <lee.jones@linaro.org> Cc: Hans de Goede <hdegoede@redhat.com>, Cezary Rojewski <cezary.rojewski@intel.com>, Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>, Liam Girdwood <liam.r.girdwood@linux.intel.com>, Jie Yang <yang.jie@linux.intel.com>, patches@opensource.cirrus.com, linux-kernel@vger.kernel.org, Andy Shevchenko <andy.shevchenko@gmail.com>, Charles Keepax <ckeepax@opensource.cirrus.com>, alsa-devel@alsa-project.org Subject: Re: [PATCH v4 0/5] MFD/ASoC: Add support for Intel Bay Trail boards with WM5102 codec Date: Thu, 4 Feb 2021 16:48:26 +0000 [thread overview] Message-ID: <20210204164826.GF4288@sirena.org.uk> (raw) In-Reply-To: <20210204152124.GO2789116@dell> [-- Attachment #1: Type: text/plain, Size: 1537 bytes --] On Thu, Feb 04, 2021 at 03:21:24PM +0000, Lee Jones wrote: > The default point-of-view is; if a patch was submitted as part of a > set, it's likely that it makes the most sense to merge it as a set. Blocking the whole series is itself not ideal since it means the whole large series keeps on getting resent for minor changes in individual patches where it's only a small number of leaf patches that have issues, with a lot of these serieses the reason they're bundled together is that there's some constants being added in one of the early patches that gets used everywhere else, not that there's a really a particularly strong relationship. > Very often sets will have inter-dependencies (usually headers) which > would otherwise require the base patches to be applied (perhaps the > MFD core and the headers) in one release, followed by the accompanying > child device changes during a subsequent release. This doesn't scale > well and puts the contributor in an unfair position. You had been sharing pull requests for the common bits in the past which had resolved the dependency issues? > This is how we usually work together. Why is ASoC so different? Like I say we've got active work that ends up doing subsystem wide interface changes on a fairly frequent basis which then creates issues if a new user pops up that's still trying to use the old API. Often it's fine but coordinating near the time is safer than just acking with a potentially long lead time on things actually going through. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@kernel.org> To: Lee Jones <lee.jones@linaro.org> Cc: Cezary Rojewski <cezary.rojewski@intel.com>, Charles Keepax <ckeepax@opensource.cirrus.com>, alsa-devel@alsa-project.org, patches@opensource.cirrus.com, Jie Yang <yang.jie@linux.intel.com>, Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>, linux-kernel@vger.kernel.org, Liam Girdwood <liam.r.girdwood@linux.intel.com>, Hans de Goede <hdegoede@redhat.com>, Andy Shevchenko <andy.shevchenko@gmail.com> Subject: Re: [PATCH v4 0/5] MFD/ASoC: Add support for Intel Bay Trail boards with WM5102 codec Date: Thu, 4 Feb 2021 16:48:26 +0000 [thread overview] Message-ID: <20210204164826.GF4288@sirena.org.uk> (raw) In-Reply-To: <20210204152124.GO2789116@dell> [-- Attachment #1: Type: text/plain, Size: 1537 bytes --] On Thu, Feb 04, 2021 at 03:21:24PM +0000, Lee Jones wrote: > The default point-of-view is; if a patch was submitted as part of a > set, it's likely that it makes the most sense to merge it as a set. Blocking the whole series is itself not ideal since it means the whole large series keeps on getting resent for minor changes in individual patches where it's only a small number of leaf patches that have issues, with a lot of these serieses the reason they're bundled together is that there's some constants being added in one of the early patches that gets used everywhere else, not that there's a really a particularly strong relationship. > Very often sets will have inter-dependencies (usually headers) which > would otherwise require the base patches to be applied (perhaps the > MFD core and the headers) in one release, followed by the accompanying > child device changes during a subsequent release. This doesn't scale > well and puts the contributor in an unfair position. You had been sharing pull requests for the common bits in the past which had resolved the dependency issues? > This is how we usually work together. Why is ASoC so different? Like I say we've got active work that ends up doing subsystem wide interface changes on a fairly frequent basis which then creates issues if a new user pops up that's still trying to use the old API. Often it's fine but coordinating near the time is safer than just acking with a potentially long lead time on things actually going through. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2021-02-04 16:51 UTC|newest] Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-20 21:49 [PATCH v4 0/5] MFD/ASoC: Add support for Intel Bay Trail boards with WM5102 codec Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-01-20 21:49 ` [PATCH v4 1/5] mfd: arizona: Add MODULE_SOFTDEP("pre: arizona_ldo1") Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-02-04 13:55 ` Lee Jones 2021-02-04 13:55 ` Lee Jones 2021-01-20 21:49 ` [PATCH v4 2/5] mfd: arizona: Replace arizona_of_get_type() with device_get_match_data() Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-02-04 13:55 ` Lee Jones 2021-02-04 13:55 ` Lee Jones 2021-01-20 21:49 ` [PATCH v4 3/5] mfd: arizona: Add support for ACPI enumeration of WM5102 connected over SPI Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-01-21 10:34 ` Charles Keepax 2021-01-21 10:34 ` Charles Keepax 2021-02-04 13:55 ` Lee Jones 2021-02-04 13:55 ` Lee Jones 2021-01-20 21:49 ` [PATCH v4 4/5] ASoC: Intel: Add DMI quirk table to soc_intel_is_byt_cr() Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-02-04 13:56 ` Lee Jones 2021-02-04 13:56 ` Lee Jones 2021-02-04 14:05 ` Mark Brown 2021-02-04 14:05 ` Mark Brown 2021-02-04 15:04 ` Lee Jones 2021-02-04 15:04 ` Lee Jones 2021-02-04 15:11 ` Mark Brown 2021-02-04 15:11 ` Mark Brown 2021-02-04 15:40 ` Lee Jones 2021-02-04 15:40 ` Lee Jones 2021-02-04 19:42 ` Mark Brown 2021-02-04 19:42 ` Mark Brown 2021-02-05 8:34 ` Lee Jones 2021-02-05 8:34 ` Lee Jones 2021-02-05 21:11 ` Mark Brown 2021-02-05 21:11 ` Mark Brown 2021-02-08 8:33 ` Lee Jones 2021-02-08 8:33 ` Lee Jones 2021-02-08 15:24 ` Mark Brown 2021-02-08 15:24 ` Mark Brown 2021-01-20 21:49 ` [PATCH v4 5/5] ASoC: Intel: bytcr_wm5102: Add machine driver for BYT/WM5102 Hans de Goede 2021-01-20 21:49 ` Hans de Goede 2021-01-21 10:37 ` Charles Keepax 2021-01-21 10:37 ` Charles Keepax 2021-02-04 13:56 ` Lee Jones 2021-02-04 13:56 ` Lee Jones 2021-02-04 10:25 ` [PATCH v4 0/5] MFD/ASoC: Add support for Intel Bay Trail boards with WM5102 codec Hans de Goede 2021-02-04 10:25 ` Hans de Goede 2021-02-04 10:57 ` Lee Jones 2021-02-04 10:57 ` Lee Jones 2021-02-04 11:07 ` Hans de Goede 2021-02-04 11:07 ` Hans de Goede 2021-02-04 12:43 ` Mark Brown 2021-02-04 12:43 ` Mark Brown 2021-02-04 13:18 ` Hans de Goede 2021-02-04 13:18 ` Hans de Goede 2021-02-04 14:04 ` Mark Brown 2021-02-04 14:04 ` Mark Brown 2021-02-04 13:46 ` Lee Jones 2021-02-04 13:46 ` Lee Jones 2021-02-04 15:09 ` Mark Brown 2021-02-04 15:09 ` Mark Brown 2021-02-04 15:21 ` Lee Jones 2021-02-04 15:21 ` Lee Jones 2021-02-04 16:48 ` Mark Brown [this message] 2021-02-04 16:48 ` Mark Brown 2021-02-08 13:52 ` [GIT PULL] Immutable branch from MFD due for the v5.12 merge window Lee Jones 2021-02-08 13:52 ` Lee Jones 2021-02-08 18:38 ` (subset) [PATCH v4 0/5] MFD/ASoC: Add support for Intel Bay Trail boards with WM5102 codec Mark Brown 2021-02-08 18:38 ` Mark Brown 2021-02-08 19:12 ` Hans de Goede 2021-02-08 19:12 ` Hans de Goede
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=20210204164826.GF4288@sirena.org.uk \ --to=broonie@kernel.org \ --cc=alsa-devel@alsa-project.org \ --cc=andy.shevchenko@gmail.com \ --cc=cezary.rojewski@intel.com \ --cc=ckeepax@opensource.cirrus.com \ --cc=hdegoede@redhat.com \ --cc=lee.jones@linaro.org \ --cc=liam.r.girdwood@linux.intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=patches@opensource.cirrus.com \ --cc=pierre-louis.bossart@linux.intel.com \ --cc=yang.jie@linux.intel.com \ /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: linkBe 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.