Message ID | 20230711200141.239025-1-gnstark@sberdevices.ru |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:a6b2:0:b0:3e4:2afc:c1 with SMTP id c18csp718805vqm; Tue, 11 Jul 2023 13:14:53 -0700 (PDT) X-Google-Smtp-Source: APBJJlF1Se6Ik0rC86JiqRxZM6D8kXR7qwrzHLldEEV3EXbg1iiRFdsN/AlZMpmvbPG3UaTPlKNB X-Received: by 2002:a17:90b:3744:b0:262:d4cf:9f71 with SMTP id ne4-20020a17090b374400b00262d4cf9f71mr12443990pjb.39.1689106493257; Tue, 11 Jul 2023 13:14:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689106493; cv=none; d=google.com; s=arc-20160816; b=d8cZZqC9/a9TA/B6kJLpFqPXKee/sAJpxCWNcSMZCAUXZiFRvmOjqT+x6aPUAoUVYe vnAeQ8t1wl14o3fkEW7t+rojscZi+6pOKk4rbOQ4E23VZAk3JMfbnrbaQUEGvPszkdQC jJJlij2ozE/6PHJusEP+1kW88Af97CapeWd2f39Xjn4+j9isjmCssTb+ztMSleCAJp/l U2W8vLNb9OkIRqghZS9pVsKgRKp5qxwOueKWOkADJCJHxnP77hxDRpzWT0HSJFjzv799 5Wcy9kCWn2u8xv0iEe8h1gxybUI4BhuxkK9+POwUnMSucP8WMPR4jaAfq2/u3pA1Q072 7Tow== 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 :message-id:date:subject:cc:to:from:dkim-signature:dkim-filter; bh=yzS1NWXRo9VE0h7AmNisIIAscgwxAKFA0/jyaDwnvn8=; fh=VM+ke2h5JKUfvXdNdLjgF4bw6fPLt1uDttKmal/DDbQ=; b=CtnrWiK7gLsHtcpjM47xO8XoSkPQNwmX7CVO4corC8XPmBI/a7drBs7hNamwl5ohIw vG/4UKFC3GoymBchZRpRqA1WhWqph+dZUC5ozG4EPD3SnPoSiK+7g5hp+m19NUNnnqfD dcaI1nttbV+bR1bwz77R4aEIXfdFI8rQImd8Fh6p2cY7Q08vJUOuHRnbtJzxIZFtAnMP 0SWhW+KA0ELh/vWk75UfelgHWy1x9407LtuCKgPLujPGR9/R0MfoovBs2+nHynJ3M4Li vCxQr/Znb/ljdC9vZvBOFjCT2P8cnn48A3r/RJg0s/0DY76gJ38X3PfVXcEkCrZcggLj 5q9A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sberdevices.ru header.s=mail header.b=Cz6ypHI7; 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=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id d22-20020a63d656000000b0055390839e3dsi1825237pgj.528.2023.07.11.13.14.40; Tue, 11 Jul 2023 13:14:53 -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=@sberdevices.ru header.s=mail header.b=Cz6ypHI7; 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=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231751AbjGKUCM (ORCPT <rfc822;gnulinuxfreebsd@gmail.com> + 99 others); Tue, 11 Jul 2023 16:02:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33130 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231544AbjGKUB4 (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Tue, 11 Jul 2023 16:01:56 -0400 Received: from mx1.sberdevices.ru (mx1.sberdevices.ru [37.18.73.165]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C570A170E; Tue, 11 Jul 2023 13:01:53 -0700 (PDT) Received: from p-infra-ksmg-sc-msk01 (localhost [127.0.0.1]) by mx1.sberdevices.ru (Postfix) with ESMTP id 73991100004; Tue, 11 Jul 2023 23:01:50 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.sberdevices.ru 73991100004 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sberdevices.ru; s=mail; t=1689105710; bh=yzS1NWXRo9VE0h7AmNisIIAscgwxAKFA0/jyaDwnvn8=; h=From:To:Subject:Date:Message-ID:MIME-Version:Content-Type:From; b=Cz6ypHI7KLOI/xmQDkUjQKRJGEwA2gYUC39UB3QZ4GITVjMemdoXYmA59GlsAJds7 cUhLUWbY/hrJXygg6izSsq3I/W6FUtnA2cSn24JanXTaiDLze2ZgX1Qm+hmiu/Hhfi JfmSR1kzbmi4QSraIcyOLVfdQm81JtHTfTD7++Khewzx+uP1TCa1LCrBExXKVHkQg5 2b5Mnex5Hxta4jJi0f0P9cZVY51FB1a7lI4B+tgh9vxbCYdd4asrFhAELDI3GpmRoO sMuAo64gYzv8yrbiq0LpbcVhNuMpna0DJyguvcz2bhkZ36fwid8OhbXQAHfBeIbKZt sowu9sTbx7HXQ== Received: from p-i-exch-sc-m01.sberdevices.ru (p-i-exch-sc-m01.sberdevices.ru [172.16.192.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.sberdevices.ru (Postfix) with ESMTPS; Tue, 11 Jul 2023 23:01:50 +0300 (MSK) Received: from localhost.localdomain (100.64.160.123) by p-i-exch-sc-m01.sberdevices.ru (172.16.192.107) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.30; Tue, 11 Jul 2023 23:01:39 +0300 From: George Stark <gnstark@sberdevices.ru> To: <jic23@kernel.org>, <lars@metafoo.de>, <neil.armstrong@linaro.org>, <khilman@baylibre.com>, <jbrunet@baylibre.com>, <martin.blumenstingl@googlemail.com>, <andriy.shevchenko@linux.intel.com>, <nuno.sa@analog.com>, <gnstark@sberdevices.ru> CC: <linux-iio@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <linux-amlogic@lists.infradead.org>, <kernel@sberdevices.ru> Subject: [PATCH v5 0/6] iio: adc: meson: add iio channels to read channel 7 mux inputs Date: Tue, 11 Jul 2023 23:00:16 +0300 Message-ID: <20230711200141.239025-1-gnstark@sberdevices.ru> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [100.64.160.123] X-ClientProxiedBy: p-i-exch-sc-m01.sberdevices.ru (172.16.192.107) To p-i-exch-sc-m01.sberdevices.ru (172.16.192.107) X-KSMG-Rule-ID: 10 X-KSMG-Message-Action: clean X-KSMG-AntiSpam-Lua-Profiles: 178558 [Jul 11 2023] X-KSMG-AntiSpam-Version: 5.9.59.0 X-KSMG-AntiSpam-Envelope-From: GNStark@sberdevices.ru X-KSMG-AntiSpam-Rate: 0 X-KSMG-AntiSpam-Status: not_detected X-KSMG-AntiSpam-Method: none X-KSMG-AntiSpam-Auth: dkim=none X-KSMG-AntiSpam-Info: LuaCore: 521 521 0c3391dd6036774f2e1052158c81e48587b96e95, {Tracking_uf_ne_domains}, {Tracking_from_domain_doesnt_match_to}, lore.kernel.org:7.1.1;d41d8cd98f00b204e9800998ecf8427e.com:7.1.1;p-i-exch-sc-m01.sberdevices.ru:5.0.1,7.1.1;www.spinics.net:7.1.1;127.0.0.199:7.1.2;sberdevices.ru:5.0.1,7.1.1;100.64.160.123:7.1.2, FromAlignment: s, {Tracking_white_helo}, ApMailHostAddress: 100.64.160.123 X-MS-Exchange-Organization-SCL: -1 X-KSMG-AntiSpam-Interceptor-Info: scan successful X-KSMG-AntiPhishing: Clean, bases: 2023/07/11 15:25:00 X-KSMG-LinksScanning: Clean, bases: 2023/07/11 13:47:00 X-KSMG-AntiVirus: Kaspersky Secure Mail Gateway, version 2.0.1.6960, bases: 2023/07/11 13:07:00 #21597915 X-KSMG-AntiVirus-Status: Clean, skipped X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771156529943354230 X-GMAIL-MSGID: 1771156529943354230 |
Series |
iio: adc: meson: add iio channels to read channel 7 mux inputs
|
|
Message
George Stark
July 11, 2023, 8 p.m. UTC
Changelog: v1->v2: split refactoring patch [1] into 4 smaller patches, fix comment style [1] https://lore.kernel.org/lkml/20230621062715.455652-2-gnstark@sberdevices.ru/ v2->v3: remove patch 'meson saradc: unite iio channel array definitions' [1] after discussion patch 'meson saradc: add enum for iio channel array indexes' - change enum items prefix from INDEX_ to NUM_ since name 'channel index' is more relevant to channel array index in iio world and with 2 tables our array index is not always equal to channel number - resolve conflicts after deleting [1] - update commit message, previous patch [2] - return channel number for temp channel. It wasn't used and isn't used currently but may need later patch meson saradc: support reading from channel 7 mux inputs - resolve conflicts after deleting [1] - update commit message, previous patch [3] - add routine find_channel_by_num to get channel by channel number [1] https://lore.kernel.org/lkml/20230623022334.791026-4-gnstark@sberdevices.ru/ [2] https://lore.kernel.org/lkml/20230623022334.791026-5-gnstark@sberdevices.ru/ [3] https://lore.kernel.org/lkml/20230623022334.791026-7-gnstark@sberdevices.ru/ v3->v4: add new patch 'iio: adc: meson: remove unused timestamp channel' [1] patch 'iio: adc: meson: move enums declaration before' update commit message, previous patch [2] patch 'iio: adc: meson: move meson_sar_adc_set_chan7_mux' update commit message, previous patch [3] patch 'iio: adc: meson: add enum for iio channel numbers' update commit message, previous patch [4] patch 'iio: adc: meson: add channel labels' update commit message, previous patch [5] change sprintf(label, "%s\n", "temp-sensor") to sprintf(label, "temp-sensor\n") patch 'iio: adc: meson: support reading from channel 7 mux' rewrite enum meson_sar_adc_chan7_mux_sel definition and read_label routine proposed by Andy [7], previous patch [6] [1] https://lore.kernel.org/lkml/20230705160413.000062e7@Huawei.com/ [2] https://lore.kernel.org/lkml/20230627224017.1724097-2-gnstark@sberdevices.ru/ [3] https://lore.kernel.org/lkml/20230627224017.1724097-3-gnstark@sberdevices.ru/ [4] https://lore.kernel.org/lkml/20230627224017.1724097-4-gnstark@sberdevices.ru/ [5] https://lore.kernel.org/lkml/20230627224017.1724097-5-gnstark@sberdevices.ru/ [6] https://lore.kernel.org/lkml/20230627224017.1724097-6-gnstark@sberdevices.ru/ [7] https://lore.kernel.org/lkml/ZJwGCNA+ZURri24i@smile.fi.intel.com/ v4->v5: patch 'iio: adc: meson: move enums declaration before variables declaration' update commit message, previous patch [1] patch 'iio: adc: meson: move meson_sar_adc_set_chan7_mux routine upper' update commit message, previous patch [2] patch 'iio: adc: meson: add channel labels' update commit message, previous patch [3] patch 'iio: adc: meson: support reading from channel 7 mux inputs' update commit message, previous patch [4] [1] https://lore.kernel.org/lkml/20230707153322.114302-3-gnstark@sberdevices.ru/ [2] https://www.spinics.net/lists/linux-iio/msg80226.html [3] https://www.spinics.net/lists/linux-iio/msg80227.html [4] https://www.spinics.net/lists/linux-iio/msg80230.html George Stark (6): iio: adc: meson: remove unused timestamp channel iio: adc: meson: move enums declaration before variables declaration iio: adc: meson: move meson_sar_adc_set_chan7_mux routine upper iio: adc: meson: add enum for iio channel numbers iio: adc: meson: add channel labels iio: adc: meson: support reading from channel 7 mux inputs drivers/iio/adc/meson_saradc.c | 173 +++++++++++++++++++++++++-------- 1 file changed, 134 insertions(+), 39 deletions(-)
Comments
On Tue, Jul 11, 2023 at 11:00:16PM +0300, George Stark wrote:
> Changelog:
I gave you a tag, any reason why you haven't applied it to your patches?
Hello Andy On 7/12/23 18:26, Andy Shevchenko wrote: > On Tue, Jul 11, 2023 at 11:00:16PM +0300, George Stark wrote: >> Changelog: > > I gave you a tag, any reason why you haven't applied it to your patches? if you're talking about patch 6 commit message - yes, I missed the last line in your letter, I'll fix it. -- Best regards George
On Thu, Jul 13, 2023 at 02:04:05AM +0300, George Stark wrote: > Hello Andy > > On 7/12/23 18:26, Andy Shevchenko wrote: > > On Tue, Jul 11, 2023 at 11:00:16PM +0300, George Stark wrote: > > > Changelog: > > > > I gave you a tag, any reason why you haven't applied it to your patches? > > if you're talking about patch 6 commit message - yes, I missed the last line > in your letter, I'll fix it. I believe I answered to the cover letter [1], which means that it applies to the entire series. If you don't include tag of the reviewer(s) who spent time on your code, you have to well justify WHY you did so. [1]: https://lore.kernel.org/lkml/ZKvQaoFxO5gpAFdM@smile.fi.intel.com/
On 7/13/23 15:08, Andy Shevchenko wrote: > On Thu, Jul 13, 2023 at 02:04:05AM +0300, George Stark wrote: >> Hello Andy >> >> On 7/12/23 18:26, Andy Shevchenko wrote: >>> On Tue, Jul 11, 2023 at 11:00:16PM +0300, George Stark wrote: >>> >>> I gave you a tag, any reason why you haven't applied it to your patches? >> >> if you're talking about patch 6 commit message - yes, I missed the last line >> in your letter, I'll fix it. > > I believe I answered to the cover letter [1], which means that it applies to > the entire series. If you don't include tag of the reviewer(s) who spent time > on your code, you have to well justify WHY you did so. > > [1]: https://lore.kernel.org/lkml/ZKvQaoFxO5gpAFdM@smile.fi.intel.com/ > Hello Andy I think I've understood at last what you meant by "not applied tag". I'm not experienced kernel contributor yet and believed that RvBs and Acks are added by one who merges patches or by some kind of magic :) Of course I'll add the tag, I missed it not on purpose. Thank you very much for help and your work for the common good.
On Fri, Jul 14, 2023 at 02:11:19AM +0300, George Stark wrote: > On 7/13/23 15:08, Andy Shevchenko wrote: > > On Thu, Jul 13, 2023 at 02:04:05AM +0300, George Stark wrote: > > > On 7/12/23 18:26, Andy Shevchenko wrote: > > > > On Tue, Jul 11, 2023 at 11:00:16PM +0300, George Stark wrote: > > > > > > > > I gave you a tag, any reason why you haven't applied it to your patches? > > > > > > if you're talking about patch 6 commit message - yes, I missed the last line > > > in your letter, I'll fix it. > > > > I believe I answered to the cover letter [1], which means that it applies to > > the entire series. If you don't include tag of the reviewer(s) who spent time > > on your code, you have to well justify WHY you did so. > > > > [1]: https://lore.kernel.org/lkml/ZKvQaoFxO5gpAFdM@smile.fi.intel.com/ > > Hello Andy > > I think I've understood at last what you meant by "not applied tag". I'm not > experienced kernel contributor yet and believed that RvBs and Acks are added > by one who merges patches or by some kind of magic :) If you (not a maintainer) respin the series, it's your responsibility to add / drop / update tags. And hence to explain any modifications to that. > Of course I'll add the tag, I missed it not on purpose. > Thank you very much for help and your work for the common good. No problem, as I said, I'm in favour of your good contribution, just some processing issues which can be easily fixed.