Hi Sylvain, and thanks for your comments.<div>I've addressed your points. Let me know if I can improve the patch furter.<br><br><div class="gmail_quote">On Wed, Jan 2, 2013 at 4:14 PM, Sylvain Munaut <span dir="ltr"><<a href="mailto:246tnt@gmail.com" target="_blank">246tnt@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
> Please find attached a patch to cell_log that allows a user to specify a<br>
> custom frequency range instead of the standard one.<br>
<br>
Thanks.<br>
<br>
> In the past I've done the same job changing the code by hand and<br>
> recompiling, but a command line switch is more feasible.<br>
> This is useful when one wants to monitor a subset of arfcns (like in GSM900)<br>
> or even a single one.<br>
<br>
Indeed, looks like a very useful functionality.<br>
<br>
> Any comment appreciated.<br>
<br>
- print_band_range & parse_band_range  should be static<br>
- Check coding style rules, (tab vs space). We use the kernel coding<br>
style (which you can view in the kernel tree under<br>
Documentation/CodingStyle )<br>
<br>
I'm not a fan of the method used to pass the new range, but I can't<br>
think of anything better so good enough. I'm not exactly sure why the<br>
app is split into two files to begin with.<br>
<br>
Cheers,<br>
<br>
    Sylvain<br>
</blockquote></div><br></div>