Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp952171imm; Fri, 3 Aug 2018 14:49:03 -0700 (PDT) X-Google-Smtp-Source: AAOMgpceHt4u/ZknQJtSp94Puox6bY1QYV+/2Mto1c3t4NjgwkyCvmg1CMr9z1WNJ3l6qFREB7M7 X-Received: by 2002:a63:f002:: with SMTP id k2-v6mr5296758pgh.8.1533332943200; Fri, 03 Aug 2018 14:49:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533332943; cv=none; d=google.com; s=arc-20160816; b=gm6xjabOPTKqf2Y8vTbV/6cSt5mHKSiwdSvgQOa7ZR/qNpX+rWbfbASPzirWlXgDGG 4tj5axT9nMzK4+IwHTcuG1gX/53r376DNMZy1DwKIj1e6l0yFgbPxFhZt8bpgjEeygc0 IY6/ZSXG21Oib15UvBKAXLbi+PWSt5HWZ6kf2hIcijfVKzuxEyZc1Ptq9k7uGVeWFXe3 G9ITV96RL6jVW+fg+4BLnsPOl25NfQMu1oEi9w93tsxLKSXH/qHjeuaBpC5XpS51H+uN d6Un2V65ih/TDwWl2Pkcl8w/F0NZg000HeZ8DcK26AvdegTRVIYOBlEt1zCZoylhlWot XGNA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :references:in-reply-to:mime-version:arc-authentication-results; bh=jCJCOmxbbpD6kxvsu4TZl3clHrzkpWK57tuw2hCa7MY=; b=kPWGFcsC31D2S3ynGa2atoajg6N/XO8+y2fKYK9V11R/xdVDxOidm2cgEbkvvfsjAD OtiAbpx27ubXSWjWdc3xGNB1LXbtSw/8jyYyR20qucj+OWpku35G/JSHynwsOWOMADtb bEnl1EZ6U8r5TV1vIgz0OUEEIHiR1LUe1KvdZQ10V3+OCv3L3sg9J7reBJJbJkYIxKrj BlVcXIdeRhm9vIuHnhrvGUHzyT0ikTseVniDvpzfqN4H6+2UOvS94d43ASYcXt3q88y0 ygkY+pb+fF9EAFWApMJu0z9iGMpAJf5fF6/W8LYa0E5a9hkW68j2fYqKRiRnDmJKrKV8 ezKA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f68-v6si5438326pfj.121.2018.08.03.14.48.47; Fri, 03 Aug 2018 14:49:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732072AbeHCXqA (ORCPT + 99 others); Fri, 3 Aug 2018 19:46:00 -0400 Received: from smtp06.smtpout.orange.fr ([80.12.242.128]:37284 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729828AbeHCXqA (ORCPT ); Fri, 3 Aug 2018 19:46:00 -0400 Received: from mail-qt0-f174.google.com ([209.85.216.174]) by mwinf5d63 with ME id Jlns1y0023mLd2903lnssU; Fri, 03 Aug 2018 23:47:53 +0200 X-ME-Helo: mail-qt0-f174.google.com X-ME-Auth: bWF4aS5qb3VyZGFuQHdhbmFkb28uZnI= X-ME-Date: Fri, 03 Aug 2018 23:47:53 +0200 X-ME-IP: 209.85.216.174 Received: by mail-qt0-f174.google.com with SMTP id c15-v6so7824760qtp.0 for ; Fri, 03 Aug 2018 14:47:52 -0700 (PDT) X-Gm-Message-State: AOUpUlHva6rKpiq/79TWc8YV3iHdsGNVSZcmgu+gF7KUXNRUG3tWV9gR J3lyuABwnKe3dam07PpMNu20INpQuYChaD99hS8= X-Received: by 2002:ac8:3351:: with SMTP id u17-v6mr5718821qta.126.1533332872012; Fri, 03 Aug 2018 14:47:52 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:aed:278a:0:0:0:0:0 with HTTP; Fri, 3 Aug 2018 14:47:51 -0700 (PDT) In-Reply-To: References: <20180801185128.23440-1-maxi.jourdan@wanadoo.fr> <20180801185128.23440-2-maxi.jourdan@wanadoo.fr> From: Maxime Jourdan Date: Fri, 3 Aug 2018 23:47:51 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 1/4] soc: amlogic: add meson-canvas driver To: Yixun Lan Cc: Maxime Jourdan , Kevin Hilman , Neil Armstrong , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Yixun, thanks for the review 2018-08-03 16:14 GMT+02:00 Yixun Lan : >> +config MESON_CANVAS >> + bool "Amlogic Meson Canvas driver" > shouldn't this a 'tristate'? since you'd make the driver a kernel module.. Yep it should! I well noted your other feedback and v2 will have fixes for them :) .