Message ID | 1464369565-12259-4-git-send-email-kieran@bingham.xyz (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Laurent Pinchart |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from <linux-media-owner@vger.kernel.org>) id 1b6LRf-0001yj-EL; Fri, 27 May 2016 17:21:11 +0000 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.84_2/mailfrontend-8) with esmtp id 1b6LRd-000560-kG; Fri, 27 May 2016 19:21:11 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932914AbcE0RUs (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 27 May 2016 13:20:48 -0400 Received: from mail-wm0-f65.google.com ([74.125.82.65]:34818 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932811AbcE0RTc (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 27 May 2016 13:19:32 -0400 Received: by mail-wm0-f65.google.com with SMTP id e3so239825wme.2 for <linux-media@vger.kernel.org>; Fri, 27 May 2016 10:19:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ksquared.org.uk; s=google; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=CzuEX+CBwjUvhNT14S1I+hdN2DssTMxNfpOLPUrgeH4=; b=NhQhnKwTcBfHVDxBnT3p7B7Czx4k/sQqSyOC9wuz24QR83VavtatA1FKh7jH0av/xa MW1UqyzTrgOFBTspGbYZEwTtLq2u8oTP3gPvCJPXfzVJF/HlloogjTSALkby7ad1a9iH 0DvmBYogr383wV8+lGatSXuDcU/E2EqJ4U6UI= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bingham-xyz.20150623.gappssmtp.com; s=20150623; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=CzuEX+CBwjUvhNT14S1I+hdN2DssTMxNfpOLPUrgeH4=; b=124Dofwvvtx19TqYswpRtQS5PAuvLoJNTwVHmZVSmjUqHBoWhya9kZsASbxNspfLf1 22Fngn6CVYVxdVyaxk5Eg1ba0CErOKYtgL7azclaT3TGIsDQFIKKsfSHL0YfHcTloiSe mY1SFk6Nd+dlmzc+mcEFH6C7cYdOWlVyU1z/XEOsJOJSVsloY9ZiWUfr1j9o6TTDs1HG ydYoEtuXN9HgolDsvA6BHP+nrYaRVfIEZUOkEVp1+vhgeesJNOlnua7sBNJ1e8VOMK8r KPhJwPFin5R5EUdkl06wbaqHAu4QLASwpro5ZR881lWhtfJgMTokP/onwE9ZnpzRTzw9 NBQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=CzuEX+CBwjUvhNT14S1I+hdN2DssTMxNfpOLPUrgeH4=; b=gsQNb0ddqQh3iivUExCcVperOuKhfuXqpQjSYFMIBt+fxF4kBQ6CIGTXSPNKOkcbBm GKYEjC5lh60X1zr0JLn+4hdRwV7kHxV25vRzqH6mVo0zXw1qYltWWY021Lk2kSBXThqi uU7Zaq1VN8cjtU1mm1PA5lWpjQZjf5EEdBdH1B/UBMydDv6KspN+ibsJhnyyS2U87M0w 516tYgY/buyQKiFbQxF1LHaeM5yuJGhRBogFPN7XF5zu12Nj3svPthFwV25Z1cWgk9ep /2UJwe7vYhYcLtn121UkoejfmqvJiB0UTQXTbqdLAU6NH9dmdrOeaItz/XHlW6eyNGxx TVKw== X-Gm-Message-State: ALyK8tLSs9NIa9GhT2dY2ty7FMQ2oIZZZQDd7Harg5nLa5+tBhMi6mO1kmdXXIISLz1Vlw== X-Received: by 10.28.143.82 with SMTP id r79mr10092091wmd.40.1464369571065; Fri, 27 May 2016 10:19:31 -0700 (PDT) Received: from CookieMonster.cookiemonster.local (cpc87017-aztw30-2-0-cust65.18-1.cable.virginm.net. [92.232.232.66]) by smtp.gmail.com with ESMTPSA id wb10sm20354645wjc.8.2016.05.27.10.19.30 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 27 May 2016 10:19:30 -0700 (PDT) From: Kieran Bingham <kieran@ksquared.org.uk> To: laurent.pinchart@ideasonboard.com, linux-renesas-soc@vger.kernel.org, kieran@ksquared.org.uk Cc: linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org Subject: [PATCH 2/4] dt-bindings: Update Renesas R-Car FCP DT binding Date: Fri, 27 May 2016 18:19:23 +0100 Message-Id: <1464369565-12259-4-git-send-email-kieran@bingham.xyz> X-Mailer: git-send-email 2.5.0 In-Reply-To: <1464369565-12259-1-git-send-email-kieran@bingham.xyz> References: <1464369565-12259-1-git-send-email-kieran@bingham.xyz> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2016.5.27.171218 X-PMX-Spam: Gauge=IIIIIIIII, Probability=9%, Report=' MULTIPLE_RCPTS 0.1, XYZ_TLD 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1500_1599 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, CS_SUSP_TLD_BODY 0, DKIM_SIGNATURE 0, IN_REP_TO 0, LEGITIMATE_NEGATE 0, MSG_THREAD 0, MULTIPLE_RCPTS_RND 0, NO_URI_HTTPS 0, REFERENCES 0, SINGLE_URI_IN_BODY 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_URI_IN_BODY 0, __HAS_CC_HDR 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MULTIPLE_RCPTS_CC_X2 0, __REFERENCES 0, __SANE_MSGID 0, __SINGLE_URI_TEXT 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_IN_BODY 0, __URI_NO_WWW 0, __URI_NS , __URI_WITH_PATH 0, __YOUTUBE_RCVD 0' |
Commit Message
Kieran Bingham
May 27, 2016, 5:19 p.m. UTC
The FCP driver, can also support the FCPF variant for FDP1 compatible
processing.
Signed-off-by: Kieran Bingham <kieran@bingham.xyz>
---
Documentation/devicetree/bindings/media/renesas,fcp.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Cc: devicetree@vger.kernel.org
Comments
Hi Kieran, On Fri, May 27, 2016 at 7:19 PM, Kieran Bingham <kieran@ksquared.org.uk> wrote: > The FCP driver, can also support the FCPF variant for FDP1 compatible > processing. > > Signed-off-by: Kieran Bingham <kieran@bingham.xyz> Thanks for your patch! > --- > Documentation/devicetree/bindings/media/renesas,fcp.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > Cc: devicetree@vger.kernel.org > > diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.txt b/Documentation/devicetree/bindings/media/renesas,fcp.txt > index 6a12960609d8..1c0718b501ef 100644 > --- a/Documentation/devicetree/bindings/media/renesas,fcp.txt > +++ b/Documentation/devicetree/bindings/media/renesas,fcp.txt > @@ -7,11 +7,12 @@ conversion of AXI transactions in order to reduce the memory bandwidth. > > There are three types of FCP: FCP for Codec (FCPC), FCP for VSP (FCPV) and FCP > for FDP (FCPF). Their configuration and behaviour depend on the module they > -are paired with. These DT bindings currently support the FCPV only. > +are paired with. These DT bindings currently support the FCPV and FCPF. > > - compatible: Must be one or more of the following > > - "renesas,r8a7795-fcpv" for R8A7795 (R-Car H3) compatible 'FCP for VSP' > + - "renesas,r8a7795-fcpf" for R8A7795 (R-Car H3) compatible 'FCP for FDP' > - "renesas,fcpv" for generic compatible 'FCP for VSP' I guess checkpatch complained about your dtsi additions because you forgot to add "renesas,fcpf" here? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Geert! On 28/05/16 20:06, Geert Uytterhoeven wrote: > Hi Kieran, > > On Fri, May 27, 2016 at 7:19 PM, Kieran Bingham <kieran@ksquared.org.uk> wrote: >> The FCP driver, can also support the FCPF variant for FDP1 compatible >> processing. >> >> Signed-off-by: Kieran Bingham <kieran@bingham.xyz> > > Thanks for your patch! > >> --- >> Documentation/devicetree/bindings/media/renesas,fcp.txt | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> Cc: devicetree@vger.kernel.org >> >> diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.txt b/Documentation/devicetree/bindings/media/renesas,fcp.txt >> index 6a12960609d8..1c0718b501ef 100644 >> --- a/Documentation/devicetree/bindings/media/renesas,fcp.txt >> +++ b/Documentation/devicetree/bindings/media/renesas,fcp.txt >> @@ -7,11 +7,12 @@ conversion of AXI transactions in order to reduce the memory bandwidth. >> >> There are three types of FCP: FCP for Codec (FCPC), FCP for VSP (FCPV) and FCP >> for FDP (FCPF). Their configuration and behaviour depend on the module they >> -are paired with. These DT bindings currently support the FCPV only. >> +are paired with. These DT bindings currently support the FCPV and FCPF. >> >> - compatible: Must be one or more of the following >> >> - "renesas,r8a7795-fcpv" for R8A7795 (R-Car H3) compatible 'FCP for VSP' >> + - "renesas,r8a7795-fcpf" for R8A7795 (R-Car H3) compatible 'FCP for FDP' >> - "renesas,fcpv" for generic compatible 'FCP for VSP' > > I guess checkpatch complained about your dtsi additions because you forgot > to add "renesas,fcpf" here? Yes. Looks quite obvious doesn't it :) I thought CP.pl was complaining because the file was renesas,fcp.txt, and I was adding in fcpf!. I was clearly a bit too keen to get those patches out on a friday evening. I must remember that I'm not smarter than checkpatch! I'll add the missing line :) > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds >
diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.txt b/Documentation/devicetree/bindings/media/renesas,fcp.txt index 6a12960609d8..1c0718b501ef 100644 --- a/Documentation/devicetree/bindings/media/renesas,fcp.txt +++ b/Documentation/devicetree/bindings/media/renesas,fcp.txt @@ -7,11 +7,12 @@ conversion of AXI transactions in order to reduce the memory bandwidth. There are three types of FCP: FCP for Codec (FCPC), FCP for VSP (FCPV) and FCP for FDP (FCPF). Their configuration and behaviour depend on the module they -are paired with. These DT bindings currently support the FCPV only. +are paired with. These DT bindings currently support the FCPV and FCPF. - compatible: Must be one or more of the following - "renesas,r8a7795-fcpv" for R8A7795 (R-Car H3) compatible 'FCP for VSP' + - "renesas,r8a7795-fcpf" for R8A7795 (R-Car H3) compatible 'FCP for FDP' - "renesas,fcpv" for generic compatible 'FCP for VSP' When compatible with the generic version, nodes must list the