Re: [RFC PATCH] mmc: block: Support the fixed index for mmcblk with aliases nodes

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

 



Hi Jaehoon,

On 02/13/2017 02:34 AM, Jaehoon Chung wrote:
> Hi,
> 
> On 02/10/2017 08:35 PM, Tomas Melin wrote:
>> Hi,
>>
>> It looks as this RFC was never posted as a real patch.
>> For embedded devices with several memory devices, some being removable,
>> providing a way to define aliases for the nodes in the device tree is a benefit.
>> I have tested the patch from Jaehoon and it works nicely.
> 
> I'm not sure there is more better solution than this.
The same approach to use aliases in DT and parse them is if I understand
correctly also how serial ports are sorted in uart drivers.

> But I guess it's useful to someone..
> If someone want to use this, i will resend the patch based on latest mmc.

That would be great.

Tomas

> 
> Best Regards,
> Jaehoon Chung
> 
>>
>> Could this be reposted and considered for inclusion?
>>
>> Found the original patch here: https://patchwork.kernel.org/patch/8685711/
>>
>> Best Regards
>> Tomas
>>
>>
>> On 04/04/2016 09:14 PM, Ulf Hansson wrote:
>>> On 4 April 2016 at 04:08, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote:
>>>> Hi All,
>>>>
>>>> Are there any other opinion for this RFC patch?
>>>>
>>>> Best Regards,
>>>> Jaehoon Chung
>>>>
>>>> On 03/30/2016 09:15 AM, Shawn Lin wrote:
>>>>> 在 2016/3/29 18:49, Jisheng Zhang 写道:
>>>>>> Hi Jaehoon,
>>>>>>
>>>>>> On Tue, 29 Mar 2016 19:43:34 +0900 Jaehoon Chung wrote:
>>>>>>
>>>>>>> This patch is just RFC. I want to know opinions.
>>>>>>>
>>>>>>> Now, index of mmcblk is allocated in accordance with probing time.
>>>>>>> If want to use the mmcblk1 for some device, it can use alias.
>>>>>>>
>>>>>>> aliases {
>>>>>>>     mmc0 = &mmc0;    /* mmcblk0 for eMMC */
>>>>>>>     mmc1 = &mmc2;    /* mmcblk1 for SD */
>>>>>>>     mmc2 = &mmc1;    /* mmcblk2 for SDIO*/
>>>>>>> };
>>>>>>
>>>>>> I like this feature and we do need it. In the past, we have to put emmc dt node
>>>>>> before the sd node, but on our platform, the register base of sdhc host for
>> emmc
>>>>>> is higher than the one for sd, putting emmc dt node before sd looks a bit
>>>>>> strange, but we have no better solutions.
>>>>>>
>>>>>
>>>>> :) That is also what I suffered currently. I have to prepare two
>>>>> seperate firmware with two diff fstab files to assign the mount point
>>>>> since sometime I need to boot from sd, but sometimes from emmc....
>>>>> With this patch, I only need to dtc a new dtb... quite simple.
>>>>>
>>>>>> Thanks,
>>>>>> Jisheng
>>>
>>> I am not immediately opposed to this patch, although let me think a
>>> bit more about it.
>>>
>>> What I would like to understand is why UUID/PARTUUID isn't working for
>>> your case. Did you try to use that?
>>
>> I have used with UUID/PARTUUID/PARTLABEL and etc.. I think it's not problem.
>> But that is why some guys wants to use the fixed index.
>> (Almost all SoCs are still using "root=mmcblk0pX" in bootloader.)
>>
>> I'm considering more what is better. :)
>>
>> Best Regards,
>> Jaehoon Chung
>>
>>>
>>> Kind regards
>>> Uffe
>>> --
>>> 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
>>>
>>>
>>
>> --
>> 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
>>
>>
> 
> --
> 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
> 
--
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



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux