From patchwork Fri Jun 1 13:47:17 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jarkko Nikula X-Patchwork-Id: 924010 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-i2c-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=linux.intel.com Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 40y5Gs6J7qz9rxs for ; Fri, 1 Jun 2018 23:47:01 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751598AbeFANrA (ORCPT ); Fri, 1 Jun 2018 09:47:00 -0400 Received: from mga14.intel.com ([192.55.52.115]:65429 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751327AbeFANrA (ORCPT ); Fri, 1 Jun 2018 09:47:00 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Jun 2018 06:46:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,467,1520924400"; d="scan'208";a="233576226" Received: from mylly.fi.intel.com (HELO mylly.fi.intel.com.) ([10.237.72.76]) by fmsmga005.fm.intel.com with ESMTP; 01 Jun 2018 06:46:58 -0700 From: Jarkko Nikula To: linux-i2c@vger.kernel.org Cc: Wolfram Sang , Andy Shevchenko , Mika Westerberg , Jarkko Nikula Subject: [PATCH v2 0/8] i2c: designware: Improve debug prints Date: Fri, 1 Jun 2018 16:47:17 +0300 Message-Id: <20180601134725.1375-1-jarkko.nikula@linux.intel.com> X-Mailer: git-send-email 2.17.0 Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org Motivation here is to improve debug prints and while doing so to remove some duplication and separate timing parameter validation from actual register writes as this parameter validation and/or calculation is needed to do only during probe. Currently debug code prints SCL timing parameters whenever HW is reinitialized but doesn't print importand SDA hold time nor actual bus speed the controller is going to operate. v2: - SDA hold time configuration moved from "Separate timing parameter setting from HW initalization" patch to a new patch as well as i2c_dw_clk_rate() cleanup. - New patch replacing ___constant_swab32() with swab32(). - Added Andys review by tags to patches 1-2. Jarkko Nikula (8): i2c: designware: Remove needless variable from i2c_dw_init_slave() i2c: designware: Move register access detection to common code i2c: designware: Don't use internal ___constant_swab32 i2c: designware: Move SDA hold time configuration to common code i2c: designware: Call i2c_dw_clk_rate() only once in i2c_dw_init_master() i2c: designware: Separate timing parameter setting from HW initalization i2c: designware: Add debug print for SDA hold time value i2c: designware: Add debug print for bus speed drivers/i2c/busses/i2c-designware-common.c | 73 ++++++++ drivers/i2c/busses/i2c-designware-core.h | 2 + drivers/i2c/busses/i2c-designware-master.c | 185 ++++++++++++--------- drivers/i2c/busses/i2c-designware-slave.c | 44 +---- 4 files changed, 185 insertions(+), 119 deletions(-)