From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Thu, 29 Oct 2015 07:54:32 +0100 Subject: [U-Boot] [PATCH 6/8] sf: probe: Minor cleanup In-Reply-To: <1444679655-30349-6-git-send-email-jteki@openedev.com> References: <1444679655-30349-1-git-send-email-jteki@openedev.com> <1444679655-30349-6-git-send-email-jteki@openedev.com> Message-ID: <5631C2A8.3050107@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Jagan, Am 12.10.2015 um 21:54 schrieb Jagan Teki: > - Use static for file-scope function > - Remove unneeded header file > - Use spi instead of slave notation for spi_slave {} > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_probe.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) Reviewed-by: Heiko Schocher bye, Heiko > > diff --git a/drivers/mtd/spi/sf_probe.c b/drivers/mtd/spi/sf_probe.c > index 60abaf2..112ab27 100644 > --- a/drivers/mtd/spi/sf_probe.c > +++ b/drivers/mtd/spi/sf_probe.c > @@ -24,7 +24,7 @@ struct spi_flash_priv { > }; > > #ifndef CONFIG_DM_SPI_FLASH > -struct spi_flash *spi_flash_probe_tail(struct spi_slave *bus) > +static struct spi_flash *spi_flash_probe_tail(struct spi_slave *bus) > { > struct spi_flash_priv *priv; > struct spi_flash *flash; > @@ -106,24 +106,24 @@ void spi_flash_free(struct spi_flash *flash) > > #else /* defined CONFIG_DM_SPI_FLASH */ > > -int spi_flash_std_probe(struct udevice *dev) > +static int spi_flash_std_probe(struct udevice *dev) > { > struct spi_flash_priv *priv = dev_get_uclass_priv(dev); > - struct spi_slave *slave = dev_get_parentdata(dev); > + struct spi_slave *spi = dev_get_parentdata(dev); > struct spi_flash *flash; > int ret; > > flash = &priv->flash; > flash->mtd = &priv->mtd; > > - flash->spi = slave; > + flash->spi = spi; > flash->priv = priv; > > priv->mtd.priv = flash; > flash->dev = dev; > > /* Claim spi bus */ > - ret = spi_claim_bus(slave); > + ret = spi_claim_bus(spi); > if (ret) { > debug("SF: Failed to claim SPI bus: %d\n", ret); > return ret; > @@ -144,9 +144,9 @@ int spi_flash_std_probe(struct udevice *dev) > return ret; > > err_mtd: > - spi_free_slave(slave); > + spi_free_slave(spi); > err_scan: > - spi_release_bus(slave); > + spi_release_bus(spi); > return ret; > } > > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany