From: "André Przywara" <andre.przywara@arm.com>
To: Amit Tomer <amittomer25@gmail.com>
Cc: "Andreas Färber" <afaerber@suse.de>,
"Manivannan Sadhasivam" <manivannan.sadhasivam@linaro.org>,
"Rob Herring" <robh+dt@kernel.org>,
cristian.ciocaltea@gmail.com,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
linux-actions@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v1 8/9] arm64: dts: actions: Add MMC controller support for S700
Date: Sun, 17 May 2020 17:42:06 +0100 [thread overview]
Message-ID: <2cd3cdaf-826e-9d12-9fd4-9f7e2a517ecd@arm.com> (raw)
In-Reply-To: <CABHD4K9yjUGuo0w-RfhdZQJm3Wtj6bU2H4DXcp4Jjp=e0fFeyA@mail.gmail.com>
On 17/05/2020 12:56, Amit Tomer wrote:
> Hi,
>
>> So, using "actions,s700-mmc", "actions,owl-mmc" here, adding this combo
>> to the binding, but leaving the driver alone for now.
>
> But if we leave this new string from driver , there would be DT
> validation issue.
I don't understand what this has to do with the driver, but I asked
above to also change the binding, allowing this compatible string
combination.
Cheers,
Andre
> Are we okay with it ?
>
> Thanks
> -Amit
>
WARNING: multiple messages have this Message-ID (diff)
From: "André Przywara" <andre.przywara@arm.com>
To: Amit Tomer <amittomer25@gmail.com>
Cc: devicetree@vger.kernel.org, linux-actions@lists.infradead.org,
cristian.ciocaltea@gmail.com, "Rob Herring" <robh+dt@kernel.org>,
"Manivannan Sadhasivam" <manivannan.sadhasivam@linaro.org>,
"Andreas Färber" <afaerber@suse.de>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v1 8/9] arm64: dts: actions: Add MMC controller support for S700
Date: Sun, 17 May 2020 17:42:06 +0100 [thread overview]
Message-ID: <2cd3cdaf-826e-9d12-9fd4-9f7e2a517ecd@arm.com> (raw)
In-Reply-To: <CABHD4K9yjUGuo0w-RfhdZQJm3Wtj6bU2H4DXcp4Jjp=e0fFeyA@mail.gmail.com>
On 17/05/2020 12:56, Amit Tomer wrote:
> Hi,
>
>> So, using "actions,s700-mmc", "actions,owl-mmc" here, adding this combo
>> to the binding, but leaving the driver alone for now.
>
> But if we leave this new string from driver , there would be DT
> validation issue.
I don't understand what this has to do with the driver, but I asked
above to also change the binding, allowing this compatible string
combination.
Cheers,
Andre
> Are we okay with it ?
>
> Thanks
> -Amit
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-05-17 16:42 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-14 16:10 [PATCH v1 0/9] Add MMC and DMA support for Actions S700 Amit Singh Tomar
2020-05-14 16:10 ` [PATCH v1 1/9] dmaengine: Actions: get rid of bit fields from dma descriptor Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-14 18:27 ` Vinod Koul
2020-05-14 18:27 ` Vinod Koul
2020-05-14 18:34 ` Amit Tomer
2020-05-14 18:34 ` Amit Tomer
2020-05-15 6:58 ` Vinod Koul
2020-05-15 6:58 ` Vinod Koul
2020-05-15 7:46 ` Amit Tomer
2020-05-15 7:46 ` Amit Tomer
2020-05-15 11:11 ` Amit Tomer
2020-05-15 11:11 ` Amit Tomer
2020-05-14 16:10 ` [PATCH v1 2/9] dmaengine: Actions: Add support for S700 DMA engine Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-14 17:02 ` André Przywara
2020-05-14 17:02 ` André Przywara
2020-05-14 16:10 ` [PATCH v1 3/9] clk: actions: Add MMC clock-register reset bits Amit Singh Tomar
2020-05-14 16:10 ` [PATCH v1 4/9] arm64: dts: actions: do not merge disable sps node from S700 Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-14 16:10 ` [PATCH v1 5/9] dt-bindings: dmaengine: convert Actions Semi Owl SoCs bindings to yaml Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-15 15:54 ` André Przywara
2020-05-15 15:54 ` André Przywara
2020-05-15 19:05 ` Amit Tomer
2020-05-15 19:05 ` Amit Tomer
2020-05-15 19:24 ` Amit Tomer
2020-05-15 19:24 ` Amit Tomer
2020-05-16 18:21 ` Amit Tomer
2020-05-16 18:21 ` Amit Tomer
2020-05-14 16:10 ` [PATCH v1 6/9] arm64: dts: actions: Add DMA Controller for S700 Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-14 16:10 ` [PATCH v1 7/9] dt-bindings: reset: s700: Add binding constants for mmc Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-14 16:10 ` [PATCH v1 8/9] arm64: dts: actions: Add MMC controller support for S700 Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
2020-05-15 15:01 ` André Przywara
2020-05-15 15:01 ` André Przywara
2020-05-15 18:41 ` Amit Tomer
2020-05-15 18:41 ` Amit Tomer
2020-05-17 11:56 ` Amit Tomer
2020-05-17 11:56 ` Amit Tomer
2020-05-17 16:42 ` André Przywara [this message]
2020-05-17 16:42 ` André Przywara
2020-05-17 17:12 ` Amit Tomer
2020-05-17 17:12 ` Amit Tomer
2020-05-17 21:30 ` André Przywara
2020-05-17 21:30 ` André Przywara
2020-05-18 3:06 ` Amit Tomer
2020-05-18 3:06 ` Amit Tomer
2020-05-18 6:17 ` Manivannan Sadhasivam
2020-05-18 6:17 ` Manivannan Sadhasivam
2020-05-18 8:29 ` André Przywara
2020-05-18 8:29 ` André Przywara
2020-05-18 9:48 ` Andreas Färber
2020-05-18 9:48 ` Andreas Färber
2020-05-14 16:10 ` [PATCH v1 9/9] arm64: dts: actions: Add uSD support for Cubieboard7 Amit Singh Tomar
2020-05-14 16:10 ` Amit Singh Tomar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2cd3cdaf-826e-9d12-9fd4-9f7e2a517ecd@arm.com \
--to=andre.przywara@arm.com \
--cc=afaerber@suse.de \
--cc=amittomer25@gmail.com \
--cc=cristian.ciocaltea@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-actions@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.