Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936726Ab3DILsb (ORCPT ); Tue, 9 Apr 2013 07:48:31 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:43862 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933173Ab3DILsa (ORCPT ); Tue, 9 Apr 2013 07:48:30 -0400 Message-ID: <51640004.6020604@ti.com> Date: Tue, 9 Apr 2013 17:18:20 +0530 From: Sekhar Nori User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130328 Thunderbird/17.0.5 MIME-Version: 1.0 To: Prabhakar lad CC: DLOS , LAK , LKML Subject: Re: [PATCH v9 1/2] ARM: davinci: dm365: add support for v4l2 video display References: <1365412636-3468-1-git-send-email-prabhakar.csengg@gmail.com> <1365412636-3468-2-git-send-email-prabhakar.csengg@gmail.com> In-Reply-To: <1365412636-3468-2-git-send-email-prabhakar.csengg@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1538 Lines: 43 On 4/8/2013 2:47 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm365. > > Signed-off-by: Lad, Prabhakar > Acked-by: Sekhar Nori > --- > diff --git a/arch/arm/mach-davinci/davinci.h b/arch/arm/mach-davinci/davinci.h > index 12d544b..a9de512 100644 > --- a/arch/arm/mach-davinci/davinci.h > +++ b/arch/arm/mach-davinci/davinci.h > @@ -36,8 +36,13 @@ > #include > > #define DAVINCI_SYSTEM_MODULE_BASE 0x01c40000 > +#define SYSMOD_VDAC_CONFIG 0x2c > #define SYSMOD_VIDCLKCTL 0x38 > #define SYSMOD_VPSS_CLKCTL 0x44 > +#define VPSS_VENCCLKEN_ENABLE BIT(3) > +#define VPSS_DACCLKEN_ENABLE BIT(4) > +#define VPSS_PLLC2SYSCLK5_ENABLE BIT(5) > + Prabhakar, I noticed it only after I applied these patches, but these bit definitions in between list of register offsets is distracting. Can you please move them down after the register offsets when you send the pull request? You can add a comment /* VPSS CLKCTL bit definitions */ before you start the bit definitions. > #define SYSMOD_VDD3P3VPWDN 0x48 > #define SYSMOD_VSCLKDIS 0x6c > #define SYSMOD_PUPDCTL1 0x7c Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/