Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753453AbbGIRbG (ORCPT ); Thu, 9 Jul 2015 13:31:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57952 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751164AbbGIRbC (ORCPT ); Thu, 9 Jul 2015 13:31:02 -0400 From: Benjamin Tissoires To: Wolfram Sang , Jean Delvare , Dmitry Torokhov Cc: benjamin.tissoires@gmail.com, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, Benjamin Tissoires Subject: [PATCH v2 0/3] I2C/SMBus: add support for Host Notify (in i2c_i801) Date: Thu, 9 Jul 2015 13:30:49 -0400 Message-Id: <1436463052-6944-1-git-send-email-benjamin.tissoires@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1379 Lines: 39 Hi, this is a v2 of this patch series. Things are starting to shape better I think, but I am still not sure about one or two things: - do we want to #ifdef i2c_propagate_smbus_host_notify() so that we do not add a dependency in the KConfig for i2c_i801, or do actually want the dependency? - do we want to have i2c_propagate_smbus_host_notify() to actually be able to sleep or do we prefer calling it directly a worker? Note that I only compile-tested the ipmi_ssif changes, but this seems straightforward enough. Thanks for your inputs. Cheers, Benjamin Benjamin Tissoires (3): i2c: add a protocol parameter to the alert callback i2c-smbus: add SMBus Host Notify support i2c: i801: add support of Host Notify Documentation/i2c/smbus-protocol | 3 + drivers/char/ipmi/ipmi_ssif.c | 6 +- drivers/i2c/busses/i2c-i801.c | 172 +++++++++++++++++++++++++++++---------- drivers/i2c/i2c-smbus.c | 35 +++++++- include/linux/i2c-smbus.h | 8 ++ include/linux/i2c.h | 10 ++- include/uapi/linux/i2c.h | 1 + 7 files changed, 186 insertions(+), 49 deletions(-) -- 2.4.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/