From: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
To: David Miller <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
Cc: socketcan-fJ+pQTUTwRTk1uMJSBkQmQ@public.gmane.org,
linux-can-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
socketcan-users-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org
Subject: Re: [PATCH next-next 0/2] can: cc770: add support for the Bosch CC770 and Intel AN82527
Date: Thu, 01 Dec 2011 09:26:42 +0100 [thread overview]
Message-ID: <4ED73A42.7060602@grandegger.com> (raw)
In-Reply-To: <20111130.160727.1989923062226789802.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
On 11/30/2011 10:07 PM, David Miller wrote:
> From: Oliver Hartkopp <socketcan-fJ+pQTUTwRTk1uMJSBkQmQ@public.gmane.org>
> Date: Wed, 30 Nov 2011 07:37:08 +0100
>
>> On 30.11.2011 00:39, David Miller wrote:
>>
>>> From: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
>>> Date: Thu, 24 Nov 2011 13:07:26 +0100
>>>
>>>> Already since a while we have support for the Bosch CC770 and Intel
>>>> AN82527 CAN controllers in our out-of-tree Socket-CAN repository.
>>>> It would be nice if somebody could test the driver. Unfortunately,
>>>> I currently do not have hardware at hand. I have not yet ported the
>>>> OF platform driver as it needs further attaention due to the merge
>>>> of the platform and OF platform interface.
>>>>
>>>> Wolfgang Grandegger (2):
>>>> can: cc770: add driver core for the Bosch CC770 and Intel AN82527
>>>> can: cc770: legacy CC770 ISA bus driver
>>>
>>> All applied, thank you.
>>
>>
>> Hello Dave,
>>
>> this patchset was superseded for some time.
>>
>> We're currently in the v4 patchset (dated 2011-11-29).
>>
>> See http://patchwork.ozlabs.org/patch/128244/
>>
>> Do you want to revert the commit or should Wolfgang post a diff patch to get
>> to v4 ??
>
> A not very good job was done here communicating to me what is happening.
> Endless revisions, and not clear indication to me what should or should
> not be applied as a result.
Well, not sure what I should have communicated to you, but ...
> I want you guys to appoint someone to be the defacto CAN driver and
> subsystem maintainer who collects and merges all the driver and
> protocol patches into his tree, and acts as the one and only interface
> for me when changes are ready to be included.
... I agree that this will work better and it hopefully will then also
save your valuable time. We will discuss in the Socket-CAN community who
will/can act as the one and only interface to you.
> I sorted this out by reverting the older changes and applying V5.
>
> But I had to fix things up, when applying patch #3 there are empty
> trailing lines in some of the new files generated, and that causes
> git to complain.
Sorry for the bad quality of my patch series. I will try to do a better
job next time.
Thanks,
Wolfgang.
prev parent reply other threads:[~2011-12-01 8:26 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-24 12:07 [PATCH next-next 0/2] can: cc770: add support for the Bosch CC770 and Intel AN82527 Wolfgang Grandegger
2011-11-24 12:07 ` [PATCH net-next 1/2] can: cc770: add driver core " Wolfgang Grandegger
2011-11-24 12:07 ` [PATCH next-next 2/2] can: cc770: legacy CC770 ISA bus driver Wolfgang Grandegger
2011-11-29 23:39 ` [PATCH next-next 0/2] can: cc770: add support for the Bosch CC770 and Intel AN82527 David Miller
[not found] ` <20111129.183950.1327835803051184146.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2011-11-30 6:37 ` Oliver Hartkopp
[not found] ` <4ED5CF14.4020502-fJ+pQTUTwRTk1uMJSBkQmQ@public.gmane.org>
2011-11-30 8:29 ` Wolfgang Grandegger
[not found] ` <4ED5E94F.8010403-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>
2011-11-30 12:10 ` Wolfgang Grandegger
2011-11-30 21:07 ` David Miller
2011-11-30 21:14 ` David Miller
2011-12-01 9:43 ` Wolfgang Grandegger
2011-12-01 7:47 ` Oliver Hartkopp
2011-12-01 18:09 ` David Miller
2011-12-01 20:04 ` Marc Kleine-Budde
2011-12-12 9:13 ` the linux-can-next tree (was: Re: [PATCH next-next 0/2] can: cc770: add support for the Bosch CC770 and Intel AN82527) Marc Kleine-Budde
2011-12-16 21:14 ` the linux-can-next tree David Miller
[not found] ` <20111130.160727.1989923062226789802.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2011-12-01 8:26 ` Wolfgang Grandegger [this message]
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=4ED73A42.7060602@grandegger.com \
--to=wg-5yr1bzd7o62+xt7jha+gda@public.gmane.org \
--cc=davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org \
--cc=linux-can-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=socketcan-fJ+pQTUTwRTk1uMJSBkQmQ@public.gmane.org \
--cc=socketcan-users-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).