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=-15.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 375FDC4361B for ; Tue, 8 Dec 2020 16:01:45 +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 CF55323A9B for ; Tue, 8 Dec 2020 16:01:44 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CF55323A9B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.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=BMLQxmSl7nmi+DNWM+87OxNtqzDWlAYgESVeN5hG8/U=; b=sJXesWxZ9ziEn90GWea3ncLJe P/1CpYuOadLnEPg3JGJsKbLa6ZupDgigbk9kP9XYfUTQUiB5dkIHr9n2ikanLlnvC+eD0yD4cy7NM mx4fCkCO4dKqhiJd5KYBhPZS3NBBn4wJbf+WeqvEHXrjNhttGHpm/60ieu2GNGn54RzoK+t8WMuhJ vW3YiT9x2/HzuKa8mlyGS7Qk22Syq39HfJ3jgvBR+MYivwlCv4LiYkYL+djThXUVcxNWo8awNBSId ORxXpIzNK00qFR68+sYkR5ZQ8uP7cUbfizzYT2ZGyiY+OFmE07P4IGk+m7aXiD1KTEh0Rq1VDLNcs SFlpLOVzQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kmfPs-00034o-8o; Tue, 08 Dec 2020 16:00:40 +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 1kmfPq-00034G-2t for linux-arm-kernel@lists.infradead.org; Tue, 08 Dec 2020 16:00:38 +0000 Date: Tue, 8 Dec 2020 16:00:29 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1607443237; bh=rgEunstMpibfhWp7GUnqRGdLLMgBZwXcZaxMILL0EII=; h=From:To:Cc:Subject:References:In-Reply-To:From; b=pIKqcHzTM1UvXwH8uSjPdHgySB79HwWsQFruel6Dt5/oB4Bnmx0SZwzHcM57S8WNr YvNeQC0ScX+PX2j6S4DxYz06eZJEyydRDkvGrcAo2LEl7QdHB7X17Src/kmQw3znzn 5CTSkV/aXTZYsuq9mMp/Cxqopzb1mKQWNgb8ILrwCNDFhqMiWDbx/5bDpC1wapl81u YvYQ7gg8jS/SBwwsfND4MrUA6+2P0Lbp2yN79dn5kXzj+yOqeAG9V+6fekSbQgJq6I cu661oYFIZ5qbqy/GTQUOtM5vBipbHFHiSF01AfN9T+U4JWTlQZyIRKbrqmtSUbCiU tirCf7gj/WmOQ== From: Will Deacon To: Doug Anderson Subject: Re: [PATCH v6 0/4] HID: i2c-hid: Reorganize to allow supporting goodix,gt7375p Message-ID: <20201208160029.GA6461@willie-the-truck> References: <20201112004130.17290-1-dianders@chromium.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201208_110038_275468_B0904A03 X-CRM114-Status: GOOD ( 26.99 ) 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: Geert Uytterhoeven , Catalin Marinas , Bjorn Andersson , Kai-Heng Feng , Anson Huang , Masahiro Yamada , Vinod Koul , Krzysztof Kozlowski , You-Sheng Yang , Linux ARM , "open list:HID CORE LAYER" , Aaron Ma , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Xiaofei Tan , Jiri Kosina , Stephen Boyd , Hans de Goede , Rob Herring , Andrea Borgia , Jiri Kosina , Dmitry Torokhov , LKML , Li Yang , Michael Walle , Benjamin Tissoires , Greg Kroah-Hartman , Olof Johansson , Shawn Guo , Pavel Balan 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 Wed, Dec 02, 2020 at 07:54:40AM -0800, Doug Anderson wrote: > On Wed, Dec 2, 2020 at 7:20 AM Benjamin Tissoires > wrote: > > On Tue, Dec 1, 2020 at 10:12 PM Doug Anderson wrote: > > > On Wed, Nov 11, 2020 at 4:41 PM Douglas Anderson wrote: > > > > .../bindings/input/goodix,gt7375p.yaml | 65 +++++ > > > > arch/arm64/configs/defconfig | 3 +- > > > > drivers/hid/Makefile | 2 +- > > > > drivers/hid/i2c-hid/Kconfig | 47 +++- > > > > drivers/hid/i2c-hid/Makefile | 6 +- > > > > drivers/hid/i2c-hid/i2c-hid-acpi.c | 159 +++++++++++ > > > > drivers/hid/i2c-hid/i2c-hid-core.c | 254 +++--------------- > > > > drivers/hid/i2c-hid/i2c-hid-of-goodix.c | 116 ++++++++ > > > > drivers/hid/i2c-hid/i2c-hid-of.c | 143 ++++++++++ > > > > drivers/hid/i2c-hid/i2c-hid.h | 22 ++ > > > > include/linux/platform_data/i2c-hid.h | 41 --- > > > > 11 files changed, 596 insertions(+), 262 deletions(-) > > > > create mode 100644 Documentation/devicetree/bindings/input/goodix,gt7375p.yaml > > > > create mode 100644 drivers/hid/i2c-hid/i2c-hid-acpi.c > > > > create mode 100644 drivers/hid/i2c-hid/i2c-hid-of-goodix.c > > > > create mode 100644 drivers/hid/i2c-hid/i2c-hid-of.c > > > > delete mode 100644 include/linux/platform_data/i2c-hid.h > > > > > > Are there any additional changes that folks would like with this > > > series? It's not crazy urgent to get it in, but it touches enough > > > lines of code that it'd be nice to get it in before other patches land > > > and it gets merge conflicts. > > > > Sorry for the delay. I was having an internal deadline last week. I > > just re-read the code, and I am quite happy with it. I also just > > tested it on the i2c-hid w/ acpi machine I have here, and it seems OK. > > > > So other than that, do we need to have approvals for patch 2/4 > > (arch/arm64/configs/defconfig)? I can easily take that in the HID > > tree, but I prefer having the approval from the maintainers first. > > Catalin, Will? > > From my past knowledge of the arm64 defconfig, I think it's a bit of a > free-for-all, sort of like updates to the "MAINTAINERS" file. Doing a > "git log" on it I see commits happen from every corner and very few of > them have Acks. I think many (but not all) of the commits to this > file go through trees that feed into the SoC tree (Arnd and Olof) > because those maintainers care about enabling drivers for boards that > they're supporting, but changes come from elsewhere too. > > Obviously an Ack wouldn't hurt, though. Since get_maintainer points > at Will and Catalin I wouldn't say no if one of them wanted to Ack > patch #2 in the series. ;-) For the avoidance of doubt: Acked-by: Will Deacon on patch 2. But yes, although there are a few things I really care about in defconfig (e.g. things like page size!), generally speaking we don't need to Ack everything that changes in there. That said, might be worth checking whether arm-soc have any defconfig changes queued in -next so you don't end up with conflicts. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel