Re: [PATCH] add driver for DesignWare UFS Host Controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 1/8/2016 2:20 PM, Akinobu Mita wrote:
> 2016-01-07 18:44 GMT+09:00 Christoph Hellwig <hch@xxxxxxxxxxxxx>:
>>> This driver patch includes a core driver and glue drivers for pci and platform
>>> for the DesignWare UFS Host IP.
>>
>> Why doesn't this use the existing ufs core?  The architecture looks
>> completely backwards to me.
> 
> I agree.  The existing ufs driver can have variant specific operations
> (hba->vops) and also can define quirks.  So if DesignWare UFS host
> controller requires vendor specific register settings or DME operations
> in initialization, or needs special workarounds for the specific versions,
> it can use those mechanisms and shares common parts between other host
> controllers.
> 

I am planning to make some tests with the ufs kernel existing core driver in our
setup, but I noticed that despite the scsi/ufs driver package contains a
platform driver, this is not the typical platform driver (glue driver) which can
be addressed from device tree. The only one available is the qcom's.
Would I need to submit a designware platform glue driver also as done by QCom?

We also need a pci glue driver, but you already have one. In your opinion the
best approach is to add synopsys device id to the pci device list in the driver
and add quirks or add a new designware pci glue driver?

Thanks,
Joao
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux