Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 18:14:33 +0100

> I asked you about this in private mail in August 2015, you told me to
> bring this on the list.  I did it (with these patches) in February
> 2016.  After two pings and months of waiting for a reply all I get is
> is a quick NAK?

Yep, that's how much effort, consideration, and time a deprecated
subsystem deserves.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 18:14:33 +0100

> I asked you about this in private mail in August 2015, you told me to
> bring this on the list.  I did it (with these patches) in February
> 2016.  After two pings and months of waiting for a reply all I get is
> is a quick NAK?

Yep, that's how much effort, consideration, and time a deprecated
subsystem deserves.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

On Thursday, December 08, 2016 11:39:57 AM David Miller wrote:
> From: Bartlomiej Zolnierkiewicz 
> Date: Thu, 08 Dec 2016 17:15:16 +0100
> 
> > On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
> >> From: Bartlomiej Zolnierkiewicz 
> >> Date: Thu, 08 Dec 2016 14:42:12 +0100
> >> 
> >> > Ping.. since there was no negative (or any other) feedback I think that
> >> > this can be put into -next for some wider exposure..
> >> 
> >> I'm not going to apply this and break things on people, sorry.
> > 
> > People building their own kernels with their own kernel config files
> > and still using IDE drivers (deprecated in 2009 BTW) will have to
> > update their setups to libata but otherwise nothing is supposed to
> > break.  So could you please explain in more detail what do you mean
> > by "break things on people"?
> 
> There is no proof that the PATA drivers work %100 reliably as well as
> the IDE driver they replace for every possible chip and architecture.

This is why only some IDE host drivers were nominated for removal.

Please see cover-letter and patch descriptions for details.

> Therefore the only safe thing is to keep the IDE drivers around
> forever.

Uh...

> They are not a maintainence burdon, I rarely get more than 1 patch
> each merge window and most of the time those are cleanups or for the
> handling of a kernel wide API change rather than bug fixes.

In the long-term perspective having two drivers for the same hardware
is not good for the whole kernel as:

- testing efforts are divided (while number of PATA systems goes down)

- bugs/missing features in the new subsystem are not getting reported
  and fixed (since it is easier to just switch back to the old stack)

- having duplicated support for the same hardware confuses users

I was hoping for incremental removal of IDE host drivers and keeping
only these that are really needed (while at the same time migrating
them slowly to libata).

I asked you about this in private mail in August 2015, you told me to
bring this on the list.  I did it (with these patches) in February
2016.  After two pings and months of waiting for a reply all I get is
is a quick NAK?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

On Thursday, December 08, 2016 11:39:57 AM David Miller wrote:
> From: Bartlomiej Zolnierkiewicz 
> Date: Thu, 08 Dec 2016 17:15:16 +0100
> 
> > On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
> >> From: Bartlomiej Zolnierkiewicz 
> >> Date: Thu, 08 Dec 2016 14:42:12 +0100
> >> 
> >> > Ping.. since there was no negative (or any other) feedback I think that
> >> > this can be put into -next for some wider exposure..
> >> 
> >> I'm not going to apply this and break things on people, sorry.
> > 
> > People building their own kernels with their own kernel config files
> > and still using IDE drivers (deprecated in 2009 BTW) will have to
> > update their setups to libata but otherwise nothing is supposed to
> > break.  So could you please explain in more detail what do you mean
> > by "break things on people"?
> 
> There is no proof that the PATA drivers work %100 reliably as well as
> the IDE driver they replace for every possible chip and architecture.

This is why only some IDE host drivers were nominated for removal.

Please see cover-letter and patch descriptions for details.

> Therefore the only safe thing is to keep the IDE drivers around
> forever.

Uh...

> They are not a maintainence burdon, I rarely get more than 1 patch
> each merge window and most of the time those are cleanups or for the
> handling of a kernel wide API change rather than bug fixes.

In the long-term perspective having two drivers for the same hardware
is not good for the whole kernel as:

- testing efforts are divided (while number of PATA systems goes down)

- bugs/missing features in the new subsystem are not getting reported
  and fixed (since it is easier to just switch back to the old stack)

- having duplicated support for the same hardware confuses users

I was hoping for incremental removal of IDE host drivers and keeping
only these that are really needed (while at the same time migrating
them slowly to libata).

I asked you about this in private mail in August 2015, you told me to
bring this on the list.  I did it (with these patches) in February
2016.  After two pings and months of waiting for a reply all I get is
is a quick NAK?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Christoph Hellwig]

On Thu, Dec 08, 2016 at 11:39:57AM -0500, David Miller wrote:
> They are not a maintainence burdon, I rarely get more than 1 patch
> each merge window and most of the time those are cleanups or for the
> handling of a kernel wide API change rather than bug fixes.

They are a major maintainence burden for the block layer due to horrible
abuse of all kinds of block layer APIs.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Christoph Hellwig]

On Thu, Dec 08, 2016 at 11:39:57AM -0500, David Miller wrote:
> They are not a maintainence burdon, I rarely get more than 1 patch
> each merge window and most of the time those are cleanups or for the
> handling of a kernel wide API change rather than bug fixes.

They are a major maintainence burden for the block layer due to horrible
abuse of all kinds of block layer APIs.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 17:15:16 +0100

> On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
>> From: Bartlomiej Zolnierkiewicz 
>> Date: Thu, 08 Dec 2016 14:42:12 +0100
>> 
>> > Ping.. since there was no negative (or any other) feedback I think that
>> > this can be put into -next for some wider exposure..
>> 
>> I'm not going to apply this and break things on people, sorry.
> 
> People building their own kernels with their own kernel config files
> and still using IDE drivers (deprecated in 2009 BTW) will have to
> update their setups to libata but otherwise nothing is supposed to
> break.  So could you please explain in more detail what do you mean
> by "break things on people"?

There is no proof that the PATA drivers work %100 reliably as well as
the IDE driver they replace for every possible chip and architecture.

Therefore the only safe thing is to keep the IDE drivers around
forever.

They are not a maintainence burdon, I rarely get more than 1 patch
each merge window and most of the time those are cleanups or for the
handling of a kernel wide API change rather than bug fixes.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 17:15:16 +0100

> On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
>> From: Bartlomiej Zolnierkiewicz 
>> Date: Thu, 08 Dec 2016 14:42:12 +0100
>> 
>> > Ping.. since there was no negative (or any other) feedback I think that
>> > this can be put into -next for some wider exposure..
>> 
>> I'm not going to apply this and break things on people, sorry.
> 
> People building their own kernels with their own kernel config files
> and still using IDE drivers (deprecated in 2009 BTW) will have to
> update their setups to libata but otherwise nothing is supposed to
> break.  So could you please explain in more detail what do you mean
> by "break things on people"?

There is no proof that the PATA drivers work %100 reliably as well as
the IDE driver they replace for every possible chip and architecture.

Therefore the only safe thing is to keep the IDE drivers around
forever.

They are not a maintainence burdon, I rarely get more than 1 patch
each merge window and most of the time those are cleanups or for the
handling of a kernel wide API change rather than bug fixes.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
> From: Bartlomiej Zolnierkiewicz 
> Date: Thu, 08 Dec 2016 14:42:12 +0100
> 
> > Ping.. since there was no negative (or any other) feedback I think that
> > this can be put into -next for some wider exposure..
> 
> I'm not going to apply this and break things on people, sorry.

People building their own kernels with their own kernel config files
and still using IDE drivers (deprecated in 2009 BTW) will have to
update their setups to libata but otherwise nothing is supposed to
break.  So could you please explain in more detail what do you mean
by "break things on people"?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

On Thursday, December 08, 2016 10:23:08 AM David Miller wrote:
> From: Bartlomiej Zolnierkiewicz 
> Date: Thu, 08 Dec 2016 14:42:12 +0100
> 
> > Ping.. since there was no negative (or any other) feedback I think that
> > this can be put into -next for some wider exposure..
> 
> I'm not going to apply this and break things on people, sorry.

People building their own kernels with their own kernel config files
and still using IDE drivers (deprecated in 2009 BTW) will have to
update their setups to libata but otherwise nothing is supposed to
break.  So could you please explain in more detail what do you mean
by "break things on people"?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 14:42:12 +0100

> Ping.. since there was no negative (or any other) feedback I think that
> this can be put into -next for some wider exposure..

I'm not going to apply this and break things on people, sorry.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[David Miller]

From: Bartlomiej Zolnierkiewicz 
Date: Thu, 08 Dec 2016 14:42:12 +0100

> Ping.. since there was no negative (or any other) feedback I think that
> this can be put into -next for some wider exposure..

I'm not going to apply this and break things on people, sorry.

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi,

On Friday, July 08, 2016 07:07:46 PM Bartlomiej Zolnierkiewicz wrote:
> 
> Hi Dave,
> 
> What is your opinion on this patchset?

Ping.. since there was no negative (or any other) feedback I think that
this can be put into -next for some wider exposure..

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R Institute Poland
> Samsung Electronics
> 
> On Thursday, February 04, 2016 02:43:57 PM Bartlomiej Zolnierkiewicz wrote:
> > Hi,
> > 
> > IDE subsystem has been deprecated since 2009 and the majority
> > (if not all) of Linux distributions have switched to use
> > libata for ATA support exclusively.
> > 
> > This patchset removes IDE host drivers that:
> > - have been replaced by their corresponding libata PATA host
> >   drivers (which have been available since 2006/2007)
> > - are not used by any defconfig
> > 
> > Having duplicated drivers for the same hardware doesn't seem
> > to be good thing in the long-term perspective for Linux as
> > - testing efforts are divided between two drivers/subsystems
> >   (while there is less and less PATA systems left in use)
> > - having two drivers for the same hardware confuses users
> > 
> > Please note that all the hardware supported by IDE host drivers
> > nominated for removal remains supported by correspoding libata
> > PATA host drivers.
> > 
> > PS If this patchset is accepted I would like to continue
> > removal of IDE host drivers in part 2 which will happen after
> > remaining defconfigs which still are using IDE host drivers
> > get converted to use corresponding libata PATA host drivers.
> > 
> > Best regards,
> > --
> > Bartlomiej Zolnierkiewicz
> > Samsung R Institute Poland
> > Samsung Electronics
> > 
> > 
> > Bartlomiej Zolnierkiewicz (19):
> >   ide: remove deprecated atiixp host driver
> >   ide: remove deprecated cs5520 host driver
> >   ide: remove deprecated cs5530 host driver
> >   ide: remove deprecated cs5535 host driver
> >   ide: remove deprecated cs5536 host driver
> >   ide: remove deprecated sc1200 host driver
> >   ide: remove deprecated delkin_cb host driver
> >   ide: remove deprecated it8213 host driver
> >   ide: remove deprecated it821x host driver
> >   ide: remove deprecated jmicron host driver
> >   ide: remove deprecated opti621 host driver
> >   ide: remove deprecated pdc202xx_old host driver
> >   ide: remove deprecated rz1000 host driver
> >   ide: remove deprecated serverworks host driver
> >   ide: remove deprecated sis5513 host driver
> >   ide: remove deprecated slc90e66 host driver
> >   ide: remove deprecated triflex host driver
> >   ide: remove deprecated ide-pnp host driver
> >   ide: remove deprecated icside host driver
> > 
> >  drivers/ide/Kconfig| 200 +
> >  drivers/ide/Makefile   |  19 --
> >  drivers/ide/atiixp.c   | 211 -
> >  drivers/ide/cs5520.c   | 168 ---
> >  drivers/ide/cs5530.c   | 295 ---
> >  drivers/ide/cs5535.c   | 219 --
> >  drivers/ide/cs5536.c   | 306 ---
> >  drivers/ide/delkin_cb.c| 181 
> >  drivers/ide/icside.c   | 691 
> > ---
> >  drivers/ide/ide-pnp.c  | 100 ---
> >  drivers/ide/it8213.c   | 216 --
> >  drivers/ide/it821x.c   | 715 
> > -
> >  drivers/ide/jmicron.c  | 176 ---
> >  drivers/ide/opti621.c  | 178 ---
> >  drivers/ide/pdc202xx_old.c | 361 ---
> >  drivers/ide/rz1000.c   |  99 ---
> >  drivers/ide/sc1200.c   | 355 --
> >  drivers/ide/serverworks.c  | 449 
> >  drivers/ide/sis5513.c  | 636 
> >  drivers/ide/slc90e66.c | 181 
> >  drivers/ide/triflex.c  | 155 --
> >  21 files changed, 2 insertions(+), 5909 deletions(-)
> >  delete mode 100644 drivers/ide/atiixp.c
> >  delete mode 100644 drivers/ide/cs5520.c
> >  delete mode 100644 drivers/ide/cs5530.c
> >  delete mode 100644 drivers/ide/cs5535.c
> >  delete mode 100644 drivers/ide/cs5536.c
> >  delete mode 100644 drivers/ide/delkin_cb.c
> >  delete mode 100644 drivers/ide/icside.c
> >  delete mode 100644 drivers/ide/ide-pnp.c
> >  delete mode 100644 drivers/ide/it8213.c
> >  delete mode 100644 drivers/ide/it821x.c
> >  delete mode 100644 drivers/ide/jmicron.c
> >  delete mode 100644 drivers/ide/opti621.c
> >  delete mode 100644 drivers/ide/pdc202xx_old.c
> >  delete mode 100644 drivers/ide/rz1000.c
> >  delete mode 100644 drivers/ide/sc1200.c
> >  delete mode 100644 drivers/ide/serverworks.c
> >  delete mode 100644 drivers/ide/sis5513.c
> >  delete mode 100644 drivers/ide/slc90e66.c
> >  delete mode 100644 drivers/ide/triflex.c

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi,

On Friday, July 08, 2016 07:07:46 PM Bartlomiej Zolnierkiewicz wrote:
> 
> Hi Dave,
> 
> What is your opinion on this patchset?

Ping.. since there was no negative (or any other) feedback I think that
this can be put into -next for some wider exposure..

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R Institute Poland
> Samsung Electronics
> 
> On Thursday, February 04, 2016 02:43:57 PM Bartlomiej Zolnierkiewicz wrote:
> > Hi,
> > 
> > IDE subsystem has been deprecated since 2009 and the majority
> > (if not all) of Linux distributions have switched to use
> > libata for ATA support exclusively.
> > 
> > This patchset removes IDE host drivers that:
> > - have been replaced by their corresponding libata PATA host
> >   drivers (which have been available since 2006/2007)
> > - are not used by any defconfig
> > 
> > Having duplicated drivers for the same hardware doesn't seem
> > to be good thing in the long-term perspective for Linux as
> > - testing efforts are divided between two drivers/subsystems
> >   (while there is less and less PATA systems left in use)
> > - having two drivers for the same hardware confuses users
> > 
> > Please note that all the hardware supported by IDE host drivers
> > nominated for removal remains supported by correspoding libata
> > PATA host drivers.
> > 
> > PS If this patchset is accepted I would like to continue
> > removal of IDE host drivers in part 2 which will happen after
> > remaining defconfigs which still are using IDE host drivers
> > get converted to use corresponding libata PATA host drivers.
> > 
> > Best regards,
> > --
> > Bartlomiej Zolnierkiewicz
> > Samsung R Institute Poland
> > Samsung Electronics
> > 
> > 
> > Bartlomiej Zolnierkiewicz (19):
> >   ide: remove deprecated atiixp host driver
> >   ide: remove deprecated cs5520 host driver
> >   ide: remove deprecated cs5530 host driver
> >   ide: remove deprecated cs5535 host driver
> >   ide: remove deprecated cs5536 host driver
> >   ide: remove deprecated sc1200 host driver
> >   ide: remove deprecated delkin_cb host driver
> >   ide: remove deprecated it8213 host driver
> >   ide: remove deprecated it821x host driver
> >   ide: remove deprecated jmicron host driver
> >   ide: remove deprecated opti621 host driver
> >   ide: remove deprecated pdc202xx_old host driver
> >   ide: remove deprecated rz1000 host driver
> >   ide: remove deprecated serverworks host driver
> >   ide: remove deprecated sis5513 host driver
> >   ide: remove deprecated slc90e66 host driver
> >   ide: remove deprecated triflex host driver
> >   ide: remove deprecated ide-pnp host driver
> >   ide: remove deprecated icside host driver
> > 
> >  drivers/ide/Kconfig| 200 +
> >  drivers/ide/Makefile   |  19 --
> >  drivers/ide/atiixp.c   | 211 -
> >  drivers/ide/cs5520.c   | 168 ---
> >  drivers/ide/cs5530.c   | 295 ---
> >  drivers/ide/cs5535.c   | 219 --
> >  drivers/ide/cs5536.c   | 306 ---
> >  drivers/ide/delkin_cb.c| 181 
> >  drivers/ide/icside.c   | 691 
> > ---
> >  drivers/ide/ide-pnp.c  | 100 ---
> >  drivers/ide/it8213.c   | 216 --
> >  drivers/ide/it821x.c   | 715 
> > -
> >  drivers/ide/jmicron.c  | 176 ---
> >  drivers/ide/opti621.c  | 178 ---
> >  drivers/ide/pdc202xx_old.c | 361 ---
> >  drivers/ide/rz1000.c   |  99 ---
> >  drivers/ide/sc1200.c   | 355 --
> >  drivers/ide/serverworks.c  | 449 
> >  drivers/ide/sis5513.c  | 636 
> >  drivers/ide/slc90e66.c | 181 
> >  drivers/ide/triflex.c  | 155 --
> >  21 files changed, 2 insertions(+), 5909 deletions(-)
> >  delete mode 100644 drivers/ide/atiixp.c
> >  delete mode 100644 drivers/ide/cs5520.c
> >  delete mode 100644 drivers/ide/cs5530.c
> >  delete mode 100644 drivers/ide/cs5535.c
> >  delete mode 100644 drivers/ide/cs5536.c
> >  delete mode 100644 drivers/ide/delkin_cb.c
> >  delete mode 100644 drivers/ide/icside.c
> >  delete mode 100644 drivers/ide/ide-pnp.c
> >  delete mode 100644 drivers/ide/it8213.c
> >  delete mode 100644 drivers/ide/it821x.c
> >  delete mode 100644 drivers/ide/jmicron.c
> >  delete mode 100644 drivers/ide/opti621.c
> >  delete mode 100644 drivers/ide/pdc202xx_old.c
> >  delete mode 100644 drivers/ide/rz1000.c
> >  delete mode 100644 drivers/ide/sc1200.c
> >  delete mode 100644 drivers/ide/serverworks.c
> >  delete mode 100644 drivers/ide/sis5513.c
> >  delete mode 100644 drivers/ide/slc90e66.c
> >  delete mode 100644 drivers/ide/triflex.c

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi Dave,

What is your opinion on this patchset?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

On Thursday, February 04, 2016 02:43:57 PM Bartlomiej Zolnierkiewicz wrote:
> Hi,
> 
> IDE subsystem has been deprecated since 2009 and the majority
> (if not all) of Linux distributions have switched to use
> libata for ATA support exclusively.
> 
> This patchset removes IDE host drivers that:
> - have been replaced by their corresponding libata PATA host
>   drivers (which have been available since 2006/2007)
> - are not used by any defconfig
> 
> Having duplicated drivers for the same hardware doesn't seem
> to be good thing in the long-term perspective for Linux as
> - testing efforts are divided between two drivers/subsystems
>   (while there is less and less PATA systems left in use)
> - having two drivers for the same hardware confuses users
> 
> Please note that all the hardware supported by IDE host drivers
> nominated for removal remains supported by correspoding libata
> PATA host drivers.
> 
> PS If this patchset is accepted I would like to continue
> removal of IDE host drivers in part 2 which will happen after
> remaining defconfigs which still are using IDE host drivers
> get converted to use corresponding libata PATA host drivers.
> 
> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R Institute Poland
> Samsung Electronics
> 
> 
> Bartlomiej Zolnierkiewicz (19):
>   ide: remove deprecated atiixp host driver
>   ide: remove deprecated cs5520 host driver
>   ide: remove deprecated cs5530 host driver
>   ide: remove deprecated cs5535 host driver
>   ide: remove deprecated cs5536 host driver
>   ide: remove deprecated sc1200 host driver
>   ide: remove deprecated delkin_cb host driver
>   ide: remove deprecated it8213 host driver
>   ide: remove deprecated it821x host driver
>   ide: remove deprecated jmicron host driver
>   ide: remove deprecated opti621 host driver
>   ide: remove deprecated pdc202xx_old host driver
>   ide: remove deprecated rz1000 host driver
>   ide: remove deprecated serverworks host driver
>   ide: remove deprecated sis5513 host driver
>   ide: remove deprecated slc90e66 host driver
>   ide: remove deprecated triflex host driver
>   ide: remove deprecated ide-pnp host driver
>   ide: remove deprecated icside host driver
> 
>  drivers/ide/Kconfig| 200 +
>  drivers/ide/Makefile   |  19 --
>  drivers/ide/atiixp.c   | 211 -
>  drivers/ide/cs5520.c   | 168 ---
>  drivers/ide/cs5530.c   | 295 ---
>  drivers/ide/cs5535.c   | 219 --
>  drivers/ide/cs5536.c   | 306 ---
>  drivers/ide/delkin_cb.c| 181 
>  drivers/ide/icside.c   | 691 ---
>  drivers/ide/ide-pnp.c  | 100 ---
>  drivers/ide/it8213.c   | 216 --
>  drivers/ide/it821x.c   | 715 
> -
>  drivers/ide/jmicron.c  | 176 ---
>  drivers/ide/opti621.c  | 178 ---
>  drivers/ide/pdc202xx_old.c | 361 ---
>  drivers/ide/rz1000.c   |  99 ---
>  drivers/ide/sc1200.c   | 355 --
>  drivers/ide/serverworks.c  | 449 
>  drivers/ide/sis5513.c  | 636 
>  drivers/ide/slc90e66.c | 181 
>  drivers/ide/triflex.c  | 155 --
>  21 files changed, 2 insertions(+), 5909 deletions(-)
>  delete mode 100644 drivers/ide/atiixp.c
>  delete mode 100644 drivers/ide/cs5520.c
>  delete mode 100644 drivers/ide/cs5530.c
>  delete mode 100644 drivers/ide/cs5535.c
>  delete mode 100644 drivers/ide/cs5536.c
>  delete mode 100644 drivers/ide/delkin_cb.c
>  delete mode 100644 drivers/ide/icside.c
>  delete mode 100644 drivers/ide/ide-pnp.c
>  delete mode 100644 drivers/ide/it8213.c
>  delete mode 100644 drivers/ide/it821x.c
>  delete mode 100644 drivers/ide/jmicron.c
>  delete mode 100644 drivers/ide/opti621.c
>  delete mode 100644 drivers/ide/pdc202xx_old.c
>  delete mode 100644 drivers/ide/rz1000.c
>  delete mode 100644 drivers/ide/sc1200.c
>  delete mode 100644 drivers/ide/serverworks.c
>  delete mode 100644 drivers/ide/sis5513.c
>  delete mode 100644 drivers/ide/slc90e66.c
>  delete mode 100644 drivers/ide/triflex.c

Re: [RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi Dave,

What is your opinion on this patchset?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

On Thursday, February 04, 2016 02:43:57 PM Bartlomiej Zolnierkiewicz wrote:
> Hi,
> 
> IDE subsystem has been deprecated since 2009 and the majority
> (if not all) of Linux distributions have switched to use
> libata for ATA support exclusively.
> 
> This patchset removes IDE host drivers that:
> - have been replaced by their corresponding libata PATA host
>   drivers (which have been available since 2006/2007)
> - are not used by any defconfig
> 
> Having duplicated drivers for the same hardware doesn't seem
> to be good thing in the long-term perspective for Linux as
> - testing efforts are divided between two drivers/subsystems
>   (while there is less and less PATA systems left in use)
> - having two drivers for the same hardware confuses users
> 
> Please note that all the hardware supported by IDE host drivers
> nominated for removal remains supported by correspoding libata
> PATA host drivers.
> 
> PS If this patchset is accepted I would like to continue
> removal of IDE host drivers in part 2 which will happen after
> remaining defconfigs which still are using IDE host drivers
> get converted to use corresponding libata PATA host drivers.
> 
> Best regards,
> --
> Bartlomiej Zolnierkiewicz
> Samsung R Institute Poland
> Samsung Electronics
> 
> 
> Bartlomiej Zolnierkiewicz (19):
>   ide: remove deprecated atiixp host driver
>   ide: remove deprecated cs5520 host driver
>   ide: remove deprecated cs5530 host driver
>   ide: remove deprecated cs5535 host driver
>   ide: remove deprecated cs5536 host driver
>   ide: remove deprecated sc1200 host driver
>   ide: remove deprecated delkin_cb host driver
>   ide: remove deprecated it8213 host driver
>   ide: remove deprecated it821x host driver
>   ide: remove deprecated jmicron host driver
>   ide: remove deprecated opti621 host driver
>   ide: remove deprecated pdc202xx_old host driver
>   ide: remove deprecated rz1000 host driver
>   ide: remove deprecated serverworks host driver
>   ide: remove deprecated sis5513 host driver
>   ide: remove deprecated slc90e66 host driver
>   ide: remove deprecated triflex host driver
>   ide: remove deprecated ide-pnp host driver
>   ide: remove deprecated icside host driver
> 
>  drivers/ide/Kconfig| 200 +
>  drivers/ide/Makefile   |  19 --
>  drivers/ide/atiixp.c   | 211 -
>  drivers/ide/cs5520.c   | 168 ---
>  drivers/ide/cs5530.c   | 295 ---
>  drivers/ide/cs5535.c   | 219 --
>  drivers/ide/cs5536.c   | 306 ---
>  drivers/ide/delkin_cb.c| 181 
>  drivers/ide/icside.c   | 691 ---
>  drivers/ide/ide-pnp.c  | 100 ---
>  drivers/ide/it8213.c   | 216 --
>  drivers/ide/it821x.c   | 715 
> -
>  drivers/ide/jmicron.c  | 176 ---
>  drivers/ide/opti621.c  | 178 ---
>  drivers/ide/pdc202xx_old.c | 361 ---
>  drivers/ide/rz1000.c   |  99 ---
>  drivers/ide/sc1200.c   | 355 --
>  drivers/ide/serverworks.c  | 449 
>  drivers/ide/sis5513.c  | 636 
>  drivers/ide/slc90e66.c | 181 
>  drivers/ide/triflex.c  | 155 --
>  21 files changed, 2 insertions(+), 5909 deletions(-)
>  delete mode 100644 drivers/ide/atiixp.c
>  delete mode 100644 drivers/ide/cs5520.c
>  delete mode 100644 drivers/ide/cs5530.c
>  delete mode 100644 drivers/ide/cs5535.c
>  delete mode 100644 drivers/ide/cs5536.c
>  delete mode 100644 drivers/ide/delkin_cb.c
>  delete mode 100644 drivers/ide/icside.c
>  delete mode 100644 drivers/ide/ide-pnp.c
>  delete mode 100644 drivers/ide/it8213.c
>  delete mode 100644 drivers/ide/it821x.c
>  delete mode 100644 drivers/ide/jmicron.c
>  delete mode 100644 drivers/ide/opti621.c
>  delete mode 100644 drivers/ide/pdc202xx_old.c
>  delete mode 100644 drivers/ide/rz1000.c
>  delete mode 100644 drivers/ide/sc1200.c
>  delete mode 100644 drivers/ide/serverworks.c
>  delete mode 100644 drivers/ide/sis5513.c
>  delete mode 100644 drivers/ide/slc90e66.c
>  delete mode 100644 drivers/ide/triflex.c

[RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi,

IDE subsystem has been deprecated since 2009 and the majority
(if not all) of Linux distributions have switched to use
libata for ATA support exclusively.

This patchset removes IDE host drivers that:
- have been replaced by their corresponding libata PATA host
  drivers (which have been available since 2006/2007)
- are not used by any defconfig

Having duplicated drivers for the same hardware doesn't seem
to be good thing in the long-term perspective for Linux as
- testing efforts are divided between two drivers/subsystems
  (while there is less and less PATA systems left in use)
- having two drivers for the same hardware confuses users

Please note that all the hardware supported by IDE host drivers
nominated for removal remains supported by correspoding libata
PATA host drivers.

PS If this patchset is accepted I would like to continue
removal of IDE host drivers in part 2 which will happen after
remaining defconfigs which still are using IDE host drivers
get converted to use corresponding libata PATA host drivers.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics


Bartlomiej Zolnierkiewicz (19):
  ide: remove deprecated atiixp host driver
  ide: remove deprecated cs5520 host driver
  ide: remove deprecated cs5530 host driver
  ide: remove deprecated cs5535 host driver
  ide: remove deprecated cs5536 host driver
  ide: remove deprecated sc1200 host driver
  ide: remove deprecated delkin_cb host driver
  ide: remove deprecated it8213 host driver
  ide: remove deprecated it821x host driver
  ide: remove deprecated jmicron host driver
  ide: remove deprecated opti621 host driver
  ide: remove deprecated pdc202xx_old host driver
  ide: remove deprecated rz1000 host driver
  ide: remove deprecated serverworks host driver
  ide: remove deprecated sis5513 host driver
  ide: remove deprecated slc90e66 host driver
  ide: remove deprecated triflex host driver
  ide: remove deprecated ide-pnp host driver
  ide: remove deprecated icside host driver

 drivers/ide/Kconfig| 200 +
 drivers/ide/Makefile   |  19 --
 drivers/ide/atiixp.c   | 211 -
 drivers/ide/cs5520.c   | 168 ---
 drivers/ide/cs5530.c   | 295 ---
 drivers/ide/cs5535.c   | 219 --
 drivers/ide/cs5536.c   | 306 ---
 drivers/ide/delkin_cb.c| 181 
 drivers/ide/icside.c   | 691 ---
 drivers/ide/ide-pnp.c  | 100 ---
 drivers/ide/it8213.c   | 216 --
 drivers/ide/it821x.c   | 715 -
 drivers/ide/jmicron.c  | 176 ---
 drivers/ide/opti621.c  | 178 ---
 drivers/ide/pdc202xx_old.c | 361 ---
 drivers/ide/rz1000.c   |  99 ---
 drivers/ide/sc1200.c   | 355 --
 drivers/ide/serverworks.c  | 449 
 drivers/ide/sis5513.c  | 636 
 drivers/ide/slc90e66.c | 181 
 drivers/ide/triflex.c  | 155 --
 21 files changed, 2 insertions(+), 5909 deletions(-)
 delete mode 100644 drivers/ide/atiixp.c
 delete mode 100644 drivers/ide/cs5520.c
 delete mode 100644 drivers/ide/cs5530.c
 delete mode 100644 drivers/ide/cs5535.c
 delete mode 100644 drivers/ide/cs5536.c
 delete mode 100644 drivers/ide/delkin_cb.c
 delete mode 100644 drivers/ide/icside.c
 delete mode 100644 drivers/ide/ide-pnp.c
 delete mode 100644 drivers/ide/it8213.c
 delete mode 100644 drivers/ide/it821x.c
 delete mode 100644 drivers/ide/jmicron.c
 delete mode 100644 drivers/ide/opti621.c
 delete mode 100644 drivers/ide/pdc202xx_old.c
 delete mode 100644 drivers/ide/rz1000.c
 delete mode 100644 drivers/ide/sc1200.c
 delete mode 100644 drivers/ide/serverworks.c
 delete mode 100644 drivers/ide/sis5513.c
 delete mode 100644 drivers/ide/slc90e66.c
 delete mode 100644 drivers/ide/triflex.c

-- 
1.9.1

[RFC PATCH 00/19] ide: remove deprecated host drivers (part 1)

[Bartlomiej Zolnierkiewicz]

Hi,

IDE subsystem has been deprecated since 2009 and the majority
(if not all) of Linux distributions have switched to use
libata for ATA support exclusively.

This patchset removes IDE host drivers that:
- have been replaced by their corresponding libata PATA host
  drivers (which have been available since 2006/2007)
- are not used by any defconfig

Having duplicated drivers for the same hardware doesn't seem
to be good thing in the long-term perspective for Linux as
- testing efforts are divided between two drivers/subsystems
  (while there is less and less PATA systems left in use)
- having two drivers for the same hardware confuses users

Please note that all the hardware supported by IDE host drivers
nominated for removal remains supported by correspoding libata
PATA host drivers.

PS If this patchset is accepted I would like to continue
removal of IDE host drivers in part 2 which will happen after
remaining defconfigs which still are using IDE host drivers
get converted to use corresponding libata PATA host drivers.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics


Bartlomiej Zolnierkiewicz (19):
  ide: remove deprecated atiixp host driver
  ide: remove deprecated cs5520 host driver
  ide: remove deprecated cs5530 host driver
  ide: remove deprecated cs5535 host driver
  ide: remove deprecated cs5536 host driver
  ide: remove deprecated sc1200 host driver
  ide: remove deprecated delkin_cb host driver
  ide: remove deprecated it8213 host driver
  ide: remove deprecated it821x host driver
  ide: remove deprecated jmicron host driver
  ide: remove deprecated opti621 host driver
  ide: remove deprecated pdc202xx_old host driver
  ide: remove deprecated rz1000 host driver
  ide: remove deprecated serverworks host driver
  ide: remove deprecated sis5513 host driver
  ide: remove deprecated slc90e66 host driver
  ide: remove deprecated triflex host driver
  ide: remove deprecated ide-pnp host driver
  ide: remove deprecated icside host driver

 drivers/ide/Kconfig| 200 +
 drivers/ide/Makefile   |  19 --
 drivers/ide/atiixp.c   | 211 -
 drivers/ide/cs5520.c   | 168 ---
 drivers/ide/cs5530.c   | 295 ---
 drivers/ide/cs5535.c   | 219 --
 drivers/ide/cs5536.c   | 306 ---
 drivers/ide/delkin_cb.c| 181 
 drivers/ide/icside.c   | 691 ---
 drivers/ide/ide-pnp.c  | 100 ---
 drivers/ide/it8213.c   | 216 --
 drivers/ide/it821x.c   | 715 -
 drivers/ide/jmicron.c  | 176 ---
 drivers/ide/opti621.c  | 178 ---
 drivers/ide/pdc202xx_old.c | 361 ---
 drivers/ide/rz1000.c   |  99 ---
 drivers/ide/sc1200.c   | 355 --
 drivers/ide/serverworks.c  | 449 
 drivers/ide/sis5513.c  | 636 
 drivers/ide/slc90e66.c | 181 
 drivers/ide/triflex.c  | 155 --
 21 files changed, 2 insertions(+), 5909 deletions(-)
 delete mode 100644 drivers/ide/atiixp.c
 delete mode 100644 drivers/ide/cs5520.c
 delete mode 100644 drivers/ide/cs5530.c
 delete mode 100644 drivers/ide/cs5535.c
 delete mode 100644 drivers/ide/cs5536.c
 delete mode 100644 drivers/ide/delkin_cb.c
 delete mode 100644 drivers/ide/icside.c
 delete mode 100644 drivers/ide/ide-pnp.c
 delete mode 100644 drivers/ide/it8213.c
 delete mode 100644 drivers/ide/it821x.c
 delete mode 100644 drivers/ide/jmicron.c
 delete mode 100644 drivers/ide/opti621.c
 delete mode 100644 drivers/ide/pdc202xx_old.c
 delete mode 100644 drivers/ide/rz1000.c
 delete mode 100644 drivers/ide/sc1200.c
 delete mode 100644 drivers/ide/serverworks.c
 delete mode 100644 drivers/ide/sis5513.c
 delete mode 100644 drivers/ide/slc90e66.c
 delete mode 100644 drivers/ide/triflex.c

-- 
1.9.1