From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935559AbdADKS3 convert rfc822-to-8bit (ORCPT ); Wed, 4 Jan 2017 05:18:29 -0500 Received: from mx2.suse.de ([195.135.220.15]:56509 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752396AbdADKSW (ORCPT ); Wed, 4 Jan 2017 05:18:22 -0500 Date: Wed, 4 Jan 2017 11:18:17 +0100 From: Jean Delvare To: Benjamin Tissoires Cc: Pali =?UTF-8?B?Um9ow6Fy?= , =?UTF-8?B?TWljaGE=?= =?UTF-8?B?xYIgS8SZcGllxYQ=?= , Wolfram Sang , Steven Honeyman , Valdis.Kletnieks@vt.edu, Jochen Eisinger , Gabriele Mazzotta , Andy Lutomirski , Mario_Limonciello@dell.com, Alex Hung , Takashi Iwai , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, Dmitry Torokhov Subject: Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines Message-ID: <20170104111817.663236ae@endymion> In-Reply-To: <20170103090641.GH5767@mail.corp.redhat.com> References: <1482843136-12838-1-git-send-email-pali.rohar@gmail.com> <201612291517.37474@pali> <20161229210932.GA1254@kmp-mobile.hq.kempniu.pl> <201612292228.18706@pali> <20170103090641.GH5767@mail.corp.redhat.com> Organization: SUSE Linux X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.31; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 3 Jan 2017 10:06:41 +0100, Benjamin Tissoires wrote: > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > So 4d5538f5882a is breaking lis3lv02d driver... > > Apologies for that. > > I could easily fix this by adding a kernel API to know whether the > provided irq is from Host Notify or if it was coming from an actual > declaration. However, I have no idea how many other drivers would > require this (hopefully only this one). > > One other solution would be to reserve the Host Notify IRQ and let the > actual drivers that need it to set it, but this was not the best > solution according to Dmitri. On my side, I am not entirely against this > given that it's a chip feature, so the driver should be able to know > that it's available. > > Dmitri, Wolfram, Jean, any preferences? I did not look at your work so I do not feel qualified to give advice on the proper way. I trust you to make the right decision :-) -- Jean Delvare SUSE L3 Support