All of lore.kernel.org
 help / color / mirror / Atom feed
From: Claudio Scordino <claudio@evidence.eu.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: alan@lxorguk.ukuu.org.uk, alan@redhat.com,
	hinko.kocevar@cetrtapot.si, janez.cufer@cetrtapot.si,
	starvik@axis.com, jesper.nilsson@axis.com, apw@canonical.com,
	linux-kernel@vger.kernel.org, dev-etrax@axis.com
Subject: Re: [PATCH]  Cleanup patch for RS485 on Cris architecture
Date: Fri, 27 Feb 2009 09:45:57 +0100	[thread overview]
Message-ID: <49A7A845.2030709@evidence.eu.com> (raw)
In-Reply-To: <20090226134924.e4d88605.akpm@linux-foundation.org>

Andrew Morton ha scritto:
> On Wed, 25 Feb 2009 11:07:52 +0100
> Claudio Scordino <claudio@evidence.eu.com> wrote:
> 
>> Dear Alan, dear Andrew,
>>
>>      around one month ago I submitted a cleanup patch to use the new 
>> general RS485 Linux data structure (introduced by Alan with commit 
>> number c26c56c0f40e200e61d1390629c806f6adaffbcc) in the Cris 
>> architecture too (currently, Cris still uses the old private data 
>> structure instead of the new one).
>>
>> The code compiled, but I couldn't test it at that time.
>>
>> Now, the code has been tested by Hinko Kocevar and Janez Cufer 
>> (CC:-ed) from the AXIS group, who also fixed a bug.
>>
>> Therefore, can you please commit the patch in attachment ?
>>
>> Many thanks,
>>
>>              Claudio Scordino
>>
>>
>> [0001-RS485-support-on-Cris-rewritten-in-order-to-use-new.patch  text/x-diff (9.9KB)]
>> >From 9d4397feff9773820e158693aa73621c0c20f017 Mon Sep 17 00:00:00 2001
>> From: Claudio Scordino <claudio@evidence.eu.com>
>> Date: Tue, 24 Feb 2009 18:00:44 +0100
>> Subject: [PATCH 1/1] RS485 support on Cris rewritten in order to use new serial_rs485 data structure
>>
>>
>> Signed-off-by: Claudio Scordino <claudio@evidence.eu.com>
>> Tested-by: Hinko Kocevar <hinko.kocevar@cetrtapot.si>
>> Tested-by: Janez Cufer <janez.cufer@cetrtapot.si>
> 
> The patch had no changelog - please always provide one.

Sorry, my apologies... Next time I'll keep in mind!

> 
> I scraped some text from your covering comment and made this:
> 
>     Use the new general RS485 Linux data structure (introduced by
>     Alan with commit number c26c56c0f40e200e61d1390629c806f6adaffbcc)
>     in the Cris architecture too (currently, Cris still uses the old
>     private data structure instead of the new one).
> 
> for a changelog.  Feel free to send better text if that is inappropriate
> or incomplete.

This changelog is perfect.

Many thanks,

          Claudio

  reply	other threads:[~2009-02-27  8:46 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-25 10:07 [PATCH] Cleanup patch for RS485 on Cris architecture Claudio Scordino
2009-02-26 21:49 ` Andrew Morton
2009-02-27  8:45   ` Claudio Scordino [this message]
2010-05-31  8:19 ` [PATCH] ioctl for getting RS485 information " Claudio Scordino
2010-05-31  8:33   ` Jesper Nilsson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49A7A845.2030709@evidence.eu.com \
    --to=claudio@evidence.eu.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=alan@redhat.com \
    --cc=apw@canonical.com \
    --cc=dev-etrax@axis.com \
    --cc=hinko.kocevar@cetrtapot.si \
    --cc=janez.cufer@cetrtapot.si \
    --cc=jesper.nilsson@axis.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=starvik@axis.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.