[PATCH 04/16] pcmcia: use dynamic debug infrastructure, deprecate CS_CHECK (ide)

Dominik Brodowski linux at dominikbrodowski.net
Mon Oct 26 11:33:33 EDT 2009


Sergei,

On Mon, Oct 26, 2009 at 06:47:49PM +0300, Sergei Shtylyov wrote:
> Dominik Brodowski wrote:
>
>> ide-cs.c is the only PCMCIA device driver making use of CONFIG_PCMCIA_DEBUG,
>> so convert it to use the dynamic debug infrastructure.
>
>> Also, remove all usages of the CS_CHECK macro and replace them with proper
>> Linux style calling and return value checking. The extra error reporting may
>> be dropped, as the PCMCIA core already complains about any (non-driver-author)
>> errors.
>
>> CC: linux-ide at vger.kernel.org
>> Signed-off-by: Dominik Brodowski <linux at dominikbrodowski.net>
>> ---
>>  drivers/ata/pata_pcmcia.c |   16 ++++++++--------
>>  drivers/ide/ide-cs.c      |   32 +++++++++++---------------------
>>  2 files changed, 19 insertions(+), 29 deletions(-)
>
>    Please submit as two separate patches as ide and libata are two 
> separate subsystems.

thanks for your comment. And I see your point -- but as I intend to push this
upstream using the linux-pcmcia tree, as the IDE and ATA subsystems use the
same (linux-ide) mailing list, and as both drivers are quire related, I
thought handling both drivers in one patch is acceptable. If it still needs
to be done otherwise, I'll re-submit this patch in two parts and udpate my
workflow accordingly.

Best,
	Dominik



More information about the linux-pcmcia mailing list