Re: [PATCH v3 2/2] i2c: change id to let i2c-at91 work

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi J,

On 10/16/2012 19:10, Jean-Christophe PLAGNIOL-VILLARD wrote:
On 18:25 Tue 16 Oct     , Bo Shen wrote:
Hi J,

On 10/16/2012 16:39, Jean-Christophe PLAGNIOL-VILLARD wrote:
On 11:13 Tue 16 Oct     , Bo Shen wrote:
Hi J,

On 10/15/2012 23:02, Jean-Christophe PLAGNIOL-VILLARD wrote:
On 17:30 Mon 15 Oct     , Bo Shen wrote:
The i2c core driver will turn the platform device ID to busnum
When using platfrom device ID as -1, it means dynamically assigned
the busnum. When writing code, we need to make sure the busnum,
and call i2c_register_board_info(int busnum, ...) to register device
if using -1, we do not know the value of busnum
	
In order to solve this issue, set the platform device ID as a fix number
Here using 0 to match the busnum used in i2c_regsiter_board_info()

Signed-off-by: Bo Shen <voice.shen@xxxxxxxxx>
---

can you check when this append for the first time to schedule a patch for stable too

OK. I will resend it and add linux stable into the cc list.
no this not how it work

I ask you to check which patch add the regression and then when this is
applied in the kernel we will request greg to apply it too to the stable

The code is using id = -1 appear 4 years ago. And after dig, I don't
find which patch cause this the regression.
so use git bisect to find the commit responsible

Thanks for your information.

Finally, I found the following commit add this:

3267c077e589bc146a0b45e220fcefafbf83fb80
[ARM] 3396/2: AT91RM9200 Platform devices update

BRs,
Bo Shen

Best Regards,
J.


--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux