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=-4.0 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS, URIBL_BLOCKED autolearn=ham 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 C957FC04AB4 for ; Tue, 14 May 2019 18:47: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 9F7FD206BF for ; Tue, 14 May 2019 18:47: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="AFeupbxL"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=googlemail.com header.i=@googlemail.com header.b="FSuDnTxa" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9F7FD206BF Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=googlemail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-amlogic-bounces+linux-amlogic=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:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=p3CubvBwBWJsLF5EMLmpYB1fJ8AWGALXegvWlpDV5VI=; b=AFeupbxLArZ4u6 PHy/0SS3SLuMo6KQID/e97kqPC1YOZXI9h9W8iJSAVPrSKcJgy4dF/dqNE1Vy2JdI+tROYSxMuJ1C SNJGVoE94wP7XSTwT0iPFiTlvRML/PeNK5mjwz3Wj2+sSzxa6EAs0neTW2os0igelvc9EyNCT1g7y PGJ1BtbZLOTJgTdH8y8Ffv5zEEhZUC5qGpx4dibLAq4IjlLPcPhoNVIsVGvxrP2IW3AggPqgkPafe DqlU7VAuNaQhOOTSt9iSnUSi+/Oj9WtyHZ+EC9SUVa2pLYDBxtGC3twWKV6rb59/+5baIS/uCN9Pg ZG2vkG2PPcQYr73fisZg==; 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 1hQcSL-0002SO-4M; Tue, 14 May 2019 18:47:17 +0000 Received: from mail-oi1-x241.google.com ([2607:f8b0:4864:20::241]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hQcSC-0002Kk-PH; Tue, 14 May 2019 18:47:10 +0000 Received: by mail-oi1-x241.google.com with SMTP id v10so12959176oib.1; Tue, 14 May 2019 11:47:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=b3wKV2yvyhNXBHezd2g3VNLzJd77vsTJn5OuHUZElkE=; b=FSuDnTxaDbNRibS/cl1V/LWshcY/nC5P8Wwb7sEKTQIA9N2sXN/aZ6qS/B9N6ZZYli iSBFSvSmeeYzin0eC/VMw0QMTnlgL6WK1P6ZSSX7g7RIkiQlQ+zVRYyuU+L6c1KQtOUm H1eZ1Jhgv6syoz7DG5/dU3a6wKSu4JpdNY90IRtdlh8byKBHrEgvzB8s0C2DAGu7u3g7 ecabg5KjKS48Qa6iSf+sF7zgW7KHddpTQClz7GvcquLTMtxqN2S6BOGnxIxUWSmrdZru E/VAnuEYZ3NTqFnB3QqeLNELRDkAwZjEk4qOK/laDM+wxE5Day3oWcT4Y4fqgcKRX2gl 3A4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=b3wKV2yvyhNXBHezd2g3VNLzJd77vsTJn5OuHUZElkE=; b=aD9KVgsoOzkLxTEwNuAsEr1c+5jd8+luPa8Fxjrvwj6bc/jWRp40pV8MzzwVbFiGn5 AAk4v5WWnZg1yyTm4HaSJubk0CaDyDzdDLcT1CZKpkC+UaIxxzacIumlVyFNzMVnzk2E x/HL8H+EPooWDyM2Z8JiAwcAjUQwx/B5BbyLpT+qTb6sFI7R/P7RQ+8O8u7MgImc9rGT rbhdpa+djwPy+gOzhDoKBKxJPsVLAxmptAo4ZeJL15u1/aP/Y0Fx0nVYdYfuQPH6O+iS iyE3RSAwsWoxl3B7D13bkyXVsd0Gt5H5PGP/Cb4Vox2qlh+TRbj4wWYHuCT/iT2Vvdt5 9w6A== X-Gm-Message-State: APjAAAUqBRUkcGOw1odKkdkVTsAx12QkNG7wiVXcGNArRaw+CdDe6x6V wPXeT/nkq4Ufih20QfLJ6p1gWn2NOj+Y4xCLoAlgHPJjkY0= X-Google-Smtp-Source: APXvYqyAsqSOHm/4PjG8rmz2z+47hObMZtkODxPLmkZxrCSNCrhSrYuN8g3dNQ1msbyt5tew1NttZ3FWwieMhIJnR2U= X-Received: by 2002:aca:ab04:: with SMTP id u4mr3815658oie.15.1557859627031; Tue, 14 May 2019 11:47:07 -0700 (PDT) MIME-Version: 1.0 References: <20190514135612.30822-1-mjourdan@baylibre.com> <20190514135612.30822-2-mjourdan@baylibre.com> In-Reply-To: <20190514135612.30822-2-mjourdan@baylibre.com> From: Martin Blumenstingl Date: Tue, 14 May 2019 20:46:56 +0200 Message-ID: Subject: Re: [PATCH v6 1/4] dt-bindings: media: add Amlogic Video Decoder Bindings To: Maxime Jourdan X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190514_114708_843266_F082739D X-CRM114-Status: GOOD ( 14.10 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Neil Armstrong , Kevin Hilman , linux-kernel@vger.kernel.org, Hans Verkuil , Jerome Brunet , linux-amlogic@lists.infradead.org, Mauro Carvalho Chehab , Rob Herring , linux-arm-kernel@lists.infradead.org, linux-media Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org Hi Maxime, On Tue, May 14, 2019 at 3:56 PM Maxime Jourdan wrote: > > Add documentation for the meson vdec dts node. thank you for working on this! > Reviewed-by: Rob Herring > Signed-off-by: Maxime Jourdan Reviewed-by: Martin Blumenstingl [...] > +Required properties: > +- compatible: value should be different for each SoC family as : > + - GXBB (S905) : "amlogic,gxbb-vdec" > + - GXL (S905X, S905D) : "amlogic,gxl-vdec" > + - GXM (S912) : "amlogic,gxm-vdec" > +- reg: base address and size of he following memory-mapped regions : > + - dos > + - esparser > +- reg-names: should contain the names of the previous memory regions > +- interrupts: should contain the following IRQs: > + - vdec > + - esparser > +- interrupt-names: should contain the names of the previous interrupts Maxime already mentioned in the RFC version that there may be more interrupts in the future. I'm fine with adding the missing ones on demand > +- amlogic,ao-sysctrl: should point to the AOBUS sysctrl node my goal is to remove this property at some point and replace it with references to the power-domains. however, we don't have a power-domain driver for this yet (it's somewhere on my TODO-list - it involves splitting the current meson-gx-pwrc-vpu into two separate drivers, so we need to do it carefully so we don't break HDMI support) Regards Martin _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic