[PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Koen Kooi]

From: Koen Kooi <koen@beagleboard.org>

Changing it do 100kHz is needed to make more devices works properly. Controlling the TI DLP Pico projector[1] doesn't work properly at 400kHz, 100kHz and lower work fine. EDID readout is unaffected by this change.

[1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234

Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
 arch/arm/mach-omap2/board-omap3beagle.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index fe97bab..f7a6f7d 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -227,7 +227,9 @@ static int __init omap3_beagle_i2c_init(void)
 #ifdef CONFIG_I2C2_OMAP_BEAGLE
 	omap_register_i2c_bus(2, 400, NULL, 0);
 #endif
-	omap_register_i2c_bus(3, 400, NULL, 0);
+	/* Bus 3 is attached to the DVI port where devices like the pico DLP 
+	 * projector don't work reliably with 400kHz */
+	omap_register_i2c_bus(3, 100, NULL, 0);
 	return 0;
 }
 
-- 
1.5.6.5

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[David Brownell]

On Sunday 25 January 2009, Koen Kooi wrote:
> From: Koen Kooi <koen@beagleboard.org>
> 
> Changing it do 100kHz is needed to make more devices works properly. Controlling the TI DLP Pico projector[1] doesn't work properly at 400kHz, 100kHz and lower work fine. EDID readout is unaffected by this change.
> 
> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
> 
> Signed-off-by: Koen Kooi <koen@beagleboard.org>

Acked-by: David Brownell <dbrownell@users.sourceforge.net>

:)

> ---
>  arch/arm/mach-omap2/board-omap3beagle.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index fe97bab..f7a6f7d 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -227,7 +227,9 @@ static int __init omap3_beagle_i2c_init(void)
>  #ifdef CONFIG_I2C2_OMAP_BEAGLE
>  	omap_register_i2c_bus(2, 400, NULL, 0);
>  #endif
> -	omap_register_i2c_bus(3, 400, NULL, 0);
> +	/* Bus 3 is attached to the DVI port where devices like the pico DLP 
> +	 * projector don't work reliably with 400kHz */
> +	omap_register_i2c_bus(3, 100, NULL, 0);
>  	return 0;
>  }
>  
> -- 
> 1.5.6.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Koen Kooi]

[-- Attachment #1: Type: text/plain, Size: 603 bytes --]


Op 25 jan 2009, om 13:04 heeft David Brownell het volgende geschreven:

> On Sunday 25 January 2009, Koen Kooi wrote:
>> From: Koen Kooi <koen@beagleboard.org>
>>
>> Changing it do 100kHz is needed to make more devices works  
>> properly. Controlling the TI DLP Pico projector[1] doesn't work  
>> properly at 400kHz, 100kHz and lower work fine. EDID readout is  
>> unaffected by this change.
>>
>> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
>>
>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>
> Acked-by: David Brownell <dbrownell@users.sourceforge.net>

ping


[-- Attachment #2: Dit deel van het bericht is digitaal ondertekend --]
[-- Type: application/pgp-signature, Size: 186 bytes --]

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Koen Kooi]

[-- Attachment #1: Type: text/plain, Size: 753 bytes --]


Op 4 feb 2009, om 20:58 heeft Koen Kooi het volgende geschreven:

>
> Op 25 jan 2009, om 13:04 heeft David Brownell het volgende geschreven:
>
>> On Sunday 25 January 2009, Koen Kooi wrote:
>>> From: Koen Kooi <koen@beagleboard.org>
>>>
>>> Changing it do 100kHz is needed to make more devices works  
>>> properly. Controlling the TI DLP Pico projector[1] doesn't work  
>>> properly at 400kHz, 100kHz and lower work fine. EDID readout is  
>>> unaffected by this change.
>>>
>>> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
>>>
>>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>>
>> Acked-by: David Brownell <dbrownell@users.sourceforge.net>
>
> ping

Any more changes needed before this can go in?

regards,

Koen

[-- Attachment #2: Dit deel van het bericht is digitaal ondertekend --]
[-- Type: application/pgp-signature, Size: 186 bytes --]

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Tony Lindgren]

* Koen Kooi <k.kooi@student.utwente.nl> [090206 06:16]:
>
> Op 4 feb 2009, om 20:58 heeft Koen Kooi het volgende geschreven:
>
>>
>> Op 25 jan 2009, om 13:04 heeft David Brownell het volgende geschreven:
>>
>>> On Sunday 25 January 2009, Koen Kooi wrote:
>>>> From: Koen Kooi <koen@beagleboard.org>
>>>>
>>>> Changing it do 100kHz is needed to make more devices works  
>>>> properly. Controlling the TI DLP Pico projector[1] doesn't work  
>>>> properly at 400kHz, 100kHz and lower work fine. EDID readout is  
>>>> unaffected by this change.
>>>>
>>>> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
>>>>
>>>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>>>
>>> Acked-by: David Brownell <dbrownell@users.sourceforge.net>
>>
>> ping
>
> Any more changes needed before this can go in?

Just few more days, I'll need to get my upstream queues ready..
Otherwise I'll forget to apply it somewhere :)

Tony

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Tony Lindgren]

* Tony Lindgren <tony@atomide.com> [090206 14:56]:
> * Koen Kooi <k.kooi@student.utwente.nl> [090206 06:16]:
> >
> > Op 4 feb 2009, om 20:58 heeft Koen Kooi het volgende geschreven:
> >
> >>
> >> Op 25 jan 2009, om 13:04 heeft David Brownell het volgende geschreven:
> >>
> >>> On Sunday 25 January 2009, Koen Kooi wrote:
> >>>> From: Koen Kooi <koen@beagleboard.org>
> >>>>
> >>>> Changing it do 100kHz is needed to make more devices works  
> >>>> properly. Controlling the TI DLP Pico projector[1] doesn't work  
> >>>> properly at 400kHz, 100kHz and lower work fine. EDID readout is  
> >>>> unaffected by this change.
> >>>>
> >>>> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
> >>>>
> >>>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
> >>>
> >>> Acked-by: David Brownell <dbrownell@users.sourceforge.net>
> >>
> >> ping
> >
> > Any more changes needed before this can go in?
> 
> Just few more days, I'll need to get my upstream queues ready..
> Otherwise I'll forget to apply it somewhere :)

OK, adding to omap-fixes and pushing to linux-omap.

Tony

Re: [PATCH v2] board-omap3beagle: set i2c-3 to 100kHz

[Koen Kooi]

[-- Attachment #1: Type: text/plain, Size: 1178 bytes --]


Op 20 feb 2009, om 17:25 heeft Tony Lindgren het volgende geschreven:

> * Tony Lindgren <tony@atomide.com> [090206 14:56]:
>> * Koen Kooi <k.kooi@student.utwente.nl> [090206 06:16]:
>>>
>>> Op 4 feb 2009, om 20:58 heeft Koen Kooi het volgende geschreven:
>>>
>>>>
>>>> Op 25 jan 2009, om 13:04 heeft David Brownell het volgende  
>>>> geschreven:
>>>>
>>>>> On Sunday 25 January 2009, Koen Kooi wrote:
>>>>>> From: Koen Kooi <koen@beagleboard.org>
>>>>>>
>>>>>> Changing it do 100kHz is needed to make more devices works
>>>>>> properly. Controlling the TI DLP Pico projector[1] doesn't work
>>>>>> properly at 400kHz, 100kHz and lower work fine. EDID readout is
>>>>>> unaffected by this change.
>>>>>>
>>>>>> [1] http://focus.ti.com/dlpdmd/docs/dlpdiscovery.tsp?sectionId=60&tabId=2234
>>>>>>
>>>>>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>>>>>
>>>>> Acked-by: David Brownell <dbrownell@users.sourceforge.net>
>>>>
>>>> ping
>>>
>>> Any more changes needed before this can go in?
>>
>> Just few more days, I'll need to get my upstream queues ready..
>> Otherwise I'll forget to apply it somewhere :)
>
> OK, adding to omap-fixes and pushing to linux-omap.

Thanks!

[-- Attachment #2: Dit deel van het bericht is digitaal ondertekend --]
[-- Type: application/pgp-signature, Size: 186 bytes --]