[U-Boot] [PATCH 3/5] arm:imx-common introduce a new function to set gpr

Wolfgang Denk wd at denx.de
Sat Nov 1 14:44:50 CET 2014


Dear Peng Fan,

In message <1414808359-4750-4-git-send-email-Peng.Fan at freescale.com> you wrote:
> Add a new function mxc_iomux_set_gpr_register to
> set the iomux gpr register.
> 
> 32-bit general purpose registers according to SoC
> requirements for any usage.
...
> +/*
> + * Configure the IOMUX General Purpose Registers.
> + *
> + * @group, which gpr register to configure.
> + * @start_bit, the first bit to set
> + * @num_bits, how many bits to set
> + * @value, the value will be set to [start_bits...start_bits+num_bits-1]
> + */

Can you please rework the API and use a mask instead of counting
bits?  Relying on bit numbers is error prone and inherently
non-portable.

Use the standard register access macros (like setbits, clrbits,
clrsetbits) as example, please.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
All I ask is a chance to prove that money can't make me happy.


More information about the U-Boot mailing list