Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1343293 > unrolled thread

Re: [PATCH V8 0/8] Add support for MAXIM MAX77620/MAX20024 PMIC

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH V8 0/8] Add support for MAXIM MAX77620/MAX20024 PMIC Thierry Reding <treding@nvidia.com> - 2016-02-25 17:20 +0100
    Re: [PATCH V8 0/8] Add support for MAXIM MAX77620/MAX20024 PMIC Laxman Dewangan <ldewangan@nvidia.com> - 2016-03-01 12:50 +0100

#1343293 — Re: [PATCH V8 0/8] Add support for MAXIM MAX77620/MAX20024 PMIC

[Multipart message — attachments visible in raw view] — view raw

On Sat, Feb 20, 2016 at 10:10:56AM +0530, Laxman Dewangan wrote:
> 
> On Tuesday 16 February 2016 09:30 PM, Thierry Reding wrote:
> >On Thu, Feb 11, 2016 at 05:26:26PM +0530, Laxman Dewangan wrote:
> >>Add SW support for MAXIM Semiconductor's Power Management
> >>IC (PMIC) MAX77620/MAX20024. This PMIC supports DC-DC/LDOS, GPIOs,
> >>RTC, watchdog, clocks etc.
> >Hi Laxman,
> >
> >This series is starting to look pretty good. Do you have patches in the
> >works to add the corresponding DT content for the various boards that we
> >support? I think the Jetson TX1 patch would be most interesting since
> >that's what people can get their hands on.
> >
> 
> Yes, I am just waiting for this series to be concluded and applied. Once it
> is done, I will immediately post the patches to enable this driver in config
> and dt entry for the Jetson-TX1.

Why hold back on the patches? I'm currently having trouble testing
your stash here on Jetson TX1 and I suspect that I'm using the wrong
device tree content (it was derived from downstream many moons ago).
There's nothing wrong with posting patches that depend on patches on
the list. Typically you'd note the dependency in the cover letter to
make it easier for people to find what belongs together.

Thierry

[toc] | [next] | [standalone]

#1346546

On Thursday 25 February 2016 09:45 PM, Thierry Reding wrote:
> On Sat, Feb 20, 2016 at 10:10:56AM +0530, Laxman Dewangan wrote:
>> On Tuesday 16 February 2016 09:30 PM, Thierry Reding wrote:
>>> On Thu, Feb 11, 2016 at 05:26:26PM +0530, Laxman Dewangan wrote:
>>>> Add SW support for MAXIM Semiconductor's Power Management
>>>> IC (PMIC) MAX77620/MAX20024. This PMIC supports DC-DC/LDOS, GPIOs,
>>>> RTC, watchdog, clocks etc.
>>> Hi Laxman,
>>>
>>> This series is starting to look pretty good. Do you have patches in the
>>> works to add the corresponding DT content for the various boards that we
>>> support? I think the Jetson TX1 patch would be most interesting since
>>> that's what people can get their hands on.
>>>
>> Yes, I am just waiting for this series to be concluded and applied. Once it
>> is done, I will immediately post the patches to enable this driver in config
>> and dt entry for the Jetson-TX1.
> Why hold back on the patches? I'm currently having trouble testing
> your stash here on Jetson TX1 and I suspect that I'm using the wrong
> device tree content (it was derived from downstream many moons ago).
> There's nothing wrong with posting patches that depend on patches on
> the list. Typically you'd note the dependency in the cover letter to
> make it easier for people to find what belongs together.
>

OK, posted patches as:

[PATCH 2/2] arm64: tegra: Jetson-TX1: Add DT entry for system PMIC MAX77620
[PATCH 1/2] arm64: tegra: Add DT entry for keys used in Jetson-TX1
[PATCH 1/1] arm64: defconfig: Enable Maxim PMIC max77620 config

Currently, the execution is crashing with this series + above patches as 
MFD_CELLS are not correctly defined.
Discussion is going on with patch from Rhyland Klein
[PATCH] mfd: Fix MACRO for commonly declared MFD cell attributes

Once MFD_CELL defines get fixed, either series need to be revisit or 
automatically fix the issue.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web