From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiner Kallweit Subject: Re: [PATCH 2/3] spi: fsl-(e)spi: simplify cleanup code Date: Tue, 25 Aug 2015 22:19:35 +0200 Message-ID: <55DCCDD7.5030307@gmail.com> References: <55D9BAE1.1030306@gmail.com> <20150825174508.GF12878@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: "linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" To: Mark Brown Return-path: In-Reply-To: <20150825174508.GF12878-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Am 25.08.2015 um 19:45 schrieb Mark Brown: > On Sun, Aug 23, 2015 at 02:21:53PM +0200, Heiner Kallweit wrote: > >> static int of_fsl_espi_remove(struct platform_device *dev) >> { >> - return mpc8xxx_spi_remove(&dev->dev); >> + return 0; >> } > > You should be able to remove empty functions like this one. In patch 3 a call to pm_runtime_disable is added to this function. Therefore I don't remove it in patch 2. Fine with you? -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html