mbox series

[v2,0/2] Add mmsys reset control for MT8186

Message ID 20220217082626.15728-1-rex-bc.chen@mediatek.com (mailing list archive)
Headers show
Series Add mmsys reset control for MT8186 | expand

Message

Rex-BC Chen (陳柏辰) Feb. 17, 2022, 8:26 a.m. UTC
v2:
1. Change variable type from unsigned int to u16.

v1:
1. Add a new variable in mmsys driver data to control different register
   offset for different SoCs.
2. Add MT8183 reset register offset.
3. Add mmsys reset control for MT8186.

This series is based on mmsys patch for MT8186 on [1].
[1]: https://patchwork.kernel.org/project/linux-mediatek/patch/20220216084831.14883-4-rex-bc.chen@mediatek.com/

Rex-BC Chen (2):
  soc: mediatek: mmsys: add sw0_rst_offset in mmsys driver data
  soc: mediatek: mmsys: add mmsys reset control for MT8186

 drivers/soc/mediatek/mt8183-mmsys.h | 2 ++
 drivers/soc/mediatek/mt8186-mmsys.h | 2 ++
 drivers/soc/mediatek/mtk-mmsys.c    | 7 +++++--
 drivers/soc/mediatek/mtk-mmsys.h    | 3 +--
 4 files changed, 10 insertions(+), 4 deletions(-)

Comments

Matthias Brugger March 1, 2022, 7:35 a.m. UTC | #1
On 17/02/2022 09:26, Rex-BC Chen wrote:
> v2:
> 1. Change variable type from unsigned int to u16.
> 
> v1:
> 1. Add a new variable in mmsys driver data to control different register
>     offset for different SoCs.
> 2. Add MT8183 reset register offset.
> 3. Add mmsys reset control for MT8186.
> 
> This series is based on mmsys patch for MT8186 on [1].
> [1]: https://patchwork.kernel.org/project/linux-mediatek/patch/20220216084831.14883-4-rex-bc.chen@mediatek.com/
> 
> Rex-BC Chen (2):
>    soc: mediatek: mmsys: add sw0_rst_offset in mmsys driver data
>    soc: mediatek: mmsys: add mmsys reset control for MT8186
> 
>   drivers/soc/mediatek/mt8183-mmsys.h | 2 ++
>   drivers/soc/mediatek/mt8186-mmsys.h | 2 ++
>   drivers/soc/mediatek/mtk-mmsys.c    | 7 +++++--
>   drivers/soc/mediatek/mtk-mmsys.h    | 3 +--
>   4 files changed, 10 insertions(+), 4 deletions(-)
> 

Series applied, thanks!
Rex-BC Chen (陳柏辰) March 1, 2022, 11:13 a.m. UTC | #2
Hello Matthias,

Thanks for accepting my patches, but I didn't see this patch [1]
in v5.17-next/soc.
There will be build error if [1] is not included while [2] is applied.
The sw0_rst_offset is declared in [2].

If I have mistake, please let me know.
Thanks!

[1]:
https://patchwork.kernel.org/project/linux-mediatek/patch/20220217082626.15728-2-rex-bc.chen@mediatek.com/

[2]:
https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/commit/?h=v5.17-next/soc&id=831785f0e5b919c29e1bc5f9a74e9ebd38289e24

BRs,
Rex
On Tue, 2022-03-01 at 08:35 +0100, Matthias Brugger wrote:
> 
> On 17/02/2022 09:26, Rex-BC Chen wrote:
> > v2:
> > 1. Change variable type from unsigned int to u16.
> > 
> > v1:
> > 1. Add a new variable in mmsys driver data to control different
> > register
> >     offset for different SoCs.
> > 2. Add MT8183 reset register offset.
> > 3. Add mmsys reset control for MT8186.
> > 
> > This series is based on mmsys patch for MT8186 on [1].
> > [1]: 
> > https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/patch/20220216084831.14883-4-rex-bc.chen@mediatek.com/__;!!CTRNKA9wMg0ARbw!wp2uW6phbST6ySpYRlb7iwmATAVkkCL1JBau9dCwhXj8KRasrAWnLd1EUT9gLE-oF9Yq$
> >  
> > 
> > Rex-BC Chen (2):
> >    soc: mediatek: mmsys: add sw0_rst_offset in mmsys driver data
> >    soc: mediatek: mmsys: add mmsys reset control for MT8186
> > 
> >   drivers/soc/mediatek/mt8183-mmsys.h | 2 ++
> >   drivers/soc/mediatek/mt8186-mmsys.h | 2 ++
> >   drivers/soc/mediatek/mtk-mmsys.c    | 7 +++++--
> >   drivers/soc/mediatek/mtk-mmsys.h    | 3 +--
> >   4 files changed, 10 insertions(+), 4 deletions(-)
> > 
> 
> Series applied, thanks!
Matthias Brugger March 1, 2022, 3:20 p.m. UTC | #3
Hi Rex,

On 01/03/2022 12:13, Rex-BC Chen wrote:
> Hello Matthias,
> 
> Thanks for accepting my patches, but I didn't see this patch [1]
> in v5.17-next/soc.
> There will be build error if [1] is not included while [2] is applied.
> The sw0_rst_offset is declared in [2].
> 
> If I have mistake, please let me know.

Thanks for the info. That was actually my mistake. Should be fixed by now.

Thanks a lot!
Matthias

> Thanks!
> 
> [1]:
> https://patchwork.kernel.org/project/linux-mediatek/patch/20220217082626.15728-2-rex-bc.chen@mediatek.com/
> 
> [2]:
> https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/commit/?h=v5.17-next/soc&id=831785f0e5b919c29e1bc5f9a74e9ebd38289e24
> 
> BRs,
> Rex
> On Tue, 2022-03-01 at 08:35 +0100, Matthias Brugger wrote:
>>
>> On 17/02/2022 09:26, Rex-BC Chen wrote:
>>> v2:
>>> 1. Change variable type from unsigned int to u16.
>>>
>>> v1:
>>> 1. Add a new variable in mmsys driver data to control different
>>> register
>>>      offset for different SoCs.
>>> 2. Add MT8183 reset register offset.
>>> 3. Add mmsys reset control for MT8186.
>>>
>>> This series is based on mmsys patch for MT8186 on [1].
>>> [1]:
>>> https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/patch/20220216084831.14883-4-rex-bc.chen@mediatek.com/__;!!CTRNKA9wMg0ARbw!wp2uW6phbST6ySpYRlb7iwmATAVkkCL1JBau9dCwhXj8KRasrAWnLd1EUT9gLE-oF9Yq$
>>>   
>>>
>>> Rex-BC Chen (2):
>>>     soc: mediatek: mmsys: add sw0_rst_offset in mmsys driver data
>>>     soc: mediatek: mmsys: add mmsys reset control for MT8186
>>>
>>>    drivers/soc/mediatek/mt8183-mmsys.h | 2 ++
>>>    drivers/soc/mediatek/mt8186-mmsys.h | 2 ++
>>>    drivers/soc/mediatek/mtk-mmsys.c    | 7 +++++--
>>>    drivers/soc/mediatek/mtk-mmsys.h    | 3 +--
>>>    4 files changed, 10 insertions(+), 4 deletions(-)
>>>
>>
>> Series applied, thanks!
>