[#4510] i2c_add_driver fails to probe drivers with new style drivers binding.

Document created by Aaronwu Employee on Aug 29, 2013
Version 1Show Document
  • View in full screen mode

[#4510] i2c_add_driver fails to probe drivers with new style drivers binding.

Submitted By: Michael Hennerich

Open Date

2008-10-13 12:20:18     Close Date

2008-10-13 15:34:54

Priority:

Medium     Assignee:

Michael Hennerich

Status:

Closed     Fixed In Release:

N/A

Found In Release:

N/A     Release:

trunk 2.6.26.5

Category:

Kernel Functions     Board:

N/A

Processor:

N/A     Silicon Revision:

Is this bug repeatable?:

Yes     Resolution:

Fixed

Uboot version or rev.:

    Toolchain version or rev.:

App binary format:

N/A     

Summary: i2c_add_driver fails to probe drivers with new style drivers binding.

Details:

 

i2c_add_driver fails to probe drivers with new style drivers binding.

The drivers probe function never get called.

This is typically the case when the i2c slave address can’t be detected.

However there is no i2c bus activity at all.

 

Enable:

CONFIG_JOYSTICK_AD7142 or CONFIG_TWI_KEYPAD.

Attach hardware.

 

The drivers probe function is never called.

This might be a problem in common code.

 

-Michael

 

Follow-ups

 

--- Michael Hennerich                                        2008-10-13 12:48:14

Ok - not a bug in common code.

 

Our new style drivers simply miss id_table in struct i2c_driver.

And require updating.

 

"The probe function is called when an entry in the id_table name field

matches the device's name. It is passed the entry that was matched so

the driver knows which one in the table matched."

 

--- Michael Hennerich                                        2008-10-13 15:34:54

Fixed on svn trunk.

I don't think the same problemn exists on 2008R1 kernels.

-Michael

 

 

 

    Files

    Changes

    Commits

    Dependencies

    Duplicates

    Associations

    Tags

 

File Name     File Type     File Size     Posted By

No Files Were Found

Attachments

    Outcomes