View Change

3 comments:

• File src/northbridge/intel/i945/raminit_i945gc.c:

  • Patch Set #19, Line 844: int idx0, dual_channel;

    If you do the renaming to idx0 in a separate commit, the diff would be smaller?

  • Patch Set #19, Line 854:

    Fix this before to just use one space?

  • Patch Set #19, Line 892: sdram_write_slew_rates(G7SRPUT, slew_group_lookup(dual_channel, idx1 * 8 + 6));

    Why does dual channel work with the mobile chipset? Any idea, why no separate idx1 is needed there?

To view, visit change 27590. To unsubscribe, or for help writing mail filters, visit settings.