From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754851AbbBFGfj (ORCPT ); Fri, 6 Feb 2015 01:35:39 -0500 Received: from mail-ig0-f195.google.com ([209.85.213.195]:56348 "EHLO mail-ig0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754776AbbBFGfh (ORCPT ); Fri, 6 Feb 2015 01:35:37 -0500 MIME-Version: 1.0 In-Reply-To: References: <1423128277-10297-1-git-send-email-bintian.wang@huawei.com> Date: Fri, 6 Feb 2015 14:35:35 +0800 Message-ID: Subject: Re: [PATCH 0/3] arm64,hi6220: Enable Hisilicon Hi6220 SoC From: Brent Wang To: Olof Johansson Cc: Tyler Baker , Bintian Wang , linux-arm-kernel , "linux-kernel@vger.kernel.org" , Catalin Marinas , Will Deacon , "devicetree@vger.kernel.org" , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kevin Hilman , Mike Turquette , Rob Herring , Zhangfei Gao , Haojian Zhuang , Xu Wei , Jaehoon Chung , Haifeng Yan , Stephen Boyd , xuejiancheng@huawei.com, sledge.yanwei@huawei.com, Tomeu Vizoso , Russell King - ARM Linux , Guodong Xu , dan.zhao@hisilicon.com, "huxinwei@huawei.com" , xuyiping@hisilicon.com, victor.lixin@hisilicon.com, btw@mail.itp.ac.cn, puck.chen@hisilicon.com, wangbinghui@hisilicon.com, zhenwei.wang@hisilicon.com, "Liguozhu (Kenneth)" , XinWei Kong , heyunlei@huawei.com, w.f@huawei.com, z.liuxinliang@huawei.com Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Olof, 2015-02-06 14:18 GMT+08:00 Olof Johansson : > > On Thu, Feb 5, 2015 at 8:21 PM, Brent Wang wrote: > > Hello Olof and Tyler, > > > > 2015-02-06 7:52 GMT+08:00 Tyler Baker : > >> > >> On 5 February 2015 at 11:02, Olof Johansson wrote: > >> > On Thu, Feb 5, 2015 at 10:46 AM, Tyler Baker > >> > wrote: > >> >> Hi Bintian, > >> >> > >> >> On 5 February 2015 at 01:24, Bintian Wang > >> >> wrote: > >> >>> Hello, > >> >>> > >> >>> Hi6220 is one mobile solution of Hisilicon, this patchset contains > >> >>> initial support for Hi6220 SoC and HiKey development board, which > >> >>> is based on ARM Cortex A53 architecture. Initial support is minimal > >> >>> and includes just the arch configuration, clock driver, device tree > >> >>> configuration. > >> >>> > >> >>> Many peripheral drivers will be submitted later. > >> >>> > >> >>> Any comments will be appreciated! > >> >>> > >> >>> Thanks, > >> >>> > >> >>> Bintian Wang (3): > >> >>> arm64: Enable Hisilicon ARMv8 SoC family in Kconfig and defconfig > >> >>> clk: hi6220: Clock driver support for Hisilicon hi6220 SoC > >> >>> arm64: dts: Add dts files for Hisilicon Hi6220 SoC > >> >> > >> >> Thanks for posting these! I've applied this series on top of > >> >> next-20150204, however there was some fuzz that needed to be cleaned > >> >> up on 3/3 [1]. I've confirmed the platform is booting to a basic user > >> >> space without issue. > >> > > >> > From ramdisk only, right? > >> > >> Correct, ramdisk only. > >> > >> > Given the timing of the posting of this > >> > patch set, I'm not going to merge it for 3.20. Hopefully for 3.21 > >> > there will be some more peripheral support as well -- at least some > >> > sort of storage device. > >> > >> Seem fair to me. I also hope to see more patches posted shortly. > > > > Yes, the mmc and sd drivers will be submitted soon, should they be included > > in this patchset? I have thought submitting this patch first for review, if > > there > > is no problem for this patchset and then submit other drivers, you know, > > other > > drivers will depend on this patchset. > > > The drivers should ideally not depend on the SoC patchset -- the > driver can go in independently. The DTS updates to specify the > hardware should go in through arm-soc even if the driver itself (and > the binding document update) should go in through the driver subsystem > instead. > > > So, you can choose if you want to post everything as a long series, > and cc different maintainers on the various parts of the series -- or > you can post each driver or subsystem as a patchset on its own and let > that get merged through respective maintainer. The latter is most > common these days. I think the latter is a better way, thank you! > > > > -Olof -- Best Regards, Bintian =========================== Don't be nervous, just be happy! From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brent Wang Subject: Re: [PATCH 0/3] arm64,hi6220: Enable Hisilicon Hi6220 SoC Date: Fri, 6 Feb 2015 14:35:35 +0800 Message-ID: References: <1423128277-10297-1-git-send-email-bintian.wang@huawei.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Olof Johansson Cc: Tyler Baker , Bintian Wang , linux-arm-kernel , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Catalin Marinas , Will Deacon , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kevin Hilman , Mike Turquette , Rob Herring , Zhangfei Gao , Haojian Zhuang , Xu Wei , Jaehoon Chung , Haifeng Yan , Stephen Boyd , xuejiancheng@hu List-Id: devicetree@vger.kernel.org Hello Olof, 2015-02-06 14:18 GMT+08:00 Olof Johansson : > > On Thu, Feb 5, 2015 at 8:21 PM, Brent Wang wrote: > > Hello Olof and Tyler, > > > > 2015-02-06 7:52 GMT+08:00 Tyler Baker : > >> > >> On 5 February 2015 at 11:02, Olof Johansson wrote: > >> > On Thu, Feb 5, 2015 at 10:46 AM, Tyler Baker > >> > wrote: > >> >> Hi Bintian, > >> >> > >> >> On 5 February 2015 at 01:24, Bintian Wang > >> >> wrote: > >> >>> Hello, > >> >>> > >> >>> Hi6220 is one mobile solution of Hisilicon, this patchset contains > >> >>> initial support for Hi6220 SoC and HiKey development board, which > >> >>> is based on ARM Cortex A53 architecture. Initial support is minimal > >> >>> and includes just the arch configuration, clock driver, device tree > >> >>> configuration. > >> >>> > >> >>> Many peripheral drivers will be submitted later. > >> >>> > >> >>> Any comments will be appreciated! > >> >>> > >> >>> Thanks, > >> >>> > >> >>> Bintian Wang (3): > >> >>> arm64: Enable Hisilicon ARMv8 SoC family in Kconfig and defconfig > >> >>> clk: hi6220: Clock driver support for Hisilicon hi6220 SoC > >> >>> arm64: dts: Add dts files for Hisilicon Hi6220 SoC > >> >> > >> >> Thanks for posting these! I've applied this series on top of > >> >> next-20150204, however there was some fuzz that needed to be cleaned > >> >> up on 3/3 [1]. I've confirmed the platform is booting to a basic user > >> >> space without issue. > >> > > >> > From ramdisk only, right? > >> > >> Correct, ramdisk only. > >> > >> > Given the timing of the posting of this > >> > patch set, I'm not going to merge it for 3.20. Hopefully for 3.21 > >> > there will be some more peripheral support as well -- at least some > >> > sort of storage device. > >> > >> Seem fair to me. I also hope to see more patches posted shortly. > > > > Yes, the mmc and sd drivers will be submitted soon, should they be included > > in this patchset? I have thought submitting this patch first for review, if > > there > > is no problem for this patchset and then submit other drivers, you know, > > other > > drivers will depend on this patchset. > > > The drivers should ideally not depend on the SoC patchset -- the > driver can go in independently. The DTS updates to specify the > hardware should go in through arm-soc even if the driver itself (and > the binding document update) should go in through the driver subsystem > instead. > > > So, you can choose if you want to post everything as a long series, > and cc different maintainers on the various parts of the series -- or > you can post each driver or subsystem as a patchset on its own and let > that get merged through respective maintainer. The latter is most > common these days. I think the latter is a better way, thank you! > > > > -Olof -- Best Regards, Bintian =========================== Don't be nervous, just be happy! -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: wangbintian@gmail.com (Brent Wang) Date: Fri, 6 Feb 2015 14:35:35 +0800 Subject: [PATCH 0/3] arm64,hi6220: Enable Hisilicon Hi6220 SoC In-Reply-To: References: <1423128277-10297-1-git-send-email-bintian.wang@huawei.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Olof, 2015-02-06 14:18 GMT+08:00 Olof Johansson : > > On Thu, Feb 5, 2015 at 8:21 PM, Brent Wang wrote: > > Hello Olof and Tyler, > > > > 2015-02-06 7:52 GMT+08:00 Tyler Baker : > >> > >> On 5 February 2015 at 11:02, Olof Johansson wrote: > >> > On Thu, Feb 5, 2015 at 10:46 AM, Tyler Baker > >> > wrote: > >> >> Hi Bintian, > >> >> > >> >> On 5 February 2015 at 01:24, Bintian Wang > >> >> wrote: > >> >>> Hello, > >> >>> > >> >>> Hi6220 is one mobile solution of Hisilicon, this patchset contains > >> >>> initial support for Hi6220 SoC and HiKey development board, which > >> >>> is based on ARM Cortex A53 architecture. Initial support is minimal > >> >>> and includes just the arch configuration, clock driver, device tree > >> >>> configuration. > >> >>> > >> >>> Many peripheral drivers will be submitted later. > >> >>> > >> >>> Any comments will be appreciated! > >> >>> > >> >>> Thanks, > >> >>> > >> >>> Bintian Wang (3): > >> >>> arm64: Enable Hisilicon ARMv8 SoC family in Kconfig and defconfig > >> >>> clk: hi6220: Clock driver support for Hisilicon hi6220 SoC > >> >>> arm64: dts: Add dts files for Hisilicon Hi6220 SoC > >> >> > >> >> Thanks for posting these! I've applied this series on top of > >> >> next-20150204, however there was some fuzz that needed to be cleaned > >> >> up on 3/3 [1]. I've confirmed the platform is booting to a basic user > >> >> space without issue. > >> > > >> > From ramdisk only, right? > >> > >> Correct, ramdisk only. > >> > >> > Given the timing of the posting of this > >> > patch set, I'm not going to merge it for 3.20. Hopefully for 3.21 > >> > there will be some more peripheral support as well -- at least some > >> > sort of storage device. > >> > >> Seem fair to me. I also hope to see more patches posted shortly. > > > > Yes, the mmc and sd drivers will be submitted soon, should they be included > > in this patchset? I have thought submitting this patch first for review, if > > there > > is no problem for this patchset and then submit other drivers, you know, > > other > > drivers will depend on this patchset. > > > The drivers should ideally not depend on the SoC patchset -- the > driver can go in independently. The DTS updates to specify the > hardware should go in through arm-soc even if the driver itself (and > the binding document update) should go in through the driver subsystem > instead. > > > So, you can choose if you want to post everything as a long series, > and cc different maintainers on the various parts of the series -- or > you can post each driver or subsystem as a patchset on its own and let > that get merged through respective maintainer. The latter is most > common these days. I think the latter is a better way, thank you! > > > > -Olof -- Best Regards, Bintian =========================== Don't be nervous, just be happy!