-----Original Message----- From: Ulf Hansson [mailto:ulf.hansson@xxxxxxxxxx] Sent: 2018年5月23日 19:28 To: Yinbo Zhu <yinbo.zhu@xxxxxxx> Cc: Y.b. Lu <yangbo.lu@xxxxxxx>; Adrian Hunter <adrian.hunter@xxxxxxxxx>; linux-mmc@xxxxxxxxxxxxxxx; Xiaobo Xie <xiaobo.xie@xxxxxxx> Subject: Re: [PATCH v2] mmc: sd: Macro name cleanup for full speed dtr On 23 May 2018 at 12:52, Yinbo Zhu <yinbo.zhu@xxxxxxx> wrote: > Add new macros for the full speed 25MHz case >For clarity, pleas re-word this as it's not a macro, but a define. >Moreover, I don't consider this a "cleanup", which is what the commit msg header says. Instead you are adding a new >define for SD default speed mode, please make that clear. Hi Ulf Thanks your feedback, and I will change the commit information as your description >Also, it's not clear *why* you needs this, and that you need to explain. This change just introduce the new define but >without anybody using it, it doesn't make sense. I add the define is for the sd/mmc limit the max-frequency's patch that need it. Thanks Regrds, Yinbo > > Signed-off-by: Yinbo Zhu <yinbo.zhu@xxxxxxx> > --- > Change in v2: > Modify the macro name from FULL_SPEED_MAX_DTR to DEFAULT_SPEED_MAX_DTR > include/linux/mmc/card.h | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h index > 279b390..f5be8c7 100644 > --- a/include/linux/mmc/card.h > +++ b/include/linux/mmc/card.h > @@ -156,6 +156,7 @@ struct sd_switch_caps { > #define UHS_DDR50_MAX_DTR 50000000 > #define UHS_SDR25_MAX_DTR UHS_DDR50_MAX_DTR > #define UHS_SDR12_MAX_DTR 25000000 > +#define DEFAULT_SPEED_MAX_DTR UHS_SDR12_MAX_DTR > unsigned int sd3_bus_mode; > #define UHS_SDR12_BUS_SPEED 0 > #define HIGH_SPEED_BUS_SPEED 1 > -- > 1.7.1 > Kind regards Uffe ��.n��������+%������w��{.n�����{��i��)�� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
