On 17 December 2015 at 14:30, Thomas Huth <th...@redhat.com> wrote:
> On 17/12/15 13:19, Markus Armbruster wrote:
>> Device init() methods aren't supposed to call hw_error(), they should
>> report the error and fail cleanly. Do that.
>> +
>> + for (i = 0; i < s->modulecount; i++) {
>
> You're now using modulecount here -^
>
>> + if (!s->fclk[i]) {
>> + error_report("omap2-gpio: fclk%d not connected", i);
>> + return -1;
>> + }
>> + }
>> +
>> if (s->mpu_model < omap3430) {
>> s->modulecount = (s->mpu_model < omap2430) ? 4 : 5;
>
> ... but it is initialized just here. So I think it is wrong that you
> moved the "!s->fclk[i]" before that.
Thanks for reviewing this patch more carefully than I did :-)
-- PMM
