From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f44.google.com (mail-pj1-f44.google.com [209.85.216.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2ED332750E0 for ; Tue, 15 Apr 2025 22:17:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744755477; cv=none; b=Yb/1wimCVtK+kuyFh1WMnMJkv4LdNrh2oyWHKnfZRn2Kr0dCSdM20Yn0XUsjg4agGbfulPAfsGNlp+S7ASdz//g2DWoalzVgEgBD9Hqm5Azy4Pfm9owwRaLV/Di/T65ideiHb8aOgOg7E5fwAYElBZFxL+4iYyTgM4bpoZL5gjI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744755477; c=relaxed/simple; bh=hGlTveS2616ckTcGMpFiqiYVbcqh6/+6UFd/j7gYNa0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=uoRhKRYsAJ5r9FHW4jURuQd5q4HBMLsZHDTkYGrnXQ/UvstWTcLUFCzjZGNI6fIoMzSeOhFl93sd/3SknJS1se/ZGXShlqtg8q+jqSqq/fDS89lk30fh6XXXnFSBULqYOjQEjeNuIxPZjdjEWx6qB94DbwKE+dscU/++FMgQP6A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=PJFxT1cV; arc=none smtp.client-ip=209.85.216.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="PJFxT1cV" Received: by mail-pj1-f44.google.com with SMTP id 98e67ed59e1d1-306bf444ba2so5377419a91.1 for ; Tue, 15 Apr 2025 15:17:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1744755474; x=1745360274; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=4J73IHenn0V6xQDZgF63RJqhtreeLtTKV20Qod+uY/M=; b=PJFxT1cVupe5DkeyO+H6r69bYJLrQxSOAVNcTpPkXRLSEFx+qD7tPZeMSO26BJiFkP 3GeJ78XywP38u9diAVmypwrJNNju0NjG9MX2P8b2XwIPKx9L/nqyodlsN87BghnktLNs pquXgxyfLcu0ltvOqzFqPX5Ez+FXy6i1J8AN69FnKbBipUaeHNMVMgwzux/VmrpdgEjd YqRhXYoYAYdDpS/0rQeVRv8nvxb+ORda69zIAmF7I9oN89H6tGT3MDPuxqoTHui0aktI 1SpZWEfZ28lx5dQyugoDqQev/i7pfrCNev9ABdxz5OG6EATJoQdpOsePUh1EkmAjwsc1 Bz5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744755474; x=1745360274; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=4J73IHenn0V6xQDZgF63RJqhtreeLtTKV20Qod+uY/M=; b=OSyNTcXdl8kETfyWWyclNqcDtF/ibeQMH/DcLbcziIQ/RwfiRRMySb/HtNwU1vpxCl oqLes3G6unS7SGXOcSSL/bJscKoXVCgvh0TLfAZli9mnzEURebQGEGzroXlTZB7LZxfu VW3cjOrooLd+kdIExc3qT5oWSpoNhklVfRIFIYU3iTdh4n7mnRXErDA7KgT03QgkQQ7C fnZm/0vWwDkl/+wlpb6SymHboMzvF7cFLF3M9IrjzRswJ9RKhJWRgoW3UcKsafIQtzxy PTO09qvXKJLZ+HZ2YOjjEkQMWcoLaRlKPLdGfIFXDRPb41B52W6+U0gnSKzZsoSKlJyz eegA== X-Forwarded-Encrypted: i=1; AJvYcCWZWOm39NM09ubCRR7rzD3Habw5pKId1TaNQDQIHQodEcHTrcWA4XoB/D0kv5KgIMVEg5OeNll9v0q4oig=@vger.kernel.org X-Gm-Message-State: AOJu0Yzjjzi7NBrUnVoO755C2iRV7goB3nZyXFX5qk6A/uMDAa8mp5C1 hlzaGWCR6Uk0PPz+y2D46DxLJmLNVWPClFkmLMJ1Obvb+LNCaG6sLjmw9/aQKmc= X-Gm-Gg: ASbGnct94M1cLTI5SoxwNf5SmoKVzIt2MtOG/qc5Iy/EdR9XPebQR/NhlJKLmASbssP zPbo//HPp9RUTpnWhleYDQ0kd4MrmBzBqjYbF5nV0rY54rdBQFlDax0CCG7jXG/lKKBUfT4mbB8 MjWGs0VKtZWMH23gMn+gzk+k62+lxDCgMlTPEZHUi/bcuE9jqE8DYS7Z+IrJ+veHGZ5eCvZRe8n XFCB6vFmpb28QQUO5ntlcQJ0g9fneHXQTmSQyO+X6adr2XmpzGJazLQIJ4B9Lbc/NLcviwDEUHJ ccIJiLrQEYWCJ9tqw/jCW5/eINGR+JhHJJHC/JoGr24keN3FZHj89P+sXYQeKc4oZnC+8wfQj+t F X-Google-Smtp-Source: AGHT+IFhuSJk+eLjcOi/3nG+9UyHOVKqlJoxNhC/IcW+qjXwSHez/LX3r8QEUmn6wUetBFK7viVoEg== X-Received: by 2002:a17:90b:2dcb:b0:2ef:114d:7bf8 with SMTP id 98e67ed59e1d1-3085eecc278mr1070084a91.6.1744755474088; Tue, 15 Apr 2025 15:17:54 -0700 (PDT) Received: from dev-linux (syn-076-088-115-008.res.spectrum.com. [76.88.115.8]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-30861212fbesm111911a91.23.2025.04.15.15.17.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 15 Apr 2025 15:17:53 -0700 (PDT) Date: Tue, 15 Apr 2025 15:17:46 -0700 From: Sukrut Bellary To: Jonathan Cameron Cc: Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Angelo Compagnucci , Nishanth Menon , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Matti Vaittinen Subject: Re: [PATCH 0/2] iio: adc: ti-adc128s052: Add support for adc102s021 Message-ID: References: <20250408132120.836461-1-sbellary@baylibre.com> <20250412141047.4273a8b1@jic23-huawei> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250412141047.4273a8b1@jic23-huawei> On Sat, Apr 12, 2025 at 02:10:47PM +0100, Jonathan Cameron wrote: > On Tue, 8 Apr 2025 06:21:18 -0700 > Sukrut Bellary wrote: > > > The patch series adds the support for adc102s021 and family. > > > > The family of devices are easier to > > support since they all (no matter the resolution) seem to respond in > > 12-bits with the LSBs set to 0 for the reduced resolution devices. > > This has raced against Matti's series > https://lore.kernel.org/linux-iio/cover.1744022065.git.mazziesaccount@gmail.com/ > Support ROHM BD79104 ADC > > With hindsight that wasn't obvious from the patch series name though > which should ideally have been > iio: adc: ti-adc128s052: Support ROHM BD79104 ADC > > Please rebase on the iio testing branch on kernel.org or on top of that series. > Technically I've only applied the first 7 patches so far, but the 8th > should be a simple change from that v3. > > Matti, you volunteered as maintainer :) Hence please take a look at > this one. > > One nice thing in there is we now have a __be16 buffer16 element that > can avoid at least one cast in patch 2. Ok, thanks. I will work on the top of this series. > Thanks, > > Jonathan > > > > > Changes in v3: > > Patch 1: > > - No changes in dt-bindings > > > > Patch 2: > > - used be16_to_cpu() for the endian conversion. > > - used config index enum while setting up the adc128_config[] > > > > - Link to v2: > > https://lore.kernel.org/lkml/20231022031203.632153-1-sukrut.bellary@linux.com/ > > > > Changes in v2: > > Patch 1: > > - No changes in dt-bindings > > > > Patch 2: > > - Arranged of_device_id and spi_device_id in numeric order. > > - Used enum to index into adc128_config. > > - Reorder adc128_config in alphabetical. > > - Include channel resolution information. > > - Shift is calculated per resolution and used in scaling and > > raw data read. > > > > - Link to v1: https://lore.kernel.org/all/20220701042919.18180-1-nm@ti.com/ > > > > Sukrut Bellary (2): > > dt-bindings: iio: adc: ti,adc128s052: Add adc08c and adc10c family > > iio: adc: ti-adc128s052: Add lower resolution devices support > > > > .../bindings/iio/adc/ti,adc128s052.yaml | 6 + > > drivers/iio/adc/ti-adc128s052.c | 149 +++++++++++++----- > > 2 files changed, 118 insertions(+), 37 deletions(-) > > >