Hi Stanimir,
Thanks for your review.
On 2018-07-25 15:06, Stanimir Varbanov wrote:
Hi Vikash,
On 07/04/2018 10:06 PM, Vikash Garodia wrote:
<snip>
#define VENUS_PAS_ID 9
-#define VENUS_FW_MEM_SIZE (6 * SZ_1M)
+#define VENUS_FW_MEM_SIZE (5 * SZ_1M)
This change should be subject to a separate patch.
Ok.
<snip>
+ writel_relaxed(reg, reg_base + WRAPPER_A9SS_SW_RESET);
+
+ /* Make sure reset is asserted before the mapping is removed */
+ mb();
mb() is used for compiler barrier as far as I know, isn't better to
just
read the register and keep the comment?
mb() can ensure that the instructions are ordered. This is needed to
ensure
reset before the mapping is removed.
Thanks,
Vikash
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
