From: Stephen Boyd <sboyd@kernel.org> To: andy.gross@linaro.org Cc: david.brown@linaro.org, gregkh@linuxfoundation.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, rafael@kernel.org, bjorn.andersson@linaro.org, vkoul@kernel.org, Vaishali Thakkar <vaishali.thakkar@linaro.org> Subject: Re: [PATCH v4 5/5] soc: qcom: socinfo: Expose image information Date: Thu, 28 Feb 2019 13:34:22 -0800 [thread overview] Message-ID: <155138966250.16805.8149986906489473683@swboyd.mtv.corp.google.com> (raw) In-Reply-To: <20190225065044.11023-6-vaishali.thakkar@linaro.org> Quoting Vaishali Thakkar (2019-02-24 22:50:44) > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + > + qcom_socinfo->mpss = debugfs_create_dir("mpss", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + > + qcom_socinfo->adsp = debugfs_create_dir("adsp", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + DEBUGFS_IMAGE_OEM(oem, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + > + qcom_socinfo->cnss = debugfs_create_dir("cnss", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + > + qcom_socinfo->video = debugfs_create_dir("video", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); > + DEBUGFS_IMAGE_OEM(oem, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); Couldn't these four lines be a function itself that takes the name, dbg_root, index, etc and create a directory? Looks like a lot of copy paste right now. > } > > static void socinfo_debugfs_exit(struct qcom_socinfo *qcom_socinfo) > -- > 2.17.1 >
WARNING: multiple messages have this Message-ID (diff)
From: Stephen Boyd <sboyd@kernel.org> To: Vaishali Thakkar <vaishali.thakkar@linaro.org>, andy.gross@linaro.org Cc: david.brown@linaro.org, gregkh@linuxfoundation.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, rafael@kernel.org, bjorn.andersson@linaro.org, vkoul@kernel.org, Vaishali Thakkar <vaishali.thakkar@linaro.org> Subject: Re: [PATCH v4 5/5] soc: qcom: socinfo: Expose image information Date: Thu, 28 Feb 2019 13:34:22 -0800 [thread overview] Message-ID: <155138966250.16805.8149986906489473683@swboyd.mtv.corp.google.com> (raw) In-Reply-To: <20190225065044.11023-6-vaishali.thakkar@linaro.org> Quoting Vaishali Thakkar (2019-02-24 22:50:44) > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, apps, SMEM_IMAGE_TABLE_APPS_INDEX); > + > + qcom_socinfo->mpss = debugfs_create_dir("mpss", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, mpss, SMEM_IMAGE_TABLE_MPSS_INDEX); > + > + qcom_socinfo->adsp = debugfs_create_dir("adsp", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + DEBUGFS_IMAGE_OEM(oem, adsp, SMEM_IMAGE_TABLE_ADSP_INDEX); > + > + qcom_socinfo->cnss = debugfs_create_dir("cnss", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + DEBUGFS_IMAGE_OEM(oem, cnss, SMEM_IMAGE_TABLE_CNSS_INDEX); > + > + qcom_socinfo->video = debugfs_create_dir("video", > + qcom_socinfo->dbg_root); > + > + DEBUGFS_IMAGE_NAME(name, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); > + DEBUGFS_IMAGE_VARIANT(variant, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); > + DEBUGFS_IMAGE_OEM(oem, video, SMEM_IMAGE_TABLE_VIDEO_INDEX); Couldn't these four lines be a function itself that takes the name, dbg_root, index, etc and create a directory? Looks like a lot of copy paste right now. > } > > static void socinfo_debugfs_exit(struct qcom_socinfo *qcom_socinfo) > -- > 2.17.1 >
next prev parent reply other threads:[~2019-02-28 21:34 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-25 6:50 [PATCH v4 0/5] soc: qcom: Add SoC info driver Vaishali Thakkar 2019-02-25 6:50 ` [PATCH v4 1/5] base: soc: Add serial_number attribute to soc Vaishali Thakkar 2019-02-28 19:23 ` Stephen Boyd 2019-02-28 19:23 ` Stephen Boyd 2019-02-25 6:50 ` [PATCH v4 2/5] base: soc: Export soc_device_register/unregister APIs Vaishali Thakkar 2019-02-28 19:23 ` Stephen Boyd 2019-02-28 19:23 ` Stephen Boyd 2019-03-01 19:08 ` Bjorn Andersson 2019-02-25 6:50 ` [PATCH v4 3/5] soc: qcom: Add socinfo driver Vaishali Thakkar 2019-02-28 19:34 ` Stephen Boyd 2019-02-28 19:34 ` Stephen Boyd 2019-03-01 19:23 ` Bjorn Andersson 2019-02-25 6:50 ` [PATCH v4 4/5] soc: qcom: socinfo: Expose custom attributes Vaishali Thakkar 2019-02-28 21:32 ` Stephen Boyd 2019-02-28 21:32 ` Stephen Boyd 2019-03-14 11:25 ` Vaishali Thakkar 2019-03-14 15:58 ` Stephen Boyd 2019-03-21 5:51 ` Vaishali Thakkar 2019-03-23 0:01 ` Stephen Boyd 2019-03-24 17:42 ` Vaishali Thakkar 2019-03-25 16:01 ` Stephen Boyd 2019-03-25 20:58 ` Vaishali Thakkar 2019-03-01 19:42 ` Bjorn Andersson 2019-02-25 6:50 ` [PATCH v4 5/5] soc: qcom: socinfo: Expose image information Vaishali Thakkar 2019-02-28 21:34 ` Stephen Boyd [this message] 2019-02-28 21:34 ` Stephen Boyd 2019-03-01 19:51 ` Bjorn Andersson
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=155138966250.16805.8149986906489473683@swboyd.mtv.corp.google.com \ --to=sboyd@kernel.org \ --cc=andy.gross@linaro.org \ --cc=bjorn.andersson@linaro.org \ --cc=david.brown@linaro.org \ --cc=gregkh@linuxfoundation.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=rafael@kernel.org \ --cc=vaishali.thakkar@linaro.org \ --cc=vkoul@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: linkBe 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.