Message ID | 1515482234-24716-8-git-send-email-pchandru@nvidia.com |
---|---|
State | Not Applicable |
Delegated to: | David Miller |
Headers | show |
Series | Refactor and add AHCI support for tegra210 | expand |
Tegra210 should be capitalized in the subject, and a commit message is needed. Add support for the AHCI-compliant Serial ATA host controller on the Tegra210 system-on-chip. Otherwise, Reviewed-by: Mikko Perttunen <mperttunen@nvidia.com> Mikko On 01/09/2018 09:17 AM, Preetham Chandru Ramchandra wrote: > From: Preetham Ramchandra <pchandru@nvidia.com> > > Signed-off-by: Preetham Chandru R <pchandru@nvidia.com> > --- > drivers/ata/ahci_tegra.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/drivers/ata/ahci_tegra.c b/drivers/ata/ahci_tegra.c > index 013880de6412..3f00dfa3760d 100644 > --- a/drivers/ata/ahci_tegra.c > +++ b/drivers/ata/ahci_tegra.c > @@ -256,6 +256,10 @@ static const struct tegra_ahci_soc tegra124_ahci_soc_data = { > }, > }; > > +static const struct tegra_ahci_soc tegra210_ahci_soc_data = { > + .quirks = NO_DEVSLP, > +}; > + Nit, I'd prefer this to be directly above the of_device_id table. > static int tegra_ahci_power_on(struct ahci_host_priv *hpriv) > { > struct tegra_ahci_priv *tegra = hpriv->plat_data; > @@ -471,6 +475,10 @@ static const struct of_device_id tegra_ahci_of_match[] = { > .compatible = "nvidia,tegra124-ahci", > .data = &tegra124_ahci_soc_data > }, > + { > + .compatible = "nvidia,tegra210-ahci", > + .data = &tegra210_ahci_soc_data > + }, > {} > }; > MODULE_DEVICE_TABLE(of, tegra_ahci_of_match); > @@ -585,5 +593,5 @@ static struct platform_driver tegra_ahci_driver = { > module_platform_driver(tegra_ahci_driver); > > MODULE_AUTHOR("Mikko Perttunen <mperttunen@nvidia.com>"); > -MODULE_DESCRIPTION("Tegra124 AHCI SATA driver"); > +MODULE_DESCRIPTION("Tegra AHCI SATA driver"); > MODULE_LICENSE("GPL v2"); > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
okay >-----Original Message----- >From: Mikko Perttunen [mailto:cyndis@kapsi.fi] >Sent: Tuesday, January 23, 2018 9:26 PM >To: Preetham Chandru <pchandru@nvidia.com>; thierry.reding@gmail.com; >tj@kernel.org >Cc: preetham260@gmail.com; linux-tegra@vger.kernel.org; linux- >ide@vger.kernel.org; Venu Byravarasu <vbyravarasu@nvidia.com>; Pavan >Kunapuli <pkunapuli@nvidia.com> >Subject: Re: [PATCH V6 7/7] ata: ahci_tegra: Add AHCI support for tegra210 > >Tegra210 should be capitalized in the subject, and a commit message is needed. > > Add support for the AHCI-compliant Serial ATA host controller on the > Tegra210 system-on-chip. > >Otherwise, >Reviewed-by: Mikko Perttunen <mperttunen@nvidia.com> > >Mikko > >On 01/09/2018 09:17 AM, Preetham Chandru Ramchandra wrote: >> From: Preetham Ramchandra <pchandru@nvidia.com> >> >> Signed-off-by: Preetham Chandru R <pchandru@nvidia.com> >> --- >> drivers/ata/ahci_tegra.c | 10 +++++++++- >> 1 file changed, 9 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/ata/ahci_tegra.c b/drivers/ata/ahci_tegra.c index >> 013880de6412..3f00dfa3760d 100644 >> --- a/drivers/ata/ahci_tegra.c >> +++ b/drivers/ata/ahci_tegra.c >> @@ -256,6 +256,10 @@ static const struct tegra_ahci_soc >tegra124_ahci_soc_data = { >> }, >> }; >> >> +static const struct tegra_ahci_soc tegra210_ahci_soc_data = { >> + .quirks = NO_DEVSLP, >> +}; >> + > >Nit, I'd prefer this to be directly above the of_device_id table. > >> static int tegra_ahci_power_on(struct ahci_host_priv *hpriv) >> { >> struct tegra_ahci_priv *tegra = hpriv->plat_data; @@ -471,6 +475,10 >> @@ static const struct of_device_id tegra_ahci_of_match[] = { >> .compatible = "nvidia,tegra124-ahci", >> .data = &tegra124_ahci_soc_data >> }, >> + { >> + .compatible = "nvidia,tegra210-ahci", >> + .data = &tegra210_ahci_soc_data >> + }, >> {} >> }; >> MODULE_DEVICE_TABLE(of, tegra_ahci_of_match); @@ -585,5 +593,5 @@ >> static struct platform_driver tegra_ahci_driver = { >> module_platform_driver(tegra_ahci_driver); >> >> MODULE_AUTHOR("Mikko Perttunen <mperttunen@nvidia.com>"); >> -MODULE_DESCRIPTION("Tegra124 AHCI SATA driver"); >> +MODULE_DESCRIPTION("Tegra AHCI SATA driver"); >> MODULE_LICENSE("GPL v2"); >>
diff --git a/drivers/ata/ahci_tegra.c b/drivers/ata/ahci_tegra.c index 013880de6412..3f00dfa3760d 100644 --- a/drivers/ata/ahci_tegra.c +++ b/drivers/ata/ahci_tegra.c @@ -256,6 +256,10 @@ static const struct tegra_ahci_soc tegra124_ahci_soc_data = { }, }; +static const struct tegra_ahci_soc tegra210_ahci_soc_data = { + .quirks = NO_DEVSLP, +}; + static int tegra_ahci_power_on(struct ahci_host_priv *hpriv) { struct tegra_ahci_priv *tegra = hpriv->plat_data; @@ -471,6 +475,10 @@ static const struct of_device_id tegra_ahci_of_match[] = { .compatible = "nvidia,tegra124-ahci", .data = &tegra124_ahci_soc_data }, + { + .compatible = "nvidia,tegra210-ahci", + .data = &tegra210_ahci_soc_data + }, {} }; MODULE_DEVICE_TABLE(of, tegra_ahci_of_match); @@ -585,5 +593,5 @@ static struct platform_driver tegra_ahci_driver = { module_platform_driver(tegra_ahci_driver); MODULE_AUTHOR("Mikko Perttunen <mperttunen@nvidia.com>"); -MODULE_DESCRIPTION("Tegra124 AHCI SATA driver"); +MODULE_DESCRIPTION("Tegra AHCI SATA driver"); MODULE_LICENSE("GPL v2");