Search Linux Wireless

Re: [PATCH 1/3] iwlwifi: adding channels to sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > If you managed to get along without this for so long, why is it needed
> > in upstream now?
> 
> We didn't run without it, We  were developing HW 5000 on older kernel
> now I just merging it upstream. See my announcement  Try to imagine
> that there is world on totally different planet with different
> gravitation constant,

Heh. But that still doesn't explain why you're pushing this API into the
upstream kernel.

> 
> > Remember, sysfs is API. You don't get to change it gratuitously, so
> > neither should you add to it gratuitously. And we already _have_ API for
> > this particular thing which is not dependent on a particular
> > hardware/driver to boot!
> Agree. We've just returned what was there before
> I think you understand what legacy means after your try to replace
> wext. It will take time
> till we switch to new API. Unfortunately cannot fix everything in one day.

I know, and that's why wext will stay around for a long time to come.
I'm well aware of that, and I think you're aware of that as well. I just
don't see the point of _adding_ to the problem with gratuitous API
proliferation when apparently you can very well manage to live without
this piece.

Couldn't you move it all to debugfs, which is understood to not really
be API, and change your verification scripts to use /sys/debug instead
of /sys?

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux