Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] ACPI: scan: Fix up _DEP-related terminology with
 supplier/consumer
To:     "Rafael J. Wysocki" <rjw@rjwysocki.net>,
        Linux ACPI <linux-acpi@vger.kernel.org>
Cc:     LKML <linux-kernel@vger.kernel.org>,
        Kees Cook <keescook@chromium.org>
References: <6314382.p3e4rEhblS@kreacher>
From:   Hans de Goede <hdegoede@redhat.com>
Message-ID: <054cc790-bebb-9c84-e14c-16b9460d1636@redhat.com>
Date:   Mon, 7 Dec 2020 19:21:50 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.4.0
MIME-Version: 1.0
In-Reply-To: <6314382.p3e4rEhblS@kreacher>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi,

On 12/7/20 6:46 PM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> The ACPI namespace scanning code uses the terms master/slave when
> populating the list of _DEP dependencies, but that use has no
> external exposures and is not mandated by nor associated with any
> external specifications.
> 
> Change the language used through-out to supplier/consumer.
> 
> No functional impact.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede@redhat.com>

Regards,

Hans


> ---
>  drivers/acpi/scan.c |   12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> Index: linux-pm/drivers/acpi/scan.c
> ===================================================================
> --- linux-pm.orig/drivers/acpi/scan.c
> +++ linux-pm/drivers/acpi/scan.c
> @@ -51,8 +51,8 @@ static u64 spcr_uart_addr;
>  
>  struct acpi_dep_data {
>  	struct list_head node;
> -	acpi_handle master;
> -	acpi_handle slave;
> +	acpi_handle supplier;
> +	acpi_handle consumer;
>  };
>  
>  void acpi_scan_lock_acquire(void)
> @@ -1881,8 +1881,8 @@ static void acpi_device_dep_initialize(s
>  		if (!dep)
>  			return;
>  
> -		dep->master = dep_devices.handles[i];
> -		dep->slave  = adev->handle;
> +		dep->supplier = dep_devices.handles[i];
> +		dep->consumer  = adev->handle;
>  		adev->dep_unmet++;
>  
>  		mutex_lock(&acpi_dep_list_lock);
> @@ -2058,8 +2058,8 @@ void acpi_walk_dep_device_list(acpi_hand
>  
>  	mutex_lock(&acpi_dep_list_lock);
>  	list_for_each_entry_safe(dep, tmp, &acpi_dep_list, node) {
> -		if (dep->master == handle) {
> -			acpi_bus_get_device(dep->slave, &adev);
> +		if (dep->supplier == handle) {
> +			acpi_bus_get_device(dep->consumer, &adev);
>  			if (!adev)
>  				continue;
>  
> 
> 
>