Re: [PATCH resend] Add sc16is7x2 driver

From: Mark Brown
Date: Tue Oct 26 2010 - 15:28:50 EST


On Tue, Oct 26, 2010 at 07:51:07AM -0700, Greg KH wrote:
> On Tue, Oct 26, 2010 at 10:55:24AM +0200, Manuel Stahl wrote:

> > This is the driver I took as a template. It consists of
> > drivers/serial/max3100.c and include/linux/serial_max3100.h

> > And huh, "grep -R serial_max3100.h *" gives what? Only max3100.c
> > So do we need to remove serial_max3100.h as it's not used anywhere else?

> Yup, that sounds like a great idea, I'll go queue up that patch when I
> get back from my travels at the moment.

Requiring that we have a user for all platform data provided by drivers
would be incredibly painful for embedded development. It would mean
that we would be unable to usefully merge any drivers with platform data
unless we also merge at least one board using them, meaning that we're
dependant on getting the core board support for some public board into a
state which can be merged (which would at best be likely to end up
creating logjams on support for CPUs).
--
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/