| Message ID | 1369987148-22970-1-git-send-email-sachin.kamat@linaro.org |
|---|---|
| State | Accepted |
| Headers | show |
On Fri, May 31, 2013 at 01:29:06PM +0530, Sachin Kamat wrote:
> Setting platform data to NULL is not necessary.
Applied, thanks.
On 2 June 2013 00:48, Mark Brown <broonie@kernel.org> wrote: > On Fri, May 31, 2013 at 01:29:06PM +0530, Sachin Kamat wrote: >> Setting platform data to NULL is not necessary. > > Applied, thanks. Looks like instead of $Subject patch, "spi: spi-fsl-lib: Make mpc8xxx_spi_work static" has been applied to your topic/nuc900 branch (as well as to topic/fsl branch).
On Thu, Jun 06, 2013 at 04:21:39PM +0530, Sachin Kamat wrote: > Looks like instead of $Subject patch, "spi: spi-fsl-lib: Make > mpc8xxx_spi_work static" has been applied to > your topic/nuc900 branch (as well as to topic/fsl branch). I'm not seeing any duplicaetd patches from you in -next.
On 6 June 2013 16:27, Mark Brown <broonie@kernel.org> wrote: > On Thu, Jun 06, 2013 at 04:21:39PM +0530, Sachin Kamat wrote: > >> Looks like instead of $Subject patch, "spi: spi-fsl-lib: Make >> mpc8xxx_spi_work static" has been applied to >> your topic/nuc900 branch (as well as to topic/fsl branch). > > I'm not seeing any duplicaetd patches from you in -next. Commits d0864323 and e48d29925 both are "spi: spi-fsl-lib: Make mpc8xxx_spi_work static".
On Thu, Jun 06, 2013 at 04:34:44PM +0530, Sachin Kamat wrote: > On 6 June 2013 16:27, Mark Brown <broonie@kernel.org> wrote: > > I'm not seeing any duplicaetd patches from you in -next. > Commits d0864323 and e48d29925 both are "spi: spi-fsl-lib: Make > mpc8xxx_spi_work static". Doesn't matter anyway, I rebased and repushed the nuc900 branch.
On 6 June 2013 16:45, Mark Brown <broonie@kernel.org> wrote: > On Thu, Jun 06, 2013 at 04:34:44PM +0530, Sachin Kamat wrote: >> On 6 June 2013 16:27, Mark Brown <broonie@kernel.org> wrote: > >> > I'm not seeing any duplicaetd patches from you in -next. > >> Commits d0864323 and e48d29925 both are "spi: spi-fsl-lib: Make >> mpc8xxx_spi_work static". > > Doesn't matter anyway, I rebased and repushed the nuc900 branch. OK. Thanks.
diff --git a/drivers/spi/spi-nuc900.c b/drivers/spi/spi-nuc900.c index 94efa2c..2ad3d74 100644 --- a/drivers/spi/spi-nuc900.c +++ b/drivers/spi/spi-nuc900.c @@ -446,8 +446,6 @@ static int nuc900_spi_remove(struct platform_device *dev) free_irq(hw->irq, hw); - platform_set_drvdata(dev, NULL); - spi_bitbang_stop(&hw->bitbang); clk_disable(hw->clk);
Setting platform data to NULL is not necessary. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Wan ZongShun <mcuos.com@gmail.com> --- drivers/spi/spi-nuc900.c | 2 -- 1 file changed, 2 deletions(-)