RE: [PATCH v1 2/2] scsi: ufs-mediatek: Add UFS support for Mediatek SoC chips

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

 



Hi,

> +
> +struct ufs_mtk_dev_params {
> +	u32 pwm_rx_gear;        /* pwm rx gear to work in */
> +	u32 pwm_tx_gear;        /* pwm tx gear to work in */
> +	u32 hs_rx_gear;         /* hs rx gear to work in */
> +	u32 hs_tx_gear;         /* hs tx gear to work in */
> +	u32 rx_lanes;           /* number of rx lanes */
> +	u32 tx_lanes;           /* number of tx lanes */
> +	u32 rx_pwr_pwm;         /* rx pwm working pwr */
> +	u32 tx_pwr_pwm;         /* tx pwm working pwr */
> +	u32 rx_pwr_hs;          /* rx hs working pwr */
> +	u32 tx_pwr_hs;          /* tx hs working pwr */
> +	u32 hs_rate;            /* rate A/B to work in HS */
> +	u32 desired_working_mode;
> +};
As this is essentially identical to ufs_qcom_dev_params and ufs_hisi_dev_params etc.,
Maybe move it to ufshcd-platform.h in a short preparatory patch, so it can be reused?

Also, the vast majority of the code in these platform-dependent modules is pretty much the same.
I wonder if it can be shape to fit some standard form?
But this is out of scope of your series.


> +/*
> + * PHY Adpater attributes
> + */
> +#define PA_LOCALTXLCCENABLE         0x155E
Already in unipro.h

Thanks,
Avri




[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