Hi Bin > Hi Rick, > > On Wed, Jun 24, 2020 at 1:24 PM Pragnesh Patel > <pragnesh.pa...@sifive.com> wrote: > > > > Hi Rick, > > > > >-----Original Message----- > > >From: Rick Chen <rickche...@gmail.com> > > >Sent: 24 June 2020 10:44 > > >To: Pragnesh Patel <pragnesh.pa...@sifive.com> > > >Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Atish Patra > > ><atish.pa...@wdc.com>; palmerdabb...@google.com; Bin Meng > > ><bmeng...@gmail.com>; Paul Walmsley ( Sifive) > > ><paul.walms...@sifive.com>; Anup Patel <anup.pa...@wdc.com>; Sagar > > >Kadam <sagar.ka...@sifive.com>; Palmer Dabbelt <pal...@dabbelt.com>; > > >Jagan Teki <ja...@amarulasolutions.com>; rick <r...@andestech.com>; Alan > > >Kao <alan...@andestech.com> > > >Subject: Re: [PATCH 2/2] riscv: sifive: fu540: Enable SiFive PWM driver > > > > > >[External Email] Do not click links or attachments unless you recognize the > > >sender and know the content is safe > > > > > >Hi Pragnesh > > > > > >> Hi Rick, > > >> > > >> >-----Original Message----- > > >> >From: Rick Chen <rickche...@gmail.com> > > >> >Sent: 24 June 2020 06:30 > > >> >To: Pragnesh Patel <pragnesh.pa...@sifive.com> > > >> >Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Atish Patra > > >> ><atish.pa...@wdc.com>; palmerdabb...@google.com; Bin Meng > > >> ><bmeng...@gmail.com>; Paul Walmsley ( Sifive) > > >> ><paul.walms...@sifive.com>; Anup Patel <anup.pa...@wdc.com>; Sagar > > >> >Kadam <sagar.ka...@sifive.com>; Palmer Dabbelt > > ><pal...@dabbelt.com>; > > >> >Jagan Teki <ja...@amarulasolutions.com>; rick <r...@andestech.com>; > > >> >Alan Kao <alan...@andestech.com> > > >> >Subject: Re: [PATCH 2/2] riscv: sifive: fu540: Enable SiFive PWM > > >> >driver > > >> > > > >> >[External Email] Do not click links or attachments unless you > > >> >recognize the sender and know the content is safe > > >> > > > >> >Hi Pragnesh > > >> > > > >> >> From: Pragnesh Patel [mailto:pragnesh.pa...@sifive.com] > > >> >> Sent: Friday, May 29, 2020 2:45 PM > > >> >> To: u-boot@lists.denx.de > > >> >> Cc: atish.pa...@wdc.com; palmerdabb...@google.com; > > >> >bmeng...@gmail.com; paul.walms...@sifive.com; anup.pa...@wdc.com; > > >> >sagar.ka...@sifive.com; Rick Jian-Zhi Chen(陳建志); Pragnesh Patel; > > >> >Palmer Dabbelt > > >> >> Subject: [PATCH 2/2] riscv: sifive: fu540: Enable SiFive PWM driver > > >> >> > > >> >> This patch enables SiFive PWM driver for the SiFive Unleashed board. > > >> >> > > >> >> Signed-off-by: Pragnesh Patel <pragnesh.pa...@sifive.com> > > >> >> --- > > >> >> board/sifive/fu540/Kconfig | 2 ++ > > >> >> 1 file changed, 2 insertions(+) > > >> >> > > >> >> diff --git a/board/sifive/fu540/Kconfig > > >> >> b/board/sifive/fu540/Kconfig index > > >> >86193d7668..683668d059 100644 > > >> >> --- a/board/sifive/fu540/Kconfig > > >> >> +++ b/board/sifive/fu540/Kconfig > > >> >> @@ -65,5 +65,7 @@ config BOARD_SPECIFIC_OPTIONS # dummy > > >> >> imply SMP > > >> >> imply MISC > > >> >> imply SIFIVE_OTP > > >> >> + imply DM_PWM > > >> >> + imply PWM_SIFIVE > > >> >> > > >> > > > >> >This patch shall follow [PATCH v2 0/2] Add support for PWM SiFive. > > >> >It is weird to introduce here and not appropriate to depend on another > > >patch. > > >> > > >> Do you want me to send this 2 patches separately independent of each > > >other ? > > > > > >How about merged [PATCH 2/2] riscv: sifive: fu540: Enable SiFive PWM driver > > >into [PATCH v2 0/2] Add support for PWM SiFive ? > > > > I am okay with it, you can go ahead and merge this patch into PWM series of > > Yash. > > > > They are separate patches and should keep separate. I am not sure > what's the issue we want to resolve?
Nothing about resolve. Logically if they can be put together, it will be more reasonable. Thanks, Rick > > Regards, > Bin