Hi, Ulf. On 07/10/2014 06:15 PM, Ulf Hansson wrote: > On 10 July 2014 09:08, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: >> Replaced the "disable-wp" into host's quirks. >> (Because the slot-node is removed at dt-file.) >> >> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> > > Hi Jaehoon, > > I plan to carry these patches through my mmc tree and I will share > them through a PR from an immutable branch with the ARM SoC guys. > Please tell me if you have any concern with this set up. > > Now, while I was applying this patch, I found it had checkpatch > errors. Please run checkpatch and resend a new version. Sorry for not checking the patch. I will resend a new version. Thanks for pointing out. Best Regards, Jaehoon Chung > > Kind regards > Uffe > >> --- >> drivers/mmc/host/dw_mmc.c | 12 +++++------- >> 1 file changed, 5 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c >> index 1ac227c..a3ccd07 100644 >> --- a/drivers/mmc/host/dw_mmc.c >> +++ b/drivers/mmc/host/dw_mmc.c >> @@ -997,7 +997,7 @@ static int dw_mci_get_ro(struct mmc_host *mmc) >> int gpio_ro = mmc_gpio_get_ro(mmc); >> >> /* Use platform get_ro function, else try on board write protect */ >> - if (slot->quirks & DW_MCI_SLOT_QUIRK_NO_WRITE_PROTECT) >> + if (slot->host->quirks & DW_MCI_SLOT_QUIRK_NO_WRITE_PROTECT) >> read_only = 0; >> else if (!IS_ERR_VALUE(gpio_ro)) >> read_only = gpio_ro; >> @@ -2006,12 +2006,7 @@ static struct device_node *dw_mci_of_find_slot_node(struct device *dev, u8 slot) >> static struct dw_mci_of_slot_quirks { >> char *quirk; >> int id; >> -} of_slot_quirks[] = { >> - { >> - .quirk = "disable-wp", >> - .id = DW_MCI_SLOT_QUIRK_NO_WRITE_PROTECT, >> - }, >> -}; >> +} of_slot_quirks[] = {}; >> >> static int dw_mci_of_get_slot_quirks(struct device *dev, u8 slot) >> { >> @@ -2238,6 +2233,9 @@ static struct dw_mci_of_quirks { >> { >> .quirk = "broken-cd", >> .id = DW_MCI_QUIRK_BROKEN_CARD_DETECTION, >> + },{ >> + .quirk = "disable-wp", >> + .id = DW_MCI_SLOT_QUIRK_NO_WRITE_PROTECT, >> }, >> }; >> >> -- >> 1.7.9.5 >> > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html