Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753223AbaGNGWD (ORCPT ); Mon, 14 Jul 2014 02:22:03 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:2471 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751505AbaGNGVz (ORCPT ); Mon, 14 Jul 2014 02:21:55 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Sun, 13 Jul 2014 23:14:36 -0700 Message-ID: <53C376FF.3060509@nvidia.com> Date: Mon, 14 Jul 2014 09:21:51 +0300 From: Mikko Perttunen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Hans de Goede CC: Tejun Heo , "swarren@wwwdotorg.org" , "thierry.reding@gmail.com" , Peter De Schrijver , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "linux-ide@vger.kernel.org" Subject: Re: [PATCH v2 6/7] ata: Add support for the Tegra124 SATA controller References: <1403101406-15439-1-git-send-email-mperttunen@nvidia.com> <1403101406-15439-7-git-send-email-mperttunen@nvidia.com> <20140708132216.GA4979@htj.dyndns.org> In-Reply-To: <20140708132216.GA4979@htj.dyndns.org> X-NVConfidentiality: public Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Hans, have you been able to take a look at this? Thanks, Mikko On 08/07/14 16:22, Tejun Heo wrote: > (cc'ing Hans) > > Hans, can you please review this patch? > > On Wed, Jun 18, 2014 at 05:23:25PM +0300, Mikko Perttunen wrote: >> +#define SATA_CONFIGURATION_0 0x180 >> +#define SATA_CONFIGURATION_EN_FPCI BIT(0) > > Let's just indent uniformly. The new line should give enough visual > hint on grouping. > >> +struct tegra_ahci_priv { >> + struct platform_device *pdev; >> + void __iomem *sata_regs; >> + struct reset_control *sata_rst; >> + struct reset_control *sata_oob_rst; >> + struct reset_control *sata_cold_rst; >> + struct clk *sata_clk; >> + struct clk *sata_oob_clk; >> + struct clk *cml1_clk; >> + struct clk *plle_clk; >> + struct regulator_bulk_data supplies[5]; >> + struct phy *padctl_phy; >> +}; > > And please indent the declared fields uniformly too. > > Except for the above nitpicks, generally looks good to me but let's > wait for Hans' review. > > Thanks. > -- 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/