[PATCH 1/3] h1940: use gpiolib for latch access
Marek Vasut
marek.vasut at gmail.com
Tue Aug 24 11:04:14 EDT 2010
Dne Út 24. srpna 2010 16:50:36 Vasily Khoruzhick napsal(a):
> В сообщении от 24 августа 2010 17:40:21 автор Marek Vasut написал:
> > Defining S3C2410_GPH_GPIONAME for each bit might be better then.
>
> Well, that requires massive refactoring of h1940 (and rx1950) support. And
> btw, that's common for s3c24xx-based machines to use generic s3c24xx gpio
> names, not machine specific.
I assume I see how it works now. Ok, please put a comment around it then so
others will understand what it does.
Using machine-specific GPIO names might help readability btw. -- the approach
"everyone does it this way so I'll do it this way without thinking" is bad :)
Well, it's just my suggestion for improvement of the overall code quality. Ben,
what's your opinion?
Cheers
>
> Regards
> Vasily
More information about the linux-arm-kernel
mailing list