Re: [PATCH 2.6.11-rc2 1/1] altix: Device driver support for the CX port of SGI's TIO chip

From: Christoph Hellwig <hch_at_infradead.org>
Date: 2005-02-12 01:26:55
> -#include "xtalk/xwidgetdev.h"
> +#include <asm/sn/xtalk/xwidgetdev.h>
>  #include <asm/sn/geo.h>
> -#include "xtalk/hubdev.h"
> +#include <asm/sn/xtalk/hubdev.h>

please put moving around headers in a separate patch.

>  DEFINE_PER_CPU(struct pda_s, pda_percpu);
> +EXPORT_PER_CPU_SYMBOL(pda_percpu);

As mentioned the last time you posted it please use a proper accessor.

> +static void tiocx_bus_release(struct device *dev)
> +{
> +}

this is broken.  See the tweleve gazillion times it came up on lkml.
The driver core doesn't warn about the lack of a release function because
it's a nice way to waste time but because there's an actual reason.

> +struct cx_drv {
> +	char *name;
> +	const struct cx_device_id *id_table;
> +	struct device_driver driver;
> +	int (*probe) (struct cx_dev * dev, const struct cx_device_id * id);
> +};

To make it work sanely you also need a ->remove callout.


With the above issues fixed it's fine to be queued up until you submit
an actual user of this interface.
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Fri Feb 11 09:27:07 2005

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:35 EST