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 58AE61DDC5; Sat, 22 Jun 2024 16:34:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719074070; cv=none; b=aR28hSGHqnrsr/dnUu6XmQ6+WeJWv3xiZx4Na8WUvLw3dsZtpUBTrQ+B9P4PnC1VegldBy1/dpSkseuZP5d6znmSbWzVNoiH+1rqDCvwhxbBqkGFvKPGsmiicD3Ln3E2FY3yM+Q1TkHiSwH63EG7QZP7ag/Hly6CkUlVndBKbeg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719074070; c=relaxed/simple; bh=t2xrxpY7g42zaDbzrTJjSd8KnoDUVB/HebWYFWAP4o8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=c53Z4xUGUI077+8oRMZiHrUj4M5zXBJBEhXbb6X6nAXDH9HiaNiZvqI2iucPdbUtB4CKoAgwzxf/FfgYt3YDHsVCZU+SwLXi9W3Ahf7wy/30YqnXh1ek30C8Oir/lYkFMGPf0BzCDz6QC23xuh01NeZuzNOGFdvW2xlePrilZ7c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=JJIYs7Yt; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="JJIYs7Yt" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1E24FC3277B; Sat, 22 Jun 2024 16:34:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719074069; bh=t2xrxpY7g42zaDbzrTJjSd8KnoDUVB/HebWYFWAP4o8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JJIYs7YtCwIYcOwlFMs3woKlHviODVXAHA1Xgb+VD+qZrJjrdWN8V59kY6vwlH9AJ MJ6vhkIu56Pf1u2OLb2hkg3Gdz6UFFw/bKJIxOlxYq62N+IlQjinp9U4Mpxlkg8say gLNXiUKllAHSKTU0CZG3uwOqcDyIZpoUnn+1XEpZ/fWeQI+axvbYEo9QMXJDgmHImm vzXM27JSmHk2knHUop8ec00x5+CB32fTzLu+uipn/6X9dO0cm3sdV4Ym1kH1130+Ma UxNv1f0RDw1Gc61CcAKWJ0f69Xc9fdrtZWhaXCqVnZgfwoUlI6A7v2rG0ZSN4SF/Sk DyrxuU+jmKd2A== Date: Sat, 22 Jun 2024 18:34:23 +0200 From: Niklas Cassel To: Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= Cc: Jingoo Han , Manivannan Sadhasivam , Bjorn Helgaas , Lorenzo Pieralisi , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Kishon Vijay Abraham I , Arnd Bergmann , Damien Le Moal , Jon Lin , Shawn Lin , Simon Xue , linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: Re: [PATCH v5 00/13] PCI: dw-rockchip: Add endpoint mode support Message-ID: References: <20240607-rockchip-pcie-ep-v1-v5-0-0a042d6b0049@kernel.org> <20240621193937.GB3008482@rocinante> <20240622154324.GA391376@rocinante> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240622154324.GA391376@rocinante> On Sun, Jun 23, 2024 at 12:43:24AM +0900, Krzysztof WilczyƄski wrote: > Hello, > > [...] > > Could you possibly include the commit: > > 3d2e425263e2 ("PCI: dwc: ep: Add a generic dw_pcie_ep_linkdown() API to handle Link Down event") > > from the controller/dwc branch in the controller/rockchip as well, > > or rebase the controller/rockchip branch on top of the controller/dwc branch, > > or merge the controller/dwc branch to the controller/rockchip branch? > > > > Done. > > > Additionally, since you picked up Mani's series which removes > > dw_pcie_ep_init_notify() on the controller/dwc branch: > > 9eba2f70362f ("PCI: dwc: ep: Remove dw_pcie_ep_init_notify() wrapper") > > > > You will need to pick up this patch as well: > > https://lore.kernel.org/linux-pci/20240622132024.2927799-2-cassel@kernel.org/T/#u > > Otherwise there will be a build error when merging the controller/dwc > > and the controller/rockchip branch to for-next. > > The patch that I sent out can be picked up to the controller/rockchip right > > now (since the API that Mani is switching to already exists in Linus's tree). > > Done. > > Hopefully, this settles things for a bit. Everything looks good! :) I'm glad that we got this sorted quickly, thank you Krzysztof! Kind regards, Niklas