fix: remove cache ops for coherent architectures

For cache alignment, PLATFORM_DCACHE_ALIGN is a better macro
than DCACHE_LINE_SIZE (for some compilations not even defined)

For CONFIG_INCOHERENT=0 compilation, it does not make any sense
to call writeback/invalidate operations

Signed-off-by: Marcin Szkudlinski <marcin.szkudlinski@intel.com>

Author: marcinszkudlinski

src/include/sof/coherent.h

@@ -17,7 +17,7 @@
 #include <sof/lib/memory.h>
 #include <sof/lib/cpu.h>
 
-#define __coherent __aligned(DCACHE_LINE_SIZE)
+#define __coherent __aligned(PLATFORM_DCACHE_ALIGN)
 
 /*
  * The coherent API allows optimized access to memory by multiple cores, using
@@ -332,29 +332,17 @@ static inline void __coherent_shared_thread(struct coherent *c, const size_t siz
 __must_check static inline struct coherent __sparse_cache *coherent_acquire(struct coherent *c,
 								     const size_t size)
 {
-	if (c->shared) {
-		struct coherent __sparse_cache *cc = uncache_to_cache(c);
-
+	if (c->shared)
 		c->key = k_spin_lock(&c->lock);
 
-		/* invalidate local copy */
-		dcache_invalidate_region(cc, size);
-	}
-
 	return (__sparse_force struct coherent __sparse_cache *)c;
 }
 
 static inline void coherent_release(struct coherent __sparse_cache *c,
 				    const size_t size)
 {
-	if (c->shared) {
-		struct coherent *uc = cache_to_uncache(c);
-
-		/* wtb and inv local data to coherent object */
-		dcache_writeback_invalidate_region(c, size);
-
-		k_spin_unlock(&uc->lock, uc->key);
-	}
+	if (c->shared)
+		k_spin_unlock(&c->lock, c->key);
 }
 
 static inline void *__coherent_init(size_t offset, const size_t size)