Re: [PATCH v1 0/2] Hide flash name if it is not set

Tudor Ambarus
2023-11-27T12:03:45Z

On 11/27/23 11:58, Michael Walle wrote:
> Hi,
>
>>> hide flash name if it is not set
>>
>> Is this really needed? I'm fine exposing en empty (or "(null)") name. I
>> guess it shows to people that the name might be missing somewhere and
>> might push people to fix the name instead.
>>
>> What's your use case?
>
> An empty string is not a bug. I'm trying to push the part names out of
> the kernel and just expose the jedec id.
>
> That being said, the description is really missing the point. See my
> previous suggestion:
>
>  The attribute is optional. User space shouldn't rely on it to
>  be present or even correct. Instead, user space should read the
>  jedec_id attribute.
>

+1