Hi Robert,
> static int __init bsr_init(void)
> {
> struct device_node *np;
> dev_t bsr_dev = MKDEV(bsr_major, 0); <--- ????????
You are right, this seems to be completely useless.
Go on make a patch, compile and send it.
thanks,
Daniel.