Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] sr_vendor: remove references to BLK_DEV_SR_VENDOR, leave
 it enabled.
To:     =?UTF-8?Q?Diego_Elio_Petten=c3=b2?= <flameeyes@flameeyes.com>
Cc:     linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
References: <20200223191144.726-1-flameeyes@flameeyes.com>
From:   Jens Axboe <axboe@kernel.dk>
Message-ID: <28e56c52-1f36-eaf5-0a92-2ffd494472ac@kernel.dk>
Date:   Mon, 24 Feb 2020 09:23:10 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.4.1
MIME-Version: 1.0
In-Reply-To: <20200223191144.726-1-flameeyes@flameeyes.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 2/23/20 12:11 PM, Diego Elio Pettenò wrote:
> This kernel configuration is basically enabling/disabling sr driver quirks
> detection. While these quirks are for fairly rare devices (very old CD
> burners, and a glucometer), the additional detection of these models is a
> very minimal amount of code.
> 
> The logic behind the quirks is always built into the sr driver.
> 
> This also removes the config from all the defconfig files that are enabling
> this already.

Looks fine to me, kind of silly to have that option.

Reviewed-by: Jens Axboe <axboe@kernel.dk>

Assuming the SCSI tree will pick this up.

-- 
Jens Axboe