Linux ATA/IDE development
 help / color / mirror / Atom feed
From: Marc Pignat <marc.pignat@hevs.ch>
To: Komuro <komurojun-mbn@nifty.com>
Cc: linux-pcmcia@lists.infradead.org, linux-ide@vger.kernel.org
Subject: Re: [PATCH] pata_pcmcia: another memory card support
Date: Thu, 6 Nov 2008 13:09:26 +0100	[thread overview]
Message-ID: <200811061309.26939.marc.pignat@hevs.ch> (raw)
In-Reply-To: <20081106205522.1c86a44b.komurojun-mbn@nifty.com>

Hi!

On Thursday 06 November 2008, Komuro wrote:
...
> Could you post the output of lspcmcia -vv?

Here it is:
armebs3:~# lspcmcia -vv
Socket 0 Bridge:        [at91_cf]       (bus ID: at91_cf)
        Configuration:  state: on       ready: yes
                        Voltage: 3.3V Vcc: 3.3V Vpp: 0.0V
                        Available IRQs: none
Socket 0 Device 0:      [pata_pcmcia]           (bus ID: 0.0)
        Configuration:  state: on
        Product Name:    512MB Flash Card 55LD017-A2 CFCA217I
        Identification: manf_id: 0x00bf card_id: 0x0001
                        function: 4 (fixed disk)
                        prod_id(1): " 512MB" (0xdc108d0e)
                        prod_id(2): "Flash Card" (0x5a362506)
                        prod_id(3): "55LD017-A2" (0xc0ede6dd)
                        prod_id(4): "CFCA217I" (0x8c1a6afb)
armebs3:~#        
> 
> I think it is better to use "PCMCIA_DEVICE_PROD_ID12"
> instead of "PCMCIA_DEVICE_PROD_ID2"
> 
> > +	PCMCIA_DEVICE_PROD_ID2("Flash Card", 0x5a362506),

I don't want to hardcode the size, however I'm not sure if other size have the
same prod_id2. " 512MB" with the space at the begginning let me hope there is
some card exists with 1024, 2048, ... or anything else in that place.

Best regards

Marc

  reply	other threads:[~2008-11-06 12:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-06 10:44 [PATCH] pata_pcmcia: another memory card support Marc Pignat
2008-11-06 11:55 ` Komuro
2008-11-06 12:09   ` Marc Pignat [this message]
2008-11-06 21:09     ` Komuro
2008-11-09 20:37 ` Dominik Brodowski
2008-11-11  8:02 ` Jeff Garzik
2008-11-11  8:40   ` Dominik Brodowski
2008-11-11  8:44     ` Jeff Garzik
2008-11-11 11:22       ` Dominik Brodowski
2008-11-11 11:59         ` Alan Cox
2008-11-11 12:25           ` Dominik Brodowski
2008-11-11 11:37       ` Alan Cox

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=200811061309.26939.marc.pignat@hevs.ch \
    --to=marc.pignat@hevs.ch \
    --cc=komurojun-mbn@nifty.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-pcmcia@lists.infradead.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