[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