From mboxrd@z Thu Jan 1 00:00:00 1970 From: mpa@pengutronix.de (Markus Pargmann) Date: Mon, 25 Aug 2014 09:24:55 +0200 Subject: [PATCH] watchdog: imx2_wdt: Remove seperate big endian handling In-Reply-To: <5d35f7c722e14022ba7db0221b9cfdb1@BY2PR0301MB0613.namprd03.prod.outlook.com> References: <1408949015-28759-1-git-send-email-mpa@pengutronix.de> <4b9a55951535433aada42332b08f664f@BY2PR0301MB0613.namprd03.prod.outlook.com> <20140825070550.GA11914@pengutronix.de> <5d35f7c722e14022ba7db0221b9cfdb1@BY2PR0301MB0613.namprd03.prod.outlook.com> Message-ID: <20140825072455.GB11914@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Aug 25, 2014 at 07:11:36AM +0000, Li.Xiubo at freescale.com wrote: > > On Mon, Aug 25, 2014 at 07:01:57AM +0000, Li.Xiubo at freescale.com wrote: > > > Hi Markus, > > > > > > Thanks for your work. > > > > > > And there has already one patch for this: > > > > > > "watchdog: imx2_wdt: Convert to use regmap framework's endianness method." > > > > Thanks didn't see this one. > > > > Best regards, > > > > Markus > > > > I also hoping for your comments about it if there has any problem. I just noticed that you added an include of of_address.h in your original patch: f728f4bfc495 (watchdog: imx2_wdt: adds big endianness support.) But you don't remove the include in your current patch. Was the include missing in the driver? Best regards, Markus -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: