Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752647AbaGOOGa (ORCPT ); Tue, 15 Jul 2014 10:06:30 -0400 Received: from mail-ig0-f175.google.com ([209.85.213.175]:48829 "EHLO mail-ig0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751792AbaGOOG2 (ORCPT ); Tue, 15 Jul 2014 10:06:28 -0400 MIME-Version: 1.0 In-Reply-To: References: <1404743641-16847-1-git-send-email-sviau@codeaurora.org> <1404743641-16847-2-git-send-email-sviau@codeaurora.org> Date: Tue, 15 Jul 2014 10:06:27 -0400 Message-ID: Subject: Re: [PATCH] drm/msm/mdp5: add support for MDP5 v1.3 From: Rob Clark To: divya ojha Cc: Stephane Viau , "dri-devel@lists.freedesktop.org" , linux-arm-msm , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 15, 2014 at 8:15 AM, divya ojha wrote: > Hi Stephane, > > On Mon, Jul 7, 2014 at 8:04 PM, Stephane Viau wrote: >> >> MDP5 has several functional blocks (ie: VIG/RGB pipes, LMs, ...). >> From one revision to another, these blocks' base addresses might >> change due to the number of instances present in the MDP5 hw. > .. >> -static inline uint32_t REG_MDP5_PIPE(enum mdp5_pipe i0) { return 0x00001200 + 0x400*i0; } >> +static inline uint32_t __offset_PIPE(enum mdp5_pipe idx) >> +{ > > ERROR: switch and case should be at the same indent > >> + switch (idx) { >> + case SSPP_VIG0: return (mdp5_cfg->pipe_vig.base[0]); >> + case SSPP_VIG1: return (mdp5_cfg->pipe_vig.base[1]); >> + case SSPP_VIG2: return (mdp5_cfg->pipe_vig.base[2]); > .. >> -static inline uint32_t REG_MDP5_LM(uint32_t i0) { return 0x00003200 + 0x400*i0; } >> +static inline uint32_t __offset_LM(uint32_t idx) >> +{ > > ERROR: switch and case should be at the same indent this is generated code, so it is ok. I'm not going to bother trying to make the tool that generates this compliant with any indent rules, etc. Too much PITA for no real purpose. Generally I try and split out regeneration of the generated headers, but in this case it was kind of coupled with the other changes. BR, -R > >> + switch (idx) { >> + case 0: return (mdp5_cfg->lm.base[0]); >> + case 1: return (mdp5_cfg->lm.base[1]); >> + case 2: return (mdp5_cfg->lm.base[2]); > .. > >> -static inline uint32_t REG_MDP5_DSPP(uint32_t i0) { return 0x00004600 + 0x400*i0; } >> +static inline uint32_t __offset_DSPP(uint32_t idx) >> +{ > > ERROR: switch and case should be at the same indent > >> + switch (idx) { >> + case 0: return (mdp5_cfg->dspp.base[0]); >> + case 1: return (mdp5_cfg->dspp.base[1]); >> + case 2: return (mdp5_cfg->dspp.base[2]); >> + case 3: return (mdp5_cfg->dspp.base[3]); > .. >> -static inline uint32_t REG_MDP5_INTF(uint32_t i0) { return 0x00012500 + 0x200*i0; } >> +static inline uint32_t __offset_INTF(uint32_t idx) >> +{ > > ERROR: switch and case should be at the same indent > >> + switch (idx) { >> + case 0: return (mdp5_cfg->intf.base[0]); >> + case 1: return (mdp5_cfg->intf.base[1]); >> + case 2: return (mdp5_cfg->intf.base[2]); > .. >> -static inline uint32_t REG_MDP5_AD(uint32_t i0) { return 0x00013100 + 0x200*i0; } >> +static inline uint32_t __offset_AD(uint32_t idx) >> +{ > > ERROR: switch and case should be at the same indent > >> + switch (idx) { >> + case 0: return (mdp5_cfg->ad.base[0]); >> + case 1: return (mdp5_cfg->ad.base[1]); >> + default: return INVALID_IDX(idx); >> + } >> +} > .. >> } >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> hosted by The Linux Foundation -- 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/