[PATCH] hdparm 6.6 1/2

[Tito]

On Friday 28 April 2006 03:35, Rob Landley wrote:
> On Thursday 27 April 2006 5:46 pm, Tito wrote:
> > Hi,
> > this is my previous hdparm 6.6 patch divided into 4 incremental patches
> >
> > hdparm66_usage_01.patch  for unclude/usage.h
> > hdparm66_Config_02.patch  for miscutils/Config.in
> > hdparm66_03.patch               for miscutils/hdparm.c
> >
> > this 3 patches must be applied all together for porting hdparm 6.6 to
> > busybox
> >
> > hdparm66_kernelhdrs_04.patch for miscutils/hdparm.c
> >
> > this patch removes the dependencies from some kernel headers
> >
> > Size is:
> >   27330          4     904   28238    6e4e miscutils/hdparm.o.orig       
> > previous version  with all the features enabled (if my notes are correct)
> > 27907     864     872   29643    73cb miscutils/hdparm.o               
> > with all the features of the previous version
> 
> Growing 1400 bytes to do the same thing is not cool.  It needs a space 
> trimming pass before this is going in.
> 
> > 28524     864     904   30292 
> >    7654 miscutils/hdparm.o                with all new features enabled
> > 28449     864     892   30205    75fd miscutils/hdparm.o                
> > with all new features enabled and patch 4 applied
> >
> > Please apply if you think is ok, or if you don't like it please
> > apply the alternative patch
> >
> > hdparm_alternative.patch that fixes only the output.
> 
> Alternative applied.
> 

Ok, fine.
Should we at least port the kernel headers stuff to the current svn.
If you agree I will rediff patch 04.

hdparm66_kernelhdrs_04.patch for miscutils/hdparm.c

to remove 

#include <linux/types.h>
#include <linux/hdreg.h>
#include <linux/major.h>
#include <asm/byteorder.h>

from hdparm.c

Let me know your opinion about it.

Ciao,
Tito
> > Ciao,
> > Tito
> 
> Rob