From patchwork Fri Jul 4 07:32:22 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kukjin Kim X-Patchwork-Id: 33074 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-oa0-f72.google.com (mail-oa0-f72.google.com [209.85.219.72]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id 844FA20C88 for ; Fri, 4 Jul 2014 07:32:45 +0000 (UTC) Received: by mail-oa0-f72.google.com with SMTP id eb12sf6787989oac.11 for ; Fri, 04 Jul 2014 00:32:45 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:delivered-to:from:to:cc:references:in-reply-to :subject:date:message-id:mime-version:thread-index:dlp-filter:sender :precedence:list-id:x-original-sender :x-original-authentication-results:mailing-list:list-post:list-help :list-archive:list-unsubscribe:content-type :content-transfer-encoding:content-language; bh=QZWux3BfAid2ArU9rMsqViNQLLZYu06IlGIdXdXeBqI=; b=m7X7UDiwmOO2WJZWP++t4QwprpTc0q800n75fKU7+NMycrt0kJGTLMyj/rIOF3JdSK 3yNtutK8kXDVY1lNfS+QfebM0zo1GYot42B1yQaW1EZDsk3Wqiw53UEBAEyzEkpjHCps CnQdqNTvqh8KZ0WuzgZEzG+Drb0WxZDpb/ClhfqRqdG3x1myNNwrXowumEob0RKCvmEx nQTomWY2gKsZkIuJnHE4B1bo+7nataeELtnDvsblafmASTlM1FsENvjp5wymnfgxOXjG +tDnb+IMtOuLkg9WAhD+wunACylyW0UtXtC3AYfxEHp2ElBuVqGvfW2+jGOK+M9SCmFt k1KQ== X-Gm-Message-State: ALoCoQm9GH+dOaF40BygwZuDzLo2hgUwTFHYKZ15DzwfLCZrHG4bN1bTFnR4pp7OJ6CBAnkNr866 X-Received: by 10.182.58.71 with SMTP id o7mr5041157obq.3.1404459165171; Fri, 04 Jul 2014 00:32:45 -0700 (PDT) X-BeenThere: patchwork-forward@linaro.org Received: by 10.140.27.33 with SMTP id 30ls801509qgw.13.gmail; Fri, 04 Jul 2014 00:32:45 -0700 (PDT) X-Received: by 10.52.163.208 with SMTP id yk16mr5144569vdb.36.1404459165063; Fri, 04 Jul 2014 00:32:45 -0700 (PDT) Received: from mail-ve0-f179.google.com (mail-ve0-f179.google.com [209.85.128.179]) by mx.google.com with ESMTPS id gm6si15213953vec.75.2014.07.04.00.32.45 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 04 Jul 2014 00:32:45 -0700 (PDT) Received-SPF: pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.128.179 as permitted sender) client-ip=209.85.128.179; Received: by mail-ve0-f179.google.com with SMTP id sa20so1294234veb.24 for ; Fri, 04 Jul 2014 00:32:45 -0700 (PDT) X-Received: by 10.58.220.230 with SMTP id pz6mr8252895vec.9.1404459164978; Fri, 04 Jul 2014 00:32:44 -0700 (PDT) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patch@linaro.org Received: by 10.221.37.5 with SMTP id tc5csp446205vcb; Fri, 4 Jul 2014 00:32:44 -0700 (PDT) X-Received: by 10.66.141.165 with SMTP id rp5mr9434898pab.90.1404459163747; Fri, 04 Jul 2014 00:32:43 -0700 (PDT) Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id id4si34697271pbc.140.2014.07.04.00.32.43; Fri, 04 Jul 2014 00:32:43 -0700 (PDT) Received-SPF: none (google.com: linux-samsung-soc-owner@vger.kernel.org does not designate permitted sender hosts) client-ip=209.132.180.67; Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752199AbaGDHcf (ORCPT + 8 others); Fri, 4 Jul 2014 03:32:35 -0400 Received: from mailout4.samsung.com ([203.254.224.34]:29962 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755361AbaGDHcc (ORCPT ); Fri, 4 Jul 2014 03:32:32 -0400 Received: from epcpsbgr2.samsung.com (u142.gpu120.samsung.co.kr [203.254.230.142]) by mailout4.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N8600GNIFLZ7900@mailout4.samsung.com> for linux-samsung-soc@vger.kernel.org; Fri, 04 Jul 2014 16:32:23 +0900 (KST) Received: from epcpsbgm2.samsung.com ( [203.254.230.49]) by epcpsbgr2.samsung.com (EPCPMTA) with SMTP id 2E.AD.19452.68856B35; Fri, 04 Jul 2014 16:32:22 +0900 (KST) X-AuditID: cbfee68e-b7fb96d000004bfc-a1-53b6588653d4 Received: from epmmp2 ( [203.254.227.17]) by epcpsbgm2.samsung.com (EPCPMTA) with SMTP id 04.DE.05196.68856B35; Fri, 04 Jul 2014 16:32:22 +0900 (KST) Received: from DOKGENEKIM03 ([12.36.165.190]) by mmp2.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0N8600GE4FLYPLA0@mmp2.samsung.com>; Fri, 04 Jul 2014 16:32:22 +0900 (KST) From: Kukjin Kim To: 'Tomasz Figa' , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org Cc: 'Thierry Reding' References: <1404163947-3105-1-git-send-email-kgene.kim@samsung.com> <1404163947-3105-12-git-send-email-kgene.kim@samsung.com> <53B55021.9020307@samsung.com> In-reply-to: <53B55021.9020307@samsung.com> Subject: RE: [PATCH 11/17] pwm: samsung: remove s5pc100 related pwm codes Date: Fri, 04 Jul 2014 16:32:22 +0900 Message-id: <02ca01cf975a$1848aaa0$48d9ffe0$@samsung.com> MIME-version: 1.0 X-Mailer: Microsoft Outlook 14.0 Thread-index: AQMomaed6KLZ96WD+OIPh1mQFKG6tgIz4fLVAZ+XWBmYvuDN8A== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrNIsWRmVeSWpSXmKPExsVy+t8zQ922iG3BBhufmFhsenyN1WLG+X1M FutnvGax+LlrHosDi8fOWXfZPTYvqffo27KK0ePzJrkAligum5TUnMyy1CJ9uwSujMdvlQsm qVcsPXqCpYGxT6GLkZNDQsBE4tHbqUwQtpjEhXvr2boYuTiEBJYxSiydcYS5i5EDrOj4H26I +HRGiZ1LW9hAGoQE/jJKnFgWAWKzCWhIHH7/jB3EFhEolHjdOwnMZhbQl2j99p8Ronkho0T7 l81g2zgFtCU+7XsKZgsLeErcbz3NAmKzCKhKbNjTwQ6ymFfAUmLvnUyQMK+AoMSPyfdYIGZq SWze1sQKYctLbF7zlhniAQWJHWdfM0Lc4CTxe8FNqHoRiX0v3oHdICFwil3i76epULsEJL5N PsQC8aSsxKYDUHMkJQ6uuMEygVFiFpLVs5CsnoVk9SwkKxYwsqxiFE0tSC4oTkovMtIrTswt Ls1L10vOz93ECInIvh2MNw9YH2JMBlo/kVlKNDkfGNF5JfGGxmZGFqYmpsZG5pZmpAkrifMu epgUJCSQnliSmp2aWpBaFF9UmpNafIiRiYNTqoExyth0iv2R9wbpwQeiuF9vXfRk1z+jk1LC W9ltDs9SErCKMT8275Hihk9GBorL3J6FpGXaan1d8nVt3oHJulOftje/Oq+dGhY05Y7zYqWW KTuTuvmmiNYb5s9SjORwMbgosyypkveMiNTe4Cv8QYynksWb+MvdtuZprTKuU3C50Sz0o1tE UV2JpTgj0VCLuag4EQDLiMDY3gIAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrFIsWRmVeSWpSXmKPExsVy+t9jQd22iG3BBiuOGVhsenyN1WLG+X1M FutnvGax+LlrHosDi8fOWXfZPTYvqffo27KK0ePzJrkAlqgGRpuM1MSU1CKF1Lzk/JTMvHRb Je/geOd4UzMDQ11DSwtzJYW8xNxUWyUXnwBdt8wcoJVKCmWJOaVAoYDE4mIlfTtME0JD3HQt YBojdH1DguB6jAzQQMI6xozHb5ULJqlXLD16gqWBsU+hi5GDQ0LAROL4H+4uRk4gU0ziwr31 bF2MXBxCAtMZJXYubWEDSQgJ/GWUOLEsAsRmE9CQOPz+GTuILSJQKPG6dxKYzSygL9H67T8j RPNCRon2L5uZQBKcAtoSn/Y9BbOFBTwl7reeZgGxWQRUJTbs6WAHOYJXwFJi751MkDCvgKDE j8n3WCBmakls3tbECmHLS2xe85YZ4lAFiR1nXzNC3OAk8XvBTah6EYl9L94xTmAUmoVk1Cwk o2YhGTULScsCRpZVjKKpBckFxUnpuUZ6xYm5xaV56XrJ+bmbGMHx/kx6B+OqBotDjAIcjEo8 vB5FW4OFWBPLiitzDzFKcDArifBWKW0LFuJNSaysSi3Kjy8qzUktPsRoCvToRGYp0eR8YCrK K4k3NDYxM7I0MrMwMjE3VxLnPdhqHSgkkJ5YkpqdmlqQWgTTx8TBKdXAGPVOb8O9/y8DVr/9 LavG8K/Mh7HhsYt/6+tgQVW3pIvnVdl4Z974VFZ05JND8Iu3b+YvrV7Zeb6sNmeFS8iFgOgm qTl5z811BYRfd89Q/m0QeV/kqnDd4xm65Z275RqvMCgIB97YdLJRfZZzzttYg3U2G5/s00u/ oLU3op+LfWkY54R5Ka8OKbEUZyQaajEXFScCAF40FY0NAwAA DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: list List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: kgene.kim@samsung.com X-Original-Authentication-Results: mx.google.com; spf=pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.128.179 as permitted sender) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org X-Google-Group-Id: 836684582541 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , Content-type: text/plain; charset=US-ASCII Content-transfer-encoding: 7bit Content-language: ko Tomasz Figa wrote: > > Hi Kukjin, > Hi, > On 30.06.2014 23:32, Kukjin Kim wrote: > > This patch removes supporting s5pc100 related pwm codes. > > > > Signed-off-by: Kukjin Kim > > Cc: Thierry Reding > > --- > > Documentation/devicetree/bindings/pwm/pwm-samsung.txt | 2 -- > > drivers/clocksource/samsung_pwm_timer.c | 12 ------------ > > drivers/pwm/pwm-samsung.c | 8 -------- > > 3 files changed, 22 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/pwm/pwm-samsung.txt > b/Documentation/devicetree/bindings/pwm/pwm-samsung.txt > > index 43925d3..82c7f6b 100644 > > --- a/Documentation/devicetree/bindings/pwm/pwm-samsung.txt > > +++ b/Documentation/devicetree/bindings/pwm/pwm-samsung.txt > > @@ -11,8 +11,6 @@ Required properties: > > - compatible : should be one of following: > > samsung,s3c2410-pwm - for 16-bit timers present on S3C24xx SoCs > > samsung,s3c6400-pwm - for 32-bit timers present on S3C64xx SoCs > > - samsung,s5pc100-pwm - for 32-bit timers present on S5PC100, S5PV210, > > - Exynos4210 rev0 SoCs > > As you can see here, this variant is used for more than S5PC100. It is > needed for S5PV210 and Exynos4210 rev0 SoCs as well. So this patch > should be dropped. Oh, I just now checked its datasheets and s5pv210 and early exynos4210 are using BIT(5) for tclk_mask. You're right. But I couldn't its usage in mainline when I created the patch. Anyway it can be used for s5pv210 so how about following? pwm: samsung: change s5pc100-pwm to s5pv210-pwm for support s5pv210 SoC Reported-by: Tomasz Figa Signed-off-by: Kukjin Kim Cc: Thierry Reding --- -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/Documentation/devicetree/bindings/pwm/pwm-samsung.txt b/Documentation/devicetree/bindings/pwm/pwm-samsung.txt index 43925d3..ce005c0 100644 --- a/Documentation/devicetree/bindings/pwm/pwm-samsung.txt +++ b/Documentation/devicetree/bindings/pwm/pwm-samsung.txt @@ -11,8 +11,8 @@ Required properties: - compatible : should be one of following: samsung,s3c2410-pwm - for 16-bit timers present on S3C24xx SoCs samsung,s3c6400-pwm - for 32-bit timers present on S3C64xx SoCs - samsung,s5pc100-pwm - for 32-bit timers present on S5PC100, S5PV210, - Exynos4210 rev0 SoCs + samsung,s5pv210-pwm - for 32-bit timers present on S5PV210 and + Exynos4210 rev0 SoCs samsung,exynos4210-pwm - for 32-bit timers present on Exynos4210, Exynos4x12, Exynos5250 and Exynos5420 SoCs - reg: base address and size of register area diff --git a/drivers/clocksource/samsung_pwm_timer.c b/drivers/clocksource/samsung_pwm_timer.c index e35a9b7..f0a5918 100644 --- a/drivers/clocksource/samsung_pwm_timer.c +++ b/drivers/clocksource/samsung_pwm_timer.c @@ -480,16 +480,16 @@ static void __init s3c64xx_pwm_clocksource_init(struct device_node *np) } CLOCKSOURCE_OF_DECLARE(s3c6400_pwm, "samsung,s3c6400-pwm", s3c64xx_pwm_clocksource_init); -static const struct samsung_pwm_variant s5p_variant = { +static const struct samsung_pwm_variant s5pv210_variant = { .bits = 32, .div_base = 0, .has_tint_cstat = true, .tclk_mask = (1 << 5), }; -static void __init s5p_pwm_clocksource_init(struct device_node *np) +static void __init s5pv210_pwm_clocksource_init(struct device_node *np) { - samsung_pwm_alloc(np, &s5p_variant); + samsung_pwm_alloc(np, &s5pv210_variant); } -CLOCKSOURCE_OF_DECLARE(s5pc100_pwm, "samsung,s5pc100-pwm", s5p_pwm_clocksource_init); +CLOCKSOURCE_OF_DECLARE(s5pv210_pwm, "samsung,s5pv210-pwm", s5pv210_pwm_clocksource_init); #endif diff --git a/drivers/pwm/pwm-samsung.c b/drivers/pwm/pwm-samsung.c index 68f3471..a2bafc9 100644 --- a/drivers/pwm/pwm-samsung.c +++ b/drivers/pwm/pwm-samsung.c @@ -404,24 +404,24 @@ static const struct samsung_pwm_variant s3c64xx_variant = { .tclk_mask = BIT(7) | BIT(6) | BIT(5), }; -static const struct samsung_pwm_variant exynos4210_variant = { +static const struct samsung_pwm_variant s5pv210_variant = { .bits = 32, .div_base = 0, .has_tint_cstat = true, - .tclk_mask = 0, + .tclk_mask = BIT(5), }; -static const struct samsung_pwm_variant s5pc100_variant = { +static const struct samsung_pwm_variant exynos4210_variant = { .bits = 32, .div_base = 0, .has_tint_cstat = true, - .tclk_mask = BIT(5), + .tclk_mask = 0, }; static const struct of_device_id samsung_pwm_matches[] = { { .compatible = "samsung,s3c2410-pwm", .data = &s3c24xx_variant }, { .compatible = "samsung,s3c6400-pwm", .data = &s3c64xx_variant }, - { .compatible = "samsung,s5pc100-pwm", .data = &s5pc100_variant }, + { .compatible = "samsung,s5pv210-pwm", .data = &s5pv210_variant }, { .compatible = "samsung,exynos4210-pwm", .data = &exynos4210_variant }, {}, };