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,

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.
But I guess it's useful to someone..
If someone want to use this, i will resend the patch based on latest mmc.

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



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

  Powered by Linux