Fix miscellaneous issues getting attributes from within HWPs

- Fix duplicate weak symbol issue when using range filter within multiple libs
- Fix FAPI target casting when calling HB macro with real target handle
- Fix FAPI namespacing when calling HB macro to bridge attribute

Change-Id: I6a187d1933aeb42e674e5236a81ae6145f3de583
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/352
Tested-by: Jenkins Server
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>

2 files changed, 7 insertions, 11 deletions

diff --git a/src/include/usr/hwpf/plat/fapiPlatAttributeService.H b/src/include/usr/hwpf/plat/fapiPlatAttributeService.H
index 3d38d0e79..e932bca6d 100755
--- a/src/include/usr/hwpf/plat/fapiPlatAttributeService.H
+++ b/src/include/usr/hwpf/plat/fapiPlatAttributeService.H
@@ -104,11 +104,11 @@ namespace platAttrSvc
     #define FAPI_PLAT_ATTR_SVC_GETMACRO_DIRECT(ID, PTARGET, VAL) \
         ( ((PTARGET) ? \
             static_cast<TARGETING::Target*>( \
-                static_cast<fapi::Target*>(PTARGET)->get()) \
+                static_cast<const fapi::Target*>(PTARGET)->get()) \
         : fapi::platAttrSvc::getSystemTarget())->tryGetAttr< \
             FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID>(VAL)) \
-                ? FAPI_RC_SUCCESS : fapi::platAttrSvc::createAttrAccessError( \
-                    FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID,ID,PTARGET)
+                ? fapi::FAPI_RC_SUCCESS : fapi::platAttrSvc::createAttrAccessError( \
+                    FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID,fapi::ID,PTARGET)
     /**
      *  @brief Macro which directly maps a FAPI request to set a platform
      *      attribute to the equivalent host boot request 
@@ -116,11 +116,11 @@ namespace platAttrSvc
     #define FAPI_PLAT_ATTR_SVC_SETMACRO_DIRECT(ID, PTARGET, VAL) \
         ( ((PTARGET) ? \
             static_cast<TARGETING::Target*>( \
-                static_cast<fapi::Target*>(PTARGET)->get()) \
+                static_cast<const fapi::Target*>(PTARGET)->get()) \
         : fapi::platAttrSvc::getSystemTarget())->trySetAttr< \
              FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID>(VAL)) \
-                ? FAPI_RC_SUCCESS : fapi::platAttrSvc::createAttrAccessError( \
-                    FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID,ID,PTARGET)
+                ? fapi::FAPI_RC_SUCCESS : fapi::platAttrSvc::createAttrAccessError( \
+                    FAPI_PLAT_ATTR_SVC_MACRO_DIRECT_FAPI_##ID,fapi::ID,PTARGET)
 }
 
 }
diff --git a/src/include/usr/targeting/iterators/rangefilter.H b/src/include/usr/targeting/iterators/rangefilter.H
index 21393bf2a..458bf3b9d 100644
--- a/src/include/usr/targeting/iterators/rangefilter.H
+++ b/src/include/usr/targeting/iterators/rangefilter.H
@@ -115,11 +115,7 @@ class RangeFilter
          *  @brief Dummy function used to implement ability to check if range 
          *      iterator is valid 
          */ 
-        operator fake_bool() const
-        {   
-            return (iv_current != iv_end) 
-                ? &RangeFilter::notComparable : NULL;
-        }
+        operator fake_bool() const;
 
         /**
          *  @brief Pre-increment operator which advances the range's iterator