[PATCH] AT91: Add VSYNC pin definition for AT91SAM9263 silicon

[PATCH] AT91: Add VSYNC pin definition for AT91SAM9263 silicon

[Hong Xu]

On Atmel's AT91SAM9263-EK board, the LCD panel does not need VSYNC
signal. But this signal is useful for the boards from other
vendors, as suggested by Nicholas Kinar.

This small patch adds the VSYNC pin definition by default.

Reported-by: Nicholas Kinar <n.kinar@usask.ca>
Signed-off-by: Hong Xu <hong.xu@atmel.com>
---
 arch/arm/mach-at91/at91sam9263_devices.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-at91/at91sam9263_devices.c b/arch/arm/mach-at91/at91sam9263_devices.c
index fb5c23a..7dab33e 100644
--- a/arch/arm/mach-at91/at91sam9263_devices.c
+++ b/arch/arm/mach-at91/at91sam9263_devices.c
@@ -832,6 +832,7 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_info *data)
 	if (!data)
 		return;
 
+	at91_set_A_periph(AT91_PIN_PC0, 0);	/* LCDVSYNC */
 	at91_set_A_periph(AT91_PIN_PC1, 0);	/* LCDHSYNC */
 	at91_set_A_periph(AT91_PIN_PC2, 0);	/* LCDDOTCK */
 	at91_set_A_periph(AT91_PIN_PC3, 0);	/* LCDDEN */
-- 
1.7.3.3

[PATCH] AT91: Add VSYNC pin definition for AT91SAM9263 silicon

[Jean-Christophe PLAGNIOL-VILLARD]

On 14:31 Wed 25 May     , Hong Xu wrote:
> On Atmel's AT91SAM9263-EK board, the LCD panel does not need VSYNC
> signal. But this signal is useful for the boards from other
> vendors, as suggested by Nicholas Kinar.
> 
> This small patch adds the VSYNC pin definition by default.
> 
> Reported-by: Nicholas Kinar <n.kinar@usask.ca>
> Signed-off-by: Hong Xu <hong.xu@atmel.com>
> ---
>  arch/arm/mach-at91/at91sam9263_devices.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/mach-at91/at91sam9263_devices.c b/arch/arm/mach-at91/at91sam9263_devices.c
> index fb5c23a..7dab33e 100644
> --- a/arch/arm/mach-at91/at91sam9263_devices.c
> +++ b/arch/arm/mach-at91/at91sam9263_devices.c
> @@ -832,6 +832,7 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_info *data)
>  	if (!data)
>  		return;
>  
> +	at91_set_A_periph(AT91_PIN_PC0, 0);	/* LCDVSYNC */
this must be optionnal

we need to update this with somethink like done on avr32

Best Regards,
J.

[PATCH] AT91: Add VSYNC pin definition for AT91SAM9263 silicon

[Hong Xu]

Hi,

On 05/25/2011 03:39 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 14:31 Wed 25 May     , Hong Xu wrote:
>> On Atmel's AT91SAM9263-EK board, the LCD panel does not need VSYNC
>> signal. But this signal is useful for the boards from other
>> vendors, as suggested by Nicholas Kinar.
>>
>> This small patch adds the VSYNC pin definition by default.
>>
>> Reported-by: Nicholas Kinar<n.kinar@usask.ca>
>> Signed-off-by: Hong Xu<hong.xu@atmel.com>
>> ---
>>   arch/arm/mach-at91/at91sam9263_devices.c |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-at91/at91sam9263_devices.c b/arch/arm/mach-at91/at91sam9263_devices.c
>> index fb5c23a..7dab33e 100644
>> --- a/arch/arm/mach-at91/at91sam9263_devices.c
>> +++ b/arch/arm/mach-at91/at91sam9263_devices.c
>> @@ -832,6 +832,7 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_info *data)
>>   	if (!data)
>>   		return;
>>
>> +	at91_set_A_periph(AT91_PIN_PC0, 0);	/* LCDVSYNC */
> this must be optionnal
>
> we need to update this with somethink like done on avr32
>

hm.. PC0 in SAM9263 has only "A" function (LCD VSYNC). As long as this 
pin is not used as GPIO, it's harmless to enable its peripheral function.

I just had a quick look at the AVR32 code without catching your 
"something like on AVR32". In AVR32, all ATMEL_LCDC_*_*BIT has "VSYNC" 
defined.

BR,
Eric

[PATCH] AT91: Add VSYNC pin definition for AT91SAM9263 silicon

[Jean-Christophe PLAGNIOL-VILLARD]

On 17:15 Wed 25 May     , Hong Xu wrote:
> Hi,
> 
> On 05/25/2011 03:39 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> >On 14:31 Wed 25 May     , Hong Xu wrote:
> >>On Atmel's AT91SAM9263-EK board, the LCD panel does not need VSYNC
> >>signal. But this signal is useful for the boards from other
> >>vendors, as suggested by Nicholas Kinar.
> >>
> >>This small patch adds the VSYNC pin definition by default.
> >>
> >>Reported-by: Nicholas Kinar<n.kinar@usask.ca>
> >>Signed-off-by: Hong Xu<hong.xu@atmel.com>
> >>---
> >>  arch/arm/mach-at91/at91sam9263_devices.c |    1 +
> >>  1 files changed, 1 insertions(+), 0 deletions(-)
> >>
> >>diff --git a/arch/arm/mach-at91/at91sam9263_devices.c b/arch/arm/mach-at91/at91sam9263_devices.c
> >>index fb5c23a..7dab33e 100644
> >>--- a/arch/arm/mach-at91/at91sam9263_devices.c
> >>+++ b/arch/arm/mach-at91/at91sam9263_devices.c
> >>@@ -832,6 +832,7 @@ void __init at91_add_device_lcdc(struct atmel_lcdfb_info *data)
> >>  	if (!data)
> >>  		return;
> >>
> >>+	at91_set_A_periph(AT91_PIN_PC0, 0);	/* LCDVSYNC */
> >this must be optionnal
> >
> >we need to update this with somethink like done on avr32
> >
> 
> hm.. PC0 in SAM9263 has only "A" function (LCD VSYNC). As long as
> this pin is not used as GPIO, it's harmless to enable its peripheral
> function.
> 
> I just had a quick look at the AVR32 code without catching your
> "something like on AVR32". In AVR32, all ATMEL_LCDC_*_*BIT has
> "VSYNC" defined.
my key point is not here, my key point is to make it optional
just take a look how did the avr32

Best Regards,
J.