All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <w.sang@pengutronix.de>
To: Manuel Lauss <manuel.lauss@googlemail.com>
Cc: linux-pcmcia <linux-pcmcia@lists.infradead.org>,
	Linux-MIPS <linux-mips@linux-mips.org>,
	Florian Fainelli <florian@openwrt.org>,
	Manuel Lauss <manuel.lauss@gmail.com>
Subject: Re: [PATCH] Alchemy: XXS1500 PCMCIA driver rewrite
Date: Fri, 2 Oct 2009 14:54:23 +0200	[thread overview]
Message-ID: <20091002125423.GD3179@pengutronix.de> (raw)
In-Reply-To: <f861ec6f0910020415j5125295fn6b5dff7db4bf170e@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1084 bytes --]


> >> Rewritten XXS1500 PCMCIA socket driver, standalone (doesn't
> >> depend on au1000_generic.c) and added carddetect IRQ support.
> >
> > I am not familiar with the details here. Why did you choose to drop the generic
> > au1000-part for this and the other driver?
> 
> I want to get rid of au1000_generic.[ch] eventually or at least all of its
> contents except the static mapping and resource allocation functions, which
> are board- independent.  On the other hand these are so short that I opted to
> just duplicate them into the xxs1500_ss.c and db1xxx_ss.c (other patch)
> files.  The db1xxx_ss (for the Alchemy demoboards) is supposed to be an
> example on how to set up PCMCIA on Alchemy SoCs.

Yeah, I saw that you want to remove it, still I don't know why :) Is it feature
incomplete and updating is impossible? Is the concept outdated? Could you
enlighten me on that?

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2009-10-02 12:54 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-29 18:50 [PATCH] Alchemy: XXS1500 PCMCIA driver rewrite Manuel Lauss
2009-09-29 19:46 ` Florian Fainelli
2009-10-02 10:59 ` Wolfram Sang
2009-10-02 11:15   ` Manuel Lauss
2009-10-02 12:54     ` Wolfram Sang [this message]
2009-10-02 14:32       ` Manuel Lauss
2009-10-03 10:22         ` Wolfram Sang
2009-10-03 11:49           ` Manuel Lauss
2009-10-03 14:03             ` Ralf Baechle
2009-10-03 14:36               ` Manuel Lauss

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=20091002125423.GD3179@pengutronix.de \
    --to=w.sang@pengutronix.de \
    --cc=florian@openwrt.org \
    --cc=linux-mips@linux-mips.org \
    --cc=linux-pcmcia@lists.infradead.org \
    --cc=manuel.lauss@gmail.com \
    --cc=manuel.lauss@googlemail.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.