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=-11.4 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,FROM_EXCESS_BASE64,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED 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 779E6C43387 for ; Sat, 12 Jan 2019 08:08:22 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 415E22133F for ; Sat, 12 Jan 2019 08:08:22 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="rH8RpMOc" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 415E22133F Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=siol.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=9UwFHGmwuK6NjUzKLC0NpGQc+xV//PtPWD4Qz1QqmzM=; b=rH8RpMOc7beMtS kqG0iMOx/hxhlB6JaKnqUNnpR1XgAoqwyc67oqgbPbOOQWE7YCnWHNuPlmPzgd1LdReTXtL4R+m/z v/livQBXnhQQI43+7nRkGNwRzDUHweIpiIeXtgCldQXGybxzBMGlyHqwL2YMSMADft6qzitn5P8st KsqeE0t0Xoc8XjBfC2pAIpW9/U1NUZIrZdSGx9jGFWWAQzxhTP2TYyE7cxA+awdaVCvrNmUvAsM4h 530uLMGELqm2/l4Q9Zt8Q9vnBT4QH3Wfv2Le9mAd3Z0aEKhjj8dbZszW+8a+NBLqJq3I6UCe5ezC7 0taD3r0BzIbCUbu2KLUg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1giEL5-0004Vo-7k; Sat, 12 Jan 2019 08:08:19 +0000 Received: from mailoutvs42.siol.net ([185.57.226.233] helo=mail.siol.net) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1giEL0-0004V3-TF for linux-arm-kernel@lists.infradead.org; Sat, 12 Jan 2019 08:08:17 +0000 Received: from localhost (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTP id 0B13A520991; Sat, 12 Jan 2019 09:08:10 +0100 (CET) X-Virus-Scanned: amavisd-new at psrvmta11.zcs-production.pri Received: from mail.siol.net ([127.0.0.1]) by localhost (psrvmta11.zcs-production.pri [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id wtYuQKiFF0G7; Sat, 12 Jan 2019 09:08:09 +0100 (CET) Received: from mail.siol.net (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTPS id 94B43520AF5; Sat, 12 Jan 2019 09:08:09 +0100 (CET) Received: from jernej-laptop.localnet (cpe1-8-82.cable.triera.net [213.161.8.82]) (Authenticated sender: 031275009) by mail.siol.net (Postfix) with ESMTPA id 05832520991; Sat, 12 Jan 2019 09:08:08 +0100 (CET) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Chen-Yu Tsai Subject: Re: [PATCH 1/3] media: dt: bindings: sunxi-ir: Add A64 compatible Date: Sat, 12 Jan 2019 09:08:08 +0100 Message-ID: <4866484.1b519qCrCi@jernej-laptop> In-Reply-To: References: <20190111173015.12119-1-jernej.skrabec@siol.net> <20190111173015.12119-2-jernej.skrabec@siol.net> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190112_000815_261185_194B84DF X-CRM114-Status: GOOD ( 21.01 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , devicetree , Maxime Ripard , linux-kernel , linux-sunxi , Rob Herring , Mauro Carvalho Chehab , linux-arm-kernel , Linux Media Mailing List Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Dne sobota, 12. januar 2019 ob 02:56:11 CET je Chen-Yu Tsai napisal(a): > On Sat, Jan 12, 2019 at 1:30 AM Jernej Skrabec wrote: > > A64 IR is compatible with A13, so add A64 compatible with A13 as a > > fallback. > > We ask people to add the SoC-specific compatible as a contigency, > in case things turn out to be not so "compatible". > > To be consistent with all the other SoCs and other peripherals, > unless you already spotted a "compatible" difference in the > hardware, i.e. the hardware isn't completely the same, this > patch isn't needed. On the other hand, if you did, please mention > the differences in the commit log. When comparing registers descriptions between A13 and A64, I noticed few minor differences: A13: RXINT: 11:6 RAL A64: RXINT: 13:8 RAL A13: IR_RXSTA: 12:6 RAC A64: IR_RXSTA: 14:8 RAC, 7 STAT (missing on A13) What is strange that RAL and RAC field have offset defined as 8 in driver. I'm not sure if that is a typo in A13 manual or driver issue. I assume the former, otherwise it wouldn't work. I couldn't found original BSP driver source to confirm, though. STAT bit is really not that important. It just tells if IR unit is busy or not. The biggest difference is in 0x34 register. A64 has one more clock option (without postdivider), although register values are backward compatible. A64 also has Active threshold setting (duration of CIR going from idle to active state). If we dismiss RAC and RAL differences as manual error and don't care for new clock option and active threshold, then having new compatible maybe really doesn't make sense. Best regards, Jernej > > ChenYu > > > Signed-off-by: Jernej Skrabec > > --- > > > > Documentation/devicetree/bindings/media/sunxi-ir.txt | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/media/sunxi-ir.txt > > b/Documentation/devicetree/bindings/media/sunxi-ir.txt index > > 278098987edb..ecac6964b69b 100644 > > --- a/Documentation/devicetree/bindings/media/sunxi-ir.txt > > +++ b/Documentation/devicetree/bindings/media/sunxi-ir.txt > > @@ -1,7 +1,10 @@ > > > > Device-Tree bindings for SUNXI IR controller found in sunXi SoC family > > > > Required properties: > > -- compatible : "allwinner,sun4i-a10-ir" or "allwinner,sun5i-a13-ir" > > +- compatible : value must be one of: > > + * "allwinner,sun4i-a10-ir" > > + * "allwinner,sun5i-a13-ir" > > + * "allwinner,sun50i-a64-ir", "allwinner,sun5i-a13-ir" > > > > - clocks : list of clock specifiers, corresponding to > > > > entries in clock-names property; > > > > - clock-names : should contain "apb" and "ir" entries; > > > > -- > > 2.20.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel