amd rs780 gfx lane reversal patch

[She, Kerry]

Hi, all

Amd rs780 feature Lane reversal is tested.

Please see the attachment

Thanks

Regards

Kerry She

[Bao, Zheng]

Please add Signed-off-by line in the mail text instead of the
attachment.

feature Lane reversal is tested.

Signed-off-by: Kerry She

Index: src/southbridge/amd/rs780/rs780_gfx.c

===================================================================

--- src/southbridge/amd/rs780/rs780_gfx.c (revision 5690)

+++ src/southbridge/amd/rs780/rs780_gfx.c (working copy)

@@ -1180,7 +1180,9 @@

case 0:

/* step 1, lane reversal (only need if CMOS 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");

@@ -1196,8 +1198,10 @@

case 1:

/* step 1, lane reversal (only need if CMOS 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) */

________________________________

From: [e-mail]
[mailto:[e-mail]] On Behalf Of She, Kerry
Sent: Wednesday, August 11, 2010 11:53 AM
To: [e-mail]
Subject: [coreboot] amd rs780 gfx lane reversal patch

Hi, all

Amd rs780 feature Lane reversal is tested.

Please see the attachment

Thanks

Regards

Kerry She

[andy brown]

amd rs789 gfx feature Lane reversal is tested.
thanks.

Signed-off-by: Kerry She

On 8/12/10, Bao, Zheng  wrote:
> Please add Signed-off-by line in the mail text instead of the
> attachment.
>
> feature Lane reversal is tested.
>
> Signed-off-by: Kerry She
>
> Index: src/southbridge/amd/rs780/rs780_gfx.c
>
> ===================================================================
>
> --- src/southbridge/amd/rs780/rs780_gfx.c (revision 5690)
>
> +++ src/southbridge/amd/rs780/rs780_gfx.c (working copy)
>
> @@ -1180,7 +1180,9 @@
>   case 0:
>
>         /* step 1, lane reversal (only need if CMOS 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");
>
>
>
> @@ -1196,8 +1198,10 @@
>   case 1:
>
>         /* step 1, lane reversal (only need if CMOS 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) */
>
> From: [e-mail]
> [mailto:[e-mail]] On Behalf Of She, Kerry
> Sent: Wednesday, August 11, 2010 11:53 AM
> To: [e-mail]
> Subject: [coreboot] amd rs780 gfx lane reversal patch
>
> Hi, all
>
> Amd rs780 feature Lane reversal is tested.
>
> Please see the attachment
>
> Thanks
>
> Regards
>
> Kerry She

[Marc Jones]

On Wed, Aug 11, 2010 at 11:12 PM, andy brown  wrote:
> amd rs789 gfx feature Lane reversal is tested.
> thanks.
>
> Signed-off-by: Kerry She

revision 5690)
> > /* step 1, lane reversal (only need if CMOS option is
> > enabled) */
> >
> > if (cfg->gfx_lane_reversal) {

Change the comment from CMOS to build config option.

Acked-by: Marc Jones

Zheng, do you want to commit this?

Marc