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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D237EC433F5 for ; Tue, 3 May 2022 08:51:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232877AbiECIzJ (ORCPT ); Tue, 3 May 2022 04:55:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34966 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230464AbiECIzH (ORCPT ); Tue, 3 May 2022 04:55:07 -0400 Received: from skyrocket.fabmicro.ru (skyrocket.fabmicro.ru [217.116.57.130]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 22C9A20BFD for ; Tue, 3 May 2022 01:51:34 -0700 (PDT) Received: from mail.fabmicro.ru (skyrocket.fabmicro.ru [217.116.57.130]) by skyrocket.fabmicro.ru (8.14.9/8.14.9) with ESMTP id 2438od2O057796; Tue, 3 May 2022 08:50:39 GMT (envelope-from rz@fabmicro.ru) MIME-Version: 1.0 Date: Tue, 03 May 2022 13:50:39 +0500 From: Ruslan Zalata To: Maxime Ripard Cc: Guenter Roeck , Jean Delvare , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev Subject: Re: [PATCH v2] hwmon: (sun4i-lradc) Add driver for LRADC found on Allwinner A13/A20 SoC In-Reply-To: <20220502110010.q7vvdkdpaiz5acjl@houat> References: <20220428210906.29527-1-rz@fabmicro.ru> <20220502110010.q7vvdkdpaiz5acjl@houat> User-Agent: Roundcube Webmail/1.4.3 Message-ID: <6009fbef45a8b77caa4fc137093a1e23@fabmicro.ru> X-Sender: rz@fabmicro.ru Organization: Fabmicro, LLC. Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Maxime, > I guess a better path forward would be to either register an hwmon > device in the original driver, or convert that driver to iio and use > iio-hwmon. My first idea was to add hwmon to sun4i-lradc-keys.c driver. But soon as I began hacking the driver I quickly realized that it would be a mess since keyboard and hwmon belong to two different subsystems. Besides we would need to invent a way to control which way the driver works (new bindings?). So I decided that a new independent small driver would be a better solution. --- Regards, Ruslan. Fabmicro, LLC.