Changeset 3f03199 in mainline for uspace/drv/bus


Ignore:
Timestamp:
2013-09-15T06:33:53Z (12 years ago)
Author:
Jan Vesely <jano.vesely@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
9348862
Parents:
dd7078c (diff), 1c0cef0 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.
Message:

Merge mainline changes.

Major conflicts in USB HC drivers.
Compiles and UHCI works (qemu).
OHCI has device remove problems.

Location:
uspace/drv/bus
Files:
21 edited

Legend:

Unmodified
Added
Removed

  • uspace/drv/bus/isa/i8237.c

    rdd7078c r3f03199  
    279279 *
    280280 * @return Error code.
     281 *
    281282 */
    282283static inline int dma_controller_init(dma_controller_t *controller)

  • uspace/drv/bus/isa/isa.c

    rdd7078c r3f03199  
    6565#include <ddf/log.h>
    6666#include <ops/hw_res.h>
     67#include <ops/pio_window.h>
    6768
    6869#include <device/hw_res.h>
     70#include <device/pio_window.h>
    6971
    7072#include "i8237.h"
    … …  
    7981        ddf_dev_t *dev;
    8082        ddf_fun_t *fctl;
     83        pio_window_t pio_win;
    8184        list_t functions;
    8285} isa_bus_t;
    … …  
    102105}
    103106
    104 static hw_resource_list_t *isa_get_fun_resources(ddf_fun_t *fnode)
    105 {
    106         isa_fun_t *isa = isa_fun(fnode);
    107         assert(isa);
    108 
    109         return &isa->hw_resources;
     107static hw_resource_list_t *isa_fun_get_resources(ddf_fun_t *fnode)
     108{
     109        isa_fun_t *fun = isa_fun(fnode);
     110        assert(fun);
     111
     112        return &fun->hw_resources;
    110113}
    111114
    … …  
    114117        /* This is an old ugly way, copied from pci driver */
    115118        assert(fnode);
    116         isa_fun_t *isa = isa_fun(fnode);
    117         assert(isa);
     119        isa_fun_t *fun = isa_fun(fnode);
     120        assert(fun);
    118121
    119122        sysarg_t apic;
    … …  
    131134                return false;
    132135
    133         const hw_resource_list_t *res = &isa->hw_resources;
     136        const hw_resource_list_t *res = &fun->hw_resources;
    134137        assert(res);
    135138        for (size_t i = 0; i < res->count; ++i) {
    … …  
    157160{
    158161        assert(fnode);
    159         isa_fun_t *isa = isa_fun(fnode);
    160         assert(isa);
    161         const hw_resource_list_t *res = &isa->hw_resources;
     162        isa_fun_t *fun = isa_fun(fnode);
     163        assert(fun);
     164        const hw_resource_list_t *res = &fun->hw_resources;
    162165        assert(res);
    163166
    … …  
    180183        assert(size);
    181184        assert(fnode);
    182         isa_fun_t *isa = isa_fun(fnode);
    183         assert(isa);
    184         const hw_resource_list_t *res = &isa->hw_resources;
     185        isa_fun_t *fun = isa_fun(fnode);
     186        assert(fun);
     187        const hw_resource_list_t *res = &fun->hw_resources;
    185188        assert(res);
    186189
    … …  
    199202
    200203static hw_res_ops_t isa_fun_hw_res_ops = {
    201         .get_resource_list = isa_get_fun_resources,
     204        .get_resource_list = isa_fun_get_resources,
    202205        .enable_interrupt = isa_fun_enable_interrupt,
    203206        .dma_channel_setup = isa_fun_setup_dma,
    … …  
    205208};
    206209
     210static pio_window_t *isa_fun_get_pio_window(ddf_fun_t *fnode)
     211{
     212        ddf_dev_t *dev = ddf_fun_get_dev(fnode);
     213        isa_bus_t *isa = isa_bus(dev);
     214        assert(isa);
     215
     216        return &isa->pio_win;
     217}
     218
     219static pio_window_ops_t isa_fun_pio_window_ops = {
     220        .get_pio_window = isa_fun_get_pio_window
     221};
     222
    207223static ddf_dev_ops_t isa_fun_ops= {
    208224        .interfaces[HW_RES_DEV_IFACE] = &isa_fun_hw_res_ops,
     225        .interfaces[PIO_WINDOW_DEV_IFACE] = &isa_fun_pio_window_ops,
    209226};
    210227
    … …  
    405422        hw_resource_t *resources = fun->hw_resources.resources;
    406423
     424        isa_bus_t *isa = isa_bus(ddf_fun_get_dev(fun->fnode));
     425
    407426        if (count < ISA_MAX_HW_RES) {
    408427                resources[count].type = IO_RANGE;
    409428                resources[count].res.io_range.address = addr;
     429                resources[count].res.io_range.address += isa->pio_win.io.base;
    410430                resources[count].res.io_range.size = len;
     431                resources[count].res.io_range.relative = false;
    411432                resources[count].res.io_range.endianness = LITTLE_ENDIAN;
    412433
    … …  
    604625static int isa_dev_add(ddf_dev_t *dev)
    605626{
     627        async_sess_t *sess;
     628        int rc;
     629
    606630        ddf_msg(LVL_DEBUG, "isa_dev_add, device handle = %d",
    607631            (int) ddf_dev_get_handle(dev));
    … …  
    614638        isa->dev = dev;
    615639        list_initialize(&isa->functions);
     640
     641        sess = ddf_dev_parent_sess_create(dev, EXCHANGE_SERIALIZE);
     642        if (sess == NULL) {
     643                ddf_msg(LVL_ERROR, "isa_dev_add failed to connect to the "
     644                    "parent driver.");
     645                return ENOENT;
     646        }
     647
     648        rc = pio_window_get(sess, &isa->pio_win);
     649        if (rc != EOK) {
     650                ddf_msg(LVL_ERROR, "isa_dev_add failed to get PIO window "
     651                    "for the device.");
     652                return rc;
     653        }       
    616654
    617655        /* Make the bus device more visible. Does not do anything. */

  • uspace/drv/bus/pci/pciintel/pci.c

    rdd7078c r3f03199  
    5757#include <ops/hw_res.h>
    5858#include <device/hw_res.h>
     59#include <ops/pio_window.h>
     60#include <device/pio_window.h>
    5961#include <ddi.h>
    6062#include <pci_dev_iface.h>
    … …  
    141143}
    142144
     145static pio_window_t *pciintel_get_pio_window(ddf_fun_t *fnode)
     146{
     147        pci_fun_t *fun = pci_fun(fnode);
     148       
     149        if (fun == NULL)
     150                return NULL;
     151        return &fun->pio_window;
     152}
     153
     154
    143155static int pci_config_space_write_32(ddf_fun_t *fun, uint32_t address,
    144156    uint32_t data)
    … …  
    198210        .get_resource_list = &pciintel_get_resources,
    199211        .enable_interrupt = &pciintel_enable_interrupt,
     212};
     213
     214static pio_window_ops_t pciintel_pio_window_ops = {
     215        .get_pio_window = &pciintel_get_pio_window
    200216};
    201217
    … …  
    211227static ddf_dev_ops_t pci_fun_ops = {
    212228        .interfaces[HW_RES_DEV_IFACE] = &pciintel_hw_res_ops,
     229        .interfaces[PIO_WINDOW_DEV_IFACE] = &pciintel_pio_window_ops,
    213230        .interfaces[PCI_DEV_IFACE] = &pci_dev_ops
    214231};
    … …  
    233250static void pci_conf_read(pci_fun_t *fun, int reg, uint8_t *buf, size_t len)
    234251{
     252        const uint32_t conf_addr = CONF_ADDR(fun->bus, fun->dev, fun->fn, reg);
    235253        pci_bus_t *bus = pci_bus_from_fun(fun);
     254        uint32_t val;
    236255       
    237256        fibril_mutex_lock(&bus->conf_mutex);
    238        
    239         const uint32_t conf_addr = CONF_ADDR(fun->bus, fun->dev, fun->fn, reg);
    240         void *addr = bus->conf_data_port + (reg & 3);
    241        
    242         pio_write_32(bus->conf_addr_port, host2uint32_t_le(conf_addr));
    243        
     257
     258        pio_write_32(bus->conf_addr_reg, host2uint32_t_le(conf_addr));
     259
     260        /*
     261         * Always read full 32-bits from the PCI conf_data_port register and
     262         * get the desired portion of it afterwards. Some architectures do not
     263         * support shorter PIO reads offset from this register.
     264         */
     265        val = uint32_t_le2host(pio_read_32(bus->conf_data_reg));
     266
    244267        switch (len) {
    245268        case 1:
    246                 /* No endianness change for 1 byte */
    247                 buf[0] = pio_read_8(addr);
     269                *buf = (uint8_t) (val >> ((reg & 3) * 8));
    248270                break;
    249271        case 2:
    250                 ((uint16_t *) buf)[0] = uint16_t_le2host(pio_read_16(addr));
     272                *((uint16_t *) buf) = (uint16_t) (val >> ((reg & 3)) * 8);
    251273                break;
    252274        case 4:
    253                 ((uint32_t *) buf)[0] = uint32_t_le2host(pio_read_32(addr));
     275                *((uint32_t *) buf) = (uint32_t) val;
    254276                break;
    255277        }
    … …  
    260282static void pci_conf_write(pci_fun_t *fun, int reg, uint8_t *buf, size_t len)
    261283{
     284        const uint32_t conf_addr = CONF_ADDR(fun->bus, fun->dev, fun->fn, reg);
    262285        pci_bus_t *bus = pci_bus_from_fun(fun);
     286        uint32_t val;
    263287       
    264288        fibril_mutex_lock(&bus->conf_mutex);
    265        
    266         const uint32_t conf_addr = CONF_ADDR(fun->bus, fun->dev, fun->fn, reg);
    267         void *addr = bus->conf_data_port + (reg & 3);
    268        
    269         pio_write_32(bus->conf_addr_port, host2uint32_t_le(conf_addr));
     289
     290        /*
     291         * Prepare to write full 32-bits to the PCI conf_data_port register.
     292         * Some architectures do not support shorter PIO writes offset from this
     293         * register.
     294         */
     295
     296        if (len < 4) {
     297                /*
     298                 * We have fewer than full 32-bits, so we need to read the
     299                 * missing bits first.
     300                 */
     301                pio_write_32(bus->conf_addr_reg, host2uint32_t_le(conf_addr));
     302                val = uint32_t_le2host(pio_read_32(bus->conf_data_reg));
     303        }
    270304       
    271305        switch (len) {
    272306        case 1:
    273                 /* No endianness change for 1 byte */
    274                 pio_write_8(addr, buf[0]);
     307                val &= ~(0xffU << ((reg & 3) * 8));
     308                val |= *buf << ((reg & 3) * 8);
    275309                break;
    276310        case 2:
    277                 pio_write_16(addr, host2uint16_t_le(((uint16_t *) buf)[0]));
     311                val &= ~(0xffffU << ((reg & 3) * 8));
     312                val |= *((uint16_t *) buf) << ((reg & 3) * 8);
    278313                break;
    279314        case 4:
    280                 pio_write_32(addr, host2uint32_t_le(((uint32_t *) buf)[0]));
     315                val = *((uint32_t *) buf);
    281316                break;
    282317        }
     318
     319        pio_write_32(bus->conf_addr_reg, host2uint32_t_le(conf_addr));
     320        pio_write_32(bus->conf_data_reg, host2uint32_t_le(val));
    283321       
    284322        fibril_mutex_unlock(&bus->conf_mutex);
    … …  
    411449                hw_resources[count].res.io_range.address = range_addr;
    412450                hw_resources[count].res.io_range.size = range_size;
     451                hw_resources[count].res.io_range.relative = true;
    413452                hw_resources[count].res.io_range.endianness = LITTLE_ENDIAN;
    414453        } else {
    … …  
    416455                hw_resources[count].res.mem_range.address = range_addr;
    417456                hw_resources[count].res.mem_range.size = range_size;
     457                hw_resources[count].res.mem_range.relative = false;
    418458                hw_resources[count].res.mem_range.endianness = LITTLE_ENDIAN;
    419459        }
    … …  
    433473{
    434474        /* Value of the BAR */
    435         uint32_t val, mask;
     475        uint32_t val;
     476        uint32_t bar;
     477        uint32_t mask;
     478
    436479        /* IO space address */
    437480        bool io;
    … …  
    471514        /* Get the address mask. */
    472515        pci_conf_write_32(fun, addr, 0xffffffff);
    473         mask &= pci_conf_read_32(fun, addr);
    474        
     516        bar = pci_conf_read_32(fun, addr);
     517
     518        /*
     519         * Unimplemented BARs read back as all 0's.
     520         */
     521        if (!bar)
     522                return addr + (addrw64 ? 8 : 4);
     523
     524        mask &= bar;   
     525
    475526        /* Restore the original value. */
    476527        pci_conf_write_32(fun, addr, val);
    … …  
    520571{
    521572        uint8_t irq = pci_conf_read_8(fun, PCI_BRIDGE_INT_LINE);
    522         if (irq != 0xff)
     573        uint8_t pin = pci_conf_read_8(fun, PCI_BRIDGE_INT_PIN);
     574
     575        if (pin != 0 && irq != 0xff)
    523576                pci_add_interrupt(fun, irq);
    524577}
    … …  
    583636                        pci_read_bars(fun);
    584637                        pci_read_interrupt(fun);
     638
     639                        /* Propagate the PIO window to the function. */
     640                        fun->pio_window = bus->pio_win;
    585641                       
    586642                        ddf_fun_set_ops(fun->fnode, &pci_fun_ops);
    … …  
    613669static int pci_dev_add(ddf_dev_t *dnode)
    614670{
     671        hw_resource_list_t hw_resources;
    615672        pci_bus_t *bus = NULL;
    616673        ddf_fun_t *ctl = NULL;
    … …  
    638695                goto fail;
    639696        }
    640        
    641         hw_resource_list_t hw_resources;
     697
     698        rc = pio_window_get(sess, &bus->pio_win);
     699        if (rc != EOK) {
     700                ddf_msg(LVL_ERROR, "pci_dev_add failed to get PIO window "
     701                    "for the device.");
     702                goto fail;
     703        }
    642704       
    643705        rc = hw_res_get_resource_list(sess, &hw_resources);
    … …  
    662724            hw_resources.resources[1].res.io_range.address);
    663725       
    664         bus->conf_io_addr =
    665             (uint32_t) hw_resources.resources[0].res.io_range.address;
    666         bus->conf_io_data =
    667             (uint32_t) hw_resources.resources[1].res.io_range.address;
    668        
    669         if (pio_enable((void *)(uintptr_t)bus->conf_io_addr, 4,
    670             &bus->conf_addr_port)) {
     726        if (pio_enable_resource(&bus->pio_win, &hw_resources.resources[0],
     727            (void **) &bus->conf_addr_reg)) {
    671728                ddf_msg(LVL_ERROR, "Failed to enable configuration ports.");
    672729                rc = EADDRNOTAVAIL;
    673730                goto fail;
    674731        }
    675         if (pio_enable((void *)(uintptr_t)bus->conf_io_data, 4,
    676             &bus->conf_data_port)) {
     732        if (pio_enable_resource(&bus->pio_win, &hw_resources.resources[1],
     733            (void **) &bus->conf_data_reg)) {
    677734                ddf_msg(LVL_ERROR, "Failed to enable configuration ports.");
    678735                rc = EADDRNOTAVAIL;
    … …  
    729786{
    730787        ddf_log_init(NAME);
    731         pci_fun_ops.interfaces[HW_RES_DEV_IFACE] = &pciintel_hw_res_ops;
    732         pci_fun_ops.interfaces[PCI_DEV_IFACE] = &pci_dev_ops;
    733788}
    734789

  • uspace/drv/bus/pci/pciintel/pci.h

    rdd7078c r3f03199  
    4040#include "pci_regs.h"
    4141
    42 #define PCI_MAX_HW_RES 8
     42#define PCI_MAX_HW_RES 10
    4343
    4444typedef struct pciintel_bus {
    4545        /** DDF device node */
    4646        ddf_dev_t *dnode;
    47         uint32_t conf_io_addr;
    48         uint32_t conf_io_data;
    49         void *conf_data_port;
    50         void *conf_addr_port;
     47        ioport32_t *conf_addr_reg;
     48        ioport32_t *conf_data_reg;
     49        pio_window_t pio_win;
    5150        fibril_mutex_t conf_mutex;
    5251} pci_bus_t;
    … …  
    6867        hw_resource_list_t hw_resources;
    6968        hw_resource_t resources[PCI_MAX_HW_RES];
     69        pio_window_t pio_window;
    7070} pci_fun_t;
    7171

  • uspace/drv/bus/usb/ehci/main.c

    rdd7078c r3f03199  
    6767{
    6868        assert(device);
    69 #define CHECK_RET_RETURN(ret, message...) \
    70 if (ret != EOK) { \
    71         usb_log_error(message); \
    72         return ret; \
    73 }
    7469
    75         uintptr_t reg_base = 0;
    76         size_t reg_size = 0;
     70        addr_range_t reg_range;
    7771        int irq = 0;
    7872
    79         int ret = get_my_registers(device, &reg_base, &reg_size, &irq);
    80         CHECK_RET_RETURN(ret,
    81             "Failed to get memory addresses for %" PRIun ": %s.\n",
    82             ddf_dev_get_handle(device), str_error(ret));
    83         usb_log_info("Memory mapped regs at 0x%" PRIxn " (size %zu), IRQ %d.\n",
    84             reg_base, reg_size, irq);
     73        int ret = get_my_registers(device, &reg_range, &irq);
     74        if (ret != EOK) {
     75                usb_log_error("Failed to get memory addresses for %" PRIun
     76                    ": %s.\n", ddf_dev_get_handle(device), str_error(ret));
     77                return ret;
     78        }
    8579
    86         ret = disable_legacy(device, reg_base, reg_size);
    87         CHECK_RET_RETURN(ret,
    88             "Failed to disable legacy USB: %s.\n", str_error(ret));
     80        usb_log_info("Memory mapped regs at %p (size %zu), IRQ %d.\n",
     81            RNGABSPTR(reg_range), RNGSZ(reg_range), irq);
     82
     83        ret = disable_legacy(device, &reg_range);
     84        if (ret != EOK) {
     85                usb_log_error("Failed to disable legacy USB: %s.\n",
     86                    str_error(ret));
     87                return ret;
     88        }
    8989
    9090        /* High Speed, no bandwidth */
    91         ret = hcd_ddf_setup_hc(device, USB_SPEED_HIGH, 0, NULL);       
    92         CHECK_RET_RETURN(ret,
    93             "Failed to init generci hcd driver: %s\n", str_error(ret));
     91        ret = hcd_ddf_setup_hc(device, USB_SPEED_HIGH, 0, NULL);
     92        if (ret != EOK) {
     93                usb_log_error("Failed to init generci hcd driver: %s\n",
     94                    str_error(ret));
     95                return ret;
     96        }
    9497
    9598        usb_log_info("Controlling new EHCI device `%s' (handle %" PRIun ").\n",
    … …  
    97100
    98101        return EOK;
    99 #undef CHECK_RET_RETURN
    100102}
    101103

  • uspace/drv/bus/usb/ehci/res.c

    rdd7078c r3f03199  
    7171 *
    7272 * @param[in] dev Device asking for the addresses.
    73  * @param[out] mem_reg_address Base address of the memory range.
    74  * @param[out] mem_reg_size Size of the memory range.
     73 * @param[out] mem_regs_p Pointer to the register range.
    7574 * @param[out] irq_no IRQ assigned to the device.
    7675 * @return Error code.
    7776 */
    7877int get_my_registers(ddf_dev_t *dev,
    79     uintptr_t *mem_reg_address, size_t *mem_reg_size, int *irq_no)
     78    addr_range_t *mem_regs_p, int *irq_no)
    8079{
    8180        assert(dev);
    … …  
    9998        }
    10099
    101         if (mem_reg_address)
    102                 *mem_reg_address = hw_res.mem_ranges.ranges[0].address;
    103         if (mem_reg_size)
    104                 *mem_reg_size = hw_res.mem_ranges.ranges[0].size;
     100        if (mem_regs_p)
     101                *mem_regs_p = hw_res.mem_ranges.ranges[0];
    105102        if (irq_no)
    106103                *irq_no = hw_res.irqs.irqs[0];
    … …  
    234231}
    235232
    236 int disable_legacy(ddf_dev_t *device, uintptr_t reg_base, size_t reg_size)
     233int disable_legacy(ddf_dev_t *device, addr_range_t *reg_range)
    237234{
    238235        assert(device);
    239236        usb_log_debug("Disabling EHCI legacy support.\n");
    240237
    241 #define CHECK_RET_RETURN(ret, message...) \
    242         if (ret != EOK) { \
    243                 usb_log_error(message); \
    244                 return ret; \
    245         } else (void)0
    246 
    247238        /* Map EHCI registers */
    248239        void *regs = NULL;
    249         int ret = pio_enable((void*)reg_base, reg_size, &regs);
    250         CHECK_RET_RETURN(ret, "Failed to map registers %p: %s.\n",
    251             (void *) reg_base, str_error(ret));
     240        int ret = pio_enable_range(reg_range, &regs);
     241        if (ret != EOK) {
     242                usb_log_error("Failed to map registers %p: %s.\n",
     243                    RNGABSPTR(*reg_range), str_error(ret));
     244                return ret;
     245        }
    252246
    253247        usb_log_debug2("Registers mapped at: %p.\n", regs);
    … …  
    264258
    265259        ret = disable_extended_caps(device, eecp);
    266         CHECK_RET_RETURN(ret, "Failed to disable extended capabilities: %s.\n",
    267             str_error(ret));
    268 
    269 #undef CHECK_RET_RETURN
     260        if (ret != EOK) {
     261                usb_log_error("Failed to disable extended capabilities: %s.\n",
     262                    str_error(ret));
     263                return ret;
     264        }
     265
    270266
    271267        /*

  • uspace/drv/bus/usb/ehci/res.h

    rdd7078c r3f03199  
    3737
    3838#include <ddf/driver.h>
     39#include <device/hw_res_parsed.h>
    3940
    40 int get_my_registers(ddf_dev_t *, uintptr_t *, size_t *, int *);
     41int get_my_registers(ddf_dev_t *, addr_range_t *, int *);
    4142int enable_interrupts(ddf_dev_t *);
    42 int disable_legacy(ddf_dev_t *, uintptr_t, size_t);
     43int disable_legacy(ddf_dev_t *, addr_range_t *);
    4344
    4445#endif

  • uspace/drv/bus/usb/ohci/hc.c

    rdd7078c r3f03199  
    109109 * @param[out] cmds Commands buffer.
    110110 * @param[in] cmds_size Size of the commands buffer (bytes).
    111  * @param[in] regs Physical address of device's registers.
    112  * @param[in] reg_size Size of the register area (bytes).
     111 * @param[in] regs Device's register range.
    113112 *
    114113 * @return Error code.
    … …  
    116115int
    117116hc_get_irq_code(irq_pio_range_t ranges[], size_t ranges_size, irq_cmd_t cmds[],
    118     size_t cmds_size, uintptr_t regs, size_t reg_size)
     117    size_t cmds_size, addr_range_t *regs)
    119118{
    120119        if ((ranges_size < sizeof(ohci_pio_ranges)) ||
    121120            (cmds_size < sizeof(ohci_irq_commands)) ||
    122             (reg_size < sizeof(ohci_regs_t)))
     121            (RNGSZ(*regs) < sizeof(ohci_regs_t)))
    123122                return EOVERFLOW;
    124123
    125124        memcpy(ranges, ohci_pio_ranges, sizeof(ohci_pio_ranges));
    126         ranges[0].base = regs;
     125        ranges[0].base = RNGABS(*regs);
    127126
    128127        memcpy(cmds, ohci_irq_commands, sizeof(ohci_irq_commands));
    129         ohci_regs_t *registers = (ohci_regs_t *) regs;
     128        ohci_regs_t *registers = (ohci_regs_t *) RNGABSPTR(*regs);
    130129        cmds[0].addr = (void *) &registers->interrupt_status;
    131130        cmds[3].addr = (void *) &registers->interrupt_status;
    … …  
    135134}
    136135
     136/** Register interrupt handler.
     137 *
     138 * @param[in] device Host controller DDF device
     139 * @param[in] regs Register range
     140 * @param[in] irq Interrupt number
     141 * @paran[in] handler Interrupt handler
     142 *
     143 * @return EOK on success or negative error code
     144 */
     145int hc_register_irq_handler(ddf_dev_t *device, addr_range_t *regs, int irq,
     146    interrupt_handler_t handler)
     147{
     148        int rc;
     149
     150        irq_pio_range_t irq_ranges[hc_irq_pio_range_count()];
     151        irq_cmd_t irq_cmds[hc_irq_cmd_count()];
     152
     153        irq_code_t irq_code = {
     154                .rangecount = hc_irq_pio_range_count(),
     155                .ranges = irq_ranges,
     156                .cmdcount = hc_irq_cmd_count(),
     157                .cmds = irq_cmds
     158        };
     159
     160        rc = hc_get_irq_code(irq_ranges, sizeof(irq_ranges), irq_cmds,
     161            sizeof(irq_cmds), regs);
     162        if (rc != EOK) {
     163                usb_log_error("Failed to generate IRQ code: %s.\n",
     164                    str_error(rc));
     165                return rc;
     166        }
     167
     168        /* Register handler to avoid interrupt lockup */
     169        rc = register_interrupt_handler(device, irq, handler, &irq_code);
     170        if (rc != EOK) {
     171                usb_log_error("Failed to register interrupt handler: %s.\n",
     172                    str_error(rc));
     173                return rc;
     174        }
     175
     176        return EOK;
     177}
     178
    137179/** Initialize OHCI hc driver structure
    138180 *
    139181 * @param[in] instance Memory place for the structure.
    140  * @param[in] regs Address of the memory mapped I/O registers.
    141  * @param[in] reg_size Size of the memory mapped area.
     182 * @param[in] regs Device's I/O registers range.
    142183 * @param[in] interrupts True if w interrupts should be used
    143184 * @return Error code
    144185 */
    145 int hc_init(hc_t *instance, uintptr_t regs, size_t reg_size, bool interrupts)
    146 {
    147         assert(instance);
    148 
    149         int ret =
    150             pio_enable((void*)regs, reg_size, (void**)&instance->registers);
     186int hc_init(hc_t *instance, addr_range_t *regs, bool interrupts)
     187{
     188        assert(instance);
     189
     190        int ret = pio_enable_range(regs, (void **) &instance->registers);
    151191        if (ret != EOK) {
    152                 usb_log_error("Failed to enable access to device regss: %s.\n",
     192                usb_log_error("Failed to gain access to device registers: %s.\n",
    153193                    str_error(ret));
    154194                return ret;

  • uspace/drv/bus/usb/ohci/hc.h

    rdd7078c r3f03199  
    3939#include <adt/list.h>
    4040#include <ddi.h>
     41#include <ddf/interrupt.h>
    4142
    4243#include <usb/usb.h>
    … …  
    7374size_t hc_irq_pio_range_count(void);
    7475size_t hc_irq_cmd_count(void);
    75 int hc_get_irq_code(irq_pio_range_t [], size_t, irq_cmd_t [], size_t, uintptr_t,
    76     size_t);
     76int hc_get_irq_code(irq_pio_range_t [], size_t, irq_cmd_t [], size_t,
     77    addr_range_t *);
     78int hc_register_irq_handler(ddf_dev_t *, addr_range_t *, int,
     79    interrupt_handler_t);
    7780int hc_register_hub(hc_t *instance, ddf_fun_t *hub_fun);
    78 int hc_init(hc_t *instance, uintptr_t regs, size_t reg_size, bool interrupts);
     81int hc_init(hc_t *instance, addr_range_t *regs, bool interrupts);
    7982
    8083/** Safely dispose host controller internal structures

  • uspace/drv/bus/usb/ohci/ohci.c

    rdd7078c r3f03199  
    8080int device_setup_ohci(ddf_dev_t *device)
    8181{
    82 #define CHECK_RET_RETURN(ret, message...) \
    83 if (ret != EOK) { \
    84         usb_log_error(message); \
    85         return ret; \
    86 }
    8782
    88         uintptr_t reg_base = 0;
    89         size_t reg_size = 0;
     83        addr_range_t regs;
    9084        int irq = 0;
    9185
    92         int ret = get_my_registers(device, &reg_base, &reg_size, &irq);
    93         CHECK_RET_RETURN(ret, "Failed to get register memory addresses for %"
    94             PRIun ": %s.\n", ddf_dev_get_handle(device), str_error(ret));
     86        int ret = get_my_registers(device, &regs, &irq);
     87        if (ret != EOK) {
     88                usb_log_error("Failed to get register memory addresses "
     89                    "for %" PRIun ": %s.\n", ddf_dev_get_handle(device),
     90                    str_error(ret));
     91                return ret;
     92        }
    9593
    9694        usb_log_debug("Memory mapped regs at %p (size %zu), IRQ %d.\n",
    97             (void *) reg_base, reg_size, irq);
     95            RNGABSPTR(regs), RNGSZ(regs), irq);
    9896
    99         const size_t ranges_count = hc_irq_pio_range_count();
    100         const size_t cmds_count = hc_irq_cmd_count();
    101         irq_pio_range_t irq_ranges[ranges_count];
    102         irq_cmd_t irq_cmds[cmds_count];
    103         irq_code_t irq_code = {
    104                 .rangecount = ranges_count,
    105                 .ranges = irq_ranges,
    106                 .cmdcount = cmds_count,
    107                 .cmds = irq_cmds
    108         };
     97        /* Initialize generic HCD driver */
     98        ret = hcd_ddf_setup_hc(device, USB_SPEED_FULL,
     99            BANDWIDTH_AVAILABLE_USB11, bandwidth_count_usb11);
     100        if (ret != EOK) {
     101                usb_log_error("Failedd to setup generic hcd: %s.",
     102                    str_error(ret));
     103                return ret;
     104        }
    109105
    110         ret = hc_get_irq_code(irq_ranges, sizeof(irq_ranges), irq_cmds,
    111             sizeof(irq_cmds), reg_base, reg_size);
    112         CHECK_RET_RETURN(ret, "Failed to gen IRQ code: %s.\n", str_error(ret));
    113 
    114         /* Register handler to avoid interrupt lockup */
    115         ret = register_interrupt_handler(device, irq, irq_handler, &irq_code);
    116         CHECK_RET_RETURN(ret,
    117             "Failed to register irq handler: %s.\n", str_error(ret));
     106        ret = hc_register_irq_handler(device, &regs, irq, irq_handler);
     107        if (ret != EOK) {
     108                usb_log_error("Failed to register interrupt handler: %s.\n",
     109                    str_error(ret));
     110                hcd_ddf_clean_hc(device);
     111                return ret;
     112        }
    118113
    119114        /* Try to enable interrupts */
    … …  
    130125        }
    131126
    132         /* Initialize generic HCD driver */
    133         ret = hcd_ddf_setup_hc(device, USB_SPEED_FULL,
    134             BANDWIDTH_AVAILABLE_USB11, bandwidth_count_usb11);
    135         if (ret != EOK) {
     127
     128        hc_t *hc_impl = malloc(sizeof(hc_t));
     129        if (!hc_impl) {
     130                usb_log_error("Failed to allocate driver structure.\n");
     131                hcd_ddf_clean_hc(device);
    136132                unregister_interrupt_handler(device, irq);
    137133                return ret;
    138134        }
    139135
    140 // TODO: Undo hcd_setup_device
    141 #define CHECK_RET_CLEAN_RETURN(ret, message...) \
    142 if (ret != EOK) { \
    143         unregister_interrupt_handler(device, irq); \
    144         CHECK_RET_RETURN(ret, message); \
    145 } else (void)0
    146 
    147         hc_t *hc_impl = malloc(sizeof(hc_t));
    148         ret = hc_impl ? EOK : ENOMEM;
    149         CHECK_RET_CLEAN_RETURN(ret, "Failed to allocate driver structure.\n");
    150 
    151136        /* Initialize OHCI HC */
    152         ret = hc_init(hc_impl, reg_base, reg_size, interrupts);
    153         CHECK_RET_CLEAN_RETURN(ret, "Failed to init hc: %s.\n", str_error(ret));
     137        ret = hc_init(hc_impl, &regs, interrupts);
     138        if (ret != EOK) {
     139                usb_log_error("Failed to init hc: %s.\n", str_error(ret));
     140                hcd_ddf_clean_hc(device);
     141                unregister_interrupt_handler(device, irq);
     142                return ret;
     143        }
    154144
    155145        /* Connect OHCI to generic HCD */
    … …  
    159149        /* HC should be running OK. We can add root hub */
    160150        ret = hcd_ddf_setup_root_hub(device);
    161         CHECK_RET_CLEAN_RETURN(ret,
    162             "Failed to register OHCI root hub: %s.\n", str_error(ret));
     151        if (ret != EOK) {
     152                usb_log_error("Failed to registter OHCI root hub: %s.\n",
     153                    str_error(ret));
     154                hcd_ddf_clean_hc(device);
     155                unregister_interrupt_handler(device, irq);
     156                return ret;
     157        }
    163158
    164159        return ret;

  • uspace/drv/bus/usb/ohci/res.c

    rdd7078c r3f03199  
    4848 *
    4949 * @param[in] dev Device asking for the addresses.
    50  * @param[out] mem_reg_address Base address of the memory range.
    51  * @param[out] mem_reg_size Size of the memory range.
     50 * @param[out] p_regs Pointer to register range.
    5251 * @param[out] irq_no IRQ assigned to the device.
    5352 * @return Error code.
    5453 */
    55 int get_my_registers(ddf_dev_t *dev,
    56     uintptr_t *mem_reg_address, size_t *mem_reg_size, int *irq_no)
     54int get_my_registers(ddf_dev_t *dev, addr_range_t *p_regs, int *irq_no)
    5755{
    5856        assert(dev);
    … …  
    6664        hw_res_list_parsed_t hw_res;
    6765        hw_res_list_parsed_init(&hw_res);
    68         const int ret =  hw_res_get_list_parsed(parent_sess, &hw_res, 0);
     66        const int ret = hw_res_get_list_parsed(parent_sess, &hw_res, 0);
    6967        async_hangup(parent_sess);
    7068        if (ret != EOK) {
    … …  
    7876        }
    7977
    80         if (mem_reg_address)
    81                 *mem_reg_address = hw_res.mem_ranges.ranges[0].address;
    82         if (mem_reg_size)
    83                 *mem_reg_size = hw_res.mem_ranges.ranges[0].size;
     78        if (p_regs)
     79                *p_regs = hw_res.mem_ranges.ranges[0];
    8480        if (irq_no)
    8581                *irq_no = hw_res.irqs.irqs[0];

  • uspace/drv/bus/usb/ohci/res.h

    rdd7078c r3f03199  
    3636
    3737#include <ddf/driver.h>
     38#include <device/hw_res_parsed.h>
    3839
    39 int get_my_registers(ddf_dev_t *, uintptr_t *, size_t *, int *);
     40int get_my_registers(ddf_dev_t *, addr_range_t *, int *);
    4041int enable_interrupts(ddf_dev_t *);
    4142

  • uspace/drv/bus/usb/uhci/hc.c

    rdd7078c r3f03199  
    111111 * @param[out] cmds Commands buffer.
    112112 * @param[in] cmds_size Size of the commands buffer (bytes).
    113  * @param[in] regs Physical address of device's registers.
    114  * @param[in] reg_size Size of the register area (bytes).
     113 * @param[in] regs Device's register range.
    115114 *
    116115 * @return Error code.
    … …  
    118117int
    119118hc_get_irq_code(irq_pio_range_t ranges[], size_t ranges_size, irq_cmd_t cmds[],
    120     size_t cmds_size, uintptr_t regs, size_t reg_size)
     119    size_t cmds_size, addr_range_t *regs)
    121120{
    122121        if ((ranges_size < sizeof(uhci_irq_pio_ranges)) ||
    123122            (cmds_size < sizeof(uhci_irq_commands)) ||
    124             (reg_size < sizeof(uhci_regs_t)))
     123            (RNGSZ(*regs) < sizeof(uhci_regs_t)))
    125124                return EOVERFLOW;
    126125
    127126        memcpy(ranges, uhci_irq_pio_ranges, sizeof(uhci_irq_pio_ranges));
    128         ranges[0].base = regs;
     127        ranges[0].base = RNGABS(*regs);
    129128
    130129        memcpy(cmds, uhci_irq_commands, sizeof(uhci_irq_commands));
    131         uhci_regs_t *registers = (uhci_regs_t *) regs;
     130        uhci_regs_t *registers = (uhci_regs_t *) RNGABSPTR(*regs);
    132131        cmds[0].addr = (void*)&registers->usbsts;
    133132        cmds[3].addr = (void*)&registers->usbsts;
     133
     134        return EOK;
     135}
     136
     137/** Register interrupt handler.
     138 *
     139 * @param[in] device Host controller DDF device
     140 * @param[in] regs Register range
     141 * @param[in] irq Interrupt number
     142 * @paran[in] handler Interrupt handler
     143 *
     144 * @return EOK on success or negative error code
     145 */
     146int hc_register_irq_handler(ddf_dev_t *device, addr_range_t *regs, int irq,
     147    interrupt_handler_t handler)
     148{
     149        assert(device);
     150        irq_pio_range_t irq_ranges[hc_irq_pio_range_count()];
     151        irq_cmd_t irq_cmds[hc_irq_cmd_count()];
     152
     153        int ret = hc_get_irq_code(irq_ranges, sizeof(irq_ranges), irq_cmds,
     154            sizeof(irq_cmds), regs);
     155        if (ret != EOK) {
     156                usb_log_error("Failed to generate IRQ commands: %s.\n",
     157                    str_error(ret));
     158                return ret;
     159        }
     160
     161        irq_code_t irq_code = {
     162                .rangecount = hc_irq_pio_range_count(),
     163                .ranges = irq_ranges,
     164                .cmdcount = hc_irq_cmd_count(),
     165                .cmds = irq_cmds
     166        };
     167
     168        /* Register handler to avoid interrupt lockup */
     169        ret = register_interrupt_handler(device, irq, handler, &irq_code);
     170        if (ret != EOK) {
     171                usb_log_error("Failed to register interrupt handler: %s.\n",
     172                    str_error(ret));
     173                return ret;
     174        }
    134175
    135176        return EOK;
    … …  
    196237 *
    197238 * @param[in] instance Memory place to initialize.
    198  * @param[in] regs Address of I/O control registers.
    199  * @param[in] reg_size Size of I/O control registers.
     239 * @param[in] regs Range of device's I/O control registers.
    200240 * @param[in] interrupts True if hw interrupts should be used.
    201241 * @return Error code.
    … …  
    205245 * interrupt fibrils.
    206246 */
    207 int hc_init(hc_t *instance, void *regs, size_t reg_size, bool interrupts)
    208 {
    209         assert(reg_size >= sizeof(uhci_regs_t));
     247int hc_init(hc_t *instance, addr_range_t *regs, bool interrupts)
     248{
     249        assert(instance);
     250        assert(regs);
     251        assert(regs->size >= sizeof(uhci_regs_t));
    210252
    211253        instance->hw_interrupts = interrupts;
    … …  
    214256        /* allow access to hc control registers */
    215257        uhci_regs_t *io;
    216         int ret = pio_enable(regs, reg_size, (void **)&io);
     258        int ret = pio_enable_range(regs, (void **) &io);
    217259        if (ret != EOK) {
    218260                usb_log_error("Failed to gain access to registers at %p: %s.\n",
    … …  
    223265
    224266        usb_log_debug(
    225             "Device registers at %p (%zuB) accessible.\n", io, reg_size);
     267            "Device registers at %p (%zuB) accessible.\n", io, regs->size);
    226268
    227269        ret = hc_init_mem_structures(instance);

  • uspace/drv/bus/usb/uhci/hc.h

    rdd7078c r3f03199  
    3636#define DRV_UHCI_HC_H
    3737
     38#include <ddf/interrupt.h>
     39#include <device/hw_res_parsed.h>
    3840#include <fibril.h>
    3941#include <macros.h>
    … …  
    124126size_t hc_irq_pio_range_count(void);
    125127size_t hc_irq_cmd_count(void);
    126 int hc_get_irq_code(irq_pio_range_t [], size_t, irq_cmd_t [], size_t, uintptr_t,
    127     size_t);
     128int hc_register_irq_handler(ddf_dev_t *, addr_range_t *, int,
     129    interrupt_handler_t);
     130int hc_get_irq_code(irq_pio_range_t [], size_t, irq_cmd_t [], size_t,
     131    addr_range_t *);
    128132void hc_interrupt(hc_t *instance, uint16_t status);
    129 int hc_init(hc_t *instance, void *regs, size_t reg_size, bool interupts);
     133int hc_init(hc_t *instance, addr_range_t *regs, bool interupts);
    130134int hc_schedule(hcd_t *hcd, usb_transfer_batch_t *batch);
    131135
    … …  
    136140static inline void hc_fini(hc_t *instance) {} /* TODO: implement*/
    137141#endif
     142
    138143/**
    139144 * @}

  • uspace/drv/bus/usb/uhci/res.c

    rdd7078c r3f03199  
    4646 *
    4747 * @param[in] dev Device asking for the addresses.
    48  * @param[out] io_reg_address Base address of the I/O range.
    49  * @param[out] io_reg_size Size of the I/O range.
     48 * @param[out] io_regs_p Pointer to register I/O range.
    5049 * @param[out] irq_no IRQ assigned to the device.
    5150 * @return Error code.
    5251 */
    53 int get_my_registers(ddf_dev_t *dev,
    54     uintptr_t *io_reg_address, size_t *io_reg_size, int *irq_no)
     52int get_my_registers(ddf_dev_t *dev, addr_range_t *io_regs_p, int *irq_no)
    5553{
    5654        assert(dev);
    … …  
    7674        }
    7775
    78         if (io_reg_address)
    79                 *io_reg_address = hw_res.io_ranges.ranges[0].address;
    80         if (io_reg_size)
    81                 *io_reg_size = hw_res.io_ranges.ranges[0].size;
     76        if (io_regs_p)
     77                *io_regs_p = hw_res.io_ranges.ranges[0];
    8278        if (irq_no)
    8379                *irq_no = hw_res.irqs.irqs[0];

  • uspace/drv/bus/usb/uhci/res.h

    rdd7078c r3f03199  
    3737
    3838#include <ddf/driver.h>
     39#include <device/hw_res_parsed.h>
    3940
    40 int get_my_registers(ddf_dev_t *, uintptr_t *, size_t *, int *);
     41int get_my_registers(ddf_dev_t *, addr_range_t *, int *);
    4142int enable_interrupts(ddf_dev_t *);
    4243int disable_legacy(ddf_dev_t *);

  • uspace/drv/bus/usb/uhci/uhci.c

    rdd7078c r3f03199  
    8080                return EBADMEM;
    8181
    82 #define CHECK_RET_RETURN(ret, message...) \
    83 if (ret != EOK) { \
    84         usb_log_error(message); \
    85         return ret; \
    86 } else (void)0
    87 
    88         uintptr_t reg_base = 0;
    89         size_t reg_size = 0;
     82        addr_range_t regs;
    9083        int irq = 0;
    9184
    92         int ret = get_my_registers(device, &reg_base, &reg_size, &irq);
    93         CHECK_RET_RETURN(ret, "Failed to get I/O region for %" PRIun ": %s.\n",
    94             ddf_dev_get_handle(device), str_error(ret));
    95         usb_log_debug("I/O regs at 0x%p (size %zu), IRQ %d.\n",
    96             (void *) reg_base, reg_size, irq);
     85        int ret = get_my_registers(device, &regs, &irq);
     86        if (ret != EOK) {
     87                usb_log_error("Failed to get I/O addresses for %" PRIun ": %s.\n",
     88                    ddf_dev_get_handle(device), str_error(ret));
     89                return ret;
     90        }
     91        usb_log_debug("I/O regs at %p (size %zu), IRQ %d.\n",
     92            RNGABSPTR(regs), RNGSZ(regs), irq);
    9793
    98         const size_t ranges_count = hc_irq_pio_range_count();
    99         const size_t cmds_count = hc_irq_cmd_count();
    100         irq_pio_range_t irq_ranges[ranges_count];
    101         irq_cmd_t irq_cmds[cmds_count];
    102         ret = hc_get_irq_code(irq_ranges, sizeof(irq_ranges), irq_cmds,
    103             sizeof(irq_cmds), reg_base, reg_size);
    104         CHECK_RET_RETURN(ret, "Failed to generate IRQ commands: %s.\n",
    105             str_error(ret));
     94        ret = hcd_ddf_setup_hc(device, USB_SPEED_FULL,
     95            BANDWIDTH_AVAILABLE_USB11, bandwidth_count_usb11);
     96        if (ret != EOK) {
     97                usb_log_error("Failed to setup generic HCD.\n");
     98                return ret;
     99        }
    106100
    107         irq_code_t irq_code = {
    108                 .rangecount = ranges_count,
    109                 .ranges = irq_ranges,
    110                 .cmdcount = cmds_count,
    111                 .cmds = irq_cmds
    112         };
     101        hc_t *hc = malloc(sizeof(hc_t));
     102        if (!hc) {
     103                usb_log_error("Failed to allocate UHCI HC structure.\n");
     104                hcd_ddf_clean_hc(device);
     105                return ENOMEM;
     106        }
    113107
    114         /* Register handler to avoid interrupt lockup */
    115         ret = register_interrupt_handler(device, irq, irq_handler, &irq_code);
    116         CHECK_RET_RETURN(ret, "Failed to register interrupt handler: %s.\n",
    117             str_error(ret));
    118        
    119         ret = disable_legacy(device);
    120         CHECK_RET_RETURN(ret, "Failed to disable legacy USB: %s.\n",
    121             str_error(ret));
     108        ret = hc_register_irq_handler(device, &regs, irq, irq_handler);
     109        if (ret != EOK) {
     110                usb_log_error("Failed to register interrupt handler: %s.\n",
     111                    str_error(ret));
     112                hcd_ddf_clean_hc(device);
     113                return ret;
     114        }
    122115
    123116        bool interrupts = false;
    … …  
    131124        }
    132125
    133         ret = hcd_ddf_setup_hc(device, USB_SPEED_FULL,
    134             BANDWIDTH_AVAILABLE_USB11, bandwidth_count_usb11);
    135         CHECK_RET_RETURN(ret, "Failed to setup UHCI HCD.\n");
    136        
    137         hc_t *hc = malloc(sizeof(hc_t));
    138         ret = hc ? EOK : ENOMEM;
    139         CHECK_RET_RETURN(ret, "Failed to allocate UHCI HC structure.\n");
     126        ret = disable_legacy(device);
     127        if (ret != EOK) {
     128                usb_log_error("Failed to disable legacy USB: %s.\n",
     129                    str_error(ret));
     130                hcd_ddf_clean_hc(device);
     131                return ret;
     132        }
    140133
    141         ret = hc_init(hc, (void*)reg_base, reg_size, interrupts);
    142         CHECK_RET_RETURN(ret,
    143             "Failed to init uhci_hcd: %s.\n", str_error(ret));
     134        ret = hc_init(hc, &regs, interrupts);
     135        if (ret != EOK) {
     136                usb_log_error("Failed to init uhci_hcd: %s.\n", str_error(ret));
     137                hcd_ddf_clean_hc(device);
     138                // TODO unregister interrupt handler
     139                return ret;
     140        }
    144141
    145142        hcd_set_implementation(dev_to_hcd(device), hc, hc_schedule, NULL, NULL);
    … …  
    151148        ret = hcd_ddf_setup_root_hub(device);
    152149        if (ret != EOK) {
    153                 // TODO: Undo hcd_setup_device
    154150                hc_fini(hc);
    155                 CHECK_RET_RETURN(ret, "Failed to setup UHCI root hub: %s.\n",
     151                hcd_ddf_clean_hc(device);
     152                // TODO unregister interrupt handler
     153                usb_log_error("Failed to setup UHCI root hub: %s.\n",
    156154                    str_error(ret));
    157155                return ret;

  • uspace/drv/bus/usb/uhci/utils/malloc32.h

    rdd7078c r3f03199  
    9292 */
    9393static inline void free32(void *addr)
    94         { free(addr); }
     94{
     95        free(addr);
     96}
    9597
    9698/** Create 4KB page mapping
    … …  
    98100 * @return Address of the mapped page, NULL on failure.
    99101 */
    100 static inline void * get_page(void)
     102static inline void *get_page(void)
    101103{
    102         void *address, *phys;
     104        uintptr_t phys;
     105        void *address;
     106       
    103107        const int ret = dmamem_map_anonymous(UHCI_REQUIRED_PAGE_SIZE,
    104             AS_AREA_READ | AS_AREA_WRITE, 0, &phys, &address);
    105         return ret == EOK ? address : NULL;
     108            DMAMEM_4GiB, AS_AREA_READ | AS_AREA_WRITE, 0, &phys,
     109            &address);
     110       
     111        return ((ret == EOK) ? address : NULL);
    106112}
    107113

  • uspace/drv/bus/usb/usbmid/explore.c

    rdd7078c r3f03199  
    5050static bool interface_in_list(const list_t *list, int interface_no)
    5151{
    52         list_foreach(*list, l) {
    53                 const usbmid_interface_t *iface = usbmid_interface_from_link(l);
     52        list_foreach(*list, link, const usbmid_interface_t, iface) {
    5453                if (iface->interface_no == interface_no) {
    5554                        return true;

  • uspace/drv/bus/usb/usbmid/usbmid.c

    rdd7078c r3f03199  
    140140        }
    141141
    142         list_foreach(match_ids.ids, link) {
    143                 match_id_t *match_id = list_get_instance(link, match_id_t, link);
     142        list_foreach(match_ids.ids, link, match_id_t, match_id) {
    144143                rc = ddf_fun_add_match_id(child, match_id->id, match_id->score);
    145144                if (rc != EOK) {

  • uspace/drv/bus/usb/vhc/transfer.c

    rdd7078c r3f03199  
    180180        int targets = 0;
    181181
    182         list_foreach(vhc->devices, pos) {
    183                 vhc_virtdev_t *dev = list_get_instance(pos, vhc_virtdev_t, link);
     182        list_foreach(vhc->devices, link, vhc_virtdev_t, dev) {
    184183                fibril_mutex_lock(&dev->guard);
    185184                if (dev->address == transfer->batch->ep->address) {
Note: See TracChangeset for help on using the changeset viewer.