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=-5.1 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS, URIBL_BLOCKED,USER_AGENT_MUTT autolearn=unavailable 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 C096CC43381 for ; Thu, 21 Feb 2019 01:11:19 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [203.11.71.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2CAD52086C for ; Thu, 21 Feb 2019 01:11:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Ir3kaq5O" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2CAD52086C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 444byX6lsLzDqMP for ; Thu, 21 Feb 2019 12:11:16 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=gmail.com (client-ip=2607:f8b0:4864:20::642; helo=mail-pl1-x642.google.com; envelope-from=vilhelm.gray@gmail.com; receiver=) Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="Ir3kaq5O"; dkim-atps=neutral Received: from mail-pl1-x642.google.com (mail-pl1-x642.google.com [IPv6:2607:f8b0:4864:20::642]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 444bwX12wZzDqML for ; Thu, 21 Feb 2019 12:09:28 +1100 (AEDT) Received: by mail-pl1-x642.google.com with SMTP id m10so3541761plt.10 for ; Wed, 20 Feb 2019 17:09:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=XazLWlV0OGpxiqeWsTijGEnFhzyXIMqtdbgnm6gIsFM=; b=Ir3kaq5OJ38gwHc9MbQ93aEeS78o9Jb4ne68ygRM17PEd63kJOo+HfHEWvo2v6rFiJ KNYERTWt52IYRarCIpcC1mVk1oyBJSudCfe/mrer7nGIXxOQfMaIBZgPLS5E0qfkrTuR Ufcc+iHE+jP6nCO9LsKxSD5q7SBg9FiHsm2wuwXwH1rl5CqFTnOIsTsRYhJM99KY7ViM RDtd512mwClfGkHF+cFrWtS4roAOGZlMoAUDdXJCtPTQKqqgDUvg55Iz6UpGom9V5/tM rYjA6mpl3jFJNGylRInO0RX6Q8KhsMw2lnhEeJib7zATI8G10gN3XcdVfuDdZ2ooSgqO Krag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=XazLWlV0OGpxiqeWsTijGEnFhzyXIMqtdbgnm6gIsFM=; b=PXkjwfLF8EsmeDvfYPcgOy/5nV3cWHthIwJXk+GBrPKdfOSS5IvcBhqwxQZrTv6jGY uAAAds+UzOKJB/RLIXzKIqyyyepEkA0hCgx/0eITPB8pe1N7lfPS7GhVmp6kzyvI4zWn d8qFN1DmoHpGDPtPE9ym1B6QYG19dMthjtUQoWM5MwvO+8ZGVUA7nhlqZP6Lb3EDwZsj i6ERAyId/wdNjvYlIme0hWRZbj1n1RcDq1/z0Y03hrVBMaOcB94s9yYUTMGPSDTls9pD aqjPhTGfbreFau5bB23Qt2zlovA5Fna4M4vpkEDmNStDH1aC8A5UVeq7d2fADB90Ejdh PJow== X-Gm-Message-State: AHQUAuaz8mbDd3UCvWGQp0zhzdl7BD4YKSBqdMy3JP2oxk1X/uSp5FEW 1pV2YTkpCI5eJs3bUt8zQKY= X-Google-Smtp-Source: AHgI3IZYhlTQw7CzP9sWgh7rfxqpLNlgl3p6nVnERXteNg31B4yHe6oKSgrg84EyP7emiFpwT7JXKQ== X-Received: by 2002:a17:902:9a95:: with SMTP id w21mr39384222plp.118.1550711366749; Wed, 20 Feb 2019 17:09:26 -0800 (PST) Received: from icarus ([2001:268:c0a3:577d:c70:4af9:86e2:2]) by smtp.gmail.com with ESMTPSA id d23sm26808746pfn.180.2019.02.20.17.09.16 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 20 Feb 2019 17:09:25 -0800 (PST) Date: Thu, 21 Feb 2019 10:09:54 +0900 From: William Breathitt Gray To: Jonathan Cameron Subject: Re: [PATCH 5/8] iio/counter: add FlexTimer Module Quadrature decoder counter driver Message-ID: <20190221010931.GA3471@icarus> References: <20190218140321.19166-1-patrick.havelange@essensium.com> <20190218140321.19166-5-patrick.havelange@essensium.com> <20190220164154.00547a77@archlinux> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190220164154.00547a77@archlinux> User-Agent: Mutt/1.11.3 (2019-02-01) X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree@vger.kernel.org, Lars-Peter Clausen , linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org, Patrick Havelange , Linus Walleij , Daniel Lezcano , linux-kernel@vger.kernel.org, Li Yang , linuxppc-dev@lists.ozlabs.org, Rob Herring , Thierry Reding , linux-arm-kernel@lists.infradead.org, Peter Meerwald-Stadler , Hartmut Knaack , Thomas Gleixner , Shawn Guo , Esben Haabendal Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Wed, Feb 20, 2019 at 04:41:54PM +0000, Jonathan Cameron wrote: > On Mon, 18 Feb 2019 15:03:18 +0100 > Patrick Havelange wrote: > > > This driver exposes the counter for the quadrature decoder of the > > FlexTimer Module, present in the LS1021A soc. > > > > Signed-off-by: Patrick Havelange > > Reviewed-by: Esben Haabendal > Given you cc'd William, I'm guessing you know about the counter > subsystem effort. I would really rather not take any drivers > into IIO if we have any hope of getting that upstreamed soon > (which I personally think we do and should!). The reason is > we end up having to maintain old ABI just because someone might be using > it and it makes the drivers very messy. > > I'll review as is though as may be there are some elements that will > cross over. > > Comments inline. William: Looks like a straight forward conversion if > it makes sense to get this lined up as part of your initial submission? > You have quite a few drivers so I wouldn't have said it needs to be there > at the start, but good to have it soon after. > > Jonathan I agree, we should try to merge this as part of Counter subsystem introduction rather than as another IIO Counter driver. As we determined when adding support for the STM32 timers, the existing IIO Counter API is fundamentally unsuitable for representing counter devices. So regardless of how a new Counter API is merged, the existing IIO Counter API must be deprecated. Patrick, I apologize for the confusion this has caused. Would you be able to convert this driver to use the proposed Counter subsystem API from this patchset that I believe you encountered before: https://marc.info/?l=linux-arm-kernel&m=153229982404051 Although it was last updated in October, I believe you should be able to rebase that Counter subsystem introduction patchset cleanly on top of the IIO tree (if there are any merge conflicts send me an email). Take a look at the generic-counter.rst file under the Documentation/driver-api/ directory for an overview of the API; the counter drivers under the drivers/counter/ directory also make good references. If you have any difficulties understanding the API, or any other troubles, don't hesitate to ask. Hopefully, I've made the documentation clear enough to make the conversion of this driver quick and easy -- and if not, then it's something I need to fix, so let me know. :-) William Breathitt Gray