> Sorry for the delay. I have now had a quick look through the patches. Just a
> couple of points :-)
>
> * stfsm_probe(): stfsm_fetch_platform_configs() needs to be called *before*
> config() -- config() is based on platform capabilities. Conceptually,
> stfsm_fetch_platform_configs() should b
Hi Lee,
Sorry for the delay. I have now had a quick look through the patches. Just a
couple of points :-)
* stfsm_probe(): stfsm_fetch_platform_configs() needs to be called *before*
config() -- config() is based on platform capabilities. Conceptually,
stfsm_fetch_platform_configs() should be c
Version 4:
Tended to Brian's review comments
- Checkpatch acceptance
- MODULE_DEVICE_TABLE() name slip correction
- Timeout issue(s) resolved
- Potential infinite loop mitigated
- Code clarity suggests heeded
- Duplication with MTD core code removed
- Upgraded to using
3 matches
Mail list logo