[LinuxBIOS] Correct function?

Hello,
Pardon my C, can you tell me if this function is setup correctly and  
going to return the correct value to the register??

/* This function sets the DRAM Controller Mode Register to NOP. */
static long set_drc_mode_nop(const struct mem_controller *ctrl)
{
       pci_write_config32(ctrl->f0, DRC, (1<<4));
    return 0;
}

Thanks - Joe