You could "memoize" it in a class method, so it runs only once per application start. Take advantage of the fact that Rails is a long-running process.
# in club model
def self.nfl_logos
@nfl_logos ||= where(is_disabled: false).order(city: :asc)
end
Then refer to this as Club.nfl_logos in your controller or view, wherever you build this page element. It will only ever run once (per server instance/thread) and thereafter always return from the cached instance.
Another thing to remember is Turbolinks. You are using it, right? Elements like this that rarely change can be marked out as persistent between reloads, so they will be cached by the browser and never re-requested. Assuming your layout is properly divided into areas that change and areas that do not, this could take care of this problem for you as well.
Walter