Re: patch "staging: remove nokia_hp4p driver

[Pavel Machek]

Hi!

> > "These are a lot of public functions. Are they all really needed or can
> > the code be done smart."
> > 
> > in TODO file.
> 
> so what about all the other really major details that needed fixing. I mean anybody who was compiling for OMAP platform could have fixed the tons and tons of inlines quickly.
> 

Do you have n900 nearby? You are welcome to try...

Unfortunately, it is not exactly easy. v3.18 with USB patch works with
nfsroot, which makes development possible (good).

Some core (or omap) change broke h4p between 3.15 and 3.16. Bisect
will not be fun, I suspect.

> > (Also hopefully get some testing from
> > n900 community; code is more
> > visible in staging).
> 
> The problem is that they are testing the wrong thing. And lets face it, we already knew this driver works since Nokia shipped it in products.
>

Except that it does not :-(. Apparently not even place in staging was
enough to get regular testing... but at least it will help bisect.

> > I attempted hdev->setup conversion, but could not figure it out till
> > now. Clearly it needs to be done.
> > 
> > For doing that, it would be good to have userland to work with, and
> > yes it takes time. (Debugging on 4" screen sucks.)
> 
> Actually hdev->setup does not need any userland. As long as request_firmware() works, you are just fine. And since the driver already uses that in the first place, I assume it works.
> 

Without working serial port, you can't really debug without
userland. Not on machine that lacks shift-pgup.

									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/