From patchwork Mon Mar 18 19:14:18 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: srinivas pandruvada X-Patchwork-Id: 10858405 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 619136C2 for ; Mon, 18 Mar 2019 19:15:08 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 4A72528EBD for ; Mon, 18 Mar 2019 19:15:08 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3F0E829461; Mon, 18 Mar 2019 19:15:08 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D02DD28EBD for ; Mon, 18 Mar 2019 19:15:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727476AbfCRTPG (ORCPT ); Mon, 18 Mar 2019 15:15:06 -0400 Received: from mga18.intel.com ([134.134.136.126]:38402 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726832AbfCRTPG (ORCPT ); Mon, 18 Mar 2019 15:15:06 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Mar 2019 12:15:04 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,494,1544515200"; d="scan'208";a="156107020" Received: from spandruv-desk.jf.intel.com ([10.54.75.31]) by fmsmga001.fm.intel.com with ESMTP; 18 Mar 2019 12:15:04 -0700 From: Srinivas Pandruvada To: jikos@kernel.org, benjamin.tissoires@redhat.com Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Srinivas Pandruvada Subject: [PATCH v2 00/10] HID: intel-ish-hid: Clean up external interfaces Date: Mon, 18 Mar 2019 12:14:18 -0700 Message-Id: <20190318191428.6527-1-srinivas.pandruvada@linux.intel.com> X-Mailer: git-send-email 2.17.2 Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP NOT FOR kernel v5.1. v2 - Rebased on the top of "HID: intel-ish: enable raw interface to HID devices on ISH" which Jiri already applied. - Also exported devc pointer for DMA No functional changes are expected with this series. I am posting this now because of usage of ISH in ChromeOS Embedded Controller. https://lkml.org/lkml/2019/2/24/26 I want to make sure that API is restricted before more development and posting there. Currently only one ISH client is using ISH transport. But it is changing now with the development of other clients using ISH transport. Some of these clients which are targeted for Linux based OS only laptops, are not using HID to export sensors. As more clients are getting developed it is important that the external interface for ISH transport only allows what clients need. Currently the header files used by clients "client.h" and "ishtp-dev.h" include other ISH header files. Also clients access fields from structure which also has other fields which are only used by ISH transort. So this series introduces one header file "linux/intel-ish-client-if.h". This header files doesn't include any other ISH transport header files. There are interface functions defined so that clients never have to directly access any ISH transort structures. Also clients don't have to match there GUID in probe. They will be only probbed if their GUID matches, which is passed as driver registry. Hong Liu (1): HID: intel-ish-hid: Add match callback to ishtp bus type Srinivas Pandruvada (9): HID: intel-ish-hid: Hide members of struct ishtp_cl_device HID: intel-ish-hid: Simplify ishtp_cl_link() HID: intel-ish-hid: Move driver registry functions HID: intel-ish-hid: Store ishtp_cl_device instance in device HID: intel-ish-hid: Move the common functions from client.h HID: intel-ish-hid: Add interface functions for struct ishtp_cl HID: intel-ish-hid: Move functions related to bus and device HID: intel-ish-hid: Use the new interface functions in HID ish client HID: intel-ish-hid: Add interface function for PCI device pointer drivers/hid/intel-ish-hid/ishtp-hid-client.c | 131 ++++++++++--------- drivers/hid/intel-ish-hid/ishtp-hid.c | 6 +- drivers/hid/intel-ish-hid/ishtp-hid.h | 6 +- drivers/hid/intel-ish-hid/ishtp/bus.c | 96 +++++++++++++- drivers/hid/intel-ish-hid/ishtp/bus.h | 37 +----- drivers/hid/intel-ish-hid/ishtp/client.c | 60 +++++++-- drivers/hid/intel-ish-hid/ishtp/client.h | 24 ---- drivers/hid/intel-ish-hid/ishtp/ishtp-dev.h | 31 ----- include/linux/intel-ish-client-if.h | 112 ++++++++++++++++ 9 files changed, 325 insertions(+), 178 deletions(-) create mode 100644 include/linux/intel-ish-client-if.h