Skip to content

Commit

Permalink
mtd: gpmi: do not set the default values for the extra clocks
Browse files Browse the repository at this point in the history
The default frequencies of the extra clocks are 200MHz.
The current code sets the extra clocks to 44.5MHz.

When i add the EDO feature to gpmi, i have to revert the extra clocks
to 200MHz.

So it is better that we do not set the default values for the extra
clocks. The driver runs well even when we do not set the default values for
extra clocks.

Signed-off-by: Huang Shijie <b32955@freescale.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
  • Loading branch information
Huang Shijie authored and David Woodhouse committed Sep 29, 2012
1 parent c50d35a commit e1ca95e
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions drivers/mtd/nand/gpmi-nand/gpmi-nand.c
Original file line number Diff line number Diff line change
Expand Up @@ -516,20 +516,15 @@ static int __devinit gpmi_get_clks(struct gpmi_nand_data *this)
r->clock[i] = clk;
}

if (GPMI_IS_MX6Q(this)) {
if (GPMI_IS_MX6Q(this))
/*
* Set the default values for the clocks in mx6q:
* The main clock(enfc) : 22MHz
* The others : 44.5MHz
* Set the default value for the gpmi clock in mx6q:
*
* These are just the default values. If you want to use
* the ONFI nand which is in the Synchronous Mode, you should
* change the clocks's frequencies as you need.
* If you want to use the ONFI nand which is in the
* Synchronous Mode, you should change the clock as you need.
*/
clk_set_rate(r->clock[0], 22000000);
for (i = 1; i < GPMI_CLK_MAX && r->clock[i]; i++)
clk_set_rate(r->clock[i], 44500000);
}

return 0;

err_clock:
Expand Down

0 comments on commit e1ca95e

Please sign in to comment.