Re: [PATCH 1/3] ram: rk3399: Fix .set_rate_index() error handling

Kever Yang
2022-07-04T10:49:54Z

Hi

On 2022/7/4 17:23, Lee Jones wrote:
> On Fri, 01 Jul 2022, Kever Yang wrote:
>
>> Hi Lee Jones,
>>
>>
>> On 2022/6/27 16:39, Lee Jones wrote:
>>> On Tue, 21 Jun 2022, Lee Jones wrote:
>>>
>>>> Functions pointed to by this op pointer can return non-zero values
>>>> indicating an error. Ensure any error value is propagated back up the
>>>> call-chain.
>>>>
>>>> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>>>> ---
>>>> drivers/ram/rockchip/sdram_rk3399.c | 4 +++-
>>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>> Weekly check-in:
>>>
>>> Are these still on someone's radar, or would you like me to [RESEND]?
>> It would be better to add leading message "rockchip:" in the subject.
> That would be highly unusual.
>
> Patch subjects are usually formed, as follows:

You should able to see many patches with "rockchip:" in U-Boot commit,
and also many patches

with other platform prefix like "imx:" or "sunxi:".

Not like kernel, U-Boot using the same mailing list for all soc
platform, so this can help easier

to identify which soc vendor it belongs to.

Thanks,

- Kever

>
> "<subsystem>: <device>: <description>"
>
> I only see 2 RAM-only patches (out of the 91 that touch this file)
> that do as you suggested and you were the author of both of them. :)
>