All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Vlasenko <vda.linux@googlemail.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: Adrian Bunk <bunk@kernel.org>, Andrew Morton <akpm@osdl.org>,
	linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
	hare@suse.de
Subject: Re: [PATCH] debloat aic7xxx and aic79xx drivers by deinlining
Date: Tue, 28 Aug 2007 20:55:06 +0100	[thread overview]
Message-ID: <200708282055.07411.vda.linux@googlemail.com> (raw)
In-Reply-To: <20070828081710.4687185c@laptopd505.fenrus.org>

On Tuesday 28 August 2007 16:17, Arjan van de Ven wrote:
> On Tue, 28 Aug 2007 12:56:34 +010
>
> Denys Vlasenko <vda.linux@googlemail.com> wrote:
> > >   make namespacecheck
> >
> > Thanks, nice tool.
> >
> > aic7xxx is kind of not very nice in this regard.
> >
> > See below what I get even on non-patched driver.
> >
> > I am willing to clean it up, but I still would like
> > "debloating" patch to be accepted.
>
> Fwiw I do like your debloat patch a lot; it's just only half the
> equation ... if you also do the namespace fixes, I bet the driver
> debloats even more...

Yes, I know, and I am happy to do that too. I just don't know
whether patches will be accepted.

Why this patch is not commented on by scsi people?
Am I sending patches to wrong people/lists?
--
vda

  reply	other threads:[~2007-08-28 19:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-25 21:57 [PATCH] debloat aic7xxx and aic79xx drivers by deinlining Denys Vlasenko
2007-08-25 21:57 ` Arjan van de Ven
2007-08-26 15:21   ` Denys Vlasenko
2007-08-27  2:03     ` Adrian Bunk
2007-08-28 11:56       ` Denys Vlasenko
2007-08-28 15:17         ` Arjan van de Ven
2007-08-28 19:55           ` Denys Vlasenko [this message]
2007-08-30 18:43             ` Jan Engelhardt
2007-08-30 19:02               ` Matthew Wilcox
2007-08-30 19:51                 ` Jan Engelhardt
2007-08-31 13:42                   ` Hannes Reinecke
  -- strict thread matches above, loose matches on Subject: below --
2007-07-31 10:14 Denis Vlasenko
2007-07-31  9:19 Denis Vlasenko

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=200708282055.07411.vda.linux@googlemail.com \
    --to=vda.linux@googlemail.com \
    --cc=akpm@osdl.org \
    --cc=arjan@infradead.org \
    --cc=bunk@kernel.org \
    --cc=hare@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.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 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.