Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux: Kernel

[PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info().

 

 

Linux kernel RSS feed   Index | Next | Previous | View Threaded


nicolas.ferre at atmel

Oct 9, 2009, 3:20 AM

Post #1 of 5 (150 views)
Permalink
[PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info().

From: Claudio Scordino <claudio [at] evidence>

This patch fixes a type mismatch when calling dev_info() in the
atmel_lcdfb.c driver.

Signed-off-by: Claudio Scordino <claudio [at] evidence>
Signed-off-by: Nicolas Ferre <nicolas.ferre [at] atmel>
---
drivers/video/atmel_lcdfb.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
index 2830ffd..317afab 100644
--- a/drivers/video/atmel_lcdfb.c
+++ b/drivers/video/atmel_lcdfb.c
@@ -959,7 +959,7 @@ static int __init atmel_lcdfb_probe(struct platform_device *pdev)
if (sinfo->atmel_lcdfb_power_control)
sinfo->atmel_lcdfb_power_control(1);

- dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %lu\n",
+ dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %d\n",
info->node, info->fix.mmio_start, sinfo->mmio, sinfo->irq_base);

return 0;
--
1.5.6.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


jkosina at suse

Oct 9, 2009, 8:31 AM

Post #2 of 5 (145 views)
Permalink
Re: [PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info(). [In reply to]

On Fri, 9 Oct 2009, Nicolas Ferre wrote:

> From: Claudio Scordino <claudio [at] evidence>
>
> This patch fixes a type mismatch when calling dev_info() in the
> atmel_lcdfb.c driver.
>
> Signed-off-by: Claudio Scordino <claudio [at] evidence>
> Signed-off-by: Nicolas Ferre <nicolas.ferre [at] atmel>
> ---
> drivers/video/atmel_lcdfb.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
> index 2830ffd..317afab 100644
> --- a/drivers/video/atmel_lcdfb.c
> +++ b/drivers/video/atmel_lcdfb.c
> @@ -959,7 +959,7 @@ static int __init atmel_lcdfb_probe(struct platform_device *pdev)
> if (sinfo->atmel_lcdfb_power_control)
> sinfo->atmel_lcdfb_power_control(1);
>
> - dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %lu\n",
> + dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %d\n",
> info->node, info->fix.mmio_start, sinfo->mmio, sinfo->irq_base);
>
> return 0;

Applied, thanks.

--
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


claudio at evidence

Nov 23, 2009, 12:30 AM

Post #3 of 5 (126 views)
Permalink
Re: [PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info(). [In reply to]

Jiri Kosina ha scritto:
> On Fri, 9 Oct 2009, Nicolas Ferre wrote:
>
>> From: Claudio Scordino <claudio [at] evidence>
>>
>> This patch fixes a type mismatch when calling dev_info() in the
>> atmel_lcdfb.c driver.
>>
>> Signed-off-by: Claudio Scordino <claudio [at] evidence>
>> Signed-off-by: Nicolas Ferre <nicolas.ferre [at] atmel>
>> ---
>> drivers/video/atmel_lcdfb.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c
>> index 2830ffd..317afab 100644
>> --- a/drivers/video/atmel_lcdfb.c
>> +++ b/drivers/video/atmel_lcdfb.c
>> @@ -959,7 +959,7 @@ static int __init atmel_lcdfb_probe(struct platform_device *pdev)
>> if (sinfo->atmel_lcdfb_power_control)
>> sinfo->atmel_lcdfb_power_control(1);
>>
>> - dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %lu\n",
>> + dev_info(dev, "fb%d: Atmel LCDC at 0x%08lx (mapped at %p), irq %d\n",
>> info->node, info->fix.mmio_start, sinfo->mmio, sinfo->irq_base);
>>
>> return 0;
>
> Applied, thanks.
>

Hi,

may I ask you the current status of this patch ? I noticed that it
has not been merged, yet.

Do we have any idea when it will be done ?

Since it is a trivial fix, I think it should be merged before the
release of v2.6.32 (i.e., waiting the next merge window does not make
much sense for such a trivial fix...).

Many thanks,

Claudio


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


jkosina at suse

Nov 23, 2009, 1:25 AM

Post #4 of 5 (126 views)
Permalink
Re: [PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info(). [In reply to]

On Mon, 23 Nov 2009, Claudio Scordino wrote:

> may I ask you the current status of this patch ? I noticed that it
> has not been merged, yet.
>
> Do we have any idea when it will be done ?
>
> Since it is a trivial fix, I think it should be merged before the
> release of v2.6.32 (i.e., waiting the next merge window does not make
> much sense for such a trivial fix...).

I usually don't send pull requests for trivial queue to Linus more than
once per release cycle.

--
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


claudio at evidence

Nov 23, 2009, 2:03 AM

Post #5 of 5 (129 views)
Permalink
Re: [PATCH TRIVIAL] atmel_lcdfb.c: Fix sinfo->irqbase from %lu to %d in dev_info(). [In reply to]

Jiri Kosina ha scritto:
> On Mon, 23 Nov 2009, Claudio Scordino wrote:
>
>
>> may I ask you the current status of this patch ? I noticed that it
>> has not been merged, yet.
>>
>> Do we have any idea when it will be done ?
>>
>> Since it is a trivial fix, I think it should be merged before the
>> release of v2.6.32 (i.e., waiting the next merge window does not make
>> much sense for such a trivial fix...).
>>
>
> I usually don't send pull requests for trivial queue to Linus more than
> once per release cycle.
>
Oh, I see. Sorry for the noise, then.

Many thanks again,

Claudio

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Linux kernel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.