From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 55BAFC433E0 for ; Sun, 26 Jul 2020 13:38:07 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1854E2065F for ; Sun, 26 Jul 2020 13:38:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="DsPckOeB"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="LH68B0WJ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1854E2065F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+iA1S2zNBSNgyNMGeodMBjgNa56r/kIGF1mZcplWzVA=; b=DsPckOeBy1w9OrGbb8Ji0i10N AI2vzXzgRmnFyvAd3/O7kpkPR87KnjyQW5ccE8jBoiK2PocMQhhx/6X8t7Oroywh9TuQWjOvSBsF8 US/1HCimRCs0SQh6pQwHI7HSeWMcxfPA1cy5L0oij6qL7Fi0LRvgHuasKDZMi0oTLH50FnYxwG0od lhOijFMguRiev7Xf7gqPzj0p/vC2s+OoMXakmsuOIX9y9/jaap1TGNd7cOoS9ju+uzkSfikRXwjOR D1DNmRB6+z5d/A4NBKyDv7toKcbOFXwIjRVEQizF/cLsV8ysBVWPQlHnvPdbFD0Z45cNTtIIv3dXB 2fkMvxVVw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jzgoo-0005Z9-8A; Sun, 26 Jul 2020 13:35:58 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jzgol-0005Yp-OY for linux-arm-kernel@lists.infradead.org; Sun, 26 Jul 2020 13:35:56 +0000 Received: from localhost (p5486c93f.dip0.t-ipconnect.de [84.134.201.63]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id AB5132065F; Sun, 26 Jul 2020 13:35:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1595770554; bh=jCCV8VYmRNQqNG2Se8e8SaJMyZBek/JWVtyEKyzac8k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=LH68B0WJmnEaWbSDySqWttA4aypHw/aroEDcF5ZyW4HmwzuIHR99ugDY8963RLMxQ Y7P0Rtg5N1eOaRONHKGXrqe2f37KzqbDfvSb77jK68iWRKMVeRVkOfc8gVfYho1omX 2gtnFBXQw72CRawIIPFNCzchjV6UMEx51g12JSVM= Date: Sun, 26 Jul 2020 15:35:51 +0200 From: Wolfram Sang To: Alain Volmat Subject: Re: [PATCH v2 1/2] i2c: smbus: add core function handling SMBus host-notify Message-ID: <20200726133551.GC16169@ninjato> References: <1593776168-17867-1-git-send-email-alain.volmat@st.com> <1593776168-17867-2-git-send-email-alain.volmat@st.com> <20200725202733.GA946@kunai> MIME-Version: 1.0 In-Reply-To: <20200725202733.GA946@kunai> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200726_093556_127136_F32651FD X-CRM114-Status: UNSURE ( 9.06 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alexandre.torgue@st.com, linux-kernel@vger.kernel.org, pierre-yves.mordret@st.com, linux-i2c@vger.kernel.org, fabrice.gasnier@st.com, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============6783439456142602431==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6783439456142602431== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="DSayHWYpDlRfCAAQ" Content-Disposition: inline --DSayHWYpDlRfCAAQ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable > > +void i2c_free_slave_host_notify_device(struct i2c_client *client) > > +{ > > + i2c_slave_unregister(client); > > + kfree(client->dev.platform_data); > > + i2c_unregister_device(client); > > +} > > +EXPORT_SYMBOL_GPL(i2c_free_slave_host_notify_device); >=20 > Sidenote: With my recent series "i2c: slave: improve sanity checks when > un-/registering" this code became NULL-safe (and IS_ERR safe, too). Stupid me, it is not NULL safe. The functions are. But, we deregister 'client' on our own. It probably makes sense to add some sanity checking of the parameters of the exported functions. --DSayHWYpDlRfCAAQ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAl8dhrMACgkQFA3kzBSg KbbyGxAAqCXGbq/i0jvy+gmv+JJk5HTVXYtglHm52P7iDw0zolpXRm0OEmF4+yRt 4MmbRDeEeQLeR2TvtaziN1q+LNQ2CObxbbVoyjcgNuc9pyo/zoM4No2JNnFqKWUt k5gl0nzVKXIlmJy2zb0EJT/xoyX+rae4hamgve7t3TvdQC3DS+sFJcReM9AmmMtN XHdPkrSbwHUhEUfDzmKw63CJZ46ExLmM03Ev08nkxW6YDkpZJeLMmloL54e8aVz7 KEYKO24PUZ4/fzgtvHK7H1baBmALgIn4sR2R6UqgIDE0BDGKP9a6XXUz2S+ZDuja vmJXZYwwKMgNPjFN5vFku1vxLUQRTNnQGy3cjX0PYW9AVMFcqLt7AOR7qHzNCZv7 t0MFewm19NOJMztPP+iLDPVOTACbuinF9NK4DptHwiBlEF2CIRYmBifJyLFjERW1 Q/S6T1k7u9EQ7ya9Rc6J8j9qk3GU9q6xU6DJtGlMGM94MvqNwdOrhzN9ysrJso7U tvfiv2lw0yDkQysl1QgtR68xpI2PLBZwahv5t+11M/7A14bd4LAWr+6kZ7Oc8G08 GeCpGIu1Ja1rMV/zPptp4IrxC1BA5GyVYaX6n01B59mK5GZMorzASUv18thLWNtq k64ubcie8Ka+C6YNvNPe9hs3egA3En5BHIXrZJidbM11PhZjY34= =AW42 -----END PGP SIGNATURE----- --DSayHWYpDlRfCAAQ-- --===============6783439456142602431== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6783439456142602431==--