From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.5 required=3.0 tests=BAYES_00,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 64225C433B4 for ; Tue, 27 Apr 2021 17:11:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 243E4613DA for ; Tue, 27 Apr 2021 17:11:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235647AbhD0RM2 (ORCPT ); Tue, 27 Apr 2021 13:12:28 -0400 Received: from mail.kernel.org ([198.145.29.99]:56110 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235802AbhD0RMX (ORCPT ); Tue, 27 Apr 2021 13:12:23 -0400 Received: from jic23-huawei (cpc108967-cmbg20-2-0-cust86.5-4.cable.virginm.net [81.101.6.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id CCD7660FD8; Tue, 27 Apr 2021 17:11:38 +0000 (UTC) Date: Tue, 27 Apr 2021 18:12:23 +0100 From: Jonathan Cameron To: Alexandru Ardelean Cc: linux-iio , Jonathan Cameron Subject: Re: [PATCH 0/9] iio: Move more things from iio_dev to iio_dev_opaque Message-ID: <20210427181223.59a8da9f@jic23-huawei> In-Reply-To: References: <20210426174911.397061-1-jic23@kernel.org> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Tue, 27 Apr 2021 11:08:52 +0300 Alexandru Ardelean wrote: > On Mon, Apr 26, 2021 at 8:50 PM Jonathan Cameron wrote: > > > > From: Jonathan Cameron > > > > Intent here is to clear out the low hanging fruit so we can focus on the > > more interesting elements. > > > > Hopefully this set are all uncontroversial. Lightly tested only but > > all fairly mechanical so hopefully I didn't mess up. > > > > Series looks good. > Thanks for moving this forward :) no problem. One thing. You gave tags for all but it seems patch 3. Fine if I also put your tag on that one? > > Alex > > > As for the others my current thinking is as follows > > > > mlock: Still some instances to clear out of this being used to protect things > > it should not be used for. Long run we definitely want to move this one. > > > > scan_timestamp: Can easily use a look up function in drivers that access this > > directly, but that feels like an unwanted level of indirection in > > iio_push_to_buffers_with_timestamp(). Perhaps worth doing anyway. > > scan_bytes is also used in this function. > > > > active_scan_mask and masklength: Both used in valid ways inside drivers. > > Could be wrapped up in access functions, but it does seem a little > > convoluted. > > > > trig: This is used correctly in lots of drivers, so should stay in > > struct iio_dev. > > > > Jonathan Cameron (9): > > iio: core: move @id from struct iio_dev to struct iio_dev_opaque > > iio: avoid shadowing of variable name in to_iio_dev_opaque() > > iio: core: move @driver_module from struct iio_dev to struct > > iio_dev_opaque > > iio: core: move @trig_readonly from struct iio_dev to struct > > iio_dev_opaque > > iio: core: move @scan_index_timestamp to struct iio_dev_opaque > > iio: core: move @info_exist_lock to struct iio_dev_opaque > > iio: core: move @chrdev from struct iio_dev to struct iio_dev_opaque > > iio: core: move @flags from struct iio_dev to struct iio_dev_opaque > > iio: core: move @clock_id from struct iio_dev to struct iio_dev_opaque > > > > drivers/iio/accel/adxl372.c | 4 +- > > drivers/iio/accel/bma180.c | 2 +- > > drivers/iio/accel/bmc150-accel-core.c | 4 +- > > drivers/iio/accel/kxcjk-1013.c | 4 +- > > drivers/iio/accel/mma8452.c | 2 +- > > drivers/iio/accel/mxc4005.c | 2 +- > > drivers/iio/accel/stk8312.c | 2 +- > > drivers/iio/accel/stk8ba50.c | 2 +- > > drivers/iio/adc/ad7606.c | 3 +- > > drivers/iio/adc/ad7766.c | 3 +- > > drivers/iio/adc/ad7768-1.c | 3 +- > > drivers/iio/adc/ad_sigma_delta.c | 2 +- > > drivers/iio/adc/at91_adc.c | 4 +- > > drivers/iio/adc/dln2-adc.c | 3 +- > > drivers/iio/adc/ina2xx-adc.c | 3 +- > > drivers/iio/adc/ti-ads131e08.c | 2 +- > > drivers/iio/adc/xilinx-xadc-core.c | 2 +- > > .../buffer/industrialio-triggered-buffer.c | 2 +- > > drivers/iio/chemical/atlas-sensor.c | 2 +- > > drivers/iio/chemical/ccs811.c | 2 +- > > drivers/iio/chemical/scd30_core.c | 3 +- > > .../common/hid-sensors/hid-sensor-trigger.c | 2 +- > > drivers/iio/gyro/adxrs290.c | 2 +- > > drivers/iio/gyro/bmg160_core.c | 4 +- > > drivers/iio/gyro/fxas21002c_core.c | 2 +- > > drivers/iio/gyro/itg3200_buffer.c | 2 +- > > drivers/iio/gyro/mpu3050-core.c | 2 +- > > drivers/iio/health/afe4403.c | 2 +- > > drivers/iio/health/afe4404.c | 2 +- > > drivers/iio/imu/adis_trigger.c | 3 +- > > drivers/iio/imu/bmi160/bmi160_core.c | 3 +- > > drivers/iio/imu/inv_mpu6050/inv_mpu_trigger.c | 2 +- > > drivers/iio/imu/kmx61.c | 2 +- > > drivers/iio/industrialio-buffer.c | 12 ++- > > drivers/iio/industrialio-core.c | 76 +++++++++++++------ > > drivers/iio/industrialio-trigger.c | 19 +++-- > > drivers/iio/industrialio-triggered-event.c | 2 +- > > drivers/iio/inkern.c | 46 ++++++----- > > drivers/iio/light/acpi-als.c | 3 +- > > drivers/iio/light/rpr0521.c | 2 +- > > drivers/iio/light/si1145.c | 2 +- > > drivers/iio/light/vcnl4000.c | 3 +- > > drivers/iio/light/vcnl4035.c | 2 +- > > drivers/iio/magnetometer/bmc150_magn.c | 2 +- > > drivers/iio/magnetometer/rm3100-core.c | 2 +- > > drivers/iio/potentiostat/lmp91000.c | 3 +- > > drivers/iio/pressure/zpa2326.c | 3 +- > > drivers/iio/proximity/as3935.c | 3 +- > > drivers/iio/proximity/sx9310.c | 2 +- > > drivers/iio/proximity/sx9500.c | 2 +- > > include/linux/iio/iio-opaque.h | 22 +++++- > > include/linux/iio/iio.h | 29 +------ > > 52 files changed, 188 insertions(+), 131 deletions(-) > > > > -- > > 2.31.1 > >