Received: by 2002:ac0:a582:0:0:0:0:0 with SMTP id m2-v6csp3181591imm; Sun, 14 Oct 2018 13:39:20 -0700 (PDT) X-Google-Smtp-Source: ACcGV62SUgIhyIiJcsHzY4ZT4vbUWpyPxb97BJrrddcLFSVybq1ScTUqY0N/vP71rCXFxPglmny8 X-Received: by 2002:a17:902:5e3:: with SMTP id f90-v6mr14717113plf.286.1539549560923; Sun, 14 Oct 2018 13:39:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1539549560; cv=none; d=google.com; s=arc-20160816; b=gh7B44Yt4unSxiyoHx6McYrk7Fvqyf+cJTA9aIj5Cc7yyerxyUgfspV/x4Vc1iXbxe P/WM8dEbI2IJ8AR7wMx6fGJ+LmdtLR7RFvKXXxNqucctqgmG/TFBY/9DjP3CpGfq6vyD c4xjzOYs5YMDtQqn/xqd8x5El6SIZ1tiPojwoncfBRt5HRaTGh7peXIWZiSa3nXKVgpd yOTJTtX6rNplEsYqp4miPsCYI3enzpaQLrE8Y33yojIg1cOduGf/v3+9FoT6C2UGtvZN n2ojO04wPQxdGlwM+V7Vvz5P0airpHojfY6u4A4nSHNj31JUbxcZGIpISaCEk2EHEmCq e0GA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=Muedtq7GBjkdYyk5pDNZjLjjjYAUTyQ0GAagivEO2lo=; b=A0Ah8/kuodssi7a8en6y9nmzHwb+a1zU2rO5xhBPmOK0hm3K14YFgxee6TtnQj2WXo HpJAkMo9Glx8/1H2xRojlGdJmT7D6U6hEc98ulNMorHs+YsmnXY2+XvFjpiVI4S0S0So BdQlwx36s95H6w6cHnZUTFMdKVBUrvVbLpPE2IJ+Eo8RT36fKtMCgPZ1h4dO1D/IloiE /U9DHub41nPBpmSRfMzPvU2sT6oyeO3RBDkaP/aaA0Mt7PhC5xqE9DSy68eo7DykIXOM zbu+HlW4Aknny9eAq+Q9YHjSEt/Lyj5b7dATuMXzfeh8Q4WTCa5jRAs+ceD+uFxAzhSN yxUw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=AlKxVk66; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g38-v6si8276115pgm.193.2018.10.14.13.39.03; Sun, 14 Oct 2018 13:39:20 -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; dkim=pass header.i=@linaro.org header.s=google header.b=AlKxVk66; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726356AbeJOEUx (ORCPT + 99 others); Mon, 15 Oct 2018 00:20:53 -0400 Received: from mail-wr1-f67.google.com ([209.85.221.67]:39715 "EHLO mail-wr1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726084AbeJOEUx (ORCPT ); Mon, 15 Oct 2018 00:20:53 -0400 Received: by mail-wr1-f67.google.com with SMTP id 61-v6so18908978wrb.6 for ; Sun, 14 Oct 2018 13:38:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=Muedtq7GBjkdYyk5pDNZjLjjjYAUTyQ0GAagivEO2lo=; b=AlKxVk66Ia5iTAuay+Lhlf0zJvcVogfoDWl/WMvfwSvPZppBeJyeHAsqFRBEbBpvBp AAOuNQ8jfN0nnweDKmqtr/XtecCWrjzTuOu9VBhkxCVWmy2oevPz2RFFihwCgMphtKxS OH8OspzaCfU7Q13tF46Nz9oiz5ja5RxQ5uHVw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=Muedtq7GBjkdYyk5pDNZjLjjjYAUTyQ0GAagivEO2lo=; b=E6OsadxfsrpFad8KSZZUfteVKjHhSDCsj7MRa00lYqseYsa/njGOwxUxw36QfTOGpg 7bvPNd00Lv1/+GGDNWfJyZyB3A3hA3VetG28HFI9fNxyr3T3+tWPkhYBBT57LGLB+A5G 9/l513dB69lhEXlSz2l4Lzg+5EwLky7LwJXCh344Qi6BaxCpd9sO/nr9MiEebb7JWUcM 7oyx9vcDQ/Fk+V73tRkOvXnizBBw+ff+slWdBRXXsHRWyBElJt7Hq40QonTVHZFeTksH Y12XC2u6o7OigqJKErI9fEwZaaN8YIk+/uBiLkKobtiszrSn2rIf3VJ/8/a8I8d/DAjX gFBg== X-Gm-Message-State: ABuFfojeiwcYTB1x/osqfSdhshV607eD4hTHa/OJ74+L3uB2BIi3W2nx V1xxbuSplftpZg+b1S7yszIAOg== X-Received: by 2002:adf:e808:: with SMTP id o8-v6mr11521208wrm.269.1539549518793; Sun, 14 Oct 2018 13:38:38 -0700 (PDT) Received: from [192.168.0.40] ([80.125.189.32]) by smtp.googlemail.com with ESMTPSA id c64-v6sm5993607wma.44.2018.10.14.13.38.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 14 Oct 2018 13:38:37 -0700 (PDT) Subject: Re: [PATCH V12 0/8] C-SKY(csky) Linux Kernel Driver To: Guo Ren Cc: tglx@linutronix.de, jason@lakedaemon.net, marc.zyngier@arm.com, robh+dt@kernel.org, mark.rutland@arm.com, will.deacon@arm.com, jhogan@kernel.org, paul.burton@mips.com, peterz@infradead.org, arnd@arndb.de, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org References: <20181014162631.GA13374@guoren-Inspiron-7460> From: Daniel Lezcano Message-ID: <36e2cb6d-6d9e-4764-a6dd-a5f74cab5a09@linaro.org> Date: Sun, 14 Oct 2018 22:38:36 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20181014162631.GA13374@guoren-Inspiron-7460> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 14/10/2018 18:26, Guo Ren wrote: > Hi Daniel, > > Thx for reply and review. > > On Sat, Oct 13, 2018 at 10:50:34PM +0200, Daniel Lezcano wrote: >> On 12/10/2018 14:02, Guo Ren wrote: >>> This is about 12th patchset for C-SKY linux drivers and it should pair >>> with 8th kernel patchset. >> >> Guo, >> >> I'm willing to take your timer related patches but you have to put a >> proper description. > > Ok, I've improved all my comment for the drivers. Here is my modification: Thanks Guo, the descriptions look ok for the timer part. Can you resend the DT bindings + timers patches separately with the commit log updated ? When you add the Changelog you have to insert it *after* the first '---', so when the patch is git applied, the change log will be discarded and won't appear in git. eg. commit 6b465856936cb8f7eda31b20eb59fb5c8a19d468 Author: Guo Ren Date: Tue Oct 2 16:43:18 2018 +0800 clocksource: add gx6605s SOC system timer The driver is for gx6605s SOC system timer and there are two same timers in gx6605s. We use one for clkevt and another one for clksrc. The timer is mmio map to access, so we need give mmio address in dts. The counter at 0x0 offset is clock event. The counter at 0x40 offset is clock source. Signed-off-by: Guo Ren Cc: Daniel Lezcano --- Changelog: - pass checkpatch.pl - Add COMIPLE_TEST in Kconfig - no cast is needed for "struct clock_event_device *ce = dev" - remove: extra space after (u64) - Add License and Copyright - Use timer-of framework - Change name with upstream feedback - Use clksource_mmio framework --- Thanks -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog