Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On 12/05/2013 03:17 AM, Dan Williams wrote: On Wed, Dec 4, 2013 at 3:24 AM, Qiao Zhou wrote: On 12/04/2013 07:17 PM, Haojian Zhuang wrote: Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Misunderstood it... Thanks for correcting. Please combine the patches for two reasons: 1/ patch1 by itself makes the problem worse it prevents the mmp_tdma driver from building even if CPU_MMP2 is selected. 2/ patch2 does not have a changelog and is the only user of the enabling in patch1 Dan, I updated the patch according to your suggestions. please help take a look again. Thanks. -- Best Regards Qiao -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On Wed, Dec 4, 2013 at 3:24 AM, Qiao Zhou wrote: > On 12/04/2013 07:17 PM, Haojian Zhuang wrote: >> >> Dan indicated that you could pack these two patches into one. Whatever >> it's also OK to use two patches. > > Misunderstood it... Thanks for correcting. > Please combine the patches for two reasons: 1/ patch1 by itself makes the problem worse it prevents the mmp_tdma driver from building even if CPU_MMP2 is selected. 2/ patch2 does not have a changelog and is the only user of the enabling in patch1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On 12/04/2013 07:17 PM, Haojian Zhuang wrote: Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Misunderstood it... Thanks for correcting. -- Best Regards Qiao -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On Wed, Dec 4, 2013 at 4:21 PM, Qiao Zhou wrote: > V1 -> V0: > No need for help text for MMP_SRAM in Kconfig and move it into MMP_TDMA > text in Kconfig. > > Qiao Zhou (2): > arm: mmp: build sram driver alone > dma: mmp-tdma: select sram driver > > arch/arm/mach-mmp/Kconfig |3 +++ > arch/arm/mach-mmp/Makefile |3 ++- > drivers/dma/Kconfig|2 ++ > 3 files changed, 7 insertions(+), 1 deletions(-) > Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Applied. Thanks Haojian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
V1 -> V0: No need for help text for MMP_SRAM in Kconfig and move it into MMP_TDMA text in Kconfig. Qiao Zhou (2): arm: mmp: build sram driver alone dma: mmp-tdma: select sram driver arch/arm/mach-mmp/Kconfig |3 +++ arch/arm/mach-mmp/Makefile |3 ++- drivers/dma/Kconfig|2 ++ 3 files changed, 7 insertions(+), 1 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
V1 - V0: No need for help text for MMP_SRAM in Kconfig and move it into MMP_TDMA text in Kconfig. Qiao Zhou (2): arm: mmp: build sram driver alone dma: mmp-tdma: select sram driver arch/arm/mach-mmp/Kconfig |3 +++ arch/arm/mach-mmp/Makefile |3 ++- drivers/dma/Kconfig|2 ++ 3 files changed, 7 insertions(+), 1 deletions(-) -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On Wed, Dec 4, 2013 at 4:21 PM, Qiao Zhou zhouq...@marvell.com wrote: V1 - V0: No need for help text for MMP_SRAM in Kconfig and move it into MMP_TDMA text in Kconfig. Qiao Zhou (2): arm: mmp: build sram driver alone dma: mmp-tdma: select sram driver arch/arm/mach-mmp/Kconfig |3 +++ arch/arm/mach-mmp/Makefile |3 ++- drivers/dma/Kconfig|2 ++ 3 files changed, 7 insertions(+), 1 deletions(-) Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Applied. Thanks Haojian -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On 12/04/2013 07:17 PM, Haojian Zhuang wrote: Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Misunderstood it... Thanks for correcting. -- Best Regards Qiao -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On Wed, Dec 4, 2013 at 3:24 AM, Qiao Zhou zhouq...@marvell.com wrote: On 12/04/2013 07:17 PM, Haojian Zhuang wrote: Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Misunderstood it... Thanks for correcting. Please combine the patches for two reasons: 1/ patch1 by itself makes the problem worse it prevents the mmp_tdma driver from building even if CPU_MMP2 is selected. 2/ patch2 does not have a changelog and is the only user of the enabling in patch1 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/2 V1] arm-mmp-build-sram-driver-alone
On 12/05/2013 03:17 AM, Dan Williams wrote: On Wed, Dec 4, 2013 at 3:24 AM, Qiao Zhou zhouq...@marvell.com wrote: On 12/04/2013 07:17 PM, Haojian Zhuang wrote: Dan indicated that you could pack these two patches into one. Whatever it's also OK to use two patches. Misunderstood it... Thanks for correcting. Please combine the patches for two reasons: 1/ patch1 by itself makes the problem worse it prevents the mmp_tdma driver from building even if CPU_MMP2 is selected. 2/ patch2 does not have a changelog and is the only user of the enabling in patch1 Dan, I updated the patch according to your suggestions. please help take a look again. Thanks. -- Best Regards Qiao -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/