From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?iso-8859-1?q?St=FCbner?= Subject: Re: [PATCH v2 2/2] rtc: add hym8563 rtc-driver Date: Sun, 1 Dec 2013 18:19:25 +0100 Message-ID: <201312011819.26005.heiko@sntech.de> References: <201312011437.47432.heiko@sntech.de> <201312011439.45417.heiko@sntech.de> Mime-Version: 1.0 Content-Type: Text/Plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Richard Weinberger Cc: Alessandro Zummo , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , devicetree@vger.kernel.org, Grant Likely , LKML , rtc-linux@googlegroups.com, Mike Turquette List-Id: devicetree@vger.kernel.org Am Sonntag, 1. Dezember 2013, 15:38:40 schrieb Richard Weinberger: > On Sun, Dec 1, 2013 at 2:39 PM, Heiko St=FCbner wro= te: > > +static int hym8563_probe(struct i2c_client *client, > > + const struct i2c_device_id *id) > > +{ > > + struct hym8563 *hym8563; > > + int ret, gpio_int; > > + > > + hym8563 =3D devm_kzalloc(&client->dev, sizeof(hym8563), > > GFP_KERNEL); >=20 > Shouldn't this be sizeof(*hym8563)? you're right of course, thanks for catching the missing "*", I'll send = a v3=20 shortly. Heiko