On 11/12/2012 10:00 PM, Laxman Dewangan wrote:
+static int tegra_sflash_resume(struct device *dev)Can we avoid this whole function simply by programming SPI_COMMAND at
+{
+ struct spi_master *master = dev_get_drvdata(dev);
+ struct tegra_sflash_data *tsd = spi_master_get_devdata(master);
+ int ret;
+
+ ret = pm_runtime_get_sync(dev);
+ if (ret< 0) {
+ dev_err(dev, "pm runtime failed, e = %d\n", ret);
+ return ret;
+ }
+ tegra_sflash_writel(tsd, tsd->command_reg, SPI_COMMAND);
+ pm_runtime_put(dev);
the start of each transaction? That seems simpler. I assume that
shouldn't e.g. leave any chip-selects in some bad state, or at least if
it does, that shouldn't be a problem, because before the driver probes()
at kernel boot, SPI_COMMAND won't have been programmed either.
Aside from that,
Acked-by: Stephen Warren<swarren@xxxxxxxxxx>
Tested-by: Stephen Warren<swarren@xxxxxxxxxx>