All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergei Shtylyov <sshtylyov@ru.mvista.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: Vitaly Bordug <vitb@kernel.crashing.org>,
	linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
	linuxppc-dev@ozlabs.org
Subject: Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)
Date: Wed, 25 Jul 2007 23:48:11 +0400	[thread overview]
Message-ID: <46A7A8FB.5090909@ru.mvista.com> (raw)
In-Reply-To: <Pine.LNX.4.60.0707252130090.6234@poirot.grange>

Guennadi Liakhovetski wrote:

>>This is now very similar to pata_platform.c, they both use
>>same platform data structure and same resources.

>>To achieve that, byte_lanes_swapping platform data variable
>>and platform specified iops removed from that driver. It's fine,
>>since those were never used anyway.

>>pata_platform and ide_platform are carrying same driver names,
>>to easily switch between these drivers, without need to touch
>>platform code.

> Why? There's a drivers/ide/arm/ide_arm.c IDe driver that some platforms 
> (not in the mainline) hack to access, e.g., CF cards in true-IDE mode. 
> About a month ago I submitted a patch to arm-linux-kernel switching that 

    Wrong list to submit sych stuff, post to linux-ide.

> driver to using platform-device. I got a reply, that it's not worth it now 
> that IDE is slowly becoming obsolete, and the pata_platform serves the 
> perpose perfectly well. I found this argument reasonable, I had the same

    Ignore such replies in the future. ;-)

> doubt, just wanted to double-check. So, why do we now need a new legacy 
> (a/drivers/ide/legacy/ide_platform.c) driver when a "modern" driver 
> exists?

    Good question (I know the answer but won't tell ;-).

> Thanks
> Guennadi

MBR, Sergei

WARNING: multiple messages have this Message-ID (diff)
From: Sergei Shtylyov <sshtylyov@ru.mvista.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org,
	linux-ide@vger.kernel.org
Subject: Re: [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver)
Date: Wed, 25 Jul 2007 23:48:11 +0400	[thread overview]
Message-ID: <46A7A8FB.5090909@ru.mvista.com> (raw)
In-Reply-To: <Pine.LNX.4.60.0707252130090.6234@poirot.grange>

Guennadi Liakhovetski wrote:

>>This is now very similar to pata_platform.c, they both use
>>same platform data structure and same resources.

>>To achieve that, byte_lanes_swapping platform data variable
>>and platform specified iops removed from that driver. It's fine,
>>since those were never used anyway.

>>pata_platform and ide_platform are carrying same driver names,
>>to easily switch between these drivers, without need to touch
>>platform code.

> Why? There's a drivers/ide/arm/ide_arm.c IDe driver that some platforms 
> (not in the mainline) hack to access, e.g., CF cards in true-IDE mode. 
> About a month ago I submitted a patch to arm-linux-kernel switching that 

    Wrong list to submit sych stuff, post to linux-ide.

> driver to using platform-device. I got a reply, that it's not worth it now 
> that IDE is slowly becoming obsolete, and the pata_platform serves the 
> perpose perfectly well. I found this argument reasonable, I had the same

    Ignore such replies in the future. ;-)

> doubt, just wanted to double-check. So, why do we now need a new legacy 
> (a/drivers/ide/legacy/ide_platform.c) driver when a "modern" driver 
> exists?

    Good question (I know the answer but won't tell ;-).

> Thanks
> Guennadi

MBR, Sergei

  reply	other threads:[~2007-07-25 19:46 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-25 16:53 [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver) Vitaly Bordug
2007-07-25 16:53 ` Vitaly Bordug
2007-07-25 16:53 ` Vitaly Bordug
2007-07-25 16:53 ` [PATCH 2/2] [POWERPC] MPC8349E-mITX: use platform IDE driver for CF interface Vitaly Bordug
2007-07-25 16:53   ` Vitaly Bordug
2007-07-25 16:53   ` Vitaly Bordug
2007-07-25 17:06   ` Sergei Shtylyov
2007-07-25 17:06     ` Sergei Shtylyov
2007-07-25 17:54     ` Sergei Shtylyov
2007-07-25 18:01       ` Scott Wood
2007-07-25 18:01         ` Scott Wood
2007-07-25 18:19         ` Sergei Shtylyov
2007-07-25 18:19           ` Sergei Shtylyov
2007-07-25 18:39           ` Scott Wood
2007-07-25 18:39             ` Scott Wood
2007-07-25 19:30             ` Sergei Shtylyov
2007-07-25 19:30               ` Sergei Shtylyov
2007-07-26 16:24               ` Sergei Shtylyov
2007-07-26 16:24                 ` Sergei Shtylyov
2007-07-31 22:47             ` Segher Boessenkool
2007-07-31 22:47               ` Segher Boessenkool
2007-07-25 18:29         ` Sergei Shtylyov
2007-07-25 18:29           ` Sergei Shtylyov
2007-07-25 18:46           ` Scott Wood
2007-07-25 18:46             ` Scott Wood
2007-07-26 17:21             ` Vitaly Bordug
2007-07-26 17:21               ` Vitaly Bordug
2007-07-26 17:45               ` Sergei Shtylyov
2007-07-26 17:45                 ` Sergei Shtylyov
2007-07-31 22:36       ` Segher Boessenkool
2007-07-31 22:36         ` Segher Boessenkool
2007-07-31 23:56         ` Alan Cox
2007-07-31 23:56           ` Alan Cox
2007-08-06 17:19           ` Segher Boessenkool
2007-08-06 17:19             ` Segher Boessenkool
2007-08-01 13:11         ` Sergei Shtylyov
2007-08-01 13:11           ` Sergei Shtylyov
2007-07-26 17:12     ` Vitaly Bordug
2007-07-26 17:12       ` Vitaly Bordug
2007-07-25 17:29   ` Scott Wood
2007-07-25 17:29     ` Scott Wood
2007-07-25 17:43 ` [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver) Alan Cox
2007-07-25 17:43   ` Alan Cox
2007-07-26 19:28   ` Bartlomiej Zolnierkiewicz
2007-07-26 19:28     ` Bartlomiej Zolnierkiewicz
2007-07-25 17:59 ` Kumar Gala
2007-07-25 17:59   ` Kumar Gala
2007-07-25 19:00 ` Sergei Shtylyov
2007-07-25 19:00   ` Sergei Shtylyov
2007-07-25 19:05   ` Scott Wood
2007-07-25 19:05     ` Scott Wood
2007-07-25 19:16     ` Sergei Shtylyov
2007-07-25 19:16       ` Sergei Shtylyov
2007-07-25 19:17       ` [PATCH 1/2] [IDE] Platform IDE driver Scott Wood
2007-07-25 19:17         ` Scott Wood
2007-07-25 19:35         ` Sergei Shtylyov
2007-07-25 19:35           ` Sergei Shtylyov
2007-07-31 22:31           ` Segher Boessenkool
2007-07-31 22:31             ` Segher Boessenkool
2007-08-01 12:39             ` Sergei Shtylyov
2007-08-01 12:39               ` Sergei Shtylyov
2007-08-06 17:16               ` Segher Boessenkool
2007-08-06 17:16                 ` Segher Boessenkool
2007-07-25 19:37 ` [PATCH 1/2] [IDE] Platform IDE driver (was: MMIO IDE driver) Guennadi Liakhovetski
2007-07-25 19:37   ` Guennadi Liakhovetski
2007-07-25 19:48   ` Sergei Shtylyov [this message]
2007-07-25 19:48     ` Sergei Shtylyov
2007-07-25 19:54     ` Guennadi Liakhovetski
2007-07-25 19:54       ` Guennadi Liakhovetski
2007-07-25 20:01       ` Sergei Shtylyov
2007-07-25 20:01         ` Sergei Shtylyov
2007-07-25 20:09         ` Guennadi Liakhovetski
2007-07-25 20:09           ` Guennadi Liakhovetski
2007-07-25 20:39   ` Alan Cox
2007-07-25 20:39     ` Alan Cox
2007-07-26 19:41     ` Guennadi Liakhovetski
2007-07-26 19:41       ` Guennadi Liakhovetski
2007-07-26 19:52       ` Sergei Shtylyov
2007-07-26 19:52         ` Sergei Shtylyov
2007-07-26 20:13         ` Guennadi Liakhovetski
2007-07-26 20:11       ` Bartlomiej Zolnierkiewicz
2007-07-26 20:11         ` Bartlomiej Zolnierkiewicz
2007-07-26 19:28 ` Bartlomiej Zolnierkiewicz
2007-07-26 19:28   ` Bartlomiej Zolnierkiewicz

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=46A7A8FB.5090909@ru.mvista.com \
    --to=sshtylyov@ru.mvista.com \
    --cc=g.liakhovetski@gmx.de \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=vitb@kernel.crashing.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.