On Thu, Jul 31, 2014 at 02:13:48PM +0200, Michal Simek wrote:

Mixing two drivers in the one file is not a good idea because with
more memory controllers it is just a mess and you are not able to
cover all cases.

Why is it a mess?

If this is just about providing uniq number we can easily extend
binding and provide that uniq value. That's remind me solution with
edac_mc_get_id() can caused that you won't have exact number all the
time - depends on driver loading (deferred probing too).

-ENOPARSE for this sentence.

One option via DT can be via aliases where you can easily specify
order. But all of these issues can be solved in follow-up patch.

Whatever you do, it should be designed cleanly and not introduce some
homegrown solution.

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--