Date: Wed, 5 Oct 2016 16:09:40 +0300 From: Andriy Gapon <avg@FreeBSD.org> To: Torfinn Ingolfsen <torfinn.ingolfsen@getmail.no>, freebsd-stable@FreeBSD.org Subject: Re: aibs(4) / atk0110 support for newer systems Message-ID: <4c431c8a-7c8c-4336-b948-6e9ae882c0fc@FreeBSD.org> In-Reply-To: <20161005143723.4273657959160b67637a5adf@getmail.no> References: <86cf8380-ac6f-55f0-f0f8-16000d7f04b2@FreeBSD.org> <20160930145704.4dbc9d90011154b38493964e@getmail.no> <7d498084-ec05-d4c9-5f49-6aef32495caf@FreeBSD.org> <20160930205928.77d7e74f7bd1a35fcf1aa50a@getmail.no> <7a868c22-e0bd-f677-e4ad-2bdf6f3605d0@FreeBSD.org> <20161003201511.7258687453f12c44a46a361a@getmail.no> <f9fbb905-2bd7-0b1d-3baa-85061258718d@FreeBSD.org> <20161005143723.4273657959160b67637a5adf@getmail.no>
next in thread | previous in thread | raw e-mail | index | archive | help
On 05/10/2016 15:37, Torfinn Ingolfsen wrote: > On Mon, 3 Oct 2016 23:05:48 +0300 > Andriy Gapon <avg@FreeBSD.org> wrote: > >> Yes, it does. Thank you! >> It seems like a couple of minor changes are not in the source tree that you are >> using. One is some casts in a diagnostic printf and the other is a different >> rounding of 0C in Kelvins. >> I've generated a patch that should apply to your tree: >> https://people.freebsd.org/~avg/aibs-ggrp-gitm.93.diff >> Please try. > > The patch applied cleanly (I removed the old one with patch -R first): Good! > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c: In function 'aibs_attach': > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:252: warning: 's_idx' may be used uninitialized in this function > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:252: note: 's_idx' was declared here > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:256: warning: 'so' may be used uninitialized in this function > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:256: note: 'so' was declared here > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:253: warning: 'name' may be used uninitialized in this function > /usr/src/sys/modules/acpi/aibs/../../../dev/acpi_support/atk0110.c:253: note: 'name' was declared here > *** [atk0110.o] Error code 1 > > Stop in /usr/src/sys/modules/acpi/aibs. > > Do I have to do something more in order to build the new module? > Seems like this is a clang vs gcc issue as I didn't get warnings here. Could you please simply add initialization to those variables in aibs_attach_ggrp()? E.g.: s_idx = NULL; so = NULL; Thank you! -- Andriy Gapon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4c431c8a-7c8c-4336-b948-6e9ae882c0fc>