[lkml]   [2017]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH v2 0/8] fujitsu-laptop: use device-specific data instead of module-wide globals
fujitsu-laptop registers two ACPI drivers that access each other's
module-wide structures. To improve data encapsulation and lay the
groundwork for separating the two aforementioned ACPI drivers into
separate modules, move away from module-wide global data structures by
using device-specific data instead.

To avoid breaking a working feature (backlight power synchronization
upon module load), this series leaves the module-wide struct fujitsu_bl
in place. It will be taken care of when the backlight driver is split
into a separate module.

As we agreed that grabbing a handle to an ACPI device using its absolute
path is not a truly elegant solution, this series uses a different
approach to call_fext_func() than v1. By passing that function a
pointer to a struct acpi_device instead of an acpi_handle, all relevant
static functions of the module will now use the same type for their
first argument and the acpi_handle fields of both module-wide structures
are removed altogether.

This patch series was tested on a Lifebook S7020 and a Lifebook E744.

As with v1, adhering to the "one logical change per patch" rule was
tricky. If the changes introduced are illegible, I will be happy to
further explain and/or improve the series. Using --color-words should
make reviewing much more manageable.

Changes from v1:

- Drop patch 01/10 from v1, i.e. do not introduce fext_*() helper

- Drop patch 02/10 from v1 as the acpi_handle fields of both
module-wide structures are removed altogether by other patches.

- Replace patch 03/10 from v1 with patch 6/8, passing call_fext_func()
a pointer to struct acpi_device instead of an acpi_handle.

- Drop patch 04/10 from v1, thus deferring driver separation until the
split into separate modules. Consider patch 5/8 a partial spiritual
successor ;) More information is available in the commit message of
that patch.

- Add an additional check to patch 2/8 to avoid a NULL dereference
which could happen due to patch 04/10 from v1 being dropped.

- Do not store ACPI handles in private structures. Instead, extract
them directly from struct acpi_device pointers passed as function

- Updated commit messages.

- As the above might be a bit confusing, here is the patch number
mapping from v1 to v2:

v1 | v2
01/10 | dropped
02/10 | dropped
03/10 | 6/8
04/10 | 5/8
05/10 | 1/8
06/10 | 2/8
07/10 | 3/8
08/10 | 4/8
09/10 | 7/8
10/10 | 8/8

drivers/platform/x86/fujitsu-laptop.c | 417 +++++++++++++++++-----------------
1 file changed, 213 insertions(+), 204 deletions(-)


 \ /
  Last update: 2017-05-19 09:45    [W:0.053 / U:8.524 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site