Date: Tue, 10 Jan 2017 17:00:49 -0500 From: Jung-uk Kim <jkim@FreeBSD.org> To: John Baldwin <jhb@freebsd.org>, "Conrad E. Meyer" <cem@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r311667 - in head/sys/contrib/dev/acpica: components/namespace components/tables include Message-ID: <0e8abb94-160a-7ca2-1c67-3d3ef8fa522e@FreeBSD.org> In-Reply-To: <53d948ef-9e7e-422f-aead-39437abc8352@FreeBSD.org> References: <201701080626.v086QXDx022252@repo.freebsd.org> <7685799.RHuinLcsJW@ralph.baldwin.cx> <53d948ef-9e7e-422f-aead-39437abc8352@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 01/09/2017 13:01, Jung-uk Kim wrote: > On 01/09/2017 12:08, John Baldwin wrote: >> On Sunday, January 08, 2017 06:26:33 AM Conrad E. Meyer wrote: >>> Author: cem >>> Date: Sun Jan 8 06:26:33 2017 >>> New Revision: 311667 >>> URL: https://svnweb.freebsd.org/changeset/base/311667 >>> >>> Log: >>> Add some additional ACPI methods for DRM >>> >>> Add AcpiGetDataFull and AcpiGetTableWithSize. >>> >>> Submitted by: Matt Macy >> >> Have these been submitted upstream? The Intel folks are generally quite >> responsive on freebsd-acpi@FreeBSD.org and this codebase is actively >> maintained externally. > > Please submit upstream first. > > https://github.com/acpica/acpica Since nobody responded, I just googled about it. It seems these two functions, AcpiGetDataFull() (aka. acpi_get_data_full() in Linux) and AcpiGetTableWithSize() (aka. acpi_get_table_with_size() in Linux), were only added in linux-pm tree and we only need them for "Graphics" repository. - AcpiGetDataFull() This function is only called by acpi_get_device_data(), which is, in turn, only called by acpi_bus_get_device() with *NULL* callback. Therefore, I don't see any reason to pollute contrib code for this. - AcpiGetTableWithSize() This API is now deprecated by the upstream: http://marc.info/?l=linux-acpi&m=148169906815835 http://marc.info/?l=linux-acpi&m=148169907615836 http://marc.info/?l=linux-acpi&m=148169908115837 http://marc.info/?l=linux-acpi&m=148169908615838 http://marc.info/?l=linux-acpi&m=148169909215840 The code changes were committed to both ACPICA: https://github.com/acpica/acpica/commit/cac67909 and committed to linux-pm tree: https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit?id=6b11d1d677132816252004426ef220ccd3c92d2f This means it will be removed from future Linux kernel. Therefore, we should NOT implement the deprecated API. Please back out r311667 and r311843 and let me know if you need any help in resolving this matter. Jung-uk Kim
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0e8abb94-160a-7ca2-1c67-3d3ef8fa522e>