Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752083AbbGJDw4 (ORCPT ); Thu, 9 Jul 2015 23:52:56 -0400 Received: from eusmtp01.atmel.com ([212.144.249.243]:37022 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751473AbbGJDwq (ORCPT ); Thu, 9 Jul 2015 23:52:46 -0400 Message-ID: <559F4187.8090803@atmel.com> Date: Fri, 10 Jul 2015 11:52:39 +0800 From: Josh Wu User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Guenter Roeck CC: Nicolas Ferre , , Wei Yongjun , Alexandre Belloni , Ben Dooks , , "Krzysztof Kozlowski" , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Fabian Frederick , , Maxime Ripard Subject: Re: [PATCH 1/2] power: reset: at91: add sama5d3 reset function References: <1436436947-11210-1-git-send-email-josh.wu@atmel.com> <20150709173747.GA32448@roeck-us.net> <559F2719.3060708@atmel.com> <20150710031438.GA27089@roeck-us.net> In-Reply-To: <20150710031438.GA27089@roeck-us.net> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.168.5.13] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1944 Lines: 49 Hi, Guenter On 7/10/2015 11:14 AM, Guenter Roeck wrote: > On Fri, Jul 10, 2015 at 09:59:53AM +0800, Josh Wu wrote: >> Hi, Guenter >> >> On 7/10/2015 1:37 AM, Guenter Roeck wrote: >>> On Thu, Jul 09, 2015 at 06:15:46PM +0800, Josh Wu wrote: >>>> As since sama5d3, to reset the chip, we don't need to shutdown the ddr >>>> controller. >>>> >>>> So add a new compatible string and new restart function for sama5d3 and >>>> later chips. As we don't use sama5d3 ddr controller, so remove it as >>>> well. >>>> >>> That sounds like it should be two separate patches, or am I missing something ? >> I think using one patch makes more sense. Maybe the commit log is not clear >> enough. How about put it this way: >> >> This patch introduces a new compatible string: "atmel,sama5d3-rstc" for the >> reset driver of sama5d3 and later chips. >> As in sama5d3 or later chips, we don't have to shutdown the DDR controller >> before reset. Shutdown the DDR controller before reset is a workaround to >> avoid DDR signal driving the bus, but since sama5d3 and later chips there is >> no such a conflict. >> That means: >> 1. the sama5d3 reset function only need to write the rstc register and >> return. >> 2. for sama5d3, we can remove the code related with DDR controller as we >> don't use it at all. >> > Sorry, I don't get it. Doesn't that mean there are two distinct logical > changes, which would ask for two separate patches ? The rewritten reset function for sama5d3 has no need to access the ddr controller, so this patch rewrite the reset function and remove ddr access for sama5d3. Those two changes are related and so make it as one patch is reasonable. Best Regards, Josh Wu > > Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/