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

Kever Yang kever.yang at rock-chips.com
Mon Jul 4 12:49:33 CEST 2022


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 at 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. :)
>


More information about the U-Boot mailing list