On Fri, Jul 10, 2015 at 09:59:53AM +0800, Josh Wu wrote:
Hi, GuenterSorry, I don't get it. Doesn't that mean there are two distinct logical
On 7/10/2015 1:37 AM, Guenter Roeck wrote:
On Thu, Jul 09, 2015 at 06:15:46PM +0800, Josh Wu wrote:I think using one patch makes more sense. Maybe the commit log is not clear
As since sama5d3, to reset the chip, we don't need to shutdown the ddrThat sounds like it should be two separate patches, or am I missing something ?
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.
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.
changes, which would ask for two separate patches ?
Guenter