From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Subject: Re: [PATCH 1/2] arm64: dts: r8a77990: Add SDHI device nodes Date: Tue, 6 Nov 2018 13:17:49 +0100 Message-ID: <20181106121748.k64k6ks3lafetaku@verge.net.au> References: <20181105214117.11734-1-marek.vasut+renesas@gmail.com> <7ab702f0-e78c-94c5-b74f-bac1803fb513@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Geert Uytterhoeven Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Takeshi Kihara , Geert Uytterhoeven , Wolfram Sang , Yoshihiro Shimoda , Linux-Renesas , Marek Vasut , Linux ARM , Marek Vasut List-Id: devicetree@vger.kernel.org On Tue, Nov 06, 2018 at 11:35:13AM +0100, Geert Uytterhoeven wrote: > Hi Marek, > > On Tue, Nov 6, 2018 at 11:32 AM Marek Vasut wrote: > > On 11/06/2018 06:26 AM, Yoshihiro Shimoda wrote: > > >> From: Marek Vasut, Sent: Tuesday, November 6, 2018 6:41 AM > > >> > > >> From: Takeshi Kihara > > >> > > >> This patch adds SDHI{0,1,3} device nodes for the r8a77990 SoC. > > >> > > >> Signed-off-by: Takeshi Kihara > > >> Signed-off-by: Marek Vasut > > >> Cc: Geert Uytterhoeven > > >> Cc: Simon Horman > > >> Cc: Wolfram Sang > > >> Cc: Yoshihiro Shimoda > > >> Cc: linux-renesas-soc@vger.kernel.org > > >> Cc: linux-arm-kernel@lists.infradead.org > > >> --- > > > > > > Thank you for the patch! > > > > > > Reviewed-by: Yoshihiro Shimoda > > > > > > I guess this patch and the 2/2 patch for ebisu have to be merged into a patch though. > > > > I wonder about that, 1/2 is adding SDHI DT nodes for an SoC, 2/2 is > > enabling SDHI on a particular board, which I think should be separate. > > The arm-soc maintainers tend to disagree, that's why Shimoda-san asked > to combine them. That is also my understanding. Please consider combining these patches. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 6 Nov 2018 13:17:49 +0100 From: Simon Horman To: Geert Uytterhoeven Cc: Marek Vasut , Yoshihiro Shimoda , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Takeshi Kihara , Marek Vasut , Geert Uytterhoeven , Wolfram Sang , Linux-Renesas , Linux ARM Subject: Re: [PATCH 1/2] arm64: dts: r8a77990: Add SDHI device nodes Message-ID: <20181106121748.k64k6ks3lafetaku@verge.net.au> References: <20181105214117.11734-1-marek.vasut+renesas@gmail.com> <7ab702f0-e78c-94c5-b74f-bac1803fb513@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: devicetree-owner@vger.kernel.org List-ID: On Tue, Nov 06, 2018 at 11:35:13AM +0100, Geert Uytterhoeven wrote: > Hi Marek, > > On Tue, Nov 6, 2018 at 11:32 AM Marek Vasut wrote: > > On 11/06/2018 06:26 AM, Yoshihiro Shimoda wrote: > > >> From: Marek Vasut, Sent: Tuesday, November 6, 2018 6:41 AM > > >> > > >> From: Takeshi Kihara > > >> > > >> This patch adds SDHI{0,1,3} device nodes for the r8a77990 SoC. > > >> > > >> Signed-off-by: Takeshi Kihara > > >> Signed-off-by: Marek Vasut > > >> Cc: Geert Uytterhoeven > > >> Cc: Simon Horman > > >> Cc: Wolfram Sang > > >> Cc: Yoshihiro Shimoda > > >> Cc: linux-renesas-soc@vger.kernel.org > > >> Cc: linux-arm-kernel@lists.infradead.org > > >> --- > > > > > > Thank you for the patch! > > > > > > Reviewed-by: Yoshihiro Shimoda > > > > > > I guess this patch and the 2/2 patch for ebisu have to be merged into a patch though. > > > > I wonder about that, 1/2 is adding SDHI DT nodes for an SoC, 2/2 is > > enabling SDHI on a particular board, which I think should be separate. > > The arm-soc maintainers tend to disagree, that's why Shimoda-san asked > to combine them. That is also my understanding. Please consider combining these patches. From mboxrd@z Thu Jan 1 00:00:00 1970 From: horms@verge.net.au (Simon Horman) Date: Tue, 6 Nov 2018 13:17:49 +0100 Subject: [PATCH 1/2] arm64: dts: r8a77990: Add SDHI device nodes In-Reply-To: References: <20181105214117.11734-1-marek.vasut+renesas@gmail.com> <7ab702f0-e78c-94c5-b74f-bac1803fb513@gmail.com> Message-ID: <20181106121748.k64k6ks3lafetaku@verge.net.au> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Nov 06, 2018 at 11:35:13AM +0100, Geert Uytterhoeven wrote: > Hi Marek, > > On Tue, Nov 6, 2018 at 11:32 AM Marek Vasut wrote: > > On 11/06/2018 06:26 AM, Yoshihiro Shimoda wrote: > > >> From: Marek Vasut, Sent: Tuesday, November 6, 2018 6:41 AM > > >> > > >> From: Takeshi Kihara > > >> > > >> This patch adds SDHI{0,1,3} device nodes for the r8a77990 SoC. > > >> > > >> Signed-off-by: Takeshi Kihara > > >> Signed-off-by: Marek Vasut > > >> Cc: Geert Uytterhoeven > > >> Cc: Simon Horman > > >> Cc: Wolfram Sang > > >> Cc: Yoshihiro Shimoda > > >> Cc: linux-renesas-soc at vger.kernel.org > > >> Cc: linux-arm-kernel at lists.infradead.org > > >> --- > > > > > > Thank you for the patch! > > > > > > Reviewed-by: Yoshihiro Shimoda > > > > > > I guess this patch and the 2/2 patch for ebisu have to be merged into a patch though. > > > > I wonder about that, 1/2 is adding SDHI DT nodes for an SoC, 2/2 is > > enabling SDHI on a particular board, which I think should be separate. > > The arm-soc maintainers tend to disagree, that's why Shimoda-san asked > to combine them. That is also my understanding. Please consider combining these patches.