[PATCH UPDATED] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[PATCH UPDATED] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[Patrick Ringl]

Hello,
since especially Serial ATA has it's own menu point now, I guess we can 
change the description of the deprecated SATA driver as well, since the 
new libATA subsystem is not configured through a SCSI low-level driver 
anymore, but has it's own menu point.

The following patch is against 2.6.21-rc4:

--- linux-2.6.20.old/drivers/ide/Kconfig	2007-03-18 00:05:11.000000000 +0100
+++ linux-2.6.20/drivers/ide/Kconfig	2007-03-18 00:09:47.000000000 +0100
@@ -103,7 +103,7 @@
  	---help---
  	  There are two drivers for Serial ATA controllers.

-	  The main driver, "libata", exists inside the SCSI subsystem
+	  The main driver, "libata", exists inside the ATA subsystem
  	  and supports most modern SATA controllers.

  	  The IDE driver (which you are currently configuring) supports

---

Since I am not subscribed to the list, I'd find it great if I were
personally CC'ed. :-)


Best regards
Patrick

Re: [PATCH UPDATED] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[Bartlomiej Zolnierkiewicz]

On Sunday 18 March 2007, Patrick Ringl wrote:
> Hello,

Hi,

> since especially Serial ATA has it's own menu point now, I guess we can 
> change the description of the deprecated SATA driver as well, since the 
> new libATA subsystem is not configured through a SCSI low-level driver 
> anymore, but has it's own menu point.
> 
> The following patch is against 2.6.21-rc4:
> 
> --- linux-2.6.20.old/drivers/ide/Kconfig	2007-03-18 00:05:11.000000000 +0100
> +++ linux-2.6.20/drivers/ide/Kconfig	2007-03-18 00:09:47.000000000 +0100
> @@ -103,7 +103,7 @@
>   	---help---
>   	  There are two drivers for Serial ATA controllers.
> 
> -	  The main driver, "libata", exists inside the SCSI subsystem
> +	  The main driver, "libata", exists inside the ATA subsystem

Strictly speaking libata is not a separate subsystem (it still uses SCSI
subsystem) and "ATA subsystem" may be misleading, since we now have:

* "ATA/ATAPI/MFM/RLL support" menu for drivers/ide

* "Serial ATA (prod) and Parallel ATA (experimental) drivers" menu for libata

What about replacing "exists inside" into "uses" and adding info about
the new menu instead?

Thanks,
Bart

Re: [PATCH UPDATED][2] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[Patrick Ringl]

Bartlomiej Zolnierkiewicz wrote:

Hello,
> On Sunday 18 March 2007, Patrick Ringl wrote:
>> Hello,
>
> Hi,
>
>> since especially Serial ATA has it's own menu point now, I guess we can 
>> change the description of the deprecated SATA driver as well, since the 
>> new libATA subsystem is not configured through a SCSI low-level driver 
>> anymore, but has it's own menu point.
>>
>> The following patch is against 2.6.21-rc4:
>>
>> --- linux-2.6.20.old/drivers/ide/Kconfig	2007-03-18 00:05:11.000000000 +0100
>> +++ linux-2.6.20/drivers/ide/Kconfig	2007-03-18 00:09:47.000000000 +0100
>> @@ -103,7 +103,7 @@
>>   	---help---
>>   	  There are two drivers for Serial ATA controllers.
>>
>> -	  The main driver, "libata", exists inside the SCSI subsystem
>> +	  The main driver, "libata", exists inside the ATA subsystem
>
> Strictly speaking libata is not a separate subsystem (it still uses SCSI
> subsystem) and "ATA subsystem" may be misleading, since we now have:
>
> * "ATA/ATAPI/MFM/RLL support" menu for drivers/ide
>
> * "Serial ATA (prod) and Parallel ATA (experimental) drivers" menu for libata
>
> What about replacing "exists inside" into "uses" and adding info about
> the new menu instead?
Well, that's even a better idea :-) I wasn't that sure about what to do 
.. it's just that it could be misleading since the new (s/p)ata drivers 
are not living in the scsi low-level subsystem anymore, but got their 
own menu point.

Here's a different patch >:)

--- linux-2.6.20.old/drivers/ide/Kconfig 2007-03-18 00:05:11.000000000 +0100

+++ linux-2.6.20/drivers/ide/Kconfig 2007-03-18 01:23:51.000000000 +0100

@@ -103,8 +103,10 @@

 ---help---

   There are two drivers for Serial ATA controllers.

 

-   The main driver, "libata", exists inside the SCSI subsystem

-   and supports most modern SATA controllers.

+   The main driver, "libata", uses the SCSI subsystem

+   and supports most modern SATA controllers. In order to use it

+   you may take a look at "Serial ATA (prod) and Parallel ATA

+   (experimental) drivers".

 

   The IDE driver (which you are currently configuring) supports

   a few first-generation SATA controllers.

---
> Thanks,
> Bart
>
>   

regards,
Patrick

Re: [PATCH UPDATED][2] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[Bartlomiej Zolnierkiewicz]

On Sunday 18 March 2007, Patrick Ringl wrote:
> Bartlomiej Zolnierkiewicz wrote:
> 
> Hello,
> > On Sunday 18 March 2007, Patrick Ringl wrote:
> >> Hello,
> >
> > Hi,
> >
> >> since especially Serial ATA has it's own menu point now, I guess we can 
> >> change the description of the deprecated SATA driver as well, since the 
> >> new libATA subsystem is not configured through a SCSI low-level driver 
> >> anymore, but has it's own menu point.
> >>
> >> The following patch is against 2.6.21-rc4:
> >>
> >> --- linux-2.6.20.old/drivers/ide/Kconfig	2007-03-18 00:05:11.000000000 +0100
> >> +++ linux-2.6.20/drivers/ide/Kconfig	2007-03-18 00:09:47.000000000 +0100
> >> @@ -103,7 +103,7 @@
> >>   	---help---
> >>   	  There are two drivers for Serial ATA controllers.
> >>
> >> -	  The main driver, "libata", exists inside the SCSI subsystem
> >> +	  The main driver, "libata", exists inside the ATA subsystem
> >
> > Strictly speaking libata is not a separate subsystem (it still uses SCSI
> > subsystem) and "ATA subsystem" may be misleading, since we now have:
> >
> > * "ATA/ATAPI/MFM/RLL support" menu for drivers/ide
> >
> > * "Serial ATA (prod) and Parallel ATA (experimental) drivers" menu for libata
> >
> > What about replacing "exists inside" into "uses" and adding info about
> > the new menu instead?
> Well, that's even a better idea :-) I wasn't that sure about what to do 
> .. it's just that it could be misleading since the new (s/p)ata drivers 
> are not living in the scsi low-level subsystem anymore, but got their 
> own menu point.
> 
> Here's a different patch >:)

applied

[ The patch was whitespace damaged and didn't apply et all
  so I had to manually change the Kconfig to merge your change. ]

Thanks,
Bart

Re: [PATCH UPDATED][2] cosmetic adaption of drivers/ide/Kconfig concerning SATA

[Patrick Ringl]

Bartlomiej Zolnierkiewicz wrote:
> On Sunday 18 March 2007, Patrick Ringl wrote:
>   
>> Bartlomiej Zolnierkiewicz wrote:
>>
>> Hello,
>>     
>>> On Sunday 18 March 2007, Patrick Ringl wrote:
>>>       
>>>> Hello,
>>>>         
>>> Hi,
>>>
>>>       
>>>> since especially Serial ATA has it's own menu point now, I guess we can 
>>>> change the description of the deprecated SATA driver as well, since the 
>>>> new libATA subsystem is not configured through a SCSI low-level driver 
>>>> anymore, but has it's own menu point.
>>>>
>>>> The following patch is against 2.6.21-rc4:
>>>>
>>>> --- linux-2.6.20.old/drivers/ide/Kconfig	2007-03-18 00:05:11.000000000 +0100
>>>> +++ linux-2.6.20/drivers/ide/Kconfig	2007-03-18 00:09:47.000000000 +0100
>>>> @@ -103,7 +103,7 @@
>>>>   	---help---
>>>>   	  There are two drivers for Serial ATA controllers.
>>>>
>>>> -	  The main driver, "libata", exists inside the SCSI subsystem
>>>> +	  The main driver, "libata", exists inside the ATA subsystem
>>>>         
>>> Strictly speaking libata is not a separate subsystem (it still uses SCSI
>>> subsystem) and "ATA subsystem" may be misleading, since we now have:
>>>
>>> * "ATA/ATAPI/MFM/RLL support" menu for drivers/ide
>>>
>>> * "Serial ATA (prod) and Parallel ATA (experimental) drivers" menu for libata
>>>
>>> What about replacing "exists inside" into "uses" and adding info about
>>> the new menu instead?
>>>       
>> Well, that's even a better idea :-) I wasn't that sure about what to do 
>> .. it's just that it could be misleading since the new (s/p)ata drivers 
>> are not living in the scsi low-level subsystem anymore, but got their 
>> own menu point.
>>
>> Here's a different patch >:)
>>     
>
> applied
>
> [ The patch was whitespace damaged and didn't apply et all
>   so I had to manually change the Kconfig to merge your change. ]
>   
D'Oh .. Probably due to the fact that I copied it and did not attach it 
in plain/text.

 Thanks

> Thanks,
> Bart
>
>   

regards,
Patrick