[PATCH] nvme: fix for big endian systems

David Lamparter equinox at diac24.net
Tue May 18 13:09:17 CEST 2021


Hi all,


On Thu, May 06, 2021 at 08:24:30PM +0200, David Lamparter wrote:
> writel() and co. already include the endian swap;  doing the swap twice
> is, er, unhelpful.
> 
> Tested on a P4080DS, which boots perfectly fine off NVMe with this.

is there anything to be done on my end to get this merged?  The
MAINTAINERS file has no entry for drivers/nvme/ so I'm not quite sure
who to ping on this...

Cheers,


David


> diff --git a/drivers/nvme/nvme.c b/drivers/nvme/nvme.c
> index c61dab20c5f0..d554ec54cbe0 100644
> --- a/drivers/nvme/nvme.c
> +++ b/drivers/nvme/nvme.c
> @@ -157,7 +157,7 @@ static u16 nvme_read_completion_status(struct nvme_queue *nvmeq, u16 index)
>  
>  	invalidate_dcache_range(start, stop);
>  
> -	return le16_to_cpu(readw(&(nvmeq->cqes[index].status)));
> +	return readw(&(nvmeq->cqes[index].status));
>  }
>  
>  /**
> @@ -221,7 +221,7 @@ static int nvme_submit_sync_cmd(struct nvme_queue *nvmeq,
>  	}
>  
>  	if (result)
> -		*result = le32_to_cpu(readl(&(nvmeq->cqes[head].result)));
> +		*result = readl(&(nvmeq->cqes[head].result));
>  
>  	if (++head == nvmeq->q_depth) {
>  		head = 0;
> @@ -304,7 +304,7 @@ static int nvme_enable_ctrl(struct nvme_dev *dev)
>  {
>  	dev->ctrl_config &= ~NVME_CC_SHN_MASK;
>  	dev->ctrl_config |= NVME_CC_ENABLE;
> -	writel(cpu_to_le32(dev->ctrl_config), &dev->bar->cc);
> +	writel(dev->ctrl_config, &dev->bar->cc);
>  
>  	return nvme_wait_ready(dev, true);
>  }
> @@ -313,7 +313,7 @@ static int nvme_disable_ctrl(struct nvme_dev *dev)
>  {
>  	dev->ctrl_config &= ~NVME_CC_SHN_MASK;
>  	dev->ctrl_config &= ~NVME_CC_ENABLE;
> -	writel(cpu_to_le32(dev->ctrl_config), &dev->bar->cc);
> +	writel(dev->ctrl_config, &dev->bar->cc);
>  
>  	return nvme_wait_ready(dev, false);
>  }


More information about the U-Boot mailing list