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


Groups > linux.kernel > #1196410 > unrolled thread

Re: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan

Started byScott Wood <scottwood@freescale.com>
First post2015-07-30 22:10 +0200
Last post2015-08-03 09:40 +0200
Articles 2 — 2 participants

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 v2 1/2] powerpc/mpc85xx: Create dts components for the  FSL QorIQ DPAA FMan Scott Wood <scottwood@freescale.com> - 2015-07-30 22:10 +0200
    RE: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL  QorIQ DPAA FMan Liberman Igal <Igal.Liberman@freescale.com> - 2015-08-03 09:40 +0200

#1196410 — Re: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan

FromScott Wood <scottwood@freescale.com>
Date2015-07-30 22:10 +0200
SubjectRe: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan
Message-ID<pS2jo-dO-19@gated-at.bofh.it>
On Thu, 2015-07-30 at 07:32 +0300, Igal.Liberman wrote:
> +fman0:  fman@400000{
> +     #address-cells = <1>;
> +     #size-cells = <1>;
> +     cell-index = <0>;
> +     compatible = "fsl,fman";
> +     ranges = <0 0x400000 0x100000>;
> +     reg = <0x400000 0x100000>;
> +     interrupts = <96 2 0 0>, <16 2 1 1>;
> +     clocks = <&clockgen 3 0>;
> +     clock-names = "fm0clk";

clock-names should be "fmanclk" as per the binding.  And why would you want 
the driver to have to encode the fman index into the name in order to look 
the clock up by name?

-Scott

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1198601 — RE: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan

FromLiberman Igal <Igal.Liberman@freescale.com>
Date2015-08-03 09:40 +0200
SubjectRE: [PATCH v2 1/2] powerpc/mpc85xx: Create dts components for the FSL QorIQ DPAA FMan
Message-ID<pTivM-4OW-15@gated-at.bofh.it>
In reply to#1196410
DQoNClJlZ2FyZHMsDQpJZ2FsIExpYmVybWFuLg0KDQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0t
LS0tDQo+IEZyb206IFdvb2QgU2NvdHQtQjA3NDIxDQo+IFNlbnQ6IFRodXJzZGF5LCBKdWx5IDMw
LCAyMDE1IDExOjAwIFBNDQo+IFRvOiBMaWJlcm1hbiBJZ2FsLUIzMTk1MA0KPiBDYzogZGV2aWNl
dHJlZUB2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4cHBjLWRldkBsaXN0cy5vemxhYnMub3JnOyBCdWN1
cg0KPiBNYWRhbGluLUNyaXN0aWFuLUIzMjcxNjsgbGludXgta2VybmVsQHZnZXIua2VybmVsLm9y
Zw0KPiBTdWJqZWN0OiBSZTogW1BBVENIIHYyIDEvMl0gcG93ZXJwYy9tcGM4NXh4OiBDcmVhdGUg
ZHRzIGNvbXBvbmVudHMgZm9yDQo+IHRoZSBGU0wgUW9ySVEgRFBBQSBGTWFuDQo+IA0KPiBPbiBU
aHUsIDIwMTUtMDctMzAgYXQgMDc6MzIgKzAzMDAsIElnYWwuTGliZXJtYW4gd3JvdGU6DQo+ID4g
K2ZtYW4wOiAgZm1hbkA0MDAwMDB7DQo+ID4gKyAgICAgI2FkZHJlc3MtY2VsbHMgPSA8MT47DQo+
ID4gKyAgICAgI3NpemUtY2VsbHMgPSA8MT47DQo+ID4gKyAgICAgY2VsbC1pbmRleCA9IDwwPjsN
Cj4gPiArICAgICBjb21wYXRpYmxlID0gImZzbCxmbWFuIjsNCj4gPiArICAgICByYW5nZXMgPSA8
MCAweDQwMDAwMCAweDEwMDAwMD47DQo+ID4gKyAgICAgcmVnID0gPDB4NDAwMDAwIDB4MTAwMDAw
PjsNCj4gPiArICAgICBpbnRlcnJ1cHRzID0gPDk2IDIgMCAwPiwgPDE2IDIgMSAxPjsNCj4gPiAr
ICAgICBjbG9ja3MgPSA8JmNsb2NrZ2VuIDMgMD47DQo+ID4gKyAgICAgY2xvY2stbmFtZXMgPSAi
Zm0wY2xrIjsNCj4gDQo+IGNsb2NrLW5hbWVzIHNob3VsZCBiZSAiZm1hbmNsayIgYXMgcGVyIHRo
ZSBiaW5kaW5nLiAgQW5kIHdoeSB3b3VsZCB5b3UNCj4gd2FudCB0aGUgZHJpdmVyIHRvIGhhdmUg
dG8gZW5jb2RlIHRoZSBmbWFuIGluZGV4IGludG8gdGhlIG5hbWUgaW4gb3JkZXIgdG8NCj4gbG9v
ayB0aGUgY2xvY2sgdXAgYnkgbmFtZT8NCj4gDQoNCldlbGwsIHllcywgdGhpcyBzaG91bGQgYmUg
ImZtYW5jbGsiLiBUaGUgZHJpdmVyIGRvZXNuJ3QgbmVlZCB0byBrbm93IHRoZSBpbmRleC4NCkkn
bGwgZml4IHRoYXQgYW5kIHJlLXN1Ym1pdC4NCg0KSWdhbC4NCg0KPiAtU2NvdHQNCg0K
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web