2005-03-09 14:38:57

by Bagalkote, Sreenivas

[permalink] [raw]
Subject: RE: [ANNOUNCE][PATCH 2.6.11 1/3] megaraid_sas: Announcing new mod ule for LSI Logic's SAS based MegaRAID controllers

>> >
>> >> source "drivers/scsi/megaraid/Kconfig.megaraid"
>> >> +source "drivers/scsi/megaraid/Kconfig.megaraid_sas"
>> >>
>> >
>> >why a fully separate file and not add your ONE config option to
>> >Kconfig.megaraid instead ??
>> >
>>
>> Arjan, I didn't want to needlessly couple megaraid and megaraid_sas.
>> Since they are in the same directory, I couldn't avoid having single
>> Makefile. I thought at least these two should be separate to
>be consistent
>> with their independent nature.
>>
>> If this is not a good enough reason, I will merge these two files.
>
>Please merge them.
>
>Whether they are in the same Kconfig file or not does not in any way
>imply any relation between them.
>
>E.g. drivers/scsi/Kconfig contains many drivers that are not
>in any way
>coupled to each other.
>

Understood. I will merge them.

>
>BTW: Why does the text say "(New Driver)"?
>

I thought I saw some new modules being labeled as such. I will remove
it in the next update if it not correct.

>--
>
> "Is there not promise of rain?" Ling Tan asked suddenly out
> of the darkness. There had been need of rain for many days.
> "Only a promise," Lao Er said.
> Pearl S. Buck - Dragon Seed
>
There is a 30% chance of rain here today.

Thanks,
Sreenivas
LSI Logic Corporation