Re: [PATCH v6 1/6] platform/x86: intel_pmc_ipc: fix gcr offset

2017-04-06 Thread Rajneesh Bhardwaj
On Wed, Apr 05, 2017 at 03:54:19PM -0700, Kuppuswamy Sathyanarayanan wrote: > According to Broxton APL PMC spec, gcr mem region starts > at offset 0x1000 from ipc mem base address. In this driver, > PLAT_RESOURCE_GCR_OFFSET macro defines the offset of GCR > memory region from IPC mem region. So we

[PATCH v6 1/6] platform/x86: intel_pmc_ipc: fix gcr offset

2017-04-05 Thread Kuppuswamy Sathyanarayanan
According to Broxton APL PMC spec, gcr mem region starts at offset 0x1000 from ipc mem base address. In this driver, PLAT_RESOURCE_GCR_OFFSET macro defines the offset of GCR memory region from IPC mem region. So we should use 0x1000(4K) as GCR offset. But currently this driver uses 0x1008 as GCT of