From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F332E1CABB; Fri, 24 Nov 2023 11:19:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="b1kwtZdH" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 60889C433C7; Fri, 24 Nov 2023 11:19:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700824740; bh=uv+M2uJcD6nigmAsYgStYTAS4i8Dzh4L8oQ+du2M+04=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=b1kwtZdHvVApDv9ZwK757wpvC8opAjMwEYhUCtu0QD1AA28B2iOvULCtCs35iBLxK apNIF3jq+X9ShikKZUo5wDmh69E+fGkD6p7M0azQ50mrxaZjno9+fHPWVdrDfh55F4 EoytZBi1YzuDbemZQAeqtuGZM16oOsxm3OX01w5ZQoUyE7SSbGXEsvubh4MviR3pdS bQoHeEgEmnXX0Fs2EMJTwain5VcYSk8u9CMaQBH+uGh6fdPs+sqSu3zBGoM3QJ+zH1 ZRYV50EKf55OpnFw873K+WYNBlQlZgfQaTAZvz1hUSfl1HkRC4V9jZPVSQW3IjC3mP qT3G34/O/wMvA== Received: from johan by xi.lan with local (Exim 4.96.2) (envelope-from ) id 1r6UDQ-0000cA-0j; Fri, 24 Nov 2023 12:19:20 +0100 Date: Fri, 24 Nov 2023 12:19:20 +0100 From: Johan Hovold To: Krishna Kurapati PSSNV Cc: Thinh Nguyen , Greg Kroah-Hartman , Philipp Zabel , Andy Gross , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Felipe Balbi , Wesley Cheng , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, quic_pkondeti@quicinc.com, quic_ppratap@quicinc.com, quic_jackp@quicinc.com, ahalaney@redhat.com, quic_shazhuss@quicinc.com Subject: Re: [PATCH v13 05/10] usb: dwc3: qcom: Refactor IRQ handling in QCOM Glue driver Message-ID: References: <04615205-e380-4719-aff1-f32c26004b14@quicinc.com> <6d4d959c-b155-471b-b13d-f6fda557cfe0@quicinc.com> <4fc27dbb-b0aa-437a-a48c-9deea236282d@quicinc.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Nov 24, 2023 at 04:08:42PM +0530, Krishna Kurapati PSSNV wrote: > On 11/24/2023 3:43 PM, Johan Hovold wrote: > > I'd suggest that you just send two separate series, one with binding and > > driver updates, which will eventually be merged by Greg, and one with > > the devicetree changes, which goes through Bjorn's tree. > > > > It's good if you could add a link to the binding series in the cover > > letter of the devicetree changes as they are of course going to be quite > > closely related and need to be reviewed in parallel. > > Thanks for this pointer. So for Multiport, can I do it this way: > > 1. Core bindings and Core driver changes in one series. Now that we > finalized we don't be adding the ctrl_irq[1] as discussed on: > https://lore.kernel.org/all/ZU33uWpStIobzyd6@hovoldconsulting.com/. > > 2. QC bindings and QC driver changes for Multiport to be pushed after we > clean up the current driver and DT's (an effort which is going on > currently). No, I was just referring to how to handle binding/driver vs devicetree patches for USB where we send them separately (unlike for most other subsystems). The dwc3 core and Qualcomm glue parts should still go in the same series for multiport support. Whether to do the irq cleanup before or after adding multiport support is a different question, but, yeah, it is probably best to do it before. The question of whether we can drop ACPI support should also be considered as that should also simplify your multiport series. Johan