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=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_MUTT autolearn=ham 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 9A07FC43381 for ; Fri, 22 Feb 2019 11:32:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 588682070D for ; Fri, 22 Feb 2019 11:32:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727067AbfBVLcC (ORCPT ); Fri, 22 Feb 2019 06:32:02 -0500 Received: from sauhun.de ([88.99.104.3]:44864 "EHLO pokefinder.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726131AbfBVLcB (ORCPT ); Fri, 22 Feb 2019 06:32:01 -0500 Received: from localhost (p54B331E2.dip0.t-ipconnect.de [84.179.49.226]) by pokefinder.org (Postfix) with ESMTPSA id CCE253E4AA3; Fri, 22 Feb 2019 12:31:59 +0100 (CET) Date: Fri, 22 Feb 2019 12:31:59 +0100 From: Wolfram Sang To: Charles Keepax Cc: Benjamin Tissoires , Jim Broadus , Linux I2C , lkml Subject: Re: [PATCH] i2c: Allow recovery of the initial IRQ by an I2C client device. Message-ID: <20190222113159.GB1771@kunai> References: <20190219193027.13882-1-jbroadus@gmail.com> <20190221232609.d4vxl3osj6mwooey@katana> <20190222102335.GA1771@kunai> <20190222103019.GD130153@ediswmail.ad.cirrus.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="LpQ9ahxlCli8rRTG" Content-Disposition: inline In-Reply-To: <20190222103019.GD130153@ediswmail.ad.cirrus.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --LpQ9ahxlCli8rRTG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable > > > But I still have the feeling that the problem is not solved at the > > > right place. In i2c_new_device() we are storing parts of the fields of > > > struct i2c_board_info, and when resetting the irq we are losing > > > information. This patch solves that, but I wonder if the IRQ should > > > not be 'simply' set in i2c_device_probe(). This means we also need to > > > store the .resources of info, but I have a feeling this will be less > > > error prone in the future. > > >=20 > > > But this is just my guts telling me something is not right. I would > > > perfectly understand if we want to get this merged ASAP. > > >=20 > > > So given that the code is correct, this is my: > > > Reviewed-by: Benjamin Tissoires > > >=20 > > > But at least I have expressed my feelings :) > >=20 > > Which I can relate to very much. I see the code solves the issue but my > > feeling is that we are patching around something which should be handled > > differently in general. > >=20 > > Is somebody willing to research this further? > >=20 > > Thanks for your input. > >=20 >=20 > I would be willing to have more of a look at it but am slightly > nervous I am not right person as all the systems I currently work > with are DT based so don't really exemplify the issue at all. Thank you! Well, I'll be there, too. Discussing, reviewing, testing. And if we have Benjamin for that on board as well, then I think we have a good start for that task :) Others are more than welcome to join, too, of course. --LpQ9ahxlCli8rRTG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAlxv3asACgkQFA3kzBSg Kbatew/+I8Kal9hvYRAu3wYJbW7Fpxjvnz7kloIiOb9wCli1oKEgOXy2ahg59Fq4 Cht2UljMlNDRzOhvmCiQsrb37ZfVRtXj9opXSJP/IRFFz11x5cUEl9y4nUf8EzIR LaCE9WcHRfq0S/RjwxAekEM4fWlwSiYlrw0n6tEZ63rPZkqHtNZ6b/Q0bZBzd6pC bion7WcG7pKJ3uOwEvfytZ1ouFr0NZbTKLs3ncxPy+kbTW9nqwFG6cmf9fZAErcv d8CT/1Z3z22OHJpJ3/ys99Tv7obvtcmFDspZeNKtH+QFXQ+YqfSc5S5+UlgNZ/iu /v3t5acY+QwnKmc8rc4C83htiVIGc8g91VR+tM4i+2nOiZ4CZPyOE0u7XuQwSd8c TxFB7PRqBLMAYJ+rUzcMqCYndatDEN3JCr12OPgK8hzTo7rh+YS2Nh803CQV2nHO AxjIi47sHIelMkHrQnXt300CPMe3A4AiJGbxKGaD0YNPXgJtvIRl+23JhJJlBiuW 42tjdALn3gG+iLmVSi1fbVn63vkA5jHa6wA/wChL4Nx0elnvle4a6WfcwTarbZJa wCqKWpVUFh5iPiuK1cr7/TbtzzNSn34//cxgmUyisty2JDWvIhT0c9wREAC1UqW4 3NULn8vAgNhncb+VZcQCmQHZF/iKIo8CzMOXrqrof1YEaUsdJYo= =3CRi -----END PGP SIGNATURE----- --LpQ9ahxlCli8rRTG--