[PATCH] NVMe: Add MAXIO 1602 to bogus nid list.

[Christoph Hellwig]

On Sat, May 20, 2023 at 02:56:03AM +0900, Tatsuki Sugiura wrote:
> HIKSEMI FUTURE M.2 SSD uses the same dummy nguid and eui64.
> I confirmed it with my two devices.
> 
> This patch marks the controller as NVME_QUIRK_BOGUS_NID.
> 
> ---------------------------------------------------------
> sugi at tempest:~% sudo nvme id-ctrl /dev/nvme0
> NVME Identify Controller:
> vid       : 0x1e4b
> ssvid     : 0x1e4b
> sn        : 30096022612
> mn        : HS-SSD-FUTURE 2048G
> fr        : SN10542
> rab       : 0
> ieee      : 000000
> cmic      : 0
> mdts      : 7
> cntlid    : 0
> ver       : 0x10400
> rtd3r     : 0x7a120
> rtd3e     : 0x1e8480
> oaes      : 0x200
> ctratt    : 0x2
> rrls      : 0
> cntrltype : 1
> fguid     : 00000000-0000-0000-0000-000000000000
> <snip...>
> ---------------------------------------------------------
> 
> ---------------------------------------------------------
> sugi at tempest:~% sudo nvme id-ns /dev/nvme0n1
> NVME Identify Namespace 1:
> <snip...>
> nguid   : 00000000000000000000000000000000
> eui64   : 0000000000000002
> lbaf  0 : ms:0   lbads:9  rp:0 (in use)
> ---------------------------------------------------------
> 
> Signed-off-by: Tatsuki Sugiura <sugi at nemui.org>
> ---
>  drivers/nvme/host/pci.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c
> index 7f25c0fe3a0b..dae0f8ed0af9 100644
> --- a/drivers/nvme/host/pci.c
> +++ b/drivers/nvme/host/pci.c
> @@ -3422,6 +3422,8 @@ static const struct pci_device_id nvme_id_table[] = {
>  		.driver_data = NVME_QUIRK_BOGUS_NID, },
>  	{ PCI_DEVICE(0x1e4B, 0x1202),   /* MAXIO MAP1202 */
>  		.driver_data = NVME_QUIRK_BOGUS_NID, },
> +	{ PCI_DEVICE(0x1e4B, 0x1602),   /* MAXIO MAP1602 like HIKSEMI FUTURE NVMe SSD */

It seems like the vendor ID is MAXIO, so I'd remove the
˝like HIKSEMI FUTURE NVMe SSD" here.

Otherwise this looks good:

Reviewed-by: Christoph Hellwig <hch at lst.de>