From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751693AbdBOJQB (ORCPT ); Wed, 15 Feb 2017 04:16:01 -0500 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:50021 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751517AbdBOJP7 (ORCPT ); Wed, 15 Feb 2017 04:15:59 -0500 Message-ID: <1487150153.2433.11.camel@pengutronix.de> Subject: Re: [PATCH v2] reset: Add i.MX7 SRC reset driver From: Philipp Zabel To: Andrey Smirnov Cc: Lucas Stach , Rob Herring , Mark Rutland , devicetree@vger.kernel.org, linux-kernel , linux-arm-kernel@lists.infradead.org Date: Wed, 15 Feb 2017 10:15:53 +0100 In-Reply-To: References: <20170213153338.13518-1-andrew.smirnov@gmail.com> <1487004643.2873.97.camel@pengutronix.de> <1487089919.2305.22.camel@pengutronix.de> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:3ad5:47ff:feaf:1a17 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2017-02-14 at 12:11 -0800, Andrey Smirnov wrote: [...] > > Arguably, the A7 resets should not be handled by the peripheral reset > > controller, but at least for the others I see no reason not to leave > > space for them in the index table. > > If for some bizarre reason one was to run Linux on M4 and use A7 as > applications processor, resetting A7 might be useful. But that's a > very unlikely use-case. > > > In fact, since unused reset controls > > don't use space, why not number them all? > > IMHO because it is unused code and because those numbers constitute an > interface which once set will be hard to change if need be. > > But that's not that important and I don't feel particularly strong > about that point, so I'll add those sources in v3. Thanks. > Do you insist that I split what I call IMX7_RESET_PCIEPHY into > PCIEPHY_G_RST and PCIEPHY_BTN or can I keep it as a single logical > reset? No, I say keep it as is. For now I'll assume this is not a reset, but some other interface signal that just happens to be routed out to the SRC and just happens to be toggled around the same time in the enable sequence. [...] > >> >> +#define IMX7_RESET_PCIE_CTRL_APPS 0 > >> >> +#define IMX7_RESET_PCIEPHY 1 > >> > > >> > It would expect these to be numbered in the order they appear in the > >> > register map, not starting from the end. Could you add all available > >> > peripheral resets to this list, in the correct order? > >> > >> The numbering is just a consequence of me adding only resets I could > >> exercise with my code and numbering then starting from zero. I also > >> was hesitant adding more sources since it seemed to me that some of > >> less trivial registers in that IP block might be best represented as a > >> single reset source doing something more sophisticated that just > >> setting a bit under the hood. > > > > Any in particular? > > USBPHY1/2 and maybe MIPI resets? But that's no more than a gut feeling. Is there any downstream code that already handles these resets? At least for the USB PHYs I'd expect there has to be something we could look at. regards Philipp