Remove workaround for attribute override in old HDAT

Removing the use of the workaround that was in place to support
attribute overrides with the non-RHB (legacy) version of HDAT.

Change-Id: Iae0a8f124d56db9bf3172b1d4866e865cfd763d7
CQ: SW391403
RTC: 169478
CMVC-Coreq: 1024559
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41517
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>

src/usr/targeting/runtime/attrPlatOverride_rt.C

@@ -142,20 +142,6 @@ void applyTempOverrides()
             TRACFCOMP(g_trac_targeting, "Overrides found at %.16X", l_overAddr );
             l_overPtr = reinterpret_cast<uint8_t*>(l_overAddr);
         }
-        else
-        {
-            //@fixme-RTC:169478-Remove this workaround once HDAT+PHYP is ready
-            // grab the data we stashed at the end of the targeting data
-            l_overAddr = g_hostInterfaces
-              ->get_reserved_mem("ibm,hbrt-target-image",0);
-            if( l_overAddr != 0 )
-            {
-                l_overAddr += (1*MEGABYTE - 64*KILOBYTE);
-                TRACFCOMP(g_trac_targeting, "NULL from get_reserved_mem, using stashed value at %.llX instead", l_overAddr );
-                l_overPtr = reinterpret_cast<uint8_t*>(l_overAddr);
-                l_usingStash = true;
-            }
-        }
     }
 
     // Having no overrides is a normal thing