[U-Boot] [PATCH 1/5] Tegra2: Add bitfield access macros

[Wolfgang Denk]

Dear Simon Glass,

In message <BANLkTimN4NVD43B2Le7_ci9Picjd6pmuXA at mail.gmail.com> you wrote:
> 
> But the FIELD is specified twice in the same statement. Can we
> therefore please take this a step further and write something like
> this?
> 
> clrsetfield_le32(&my_device->ctrl, FIELD, 6);

I consider this too specific on one side, and the additional effort
to write two arguments too less of an issue to introduce another
macro.


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
: ... and it's got weird formatting - Notepad, Write, Works  3  can't
: decipher it, and it's too big to go in DOS Edit. Help!
Install an operating system. :-)                  -- Tom Christiansen