Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp517136pxb; Tue, 9 Feb 2021 06:17:16 -0800 (PST) X-Google-Smtp-Source: ABdhPJw16+KLu4XeeEbRnKPGzouw3CcpMlKK3f1FqPyl2giff2HKeydArbOPnk+xIBPlueBR508Y X-Received: by 2002:a17:907:78d5:: with SMTP id kv21mr22654314ejc.461.1612880236481; Tue, 09 Feb 2021 06:17:16 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1612880236; cv=none; d=google.com; s=arc-20160816; b=YlBYMw9Gn4AcwuTWjMnN3MfucObiciZS9iutiGq8Mb1GwtrllXvmy29Gahbv42wlbf lJxIjEGhuK9udp5oUOnoz7lkK2cYemY92hHGTpGlvmn5mobhfJGa5uV5p/HMzRzNvPc7 kn+jLffpmh9xKaavTHdzGcV5WY/ddhKMRcRlcJLZEG9FP1kilRlyXCZPOtvzDoaVD0Zr EWyv8DKOhfvNzhGVUWm2I/0erfIita3obQ+ECYstGiNdyHOLwYeTmb42GkQRHaZOqL4m Y3Q6yUZvZLHqBu8coHotu9OWLcO3Gbnbx8RiJN5/ekVFJ2DAuWx7rGdHqis6iGjwzh8K sk9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=hUh/wcUnUiqWLSuZNUcSG49hoBmdkF7bh0pKMix4LK4=; b=twu8v2922jPgC3PIGkkPtEWV/pWH6OdbnHGTQXfOvtAW0mAw8WoiJx5pB2XNnPnlnA KaJ+Y89x/UyOeevbZQDQidggpeGgTxGYIk0n7kbLV29RoYSKigmeY/0muhaOs/xcMUpI 74kNPcplJG8ixXHIjs4wnvMuyMFa98tKQv3y5YiRhBTMW0NdSY19nTGPGYCi+F03a4PM KshcHesBwVpk2b0RGVaaKdLzIfj97qmtA+8c+wgTCZ8jIQ1lk9PM/rImCF3gBacf+gDW LbTGNZBDNudtxhfY1Pr2yUSmKA7uwb8KsCEyE/VG/aA1K06jU1y9GQc7k8LbI1U1raQG 3G4A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=i6yAEp9f; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id n2si17515218ejl.444.2021.02.09.06.16.33; Tue, 09 Feb 2021 06:17:16 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=i6yAEp9f; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231614AbhBIOPG (ORCPT + 99 others); Tue, 9 Feb 2021 09:15:06 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38296 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230006AbhBIOPE (ORCPT ); Tue, 9 Feb 2021 09:15:04 -0500 Received: from mail-wr1-x42c.google.com (mail-wr1-x42c.google.com [IPv6:2a00:1450:4864:20::42c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EA49FC061786 for ; Tue, 9 Feb 2021 06:14:23 -0800 (PST) Received: by mail-wr1-x42c.google.com with SMTP id n6so9168447wrv.8 for ; Tue, 09 Feb 2021 06:14:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=hUh/wcUnUiqWLSuZNUcSG49hoBmdkF7bh0pKMix4LK4=; b=i6yAEp9fsEQMPTs1jK5sCt2+BjobvrhRaHbWpE75MODFwS+yEZchD4VACw7QMIIiN6 QuHtxzioIQRtMRVVV3+3+LKHV+UNhWY2ecwdq4iA+KXPR8ewMv2cUHPeGs5yOltgw5gE Fj1eyS7FOaOWnNLK7Eo+pjKEgDrwyWqWuxUkNtQbMU7GMJYFKhutN8P2pX2qfJFYWFbS hXyRsoS3gk73nRV4AaE6RUrnowQx1tZdJeDPoi222cEew74atJCtuqiT/Hj+bF0KFZOE IGyIgfowfy69HxtyqCV4o3+BBEyveBhw7Ltm/HCkIclkhU2sPFKmLUCwKtOYfZWxhp6V gUfA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=hUh/wcUnUiqWLSuZNUcSG49hoBmdkF7bh0pKMix4LK4=; b=UJ1sFhG0eR8GodD1t72CPJpZGPczIzBWMH7sa5GEDA+IT6hCRZf4g5y0CfbrDSTEsk KzYhxH1RcG94mNvyubdBCeFvSQKE5gvBUWgGC3x/zgqVZnVfvMNhtuuJBn///ozdoL3R a4tOf+NILFa0R/pJOuV9WOfHHzxXNrxtI9D98yaQaTfLCSJz+N6nHypFR+yJwxzoxZth PiGR3Qubn59zidiKDc2msTTHNCBlE/ECZWjCZvqiFJ83vKlATUVh8Yvr9tZq+KsaJBJC ck6wCCvfKKxhfoMOMLlM6xonxSvsdfKNB/A7Fes2uYoU7z0+4dbclTG2Dh1vGfV4fIXJ QqIA== X-Gm-Message-State: AOAM530xGiTmNJamxxLILGmffqHOy2V44Ro3utSFTmGm69Bg/wQKp8TP o82RevCExzBZ3ZA40AdkQfDJiQ== X-Received: by 2002:a5d:60d0:: with SMTP id x16mr25947457wrt.269.1612880062687; Tue, 09 Feb 2021 06:14:22 -0800 (PST) Received: from dell ([91.110.221.187]) by smtp.gmail.com with ESMTPSA id h207sm5096569wme.18.2021.02.09.06.14.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 09 Feb 2021 06:14:22 -0800 (PST) Date: Tue, 9 Feb 2021 14:14:20 +0000 From: Lee Jones To: Hans de Goede Cc: MyungJoo Ham , Chanwoo Choi , Cezary Rojewski , Pierre-Louis Bossart , Liam Girdwood , Jie Yang , Mark Brown , patches@opensource.cirrus.com, linux-kernel@vger.kernel.org, Andy Shevchenko , Charles Keepax , alsa-devel@alsa-project.org Subject: Re: [PATCH v4 resend 00/13] MFD/extcon/ASoC: Rework arizona codec jack-detect support Message-ID: <20210209141420.GE4766@dell> References: <20210204112502.88362-1-hdegoede@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 08 Feb 2021, Hans de Goede wrote: > Hi Mark, Lee, > > On 2/4/21 12:24 PM, Hans de Goede wrote: > > Hi all, > > > > Here is v4 of my series to rework the arizona codec jack-detect support > > to use the snd_soc_jack helpers instead of direct extcon reporting. > > > > This is a resend with some extra *-by tags collected and with the extcon > > folks added to the "To:" list, which I somehow missed with the original > > v4 posting, sorry. > > > > This is done by reworking the extcon driver into an arizona-jackdet > > library and then modifying the codec drivers to use that directly, > > replacing the old separate extcon child-devices and extcon-driver. > > > > This brings the arizona-codec jack-detect handling inline with how > > all other ASoC codec driver do this. This was developed and tested on > > a Lenovo Yoga Tablet 1051L with a WM5102 codec. > > > > This was also tested by Charles Keepax, one of the Cirrus Codec folks. > > > > This depends on the previously posted "[PATCH v4 0/5] MFD/ASoC: Add > > support for Intel Bay Trail boards with WM5102 codec" series and there > > are various interdependencies between the patches in this series. > > > > Lee Jones, the MFD maintainer has agreed to take this series upstream > > through the MFD tree and to provide an immutable branch for the ASoC > > and extcon subsystems to merge. > > > > Mark and extcon-maintainers may we have your ack for merging these > > through the MFD tree ? > > Now that the pre-cursor (1) series to this has been merged, I guess it > is time to decide how to merge this series. > > Chanwoo Choi has given his ack to merge the extcon bits through the MFD > tree and since Mark has expressed a preference for merging ASOC patches > directly I guess that it would be best to merge 1-6 through the MFD > tree and then Lee can send Mark a pull-req and Mark can apply the others? : > > 1/13 mfd: arizona: Drop arizona-extcon cells > 2/13 extcon: arizona: Fix some issues when HPDET IRQ fires after the jack has been unplugged > 3/13 extcon: arizona: Fix various races on driver unbind > 4/13 extcon: arizona: Fix flags parameter to the gpiod_get("wlf,micd-pol") call > 5/13 extcon: arizona: Always use pm_runtime_get_sync() when we need the device to be awake > 6/14 ASoC/extcon: arizona: Move arizona jack code to sound/soc/codecs/arizona-jack.c > > 1 is: Acked-for-MFD-by: Lee Jones > 2-6 are: Acked-by: Chanwoo Choi > > Note patch 6 renames drivers/extcon/extcon-arizona.c to sound/soc/codecs/arizona-jack.c > but it does not touch any other files under sound/soc (including NOT touching > sound/soc/codecs/Makefile that is done in a later patch). So it cannot cause any > conflicts. > > Mark, would merging 1-6 through the MFD tree, and you applying the rest > (which are all ASoC patches) work for you ? What a faff. I still don't see why they can't all go in and a PR provided. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog