From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleksij Rempel Date: Sun, 10 Jul 2016 11:28:22 +0200 Subject: [ath9k-devel] [PATCH v4 3/3] ath9k: parse the device configuration from an OF node In-Reply-To: <87inwe9wfb.fsf@miraculix.mork.no> References: <20160624123430.4097-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-4-martin.blumenstingl@googlemail.com> <87inwe9wfb.fsf@miraculix.mork.no> Message-ID: <57821536.3020000@rempel-privat.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ath9k-devel@lists.ath9k.org Am 10.07.2016 um 02:19 schrieb Bj?rn Mork: > Martin Blumenstingl writes: > >> + if (of_property_read_bool(np, "qca,clk-25mhz")) >> + ah->is_clk_25mhz = true; >> + >> + if (of_property_read_bool(np, "qca,disable-2ghz")) >> + ah->disable_2ghz = true; >> + >> + if (of_property_read_bool(np, "qca,disable-5ghz")) >> + ah->disable_5ghz = true; > > This is bike-shedding, but how about > > ah->is_clk_25mhz = of_property_read_bool(np, "qca,clk-25mhz"); > ah->disable_2ghz = of_property_read_bool(np, "qca,disable-2ghz"); > ah->disable_5ghz = of_property_read_bool(np, "qca,disable-5ghz"); > > instead? Hm... i assume each WiFi hw in the world can reuse this settings, may be it i worth to use generic names? -- Regards, Oleksij -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 213 bytes Desc: OpenPGP digital signature Url : http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20160710/c6337c74/attachment.pgp From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mout.gmx.net ([212.227.17.20]:61724 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750822AbcGJJ3Z (ORCPT ); Sun, 10 Jul 2016 05:29:25 -0400 Subject: Re: [PATCH v4 3/3] ath9k: parse the device configuration from an OF node To: =?UTF-8?Q?Bj=c3=b8rn_Mork?= , Martin Blumenstingl References: <20160624123430.4097-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-4-martin.blumenstingl@googlemail.com> <87inwe9wfb.fsf@miraculix.mork.no> Cc: ath9k-devel@venema.h4ckr.net, devicetree@vger.kernel.org, linux-wireless@vger.kernel.org, ath9k-devel@qca.qualcomm.com, mcgrof@do-not-panic.com, galak@codeaurora.org, ijc+devicetree@hellion.org.uk, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org, kvalo@codeaurora.org, chunkeey@googlemail.com, arend.vanspriel@broadcom.com, julian.calaby@gmail.com From: Oleksij Rempel Message-ID: <57821536.3020000@rempel-privat.de> (sfid-20160710_112930_451916_D65FE8CB) Date: Sun, 10 Jul 2016 11:28:22 +0200 MIME-Version: 1.0 In-Reply-To: <87inwe9wfb.fsf@miraculix.mork.no> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL" Sender: linux-wireless-owner@vger.kernel.org List-ID: This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL Content-Type: multipart/mixed; boundary="Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7" From: Oleksij Rempel To: =?UTF-8?Q?Bj=c3=b8rn_Mork?= , Martin Blumenstingl Cc: ath9k-devel@venema.h4ckr.net, devicetree@vger.kernel.org, linux-wireless@vger.kernel.org, ath9k-devel@qca.qualcomm.com, mcgrof@do-not-panic.com, galak@codeaurora.org, ijc+devicetree@hellion.org.uk, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org, kvalo@codeaurora.org, chunkeey@googlemail.com, arend.vanspriel@broadcom.com, julian.calaby@gmail.com Message-ID: <57821536.3020000@rempel-privat.de> Subject: Re: [PATCH v4 3/3] ath9k: parse the device configuration from an OF node References: <20160624123430.4097-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-4-martin.blumenstingl@googlemail.com> <87inwe9wfb.fsf@miraculix.mork.no> In-Reply-To: <87inwe9wfb.fsf@miraculix.mork.no> --Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Am 10.07.2016 um 02:19 schrieb Bj=C3=B8rn Mork: > Martin Blumenstingl writes: >=20 >> + if (of_property_read_bool(np, "qca,clk-25mhz")) >> + ah->is_clk_25mhz =3D true; >> + >> + if (of_property_read_bool(np, "qca,disable-2ghz")) >> + ah->disable_2ghz =3D true; >> + >> + if (of_property_read_bool(np, "qca,disable-5ghz")) >> + ah->disable_5ghz =3D true; >=20 > This is bike-shedding, but how about >=20 > ah->is_clk_25mhz =3D of_property_read_bool(np, "qca,clk-25mhz"); > ah->disable_2ghz =3D of_property_read_bool(np, "qca,disable-2ghz"); > ah->disable_5ghz =3D of_property_read_bool(np, "qca,disable-5ghz"); >=20 > instead? Hm... i assume each WiFi hw in the world can reuse this settings, may be it i worth to use generic names? --=20 Regards, Oleksij --Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7-- --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iF4EAREIAAYFAleCFUYACgkQHwImuRkmbWkbfQD/QmWNTeL6XkKEncYbzGKQijUk Yuv3MxQ2qc5oFBiTMpYA/038A3ay4mfifoGXoi5Mm1VRVZPLMKmyYHCJu07APQng =Cglo -----END PGP SIGNATURE----- --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleksij Rempel Subject: Re: [PATCH v4 3/3] ath9k: parse the device configuration from an OF node Date: Sun, 10 Jul 2016 11:28:22 +0200 Message-ID: <57821536.3020000@rempel-privat.de> References: <20160624123430.4097-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-1-martin.blumenstingl@googlemail.com> <20160709232834.31654-4-martin.blumenstingl@googlemail.com> <87inwe9wfb.fsf@miraculix.mork.no> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL" Return-path: In-Reply-To: <87inwe9wfb.fsf-3F4PFWf5pNjpjLOzFPqGjWGXanvQGlWp@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: =?UTF-8?Q?Bj=c3=b8rn_Mork?= , Martin Blumenstingl Cc: ath9k-devel-juf53994utBLZpfksSYvnA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, ath9k-devel-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org, mcgrof-3uybbJdB1yH774rrrx3eTA@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, chunkeey-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org, arend.vanspriel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org, julian.calaby-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org List-Id: devicetree@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL Content-Type: multipart/mixed; boundary="Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7" From: Oleksij Rempel To: =?UTF-8?Q?Bj=c3=b8rn_Mork?= , Martin Blumenstingl Cc: ath9k-devel-juf53994utBLZpfksSYvnA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, ath9k-devel-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org, mcgrof-3uybbJdB1yH774rrrx3eTA@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, chunkeey-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org, arend.vanspriel-dY08KVG/lbpWk0Htik3J/w@public.gmane.org, julian.calaby-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Message-ID: <57821536.3020000-YEK0n+YFykbzxQdaRaTXBw@public.gmane.org> Subject: Re: [PATCH v4 3/3] ath9k: parse the device configuration from an OF node References: <20160624123430.4097-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> <20160709232834.31654-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> <20160709232834.31654-4-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> <87inwe9wfb.fsf-3F4PFWf5pNjpjLOzFPqGjWGXanvQGlWp@public.gmane.org> In-Reply-To: <87inwe9wfb.fsf-3F4PFWf5pNjpjLOzFPqGjWGXanvQGlWp@public.gmane.org> --Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Am 10.07.2016 um 02:19 schrieb Bj=C3=B8rn Mork: > Martin Blumenstingl writes: >=20 >> + if (of_property_read_bool(np, "qca,clk-25mhz")) >> + ah->is_clk_25mhz =3D true; >> + >> + if (of_property_read_bool(np, "qca,disable-2ghz")) >> + ah->disable_2ghz =3D true; >> + >> + if (of_property_read_bool(np, "qca,disable-5ghz")) >> + ah->disable_5ghz =3D true; >=20 > This is bike-shedding, but how about >=20 > ah->is_clk_25mhz =3D of_property_read_bool(np, "qca,clk-25mhz"); > ah->disable_2ghz =3D of_property_read_bool(np, "qca,disable-2ghz"); > ah->disable_5ghz =3D of_property_read_bool(np, "qca,disable-5ghz"); >=20 > instead? Hm... i assume each WiFi hw in the world can reuse this settings, may be it i worth to use generic names? --=20 Regards, Oleksij --Id1sqr1CI0Gfbo1lLbu8qAhs6d0UmrCV7-- --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iF4EAREIAAYFAleCFUYACgkQHwImuRkmbWkbfQD/QmWNTeL6XkKEncYbzGKQijUk Yuv3MxQ2qc5oFBiTMpYA/038A3ay4mfifoGXoi5Mm1VRVZPLMKmyYHCJu07APQng =Cglo -----END PGP SIGNATURE----- --2G07aagLxm6TmdhJgFd3JVxrEk0FhicIL-- -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html