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=-4.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 BE91DC433E0 for ; Tue, 4 Aug 2020 08:13:13 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 B21172076E for ; Tue, 4 Aug 2020 08:13:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="iZmZCC7s"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="QAOqMFQB" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B21172076E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ws6yZMNXveIw/JCMhbGaqtC0hwtAgXGnTr7UNeTVWgc=; b=iZmZCC7snsIxSScplxlNfOtX0 /eZXL0YU0/MMZ1SrSFyqqcpUom7OCaRLRb9c9H4K4/n6gkG/vdHTZuNFOF0ecOO5MUR9aAsx2wejG AIsytOweSRS15gQ8qa48iSkVMPG1K4LjR6eRcU9CkI1br9V/rcnJ8TDiJThcXC3PPCe0u/SxH8GlT +PTeC1QqTmnDB1qE4BjjMFsVNKf+8RCR6DhVkJj+S8BQTD88awG+sYMp5WrY2WuMtcOD+h987dtby NTcIwVYWKPyo0kLkYLI7NCbDIjBvWg76qbQRJxKpTP8NayduHaqFr2AmP+wngVq3be/RK37Sawm02 z4rNn1JBQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k2s35-0002pq-76; Tue, 04 Aug 2020 08:11:51 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k2s31-0002p0-ED; Tue, 04 Aug 2020 08:11:48 +0000 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (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 B82442076C; Tue, 4 Aug 2020 08:11:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1596528706; bh=vDIdMthlwvK/lkIZaP62qRcbBqyYKHtAH/TIlf2+Eh4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QAOqMFQBXE6gbzCuR+sDBsl8IRxsLNFDP6Il/GaPzZT+gvA5oW0wVuRUqzDPvDuQp r8UCTIIT31gsbQEhsvEnC79nrk+7FRDKCi1hajUnkC27OS8olMycbG4OrilGdUOySY +6nCD/Zdd2WXZi+ckAUmdh7dfFuCKqKmk6zoyxbg= Date: Tue, 4 Aug 2020 10:11:26 +0200 From: Greg Kroah-Hartman To: "hongxu.zhao" Subject: Re: [PATCH] staging: Add Mediatek High Frequency Manager Framework Message-ID: <20200804081126.GA1765831@kroah.com> References: <20200804075339.9820-1-hongxu.zhao@mediatek.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200804075339.9820-1-hongxu.zhao@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200804_041147_622465_F31393A9 X-CRM114-Status: GOOD ( 15.14 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "open list:STAGING SUBSYSTEM" , wsd_upstream@mediatek.com, Weiqi Fu , open list , Cunliang Du , "moderated list:ARM/Mediatek SoC support" , Zhen jiang , Matthias Brugger , "moderated list:ARM/Mediatek SoC support" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Aug 04, 2020 at 03:52:49PM +0800, hongxu.zhao wrote: > Add a new sensor framework into linux kernel which can support multi client request sensor data. > There are the following features: > 1.Ringbuffer between manager and client; > 2.Kernel space user interface; > 3.User space user interface with syscall; > 4.Each client hang detect mechanism; > 5.Polling timer management in framework no need driver concern; > 6.Polling kthread work intergrated into a single kthread > worker to save system resources in framework no need driver concern; > 7.Proc file system to show manager device and client details; > 8.Compitable with android and widely used in many mediatek platform products; > > Change-Id: I6361cdc2d51de50f66eede7df099c4575e7ec473 Did you not run checkpatch.pl on this? :) No need for change-id here. But, most importantly, why is this in drivers/staging? What keeps it from being in the "real" part of the kernel? I need a TODO file in the directory of the driver listing what remains to be done and who is responsible for doing this work and reviewing patches. Can you resend this with that file added and the Change-id removed? Also, why not just use the IIO interface, why are you creating yet-another api for sensors? We already have 2, making a third seems like something that guarantees this will never be mergable to the correct part of the kernel. And finally, /proc/ is not for devices, that is what sysfs is for, please use that. thanks, greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel