#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 in reply to:  description Changed on Apr 27, 2015 at 7:17:32 AM by Nick Withers

Replying to nick.withers:

(P.S., "_IO_Manager_drivers_inititalized" should probably be "_IO_Manager_drivers_initialized", too)

Never mind - that was addressed in a subsequent commit.

comment:2 Changed on Jan 26, 2017 at 7:16:00 AM by Sebastian Huber

Milestone: 4.11.14.11.2

comment:3 Changed on Feb 15, 2017 at 1:37:51 PM by Sebastian Huber

Milestone: 4.11.2Indefinite
Owner: set to Needs Funding
Status: newassigned
Note: See TracTickets for help on using tickets.