Fork us on GitHub Follow us on Facebook Follow us on Twitter

Changeset 8304889 in mainline


Ignore:
Timestamp:
2010-10-23T17:51:33Z (11 years ago)
Author:
Jiri Svoboda <jiri@…>
Branches:
lfn, master
Children:
d93aafed
Parents:
713a4b9
Message:

Get rid of Yoda comparisons in pci.c.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • uspace/drv/pciintel/pci.c

    r713a4b9 r8304889  
    6565        pci_dev_data_t *dev_data = (pci_dev_data_t *) dev->driver_data;
    6666       
    67         if (NULL == dev_data)
     67        if (dev_data == NULL)
    6868                return NULL;
    6969        return &dev_data->hw_resources;
     
    109109       
    110110        bus_data = (pci_bus_data_t *) malloc(sizeof(pci_bus_data_t));
    111         if (NULL != bus_data) {
     111        if (bus_data != NULL) {
    112112                memset(bus_data, 0, sizeof(pci_bus_data_t));
    113113                fibril_mutex_initialize(&bus_data->conf_mutex);
    114114        }
     115
    115116        return bus_data;
    116117}
     
    123124static void pci_conf_read(device_t *dev, int reg, uint8_t *buf, size_t len)
    124125{
    125         assert(NULL != dev->parent);
     126        assert(dev->parent != NULL);
    126127       
    127128        pci_dev_data_t *dev_data = (pci_dev_data_t *) dev->driver_data;
     
    153154static void pci_conf_write(device_t *dev, int reg, uint8_t *buf, size_t len)
    154155{
    155         assert(NULL != dev->parent);
     156        assert(dev->parent != NULL);
    156157       
    157158        pci_dev_data_t *dev_data = (pci_dev_data_t *) dev->driver_data;
     
    224225       
    225226        match_id = create_match_id();
    226         if (NULL != match_id) {
     227        if (match_id != NULL) {
    227228                asprintf(&match_id_str, "pci/ven=%04x&dev=%04x",
    228229                    dev_data->vendor_id, dev_data->device_id);
     
    230231                match_id->score = 90;
    231232                add_match_id(&dev->match_ids, match_id);
    232         }       
     233        }
     234
    233235        /* TODO add more ids (with subsys ids, using class id etc.) */
    234236}
     
    242244        size_t count = hw_res_list->count;
    243245       
    244         assert(NULL != hw_resources);
     246        assert(hw_resources != NULL);
    245247        assert(count < PCI_MAX_HW_RES);
    246248       
     
    354356{
    355357        uint8_t irq = pci_conf_read_8(dev, PCI_BRIDGE_INT_LINE);
    356         if (0xff != irq)
     358        if (irq != 0xff)
    357359                pci_add_interrupt(dev, irq);
    358360}
     
    415417                        create_pci_match_ids(dev);
    416418                       
    417                         if (EOK != child_device_register(dev, parent)) {
     419                        if (child_device_register(dev, parent) != EOK) {
    418420                                pci_clean_resource_list(dev);
    419421                                clean_match_ids(&dev->match_ids);
     
    424426                       
    425427                        if (header_type == PCI_HEADER_TYPE_BRIDGE ||
    426                             header_type == PCI_HEADER_TYPE_CARDBUS ) {
     428                            header_type == PCI_HEADER_TYPE_CARDBUS) {
    427429                                child_bus = pci_conf_read_8(dev,
    428430                                    PCI_BRIDGE_SEC_BUS_NUM);
    429431                                printf(NAME ": device is pci-to-pci bridge, "
    430432                                    "secondary bus number = %d.\n", bus_num);
    431                                 if(child_bus > bus_num)
     433                                if (child_bus > bus_num)
    432434                                        pci_bus_scan(parent, child_bus);
    433435                        }
     
    453455       
    454456        pci_bus_data_t *bus_data = create_pci_bus_data();
    455         if (NULL == bus_data) {
     457        if (bus_data == NULL) {
    456458                printf(NAME ": pci_add_device allocation failed.\n");
    457459                return ENOMEM;
     
    517519        pci_dev_data_t *res = (pci_dev_data_t *) malloc(sizeof(pci_dev_data_t));
    518520       
    519         if (NULL != res)
     521        if (res != NULL)
    520522                memset(res, 0, sizeof(pci_dev_data_t));
    521523        return res;
     
    531533void delete_pci_dev_data(pci_dev_data_t *d)
    532534{
    533         if (NULL != d) {
     535        if (d != NULL) {
    534536                clean_hw_resource_list(&d->hw_resources);
    535537                free(d);
     
    560562        pci_dev_data_t *dev_data = (pci_dev_data_t *) dev->driver_data;
    561563       
    562         if (NULL != dev_data->hw_resources.resources) {
     564        if (dev_data->hw_resources.resources != NULL) {
    563565                free(dev_data->hw_resources.resources);
    564566                dev_data->hw_resources.resources = NULL;
Note: See TracChangeset for help on using the changeset viewer.