[U-Boot] [PATCH] Add Boundary Devices Nitrogen6X boards
Wolfgang Denk
wd at denx.de
Wed Feb 20 15:20:41 CET 2013
Dear Eric Nelson,
In message <1361305248-24598-1-git-send-email-eric.nelson at boundarydevices.com> you wrote:
> +static int detect_hdmi(struct display_info_t const *dev)
> +{
> + return __raw_readb(HDMI_ARB_BASE_ADDR+HDMI_PHY_STAT0) & HDMI_PHY_HPD;
...
> + reg = __raw_readb(
> + HDMI_ARB_BASE_ADDR
> + +HDMI_PHY_CONF0);
etc. etc.
NAK!! We do not allow device access through base address plus offset
notation. Please use a proper C Struct to access thes device.
Also, are you absolutely sure that the __raw_* accessors are the right
thing to use here? Are you really sure no memory barriers etc. are
needed here?
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
The heart is not a logical organ.
-- Dr. Janet Wallace, "The Deadly Years", stardate 3479.4
More information about the U-Boot
mailing list