[PATCH] dt-bindings: regulator: mediatek,mt6397-regulator: convert to YAML

Macpaul Lin macpaul.lin at mediatek.com
Thu Aug 8 22:59:23 PDT 2024



On 8/8/24 20:05, Krzysztof Kozlowski wrote:
> 	
> 
> External email : Please do not click links or open attachments until you 
> have verified the sender or the content.
> 
> On 07/08/2024 12:32, Macpaul Lin wrote:
>>>> +  - |
>>>> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
>>>> +
>>>> +    pwrap {
>>>> +       pmic {
>>>> +            compatible = "mediatek,mt6397";
>>>
>>> Messed indentation.
>>>
>>> Use 4 spaces for example indentation.
>>>
>>> Anyway, drop top node or better move the example to the parent device
>>> schema making it complete.
> 
> Look at this comment. I asked you to move to the parent device schema...
> 
> Best regards,
> Krzysztof
> 

I would like to ensure that the progress can be made step by step.

Since I am not specifically responsible for this driver, I cannot 
determine when I will be able to modify the next file after completing 
one today.
Will it be tomorrow? Perhaps in a few months?
There is also the possibility that I might have to pause midway due to 
other job assignments.

Additionally, I need to obtain approval from other internal colleagues.
I am pleased to have received permission to modify 
mfd/mediatek,mt6397.yaml the day after I submitted this conversion of 
mt6397-regulator.

I was intend to split the example parts of the patches into 3 parts 
seprately, mt6397-regulator.yaml, mfd/mediatek,mt6397.yaml,i
then moving the examples from mt6397-regulator.yaml to 
mfd/mediatek,mt6397.yaml, move the content to parent device step by step.

I will include the example part in next version patch in 
mfd/mediatek,mt6397.yaml.

Thanks for the reminder.
Macpaul Lin



More information about the Linux-mediatek mailing list