Re: [PATCH 09/44 take 2] [UBI] debug unit header

Christoph Hellwig
2007-02-19T11:00:44Z

On Sat, Feb 17, 2007 at 10:18:21PM +0100, Arnd Bergmann wrote:
> On Saturday 17 February 2007 17:55, Artem Bityutskiy wrote:
> > +
> > +/**
> > + * UBI debugging unit.
> > + *
> > + * UBI provides rich debugging capabilities which are implemented in
> > + * this unit.
>
> Stop right here. You should be doing one thing and do it right.
> Since the point of your patches is to do volume management for MTD,
> it should do just that.
>
> If you feel that Linux needs rich debugging capabilities, then submit
> a patch for that independent of UBI.
>

*nod*