Opened on 04/27/15 at 07:11:13
Last modified on 02/15/17 at 13:37:51
#2335 assigned defect
MVME3100 libi2c broken by "IO_MANAGER: early dynamic driver registration" change
Reported by: | Nick Withers | Owned by: | Needs Funding |
---|---|---|---|
Priority: | normal | Milestone: | Indefinite |
Component: | bsps | Version: | 4.11 |
Severity: | normal | Keywords: | libi2c, i2c, mvme3100 |
Cc: | Blocked By: | ||
Blocking: |
Description
Commit 04cf2c5556428e836e7c024258851e24f10b2693 of the 2015-04-16 broke the MVME3100's libi2c driver.
From src/lib/libbsp/powerpc/mvme3100/i2c/i2c_init.c:
/* Initialize the library */ if ( rtems_libi2c_initialize() ) { safe_printf("Initializing I2C library failed\n"); return -1; } /* Register our bus driver */ if ( (busno=rtems_libi2c_register_bus( BSP_I2C_BUS0_NAME, BSP_I2C_BUS_DESCRIPTOR) ) < 0 ) { safe_perror("Registering mpc8540 i2c bus driver"); return -1; }
It's assumed that the i2c driver's ready to go after a successful return from rtems_libi2c_initialize(), which is no longer the case.
(P.S., "_IO_Manager_drivers_inititalized" should probably be "_IO_Manager_drivers_initialized", too)
Change History (3)
comment:1 Changed on 04/27/15 at 07:17:32 by Nick Withers
comment:2 Changed on 01/26/17 at 07:16:00 by Sebastian Huber
Milestone: | 4.11.1 → 4.11.2 |
---|
comment:3 Changed on 02/15/17 at 13:37:51 by Sebastian Huber
Milestone: | 4.11.2 → Indefinite |
---|---|
Owner: | set to Needs Funding |
Status: | new → assigned |
Note: See
TracTickets for help on using
tickets.
Replying to nick.withers:
Never mind - that was addressed in a subsequent commit.