-----Original Message----- From: Tejun Heo [mailto:htejun@xxxxxxxxx] On Behalf Of Tejun Heo Sent: Wednesday, March 21, 2018 4:12 AM To: Richard Zhu <hongxing.zhu@xxxxxxx> Cc: hdegoede@xxxxxxxxxx; linux-ide@xxxxxxxxxxxxxxx Subject: Re: [RFC] ahci: imx: fix the build warning On Tue, Mar 20, 2018 at 01:01:34PM +0800, Richard Zhu wrote: > Fix the following build warning introduced by commit. > e5878732a521 ("ahci: imx: add the imx6qp ahci sata support") > > drivers/ata/ahci_imx.c: In function 'imx_sata_disable': > drivers/ata/ahci_imx.c:478:2: warning: enumeration value 'AHCI_IMX53' > not handled in switch [-Wswitch] > switch (imxpriv->type) { > ^~~~~~ > > Signed-off-by: Richard Zhu <hongxing.zhu@xxxxxxx> > --- > drivers/ata/ahci_imx.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/ata/ahci_imx.c b/drivers/ata/ahci_imx.c index > 577458f..f898a0f 100644 > --- a/drivers/ata/ahci_imx.c > +++ b/drivers/ata/ahci_imx.c > @@ -476,6 +476,9 @@ static void imx_sata_disable(struct ahci_host_priv *hpriv) > return; > > switch (imxpriv->type) { > + case AHCI_IMX53: > + break; > + Can we just add default: as the last entry or just switch it to if/else? [Richard] Thanks for your comment. Okay, the default as the last entry in next version. Thanks. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html