Re: [PATCH v2 04/13] clk: davinci: psc-dm646x: use two lookup entries for the aemif clock

From: David Lechner
Date: Wed Jun 27 2018 - 12:55:01 EST


On 06/27/2018 05:26 AM, Sekhar Nori wrote:
On Wednesday 27 June 2018 03:33 PM, Bartosz Golaszewski wrote:
2018-06-27 12:01 GMT+02:00 Sekhar Nori <nsekhar@xxxxxx>:
On Tuesday 26 June 2018 02:55 PM, Bartosz Golaszewski wrote:
From: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>

We want to be able to get the clock both from the board file by its
con_id and from the aemif driver by dev_id.

Signed-off-by: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>

I see the DM365 and DM644x EVM board files getting reference to aemif
clock using just the con_id, but don't see that for DM646x. Am I
overlooking something?

Thanks,
Sekhar

It's being done indirectly from mach-davinci/aemif.c. I guess we can
remove it again after last patch.

I see. One way to avoid that would be to merge this into 11/13. I prefer
that, but it prevents clean separation of clock and machine patches. So
okay with current way too.

I think da850 has a con_id-only lookup that can be removed as well. I'm
in favor of doing another series to clean these up later.