On 08/16/2012 01:09 PM, IAN CHEN wrote: > MMC_QUIRK_SEC_ERASE_TRIM_BROKEN seems quite clear but I might not use "SEC" > due to it could be Samsung Electronics Corporation. I didn't think so. MMC developers didn't think that SEC means Samsung Electronics Corporation. Already we used the flags with "_SEC_" It's no problem.
Best Regards, Jaehoon Chung > So, how about MMC_QUIRK_SECDISCARD_BROKEN which was used in > mmc_blk_issue_secdiscard_rq() naming, the function we skip. > Or, MMC_QUIRK_SECERASE_SECTRIM_BROKEN? > > Regards, > Ian > -----Original Message----- > From: Linus Walleij [mailto:[email protected]] > Sent: Tuesday, August 14, 2012 5:07 PM > To: IAN CHEN > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; [email protected] > Subject: Re: [PATCH v3] mmc: card: Skip secure option for MoviNAND. > > On Tue, Aug 14, 2012 at 9:06 AM, IAN CHEN <[email protected]> wrote: > >> +#define MMC_QUIRK_SECURE_MODE_BROKEN (1<<10) /* Skip secure for >> erase/trim */ > > OK this will work and sorry for being picky, but we are working with RPMB > support and that's some other "secure mode" so to avoid any > misuderstandings it's better to name the flag like I proposed, > MMC_QUIRK_SEC_ERASE_TRIM_BROKEN > > I won't die if this goes in though :-) > > Yours, > Linus Walleij > > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
