Received: by 2002:ad5:4acb:0:0:0:0:0 with SMTP id n11csp1628471imw; Sat, 16 Jul 2022 10:51:52 -0700 (PDT) X-Google-Smtp-Source: AGRyM1uti8GzZ3QfFuCZ8dOuwl+rrxw8cNaaBwOSHOh0kEZsaJ6GlfC7S+Vdw8znlRo+E/AQ0FyZ X-Received: by 2002:a17:906:5047:b0:710:456a:695e with SMTP id e7-20020a170906504700b00710456a695emr19077691ejk.433.1657993912299; Sat, 16 Jul 2022 10:51:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1657993912; cv=none; d=google.com; s=arc-20160816; b=0CQVcZJxz6ReAzOZcpdeJEjpASf4/vVDxW4nJITvf2b7xpeBYnzvUhLccm/7zbH2Yt jTm2D3p2k7AlWLPMNiQQFZszvUfcYQ4av59VHnflkzwqkX7RCAS6hYZSnWwSYJ93QMfV fojBXOOdcTZJfohz0ZnqxUWikx3D7Ab/Llw5QMrIx68Mou6xp48A2EUmp8nXbo3OZu4I x3gSlAS7KkSfROqecnMb3FR/axp98sR+gH1ajXC+dLsLr3eNkBzOsCs5q7norG0ea8Ea ovwT/TJrRLteyTSmPd/Ua3yxTrrWWdDrOqoIZdjxWwPd48PxErQ6p8efvr0wSeO3kfuw vOMw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=ZH5wOA6hlPTaHBlonAaYMB2whU72CYha4gyw4F62c5E=; b=X1wHg/yNU2BeyWBCuqDZHU1kDSAzvUN5oYbWpRfKucXvPvDKq8yujpBh8KiIw1DFP2 Km+SbMoom3JImi2koJLqiUiRcrRR09wrX62Cip9QpkNxHmXkYC8yYPwguWtBQfiltwmH aPVVVaCqxOsprgIozUuK+uNPj27iRdpmnhyNvFXqyA/Jk5XW/feeoHD8mupgAeKfX0uR WTEQrX1my5UlWClyJJ2fAg6haG8W4lW7ksV9fGRevlyv6/0SZb7MPYX7BeAooRbhc1KN Iy+lcc8hhZi0CP+0T2FD8OdmoBxRTSVBOhHYtsY6Wh9Mn3yJwuh8Ry/5WsHWcwxwyT8E mpsg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=kHN2y5iB; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id o5-20020a170906974500b0072caf70b02fsi11837879ejy.498.2022.07.16.10.51.27; Sat, 16 Jul 2022 10:51:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=kHN2y5iB; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231859AbiGPRjM (ORCPT + 99 others); Sat, 16 Jul 2022 13:39:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46222 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230313AbiGPRjJ (ORCPT ); Sat, 16 Jul 2022 13:39:09 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B83DD10FC0; Sat, 16 Jul 2022 10:39:05 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 0D466611A6; Sat, 16 Jul 2022 17:39:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E6B92C34114; Sat, 16 Jul 2022 17:39:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1657993144; bh=k8F15ykDf5m+r4/I7RU3R8rqaaAe0MvqtE3T6MXh/to=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=kHN2y5iBaPDyznSfGD+B0FZebCnxFNJTZUfhljzHRaZFdjArTKoYH3dhOzgQhbaFj qU+pf5QqG6IX/BHcd6NeQdcNRa7InJrk1FNCpZoupQO177qxVHg6f1uWBKJWdngpKO kreFeKArz+UMOteMWfRLHt8nBDudvIlD9IKW067UzkPqHY9esrLs897JH8UNeJ0fnk sxb5mCt22PE9q6gBsDHiURYvQz3t5wNky1WYyGkVYC6C6d5sM+gP3x/2BFWY/fK6Pe +HwkYPttHxZQ7XLC2AYNiL3uMkNQ9LsnoON+Tsx7o9sI/XDX2uuhnUuYhMcbl6i2zB 1a/x1gMw282Cg== Date: Sat, 16 Jul 2022 18:48:57 +0100 From: Jonathan Cameron To: William Breathitt Gray Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Fred Eckert Subject: Re: [PATCH v2 0/2] iio: Implement and utilize register structures for ISA drivers Message-ID: <20220716184857.7ad17421@jic23-huawei> In-Reply-To: References: X-Mailer: Claws Mail 4.1.0 (GTK 3.24.34; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-7.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 7 Jul 2022 13:21:23 -0400 William Breathitt Gray wrote: > Changes in v2: > - Rename 'ad' member to 'ssr_ad' to indicate access to Software Strobe > Register as well; relevant comments updated > - Dereference DAC channel address via array subscripts in the > stx104_write_raw() function in order to match rest of code > - Replace struct cio_dac_reg with direct u16 pointer to the DAC > address; this greatly simplifies the cio-dac patch > > The STX104 and CIO-DAC drivers were updated to use I/O memory accessor > calls such as ioread8()/iowrite8() in previous patch series [1]. This > patch series is a continuation of the effort to improve the code > readability and reduce magic numbers by implementing and utilizing named > register data structures. Series applied to the togreg branch of iio.git. Given timing in cycle, I'll only push that out as testing (for 0-day etc) and then rebase it after rc1. Thanks, Jonathan > > [1] https://lore.kernel.org/all/cover.1652201921.git.william.gray@linaro.org/ > > William Breathitt Gray (2): > iio: adc: stx104: Implement and utilize register structures > iio: dac: cio-dac: Cleanup indexing for DAC writes > > drivers/iio/adc/stx104.c | 74 ++++++++++++++++++++++++++------------- > drivers/iio/dac/cio-dac.c | 10 +++--- > 2 files changed, 55 insertions(+), 29 deletions(-) > > > base-commit: 338222d8e1121bcb78a86fb39982eddcc367a5bc