[PATCH 3.16.y-ckt 118/135] ahci_xgene: Fix the endianess issue in APM X-Gene SoC AHCI SATA controller driver.

Suman Tripathi stripathi at apm.com
Fri Feb 6 12:18:35 UTC 2015


3.16.7-ckt6 -stable review patch.  If anyone has any objections, please let
me know.

------------------

From: Suman Tripathi <stripathi at apm.com>

commit 5c0b8e0de76a86edb99e46612fd9d341b4c4fa0a upstream.

This patch fixes the big endian mode issue with function
xgene_ahci_read_id.

Signed-off-by: Suman Tripathi <stripathi at apm.com>
Signed-off-by: Tejun Heo <tj at kernel.org>
Signed-off-by: Luis Henriques <luis.henriques at canonical.com>
Acked-by:  Suman Tripathi <stripathi at apm.com>
---
 drivers/ata/ahci_xgene.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/ata/ahci_xgene.c b/drivers/ata/ahci_xgene.c
index 10d524699676..dcd758be12db 100644
--- a/drivers/ata/ahci_xgene.c
+++ b/drivers/ata/ahci_xgene.c
@@ -174,7 +174,7 @@ static unsigned int xgene_ahci_read_id(struct
ata_device *dev,
         *
         * Clear reserved bit 8 (DEVSLP bit) as we don't support DEVSLP
         */
-       id[ATA_ID_FEATURE_SUPP] &= ~(1 << 8);
+       id[ATA_ID_FEATURE_SUPP] &= cpu_to_le16(~(1 << 8));

        return 0;
 }

On Fri, Feb 6, 2015 at 5:31 PM, Luis Henriques <luis.henriques at canonical.com
> wrote:

> 3.16.7-ckt6 -stable review patch.  If anyone has any objections, please
> let me know.
>
> ------------------
>
> From: Suman Tripathi <stripathi at apm.com>
>
> commit 5c0b8e0de76a86edb99e46612fd9d341b4c4fa0a upstream.
>
> This patch fixes the big endian mode issue with function
> xgene_ahci_read_id.
>
> Signed-off-by: Suman Tripathi <stripathi at apm.com>
> Signed-off-by: Tejun Heo <tj at kernel.org>
> Signed-off-by: Luis Henriques <luis.henriques at canonical.com>
> ---
>  drivers/ata/ahci_xgene.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/ata/ahci_xgene.c b/drivers/ata/ahci_xgene.c
> index 10d524699676..dcd758be12db 100644
> --- a/drivers/ata/ahci_xgene.c
> +++ b/drivers/ata/ahci_xgene.c
> @@ -174,7 +174,7 @@ static unsigned int xgene_ahci_read_id(struct
> ata_device *dev,
>          *
>          * Clear reserved bit 8 (DEVSLP bit) as we don't support DEVSLP
>          */
> -       id[ATA_ID_FEATURE_SUPP] &= ~(1 << 8);
> +       id[ATA_ID_FEATURE_SUPP] &= cpu_to_le16(~(1 << 8));
>
>         return 0;
>  }
> --
> 2.1.4
>
>


-- 
Thanks,
with regards,
Suman Tripathi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20150206/52a30a24/attachment.html>


More information about the kernel-team mailing list