switch(board_idx) {
        case chip_5080:
                hpriv->ops = &mv5xxx_ops;
 -              hp_flags |= MV_HP_50XX;
 +              hp_flags |= MV_HP_GEN_I;
  
-               switch (rev_id) {
+               switch (pdev->revision) {
                case 0x1:
                        hp_flags |= MV_HP_ERRATA_50XXB0;
                        break;
        case chip_504x:
        case chip_508x:
                hpriv->ops = &mv5xxx_ops;
 -              hp_flags |= MV_HP_50XX;
 +              hp_flags |= MV_HP_GEN_I;
  
-               switch (rev_id) {
+               switch (pdev->revision) {
                case 0x0:
                        hp_flags |= MV_HP_ERRATA_50XXB0;
                        break;
        case chip_604x:
        case chip_608x:
                hpriv->ops = &mv6xxx_ops;
 +              hp_flags |= MV_HP_GEN_II;
  
-               switch (rev_id) {
+               switch (pdev->revision) {
                case 0x7:
                        hp_flags |= MV_HP_ERRATA_60X1B2;
                        break;
        case chip_7042:
        case chip_6042:
                hpriv->ops = &mv6xxx_ops;
 -
                hp_flags |= MV_HP_GEN_IIE;
  
-               switch (rev_id) {
+               switch (pdev->revision) {
                case 0x0:
                        hp_flags |= MV_HP_ERRATA_XX42A0;
                        break;