VirtualBox

Changeset 6318

Show
Ignore:
Timestamp:
01/10/08 09:25:28 (11 months ago)
Author:
vboxsync
Message:

PDMDevHlp*, cosmetics

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/src/VBox/Devices/Storage/fdc.c

    r5999 r6318  
    709709    FLOPPY_DPRINTF("Reset interrupt\n"); 
    710710#ifdef VBOX 
    711     fdctrl->pDevIns->pDevHlp->pfnISASetIrq (fdctrl->pDevIns, 
    712                                             fdctrl->irq_lvl, 0); 
     711    PDMDevHlpISASetIrq (fdctrl->pDevIns, fdctrl->irq_lvl, 0); 
    713712#else 
    714713    pic_set_irq(fdctrl->irq_lvl, 0); 
     
    721720    if (~(fdctrl->state & FD_CTRL_INTR)) { 
    722721#ifdef VBOX 
    723         fdctrl->pDevIns->pDevHlp->pfnISASetIrq (fdctrl->pDevIns, 
    724                                                 fdctrl->irq_lvl, 1); 
     722        PDMDevHlpISASetIrq (fdctrl->pDevIns, fdctrl->irq_lvl, 1); 
    725723#else 
    726724        pic_set_irq(fdctrl->irq_lvl, 1); 
     
    997995    if (fdctrl->state & FD_CTRL_BUSY) { 
    998996#ifdef VBOX 
    999         fdctrl->pDevIns->pDevHlp->pfnDMASetDREQ (fdctrl->pDevIns, 
    1000                                                  fdctrl->dma_chann, 
    1001                                                  0); 
     997        PDMDevHlpDMASetDREQ (fdctrl->pDevIns, fdctrl->dma_chann, 0); 
    1002998#else 
    1003999        DMA_release_DREQ(fdctrl->dma_chann); 
     
    10811077        dma_mode = DMA_get_channel_mode(fdctrl->dma_chann); 
    10821078#else 
    1083         dma_mode = fdctrl->pDevIns->pDevHlp->pfnDMAGetChannelMode ( 
    1084             fdctrl->pDevIns, 
    1085             fdctrl->dma_chann 
    1086             ); 
     1079        dma_mode = PDMDevHlpDMAGetChannelMode (fdctrl->pDevIns, fdctrl->dma_chann); 
    10871080#endif 
    10881081        dma_mode = (dma_mode >> 2) & 3; 
     
    11041097            DMA_schedule(fdctrl->dma_chann); 
    11051098#else 
    1106             fdctrl->pDevIns->pDevHlp->pfnDMASetDREQ (fdctrl->pDevIns, 
    1107                                                      fdctrl->dma_chann, 
    1108                                                      1); 
    1109             fdctrl->pDevIns->pDevHlp->pfnDMASchedule (fdctrl->pDevIns); 
     1099            PDMDevHlpDMASetDREQ (fdctrl->pDevIns, fdctrl->dma_chann, 1); 
     1100            PDMDevHlpDMASchedule (fdctrl->pDevIns); 
    11101101#endif 
    11111102            return; 
     
    12481239            { 
    12491240                uint32_t read; 
    1250                 int rc = fdctrl->pDevIns->pDevHlp->pfnDMAWriteMemory( 
    1251                     fdctrl->pDevIns, 
    1252                     nchan, 
    1253                     fdctrl->fifo + rel_pos, 
    1254                     fdctrl->data_pos, 
    1255                     len, 
    1256                     &read); 
     1241                int rc = PDMDevHlpDMAWriteMemory(fdctrl->pDevIns, nchan, 
     1242                                                 fdctrl->fifo + rel_pos, 
     1243                                                 fdctrl->data_pos, 
     1244                                                 len, &read); 
    12571245                dump (fdctrl->fifo + rel_pos, len); 
    1258                 AssertMsgRC (rc, 
    1259                              ("DMAWriteMemory -> %Vrc\n", rc)); 
     1246                AssertMsgRC (rc, ("DMAWriteMemory -> %Vrc\n", rc)); 
    12601247            } 
    12611248#else 
     
    12711258            { 
    12721259                uint32_t written; 
    1273                 int rc = fdctrl->pDevIns->pDevHlp->pfnDMAReadMemory( 
    1274                     fdctrl->pDevIns, 
    1275                     nchan, 
    1276                     fdctrl->fifo + rel_pos, 
    1277                     fdctrl->data_pos, 
    1278                     len, 
    1279                     &written); 
    1280                 AssertMsgRC (rc, 
    1281                              ("DMAReadMemory -> %Vrc\n", rc)); 
     1260                int rc = PDMDevHlpDMAReadMemory(fdctrl->pDevIns, nchan, 
     1261                                                fdctrl->fifo + rel_pos, 
     1262                                                fdctrl->data_pos, 
     1263                                                len, &written); 
     1264                AssertMsgRC (rc, ("DMAReadMemory -> %Vrc\n", rc)); 
    12821265            } 
    12831266#else 
     
    13241307                uint32_t read; 
    13251308 
    1326                 rc = fdctrl->pDevIns->pDevHlp->pfnDMAReadMemory ( 
    1327                     fdctrl->pDevIns, 
    1328                     nchan, 
    1329                     tmpbuf, 
    1330                     fdctrl->data_pos, 
    1331                     len, 
    1332                     &read); 
    1333                 AssertMsg (VBOX_SUCCESS (rc), 
    1334                            ("DMAReadMemory -> %Vrc\n", rc)); 
     1309                rc = PDMDevHlpDMAReadMemory (fdctrl->pDevIns, nchan, tmpbuf, 
     1310                                             fdctrl->data_pos, len, &read); 
     1311                AssertMsg (VBOX_SUCCESS (rc), ("DMAReadMemory -> %Vrc\n", rc)); 
    13351312#else 
    13361313                DMA_read_memory (nchan, tmpbuf, fdctrl->data_pos, len); 
     
    14841461            /* READ commands */ 
    14851462#ifdef VBOX 
    1486             fdctrl->pDevIns->pDevHlp->pfnPhysWrite ( 
    1487                 fdctrl->pDevIns, 
    1488                 addr + fdctrl->data_pos, 
    1489                 fdctrl->fifo + rel_pos, 
    1490                 len); 
     1463            PDMDevHlpPhysWrite (fdctrl->pDevIns, addr + fdctrl->data_pos, 
     1464                                fdctrl->fifo + rel_pos, len); 
    14911465#else 
    14921466            cpu_physical_memory_write(addr + fdctrl->data_pos, 
     
    15001474                int rc; 
    15011475 
    1502                 fdctrl->pDevIns->pDevHlp->pfnPhysRead ( 
    1503                     fdctrl->pDevIns, 
    1504                     addr + fdctrl->data_pos, 
    1505                     fdctrl->fifo + rel_pos, 
    1506                     len); 
     1476                PDMDevHlpPhysRead (fdctrl->pDevIns, addr + fdctrl->data_pos, 
     1477                                   fdctrl->fifo + rel_pos, len); 
    15071478 
    15081479                cur_drv->Led.Asserted.s.fWriting 
     
    15441515                int ret; 
    15451516#ifdef VBOX 
    1546                 fdctrl->pDevIns->pDevHlp->pfnPhysRead ( 
    1547                     fdctrl->pDevIns, 
    1548                     addr + fdctrl->data_pos, 
    1549                     tmpbuf, 
    1550                     len); 
     1517                PDMDevHlpPhysRead (fdctrl->pDevIns, addr + fdctrl->data_pos, 
     1518                                   tmpbuf, len); 
    15511519#else 
    15521520                cpu_physical_memory_read(addr + fdctrl->data_pos, 
     
    28032771     * Validate configuration. 
    28042772     */ 
    2805     if (!CFGMR3AreValuesValid(pCfgHandle, "IRQ\0DMA\0MemMapped\0IOBase\0")) { 
     2773    if (!CFGMR3AreValuesValid(pCfgHandle, "IRQ\0DMA\0MemMapped\0IOBase\0")) 
    28062774        return VERR_PDM_DEVINS_UNKNOWN_CFG_VALUES; 
    2807     } 
    28082775 
    28092776    /* 
     
    28112778     */ 
    28122779    rc = CFGMR3QueryU8 (pCfgHandle, "IRQ", &irq_lvl); 
    2813     if (rc == VERR_CFGM_VALUE_NOT_FOUND) { 
     2780    if (rc == VERR_CFGM_VALUE_NOT_FOUND) 
    28142781        irq_lvl = 6; 
    2815     } 
    2816     else if (VBOX_FAILURE (rc))
     2782    else if (VBOX_FAILURE (rc)) 
     2783   
    28172784        AssertMsgFailed (("Configuration error: Failed to read U8 IRQ, rc=%Vrc\n", rc)); 
    28182785        return rc; 
     
    28202787 
    28212788    rc = CFGMR3QueryU8 (pCfgHandle, "DMA", &dma_chann); 
    2822     if (rc == VERR_CFGM_VALUE_NOT_FOUND) { 
     2789    if (rc == VERR_CFGM_VALUE_NOT_FOUND) 
    28232790        dma_chann = 2; 
    2824     } 
    2825     else if (VBOX_FAILURE (rc))
     2791    else if (VBOX_FAILURE (rc)) 
     2792   
    28262793        AssertMsgFailed (("Configuration error: Failed to read U8 DMA, rc=%Vrc\n", rc)); 
    28272794        return rc; 
     
    28292796 
    28302797    rc = CFGMR3QueryU16 (pCfgHandle, "IOBase", &io_base); 
    2831     if (rc == VERR_CFGM_VALUE_NOT_FOUND) { 
     2798    if (rc == VERR_CFGM_VALUE_NOT_FOUND) 
    28322799        io_base = 0x3f0; 
    2833     } 
    2834     else if (VBOX_FAILURE (rc))
     2800    else if (VBOX_FAILURE (rc)) 
     2801   
    28352802        AssertMsgFailed (("Configuration error: Failed to read U16 IOBase, rc=%Vrc\n", rc)); 
    28362803        return rc; 
     
    28382805 
    28392806    rc = CFGMR3QueryBool (pCfgHandle, "MemMapped", &mem_mapped); 
    2840     if (rc == VERR_CFGM_VALUE_NOT_FOUND) { 
     2807    if (rc == VERR_CFGM_VALUE_NOT_FOUND) 
    28412808        mem_mapped = false; 
    2842     } 
    2843     else if (VBOX_FAILURE (rc))
     2809    else if (VBOX_FAILURE (rc)) 
     2810   
    28442811        AssertMsgFailed (("Configuration error: Failed to read bool value MemMapped rc=%Vrc\n", rc)); 
    28452812        return rc; 
     
    28602827    fdctrl->ILeds.pfnQueryStatusLed = fdcStatusQueryStatusLed; 
    28612828 
    2862     for (i = 0; i < ELEMENTS(fdctrl->drives); ++i) { 
     2829    for (i = 0; i < ELEMENTS(fdctrl->drives); ++i) 
     2830    { 
    28632831        fdrive_t *drv = &fdctrl->drives[i]; 
    28642832 
     
    28762844     */ 
    28772845    rc = PDMDevHlpTMTimerCreate(pDevIns, TMCLOCK_VIRTUAL, fdc_timer, "FDC Timer", &fdctrl->result_timer); 
    2878     if (VBOX_FAILURE (rc)) { 
     2846    if (VBOX_FAILURE (rc)) 
    28792847        return rc; 
    2880     } 
    28812848 
    28822849    /* 
    28832850     * Register DMA channel. 
    28842851     */ 
    2885     if (fdctrl->dma_chann != 0xff) { 
     2852    if (fdctrl->dma_chann != 0xff) 
     2853    { 
    28862854        fdctrl->dma_en = 1; 
    2887         rc = pDevIns->pDevHlp->pfnDMARegister ( 
    2888             pDevIns, 
    2889             dma_chann, 
    2890             &fdctrl_transfer_handler, 
    2891             fdctrl); 
    2892         if (VBOX_FAILURE (rc)) { 
     2855        rc = PDMDevHlpDMARegister (pDevIns, dma_chann, &fdctrl_transfer_handler, fdctrl); 
     2856        if (VBOX_FAILURE (rc)) 
    28932857            return rc; 
    2894        
    2895     } else { 
     2858   
     2859    else 
    28962860        fdctrl->dma_en = 0; 
    2897     } 
    28982861 
    28992862    /* 
    29002863     * IO / MMIO. 
    29012864     */ 
    2902     if (mem_mapped) { 
     2865    if (mem_mapped) 
     2866    { 
    29032867        AssertMsgFailed (("Memory mapped floppy not support by now\n")); 
    29042868        return VERR_NOT_SUPPORTED; 
     
    29082872        cpu_register_physical_memory(base, 0x08, io_mem); 
    29092873#endif 
    2910     } else { 
    2911         rc = pDevIns->pDevHlp->pfnIOPortRegister ( 
    2912             pDevIns, 
    2913             io_base + 0x1, 
    2914             5, 
    2915             fdctrl, 
    2916             fdc_io_write, 
    2917             fdc_io_read, 
    2918             NULL, NULL, 
    2919             "FDC#1" 
    2920             ); 
    2921         if (VBOX_FAILURE (rc)) { 
     2874    } 
     2875    else 
     2876    { 
     2877        rc = PDMDevHlpIOPortRegister (pDevIns, io_base + 0x1, 5, fdctrl, 
     2878                                      fdc_io_write, fdc_io_read, NULL, NULL, "FDC#1"); 
     2879        if (VBOX_FAILURE (rc)) 
    29222880            return rc; 
    2923         } 
    2924  
    2925         rc = pDevIns->pDevHlp->pfnIOPortRegister ( 
    2926             pDevIns, 
    2927             io_base + 0x7, 
    2928             1, 
    2929             fdctrl, 
    2930             fdc_io_write, 
    2931             fdc_io_read, 
    2932             NULL, NULL, 
    2933             "FDC#2" 
    2934             ); 
    2935         if (VBOX_FAILURE (rc)) { 
     2881 
     2882        rc = PDMDevHlpIOPortRegister (pDevIns, io_base + 0x7, 1, fdctrl, 
     2883                                      fdc_io_write, fdc_io_read, NULL, NULL, "FDC#2"); 
     2884        if (VBOX_FAILURE (rc)) 
    29362885            return rc; 
    2937         } 
    29382886    } 
    29392887 
     
    29412889     * Register the saved state data unit. 
    29422890     */ 
    2943     rc = pDevIns->pDevHlp->pfnSSMRegister ( 
    2944         pDevIns,                /* pDevIns */ 
    2945         pDevIns->pDevReg->szDeviceName, /* pszName */ 
    2946         iInstance,              /* u32Instance */ 
    2947         1                       /* u32Version */, 
    2948         sizeof (*fdctrl),       /* cbGuess */ 
    2949         NULL,                   /* pfnSavePrep */ 
    2950         SaveExec,               /* pfnSaveExec */ 
    2951         NULL,                   /* pfnSaveDone */ 
    2952         NULL,                   /* pfnLoadPrep */ 
    2953         LoadExec,               /* pfnLoadExec */ 
    2954         NULL                    /* pfnLoadDone */ 
    2955         ); 
     2891    rc = PDMDevHlpSSMRegister (pDevIns, pDevIns->pDevReg->szDeviceName, iInstance, 1, sizeof(*fdctrl), 
     2892                               NULL, SaveExec, NULL, NULL, LoadExec, NULL); 
    29562893    if (VBOX_FAILURE(rc)) 
    29572894        return rc; 
     
    29742911     * Initialize drives. 
    29752912     */ 
    2976     for (i = 0; i < ELEMENTS(fdctrl->drives); i++) { 
     2913    for (i = 0; i < ELEMENTS(fdctrl->drives); i++) 
     2914    { 
    29772915        fdrive_t *drv = &fdctrl->drives[i]; 
    29782916        rc = fdConfig (drv, pDevIns); 
    29792917        if (    VBOX_FAILURE (rc) 
    2980             &&  rc != VERR_PDM_NO_ATTACHED_DRIVER) { 
     2918            &&  rc != VERR_PDM_NO_ATTACHED_DRIVER) 
     2919        { 
    29812920            AssertMsgFailed (("Configuration error: failed to configure drive %d, rc=%Vrc\n", rc)); 
    29822921            return rc; 
     
    29872926    fdctrl->state = FD_CTRL_ACTIVE; 
    29882927 
    2989     for (i = 0; i < ELEMENTS(fdctrl->drives); i++) { 
     2928    for (i = 0; i < ELEMENTS(fdctrl->drives); i++) 
    29902929        fd_revalidate(&fdctrl->drives[i]); 
    2991     } 
    29922930 
    29932931    return VINF_SUCCESS; 

© 2008 Sun Microsystems, Inc.
ContactPrivacy policy