Re: [PATCH] pcmcia: add another ide-cs CF card id

From: Arjan van de Ven
Date: Thu Mar 02 2006 - 08:04:42 EST


On Thu, 2006-03-02 at 07:38 -0500, Jeff Garzik wrote:
> Russell King wrote:
> > I think it's fairly safe and obvious to say that Dominik is the peer
> > review for these tables - he _is_ the PCMCIA maintainer, he _is_
> > arguably the maintainer for the ide-cs driver, he _is_ the person
> > who invented these tables, he _is_ the one taking patches from people
> > to add IDs, he _is_ the one reviewing such patches.
> >
> > If you want to know what's going on in PCMCIA land, subscribe to
> > linux-pcmcia. In the same way that if you want to know what's going
> > in in IDE land, you subscribe to linux-ide, or PCI land linux-pci,
> > SCSI land linux-scsi, network land netdev.
> >
> > Using your argument (which seems to be demanding that any patch to
> > any IDE driver no matter how trivial must be on linux-ide) that a patch
> > to a PCI network device driver must be copied to linux-pci and netdev
> > even though it may not touch the PCI specific code.
>
> IDE driver -> IDE reviewers
>
> network driver -> network reviewers

I would turn this around

IDE impacting change -> IDE reviewers

network driver impacting change -> network reviewers

the reason for the review is because of the impact, not because a file
happens to live in a certain directory.


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/