Messages in this thread | | | From | Grant Likely <> | Date | Wed, 3 Feb 2010 09:27:55 -0700 | Subject | Re: [PATCHv2 03/11] mxc: Fix Drive Strength Field in the IOMUX controller |
| |
On Tue, Feb 2, 2010 at 10:16 PM, Amit Kucheria <amit.kucheria@canonical.com> wrote: > i.MX51 defines 4 values: > > 00: Low Drive Strength > 01: Medium Drive Strength > 10: High Drive Strength > 11: Max Drive Strength > > Signed-off-by: Amit Kucheria <amit.kucheria@canonical.com> > --- > arch/arm/plat-mxc/include/mach/iomux-v3.h | 8 +++++--- > 1 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/plat-mxc/include/mach/iomux-v3.h b/arch/arm/plat-mxc/include/mach/iomux-v3.h > index 1deda01..f2f73d3 100644 > --- a/arch/arm/plat-mxc/include/mach/iomux-v3.h > +++ b/arch/arm/plat-mxc/include/mach/iomux-v3.h > @@ -81,11 +81,13 @@ struct pad_desc { > > #define PAD_CTL_ODE (1 << 3) > > -#define PAD_CTL_DSE_STANDARD (0 << 1) > -#define PAD_CTL_DSE_HIGH (1 << 1) > -#define PAD_CTL_DSE_MAX (2 << 1) > +#define PAD_CTL_DSE_LOW (0 << 1) > +#define PAD_CTL_DSE_MED (1 << 1) > +#define PAD_CTL_DSE_HIGH (2 << 1) > +#define PAD_CTL_DSE_MAX (3 << 1) > > #define PAD_CTL_SRE_FAST (1 << 0) > +#define PAD_CTL_SRE_SLOW (0 << 0)
Shouldn't the addition/change of #defines be done in patches that actually use them?
It would also help to include in the commit description what the impact of the change in values is. The description as written doesn't really give any indication on why the patch is important, which makes it hard to review.
g.
-- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. -- 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/
| |