[PATCH 02/03] ARM: shmobile: Use shmobile_clk_workaround() on Lager

[Magnus Damm]

On Thu, Mar 13, 2014 at 10:23 AM, Simon Horman <horms at verge.net.au> wrote:
> On Thu, Mar 13, 2014 at 08:59:45AM +0900, Simon Horman wrote:
>> On Thu, Mar 13, 2014 at 08:36:26AM +0900, Magnus Damm wrote:
>> > From: Magnus Damm <damm at opensource.se>
>> >
>> > Convert the Lager DT reference code to use the newly introduced
>> > function shmobile_clk_workaround().
>> >
>> > Signed-off-by: Magnus Damm <damm at opensource.se>
>> > ---
>> >
>> >  arch/arm/mach-shmobile/board-lager-reference.c |   65 +++++++++---------------
>> >  1 file changed, 25 insertions(+), 40 deletions(-)
>> >
>> > --- 0001/arch/arm/mach-shmobile/board-lager-reference.c
>> > +++ work/arch/arm/mach-shmobile/board-lager-reference.c     2014-02-24 16:17:32.000000000 +0900
>> > @@ -18,12 +18,11 @@
>> >   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
>> >   */
>> >
>> > -#include <linux/clk.h>
>> > -#include <linux/clkdev.h>
>> >  #include <linux/dma-mapping.h>
>> >  #include <linux/init.h>
>> >  #include <linux/of_platform.h>
>> >  #include <linux/platform_data/rcar-du.h>
>> > +#include <mach/clock.h>
>> >  #include <mach/common.h>
>> >  #include <mach/irqs.h>
>> >  #include <mach/rcar-gen2.h>
>> > @@ -86,46 +85,32 @@ static void __init lager_add_du_device(v
>> >     platform_device_register_full(&info);
>> >  }
>> >
>> > +/*
>> > + * This is a really crude hack to provide clkdev support to platform
>> > + * devices until they get moved to DT.
>> > + */
>> > +static const struct clk_name clk_names[] = {
>>
>> Should this be annotated as __initconst?
>> Likewise for the Koelsch variant of this patch.
>
> Btw, I'm happy to handle this as an incremental patch if you like.

Yes, that would be greatly appreciated!

Thanks,

/ magnus