<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 8, 2018 at 9:37 AM Martin Erik Werner <<a href="mailto:martinerikwerner.aac@gmail.com">martinerikwerner.aac@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2018-10-08 at 15:28 +0200, Sebastian Huber wrote:<br>
> Are the intentionally unused or is there some bug and the generic cache <br>
> manager doesn't use them accidentally?<br>
<br>
I think that when I added the range support to the or1k cache helpers I<br>
left the 1-line functions there since I was unaware that they would be<br>
completely non-visible to upper layers (right?). As far as I can tell<br>
they can be removed.<br></blockquote><div><br></div><div>I'm not pushing this change unless you are happy with it.  </div><div><br></div><div>So I am waiting for a clear yes/no. I was just fixing an obvious warning.</div><div><br></div><div>On a random or1k note, I managed to build everything with the new FSF</div><div>clean tools last week.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Whilst looking, I saw that something like this is also present in<br>
bsps/arm/shared/cache/cache-cp15.c, but<br>
CPU_CACHE_SUPPORT_PROVIDES_RANGE_FUNCTIONS is set via indirection<br>
there, so it might be a special case?<br></blockquote><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
Martin Erik Werner <<a href="mailto:martinerikwerner.aac@gmail.com" target="_blank">martinerikwerner.aac@gmail.com</a>><br>
ÅAC Microtec AB | Clyde Space Ltd.<br>
</blockquote></div></div>