Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   10 Jan 2019 12:46:29 +0900
Message-ID: <8736q1b2w8.wl-kuninori.morimoto.gx@renesas.com>
From:   Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To:     Mark Brown <broonie@kernel.org>, Jon Hunter <jonathanh@nvidia.com>
Cc:     Liam Girdwood <lgirdwood@gmail.com>, linux-tegra@vger.kernel.org,
        Matthias Reichl <hias@horus.com>, alsa-devel@alsa-project.org,
        Marcel Ziswiler <marcel.ziswiler@toradex.com>,
        Takashi Iwai <tiwai@suse.com>, linux-kernel@vger.kernel.org,
        Marcel Ziswiler <marcel@ziswiler.com>
Subject: Re: [alsa-devel] [PATCH v1 3/3] ASoC: soc-core: fix platform name vs. of_node assignement
In-Reply-To: <875zuxb9uy.wl-kuninori.morimoto.gx@renesas.com>
References: <20181218174040.k7u26vnnoplllnwb@camel2.lan>
        <952471da-b355-6471-6c19-5120d6704f81@nvidia.com>
        <87lg3vuc7p.wl-kuninori.morimoto.gx@renesas.com>
        <b5164c93-8be8-1c3d-9415-c895da6ad622@nvidia.com>
        <d1e41ed2-da7c-b46c-bc6d-64cf3e536771@nvidia.com>
        <865d2a3e-bf6b-1f30-1179-7e922c0d0641@nvidia.com>
        <87k1je38w7.wl-kuninori.morimoto.gx@renesas.com>
        <0660a471-3698-2059-4494-ad146518a4ed@nvidia.com>
        <20190109125340.GD10405@sirena.org.uk>
        <52f5f2cf-9260-f9c5-f73d-3c3d5debc86b@nvidia.com>
        <20190109141439.GE10405@sirena.org.uk>
        <875zuxb9uy.wl-kuninori.morimoto.gx@renesas.com>
User-Agent: Wanderlust/2.15.9 Emacs/24.5 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


Hi Mark, Jon again

> My posted quick-patch can solve "dirty pointer" issue,
> but it can't solve "memory leak" issue.
> This issue will be solved if all driver can switch to
> modern style, but it needs more time.
> Are these correct ?

Sorry I was very confused.
I think the issue is only "dirty pointer",
there is no "memory leak" issue (= devm_kzalloc()).

And the things Jon is worry about is that why we need to
have platform pointer.
And the answer is we need multi platform support in the future
(pointer is prepare for it).
Are these correct ?

If so my posted patch can solve all issues ?

Best regards
---
Kuninori Morimoto