It should be Signed-off-by: Kerry She <[email protected]>
> -----Original Message----- > From: [email protected] [mailto:[email protected]] > On Behalf Of repository service > Sent: Tuesday, August 17, 2010 10:15 AM > To: [email protected] > Subject: [coreboot] [commit] r5703 - trunk/src/southbridge/amd/rs780 > > Author: zbao > Date: Tue Aug 17 04:14:53 2010 > New Revision: 5703 > URL: https://tracker.coreboot.org/trac/coreboot/changeset/5703 > > Log: > Feature of lane reversal of AMD RS780 is tested. > > Signed-off-by: Zheng Bao <[email protected]> > Acked-by: Marc Jones <[email protected]> > > Modified: > trunk/src/southbridge/amd/rs780/rs780_gfx.c > > Modified: trunk/src/southbridge/amd/rs780/rs780_gfx.c > ======================================================================== == > ==== > --- trunk/src/southbridge/amd/rs780/rs780_gfx.c Mon Aug 16 22:00:49 2010 > (r5702) > +++ trunk/src/southbridge/amd/rs780/rs780_gfx.c Tue Aug 17 04:14:53 2010 > (r5703) > @@ -1178,9 +1178,11 @@ > /* Single-port/Dual-port configureation. */ > switch (cfg->gfx_dual_slot) { > case 0: > - /* step 1, lane reversal (only need if CMOS option is enabled) > */ > + /* step 1, lane reversal (only need if build config option is > enabled) */ > if (cfg->gfx_lane_reversal) { > + set_nbmisc_enable_bits(nb_dev, 0x36, 1 << 31, 1 << 31); > set_nbmisc_enable_bits(nb_dev, 0x33, 1 << 2, 1 << 2); > + set_nbmisc_enable_bits(nb_dev, 0x36, 1 << 31, 0 << 31); > } > printk(BIOS_DEBUG, "rs780_gfx_init step1.\n"); > > @@ -1194,10 +1196,12 @@ > > break; > case 1: > - /* step 1, lane reversal (only need if CMOS option is enabled) > */ > + /* step 1, lane reversal (only need if build config option is > enabled) */ > if (cfg->gfx_lane_reversal) { > + set_nbmisc_enable_bits(nb_dev, 0x36, 1 << 31, 1 << 31); > set_nbmisc_enable_bits(nb_dev, 0x33, 1 << 2, 1 << 2); > set_nbmisc_enable_bits(nb_dev, 0x33, 1 << 3, 1 << 3); > + set_nbmisc_enable_bits(nb_dev, 0x36, 1 << 31, 0 << 31); > } > printk(BIOS_DEBUG, "rs780_gfx_init step1.\n"); > /* step 1.1, dual-slot gfx configuration (only need if CMOS > option is enabled) */ > > -- > coreboot mailing list: [email protected] > http://www.coreboot.org/mailman/listinfo/coreboot -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
