Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
IronPort-SDR: /FsvKn/sNejVzwKLl7ENl8gASOa1ML8mdAlHpEzPDyev6KHhlwDPY1G93wxQjhGgUR08r1tA4i
 QxLRoOnnQFdBFM3xXh3DpcON2XBc10itemObWnmgfU0US2IgKUHMRBYzClgrWDkcIA2WqdubIM
 wdEKj++N+MaG7Q0PF7pVaEJXQOsNzGviVWNUSp4mvd+dtKKm1KMFkoTd1O6ekA9SHb4lxXOMxv
 2+Qf+XHfb0V+cIquPiAfqcK3sWHx8e6qPqwuCODJHUh9kMSrFockC9DR1eeaAZfDA5TxEFjAJT
 JDP5BTynqgFnO+SPiJR59AES
IronPort-SDR: Vzn/mGVYXxoXT3gG8ftnPoCvjZOhFH4TN6rkj84C9eaEMK195poYFJMiQmLh0x5OELxg932m6x
 CeflhM6jH89/qwnRAkS0F3CrNCdOqlUSvmRLNeEnhSMs80qkNxnBuSBrIZ1QKafUYjem5SpE0k
 JtDzKSfrQqCGluL37nCUIF+VTSRpAS7skGEaMHC5l5Qa3feBQdmsuls4M37KgN3c66z2Y7QVIG
 iT9JFQj4Yexhkt4pNZx2lcvcR47T+HW0m9JIAYKU9500UOGtKNdHFGGMrC1MMsgVNWLntKkAmC
 DDA=
WDCIronportException: Internal
Message-ID: <69ed5587-52fd-4171-b97f-091df8b377c6@opensource.wdc.com>
Date:   Wed, 29 Jun 2022 10:51:26 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: [PATCH] ata: pata_cs5535: Fix W=1 warnings
Content-Language: en-US
To:     John Garry <john.garry@huawei.com>, s.shtylyov@omp.ru
Cc:     linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
References: <1656335540-50293-1-git-send-email-john.garry@huawei.com>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <1656335540-50293-1-git-send-email-john.garry@huawei.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On 6/27/22 22:12, John Garry wrote:
> x86_64 allmodconfig build with W=3D1 gives these warnings:
>=20
> drivers/ata/pata_cs5535.c: In function =E2=80=98cs5535_set_piomode=E2=80=
=99:
> drivers/ata/pata_cs5535.c:93:11: error: variable =E2=80=98dummy=E2=80=99=
 set but not used [-Werror=3Dunused-but-set-variable]
>   u32 reg, dummy;
>            ^~~~~
> drivers/ata/pata_cs5535.c: In function =E2=80=98cs5535_set_dmamode=E2=80=
=99:
> drivers/ata/pata_cs5535.c:132:11: error: variable =E2=80=98dummy=E2=80=99=
 set but not used [-Werror=3Dunused-but-set-variable]
>   u32 reg, dummy;
>            ^~~~~
> cc1: all warnings being treated as errors
>=20
> Mark variables 'dummy' as "maybe unused" to satisfy when rdmsr() is
> stubbed, which is the same as what we already do in pata_cs5536.c .
>=20
> Signed-off-by: John Garry <john.garry@huawei.com>

Applied to for-5.20. Thanks !

>=20
> diff --git a/drivers/ata/pata_cs5535.c b/drivers/ata/pata_cs5535.c
> index 6725931f3c35..c2c3238ff84b 100644
> --- a/drivers/ata/pata_cs5535.c
> +++ b/drivers/ata/pata_cs5535.c
> @@ -90,7 +90,7 @@ static void cs5535_set_piomode(struct ata_port *ap, s=
truct ata_device *adev)
>  	static const u16 pio_cmd_timings[5] =3D {
>  		0xF7F4, 0x53F3, 0x13F1, 0x5131, 0x1131
>  	};
> -	u32 reg, dummy;
> +	u32 reg, __maybe_unused dummy;
>  	struct ata_device *pair =3D ata_dev_pair(adev);
> =20
>  	int mode =3D adev->pio_mode - XFER_PIO_0;
> @@ -129,7 +129,7 @@ static void cs5535_set_dmamode(struct ata_port *ap,=
 struct ata_device *adev)
>  	static const u32 mwdma_timings[3] =3D {
>  		0x7F0FFFF3, 0x7F035352, 0x7F024241
>  	};
> -	u32 reg, dummy;
> +	u32 reg, __maybe_unused dummy;
>  	int mode =3D adev->dma_mode;
> =20
>  	rdmsr(ATAC_CH0D0_DMA + 2 * adev->devno, reg, dummy);


--=20
Damien Le Moal
Western Digital Research