[PATCH v4] ARM/mx35/3ds: gpio: add mc9s08dz60 gpio function

Hi Wolfram:

  I changed the code to unset i2c client data just when gipochip_remove successes,
Otherwise, there will be problems next time the remove function is called.

  Please review the v5 patch.

  Thanks!

Best Regards
Wu Guoxing

-----Original Message-----
From: Wolfram Sang [mailto:w.sang at pengutronix.de] 
Sent: Thursday, November 10, 2011 6:21 PM
To: Marc Kleine-Budde
Cc: Wu Guoxing-B39297; grant.likely at secretlab.ca; s.hauer at pengutronix.de; shawn.guo at linaro.org; linux-arm-kernel at lists.infradead.org
Subject: Re: [PATCH v4] ARM/mx35/3ds: gpio: add mc9s08dz60 gpio function

On Thu, Nov 10, 2011 at 11:13:42AM +0100, Marc Kleine-Budde wrote:

On 11/10/2011 03:24 AM, wu guoxing wrote:

quoted

we only use the gpio function of mc9s08dz60 mcu chip, so just add 
the gpio driver, as this mcu will never be used in other board.

Signed-off-by: Wu Guoxing <redacted>
---
changes since v3:
	1. set i2c client data before gpiochip_add

exchange the two in the remove function, too.
First shut down the gpio chip, then unset set i2c client data.

and don't set clientdata to NULL.

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |